-
Notifications
You must be signed in to change notification settings - Fork 26
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
provide various missing unit tests #1474
Conversation
Provide tests for hideEmail, loginServices, okapiActions, okapiReducer, and processBadResponse. Tiny refactor of some loginServices functions to: * consistently test for success, else handle an error * consistently test for success in responses by checking `response.ok` rather than `response.status`. consistency is nice. it's predictable.
BigTest Unit Test Statistics 1 files ±0 1 suites ±0 10s ⏱️ ±0s Results for commit a3a63d3. ± Comparison against base commit 142ff27. This pull request removes 5 and adds 3 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Bigtest Unit Test Results193 tests ±0 188 ✅ ±0 6s ⏱️ ±0s Results for commit a9594c6. ± Comparison against base commit e93a5af. This pull request removes 5 and adds 3 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Quality Gate passedIssues Measures |
Provide tests for hideEmail, loginServices, okapiActions, okapiReducer, and processBadResponse.
Tiny refactor of some loginServices functions to:
response.ok
rather thanresponse.status
. consistency is nice. it's predictable.