From 7360ecf9c5bd593834c45b1ad60bf0b6ac30d05e Mon Sep 17 00:00:00 2001 From: Eric Biagiotti Date: Thu, 17 Jan 2019 10:04:03 -0500 Subject: [PATCH] 13306: Removes a log handler added during a test, breaking subsequent tests. Arvados-DCO-1.1-Signed-off-by: Eric Biagiotti --- sdk/cwl/tests/test_submit.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/sdk/cwl/tests/test_submit.py b/sdk/cwl/tests/test_submit.py index 7f1688c756..a1ace717bb 100644 --- a/sdk/cwl/tests/test_submit.py +++ b/sdk/cwl/tests/test_submit.py @@ -1609,8 +1609,9 @@ class TestCreateWorkflow(unittest.TestCase): @stubs def test_incompatible_api(self, stubs): capture_stderr = io.StringIO() - logging.getLogger('arvados.cwl-runner').addHandler( - logging.StreamHandler(capture_stderr)) + acr_logger = logging.getLogger('arvados.cwl-runner') + stderr_logger = logging.StreamHandler(capture_stderr) + acr_logger.addHandler(stderr_logger) exited = arvados_cwl.main( ["--update-workflow", self.existing_workflow_uuid, @@ -1622,6 +1623,7 @@ class TestCreateWorkflow(unittest.TestCase): self.assertRegexpMatches( capture_stderr.getvalue(), "--update-workflow arg '{}' uses 'containers' API, but --api='jobs' specified".format(self.existing_workflow_uuid)) + acr_logger.removeHandler(stderr_logger) @stubs def test_update(self, stubs): -- 2.30.2