Skip to content
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

Convert tests to use pytest #54

Open
devos50 opened this issue Nov 8, 2020 · 4 comments
Open

Convert tests to use pytest #54

devos50 opened this issue Nov 8, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@devos50
Copy link
Contributor

devos50 commented Nov 8, 2020

We should convert the tests to use pytest, instead of nose.

@devos50 devos50 added the enhancement New feature or request label Nov 8, 2020
@devos50
Copy link
Contributor Author

devos50 commented Feb 23, 2021

@Solomon1732 this is something that's quite high on my wish list. If you're interested in learning more about pytest, you might want to start out with converting one or more tests (e.g., test_match_queue.py), using fixtures to reduce code duplication and to increase readability 👍

@Solomon1732
Copy link
Contributor

I'll do it after the current PR 👍

@devos50
Copy link
Contributor Author

devos50 commented Apr 12, 2021

I converted a few tests in #69

@dannysepler
Copy link

hi! i wrote a pip package to help out with this sort of refactor (https://github.com/dannysepler/pytestify) and i'm just snooping around github looking for people to test it out 🙃 -- totally cool if it doesn't end up working for you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants