-
Notifications
You must be signed in to change notification settings - Fork 18
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
Rednose and processes do not work togeather #14
Comments
I'll try to reproduce this later this week using your code and dig into it! Thanks for the bug report, what an odd bug. |
hmmm it does seem like it works with everything but that library, the test output isn't even colored with |
alone breaks it. |
I will look into it. |
for example,
|
I started taking a look and this is non-trivial so I'm not sure I'll have time to do a crazy deep dive on it, it's an artifact of a really awful plugin api for nose. Colorizing the output shouldn't affect this at all, but it does, in ways I don't entirely understand just yet. |
I can comment all of rednose, except for def prepareTestResult(self, result): # noqa
"""Required to prevent others from monkey patching the add methods."""
return result and it still breaks the multiprocess package but I can't see why that is. while this works def prepareTestResult(self, result): # noqa
"""Required to prevent others from monkey patching the add methods."""
return None |
Fixed in here: #22 |
I'm not sure it actually works, I think it might just be you cannot use them together and it's running only one of the two now. |
Hi, I've run into a bizarre issue with rednose, python 3.5.2, Ubuntu 16.04.
Problem: when multiple inheritance is used rednose discards the failure. It depends on the order of defining test classes/methods.
The code to reproduce the problem:
$ cat test_rednose.py
$ cat nosetests.cfg
Run it:
Now, comment out rednose in nosetests.cfg and run again:
Voila, finally assertTrue(False) fails as it should.
This issue doesn't seem to be related to "--processes" in nosetests.
What is really bizarre is that:
a) if we just remove Test_0::test_dummy method (leaving Test_0 class empty), the assertion will properly fail.
b) The test method in Test_1 class is actually executed up to that assertion point.
I didn't debug rednose itself.
$ python -V
Python 3.5.2
$ uname -rmpov
4.4.0-62-generic #83-Ubuntu SMP Wed Jan 18 14:10:15 UTC 2017 x86_64 x86_64 GNU/Linux
The text was updated successfully, but these errors were encountered: