PYTHON-5371 - Skip test_continuous_network_errors when debug logs are… #2329
Conversation
test/asynchronous/test_client.py
Outdated
| @@ -1906,6 +1906,7 @@ async def test_direct_connection(self): | |||
| with self.assertRaises(ConfigurationError): | |||
| AsyncMongoClient(["host1", "host2"], directConnection=True) | |||
|
|
|||
| @unittest.skipIf(os.environ.get("DEBUG_LOG"), "Enabling debug logs breaks this test") | |||
There was a problem hiding this comment.
Why does this test break? I’m concerned about this pattern since it makes it harder to debug the test if it gets skipped. Could we leave a ticket open to come back to this test and make it work in debug mode?
There was a problem hiding this comment.
We were planning to replace this test with something less hacky in the future anyway. I can investigate deeper as to why this is breaking if we're planning to keep this test around instead.
|
Patch build with debug logs enabled without any failures of test_continuous_network_errors: https://spruce.mongodb.com/version/6814e3d321f66a0007561099/tasks?page=0&sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC&statuses=failed&taskName=%5E%28%28%3F%21pypy%29.%29%2A%24&variant=%5E%28%28%3F%21pypy%29.%29%2A%24 The fix was to pass |
… enabled