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

The order of test matters. I think this is a bug #157

Open
jsn1993 opened this issue Aug 25, 2016 · 0 comments
Open

The order of test matters. I think this is a bug #157

jsn1993 opened this issue Aug 25, 2016 · 0 comments

Comments

@jsn1993
Copy link
Contributor

jsn1993 commented Aug 25, 2016

When I use py.test -n 2 option I found that there is some test case failing on travis CI but previously not locally. I reproduced it locally by swapping the order of some tests.

Basically if we swap those two tests, it will fail because these two share the same hash value.
https://github.com/cornell-brg/pymtl/blob/master/pymtl/tools/simulation/SimulationTool_struct_test.py#L784
and
https://github.com/cornell-brg/pymtl/blob/master/pymtl/tools/simulation/SimulationTool_struct_test.py#L793

If it's not a bug which means these two have to be tested in a certain order, then xdist -n 2 option may not be applicable to this case.

@jsn1993 jsn1993 changed the title The order of test matters if this is correct. The order of test matters. I think this is a bug Aug 25, 2016
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

No branches or pull requests

1 participant