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

Intermittent failure from test_omitLoggerFromActionType in 1.7.0 #436

Open
exarkun opened this issue Oct 21, 2019 · 6 comments
Open

Intermittent failure from test_omitLoggerFromActionType in 1.7.0 #436

exarkun opened this issue Oct 21, 2019 · 6 comments
Labels

Comments

@exarkun
Copy link
Collaborator

exarkun commented Oct 21, 2019

Seen on my CI:

FAIL: test_omitLoggerFromActionType (eliot.tests.test_validation.EndToEndValidationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/nix-build-python2.7-eliot-1.7.0.drv-0/eliot-1.7.0/eliot/tests/test_validation.py", line 895, in test_omitLoggerFromActionType
    self.assertEqual(messages[0]["key"], 123)
AssertionError: 5 != 123

This is a somewhat older version so perhaps the problem has been fixed already. I couldn't find any tickets that seemed related, though.

@itamarst
Copy link
Owner

Hm. Will take a look.

@itamarst itamarst added the bug label Oct 22, 2019
@itamarst
Copy link
Owner

Are you running that on Python 3 too? If so, does it fail there?

@exarkun
Copy link
Collaborator Author

exarkun commented Nov 21, 2019

Alas, I'm only running on Python 2 on CI.

@itamarst
Copy link
Owner

itamarst commented Nov 21, 2019

Since I've never seen that on recent runs (Python 3.5+), I hypothesize it's something about dictionary order stability, but would need to read actual code to be sure.

@exarkun
Copy link
Collaborator Author

exarkun commented Feb 11, 2020

It looks like another test fails similarly:

======================================================================
FAIL: test_logCallsDefaultLoggerWrite (eliot.tests.test_validation.MessageTypeTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/nix-build-python2.7-eliot-1.7.0.drv-0/eliot-1.7.0/eliot/tests/test_validation.py", line 533, in test_logCallsDefaultLoggerWrite
    self.assertEqual(messages[0][u"key"], 1234)
AssertionError: 5 != 1234

@exarkun
Copy link
Collaborator Author

exarkun commented Apr 15, 2020

Here's another one :/

ERROR: test_global_cleanup (eliot.tests.test_testing.CaptureLoggingTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/nix-build-python2.7-eliot-1.7.0.drv-0/eliot-1.7.0/eliot/tests/test_testing.py", line 662, in test_global_cleanup
    self.assertEqual(messages[0][u"some_key"], 1234)
KeyError: u'some_key'

every time I mark another one as skipped, another pops up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants