-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update unit testing #38
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #38 +/- ##
===========================================
+ Coverage 19.18% 34.37% +15.18%
===========================================
Files 34 36 +2
Lines 1777 1882 +105
===========================================
+ Hits 341 647 +306
+ Misses 1436 1235 -201 ☔ View full report in Codecov by Sentry. |
@gvegayon We should add tests for other files (e.g., other model types, generic model methods, etc), but I think it would be better tackled as separate issues. If you want me to do that here, let me know. Otherwise, I'll create issues for the different areas that need unit testing. |
@gvegayon I've added a few more test files, but I think this is a good place to stop for this PR. We will add more tests down the line, but we should merge these ones. |
Strengthens unit testing:
test-sir.R
to monitor:test-lfmcmc.R
to monitor:test-sirconn.R
,test-sird.R
,test-sis.R
,test-seir.R
with checks for:test-tool.R
with checks for: