You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have status tests now thanks to @Charlottez112, but we still do not test the proper output given submission statuses and eligibility/completion changes. This limits our testing coverage and makes detecting bugs in changes to the status code difficult.
Proposed solution
Add tests using the FakeScheduler or some other means to test the different cases. These should be regression tests also that are manually verified in the PR that implements them.
Additional context
Looking at exist status tests and those that use FakeScheduler should help with implementing these tests.
The text was updated successfully, but these errors were encountered:
Description
We have status tests now thanks to @Charlottez112, but we still do not test the proper output given submission statuses and eligibility/completion changes. This limits our testing coverage and makes detecting bugs in changes to the status code difficult.
Proposed solution
Add tests using the
FakeScheduler
or some other means to test the different cases. These should be regression tests also that are manually verified in the PR that implements them.Additional context
Looking at exist status tests and those that use
FakeScheduler
should help with implementing these tests.The text was updated successfully, but these errors were encountered: