-
-
Notifications
You must be signed in to change notification settings - Fork 767
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
Make tests framework agnostic #1634
Conversation
30375f5
to
515e324
Compare
I added a new commit which uses the Starlette 62 tests still failing. |
This PR defines a request interface and aligns the `ASGIRequest` with the `WSGIRequest` class. This brings down the test errors from 62 to 25.
This PR fixes 'outside of Flask context' errors in our test suite and contains the following changes: - `Connexion.request` is now a Connexion `ASGIRequest` instead of a Flask `request` - All other tests with a Flask dependency have been updated or split Failing tests are down to 11.
This PR adds all the needed error handling for the AsyncApp. Only 5 errors remaining.
0ebc9d7
to
19bf15e
Compare
Pull Request Test Coverage Report for Build 4192692263
💛 - Coveralls |
Currently, our tests only run against the
FlaskApp
. This PR parametrizes the relevant tests to run against both theAsyncApp
andFlaskApp
.Unfortunately, the test clients provided by Flask and Starlette are not entirely aligned. This PR addresses this by refactoring the tests so they are framework agnostic.
This increased the number of tests from 483 to 709, of which 66 still fail due to issues with the
AsyncApp
. I will submit separate PRs targeting this branch to fix those.