-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Test mode #1894
Test mode #1894
Conversation
What's up with the failing tests, is it related to this one? |
Ummm.... this is super weird. At one point I had added a check here to check against But, none of that happened. I left the test the way it is. I am going to resubmit the test suite. |
Ignore the ridiculous number of commits. I was going too fast. |
Ready for review and merge. |
Codecov Report
@@ Coverage Diff @@
## master #1894 +/- ##
==========================================
+ Coverage 92.13% 92.16% +0.03%
==========================================
Files 27 27
Lines 3076 3089 +13
Branches 552 552
==========================================
+ Hits 2834 2847 +13
- Misses 168 169 +1
+ Partials 74 73 -1
Continue to review full report at Codecov.
|
One thing that I plan to add to the
sanic-test
module is the ability for Sanic app to be aware of whether or not it is being tested. This adds a variable to the Sanic app that is set by the internal test clients.