Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

first hypotests-python tests #35

Merged
merged 3 commits into from
Jul 16, 2018
Merged

first hypotests-python tests #35

merged 3 commits into from
Jul 16, 2018

Conversation

RSabet
Copy link
Collaborator

@RSabet RSabet commented Jul 16, 2018

created some hypothesis-python tests as described in #30

@RSabet RSabet requested a review from mre July 16, 2018 21:37
Copy link
Owner

@mre mre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!
Should give us more confidence in the implementation and reveal a few edge-cases. Is the hypothesis test already activated? Can't see it in the Travis build...

@RSabet
Copy link
Collaborator Author

RSabet commented Jul 16, 2018

yes, it is activated, since the filename and functions start with test they are discovered by pytest:

tests/test_hypothesis.py::test_floats PASSED
tests/test_hypothesis.py::test_text PASSED
tests/test_hypothesis.py::test_bool PASSED
tests/test_hypothesis.py::test_none PASSED
tests/test_hypothesis.py::test_list_integers PASSED
tests/test_hypothesis.py::test_list_floats PASSED
tests/test_hypothesis.py::test_list_text PASSED
tests/test_hypothesis.py::test_list_mixed PASSED
tests/test_hypothesis.py::test_json_obj PASSED

@mre
Copy link
Owner

mre commented Jul 16, 2018

Ah very nice. Sorry for overlooking that. :shipit:

@mre mre merged commit dd96e7e into mre:master Jul 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants