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

Refactor tests to reduce repetition and use py.test over nose. #834

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NDevox
Copy link

@NDevox NDevox commented Oct 20, 2019

Nose has been deprecated for several years whereas py.test is still
under active development. As a result it is more accepted nowadays to
use py.test as the test runner.

This refactor also reduces much of the repetition in the tests and moves
to the functional style of testing favoured by py.test.

Nose has been deprecated for several years whereas py.test is still
under active development. As a result it is more accepted nowadays to
use py.test as the test runner.

This refactor also reduces much of the repetition in the tests and moves
to the functional style of testing favoured by py.test.
@NDevox
Copy link
Author

NDevox commented Oct 21, 2019

Ok I'll look at upping the coverage a bit today 😇

@NDevox
Copy link
Author

NDevox commented Oct 21, 2019

@joshfriend what is the crypto lib used? I have tests for that module but don't know what lib is being used.

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.

None yet

1 participant