Yelp / Testify

A more pythonic testing framework.
Other
306 stars 67 forks source link

testsuite failures in most versions #283

Closed idella closed 9 years ago

idella commented 10 years ago

It appears it wants catbox installed for running the tetsuite yet I see no mention of it as a dep in the source. Exception: Violation collection pipeline tests require catbox. https://github.com/Yelp/catbox/wiki/Install-Catbox-with-PCRE-enabled

I get a whole lot of

ERROR    Could not remove broken import file /mnt/gen2/TmpDir/portage/dev-python/testify-0.3.6/work/testify-0.3.6/test/fake_broken_importynJXxe.pyc: OSError(2, 'No such file or directory')
.ERROR    Could not remove broken import file /mnt/gen2/TmpDir/portage/dev-python/testify-0.3.6/work/testify-0.3.6/test/fake_broken_importF_LPkF.pyc: OSError(2, 'No such file or directory')
.ERROR    test error message 2

No idea where these fake_broken_import..... files come from. Not in the source repo.

Then a whole lot of

E.........................................................ERROR    Could not remove broken import file /mnt/gen2/TmpDir/portage/dev-python/testify-0.3.6/work/testify-0.3.6/test/fake_broken_importDLsT2s.pyc: OSError(2, 'No such file or directory')
.........error: test.test_runner_server_test TestCaseFailureLimitClassTeardownErrorTestCase.test_methods_are_not_run_after_failure_limit_reached
Traceback (most recent call last):
  File "./test/test_runner_server_test.py", line 113, in teardown_server
    self.stop_server()
  File "./test/test_runner_server_test.py", line 100, in stop_server
    self.server.shutdown()
  File "/mnt/gen2/TmpDir/portage/dev-python/testify-0.3.6/work/testify-0.3.6-python2_7/lib/testify/test_runner_server.py", line 476, in shutdown
    if iol.running():
AttributeError: 'EPollIOLoop' object has no attribute 'running'

then it hangs

EERROR Could not remove broken import file /mnt/gen2/TmpDir/portage/dev-python/testify-0.3.6/work/testify-0.3.6/test/fake_broken_importZoumnL.pyc: OSError(2, 'No such file or directory') ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ERROR No client activity for 10s, shutting down. ^X^Z [13]+ Stopped ebuild testify-0.3.6.ebuild clean test

asottile commented 9 years ago

For what it's worth, 0.3.6 is pretty old and we've probably (?) fixed this problem in a newer version. Could you try with 0.5.6 and report back?

idella commented 9 years ago

Had no idea it was so old. Our app for listing new versions @ gentoo is imperfect. Some packages like this it lists one bump further than the last known, others it goes to the latest. It will be like starting from scratch I suspect. Will do it tomorrow

idella commented 9 years ago

Tornado<2.0.0 from work/testify-0.5.6/testify.egg-info/requires.txt. You have completely lost me. The addition of tornado version -2.0 was (28 Jun 2011) and you want us to use a tornado < 2.0. tornado-1.2 was removed 28 Jun 2011. tornado < 2.0. is no longer in portage since then.

asottile commented 9 years ago

@idella fwiw I've created a PR that allows Testify to run on newer tornado: https://github.com/Yelp/Testify/pull/294

bukzor commented 9 years ago

In truth, testify doesn't use tornado in many circumstances. The same holds for mysql. We should put these into an optional dependency imo.

On Mon, Oct 27, 2014 at 9:24 AM, Anthony Sottile notifications@github.com wrote:

@idella https://github.com/idella fwiw I've created a PR that allows Testify to run on newer tornado: #294 https://github.com/Yelp/Testify/pull/294

— Reply to this email directly or view it on GitHub https://github.com/Yelp/Testify/issues/283#issuecomment-60621922.

asottile commented 9 years ago

@idella testify 0.5.7 contains the fix for tornado. Closing issue, feel free to reopen.