Skip to content

Fix broken test suite #68

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

Merged
merged 6 commits into from
Jan 13, 2017
Merged

Fix broken test suite #68

merged 6 commits into from
Jan 13, 2017

Conversation

thomasw
Copy link
Contributor

@thomasw thomasw commented Jan 5, 2017

The test suite would not run under python 3.5 due to pytest incompatibilities. Additionally, the travis config for this project no longer appears to be valid which this pull also corrects.

Once I had the test suite running, I discovered a bad test which was failing. I've corrected that test so that it verifies the expected behavior and passes.

The 3.x branch of pytest drops support for <3.3 which, according, to the travis file, this project still supports. As a consequence of that, we can only go up to pytest 2.9.2.
This project is tested in that environment.
test_ResourceNotFound_connect was checking for a ResourceNotFoundError on an operation that should actually succeed. This modifies that operation so that it will fail as expected.
@zallarak
Copy link

Thanks so much for doing this @thomasw - will merge now!

@zallarak zallarak merged commit 4d6508f into plaid:master Jan 13, 2017
@thomasw
Copy link
Contributor Author

thomasw commented Jan 13, 2017

❤️ 🎆

@thomasw thomasw deleted the fix_tests branch January 13, 2017 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants