zuul/zuul
Code Issues Proposed changes

Fix stack_dump_handler test

Browse Source 
If the test suite is run in an environment with yappi or objgraph
installed, the stack_dump_handler test will activate them, and so
we need to call the handler a second time to turn them off.
Otherwise, test performance or errors can occur.

Change-Id: If073f0e46b24fc4e9f1281f911ce287f5c23d4dd
This commit is contained in:
James E. Blair 2024-03-01 14:25:39 -08:00
parent 3d30928d39
commit d1a5291882
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tests/unit/test_stack_dump.py
View File

@ -33,3 +33,6 @@ class TestStackDump(testtools.TestCase):
self.assertIn("Thread", self.log_fixture.output)
self.assertIn("MainThread", self.log_fixture.output)
self.assertIn("test_stack_dump_logs", self.log_fixture.output)
# Call it again in case yappi or objgraph are installed in the
# venv so that we turn them back off.
zuul.cmd.stack_dump_handler(signal.SIGUSR2, None)