Skip to content

Commit

Permalink
test_exit: Drop bad test_noop
Browse files Browse the repository at this point in the history
This test races with other tests because Exiter doesn't have a great way to
remove all installed handlers. This is a test-only issue, so we can drop this
test.

Signed-off-by: Zack Cerza <[email protected]>
  • Loading branch information
zmc committed Jan 31, 2024
1 parent 2a87d27 commit 61552e5
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions teuthology/test/test_exit.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
import os
import random
import signal

from unittest.mock import patch, Mock

from teuthology import exit
from teuthology.test import skipif_teuthology_process


class TestExiter(object):
Expand Down Expand Up @@ -40,13 +38,6 @@ def teardown_method(self):
self.patcher_kill.stop()
del self.m_kill

@skipif_teuthology_process
def test_noop(self):
sig = 15
obj = self.klass()
assert len(obj.handlers) == 0
assert signal.getsignal(sig) == 0

def test_basic(self):
sig = 15
obj = self.klass()
Expand Down

0 comments on commit 61552e5

Please sign in to comment.