-
Notifications
You must be signed in to change notification settings - Fork 13
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
simplify controller test cases #170
Conversation
maskarb
commented
Mar 7, 2023
•
edited
Loading
edited
- update the controller test suite so the result of one test does not affect the result of another
- each test can now be run individually
- tests should be more robust to future code changes
- should be easier to add new tests in the future
Codecov Report
@@ Coverage Diff @@
## main #170 +/- ##
==========================================
- Coverage 88.78% 88.27% -0.52%
==========================================
Files 11 11
Lines 2328 2328
==========================================
- Hits 2067 2055 -12
- Misses 186 196 +10
- Partials 75 77 +2
Flags with carried forward coverage won't be shown. Click here to find out more. see 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|