From: Zack Cerza <zack@redhat.com>
Date: Wed, 31 Jan 2024 01:56:19 +0000 (-0700)
Subject: test_exit: Drop bad test_noop
X-Git-Tag: 1.2.0~56^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=61552e565d59b8eef3c55fbd10b9d8bb7ed0b934;p=teuthology.git

test_exit: Drop bad test_noop

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 <zack@redhat.com>
---

diff --git a/teuthology/test/test_exit.py b/teuthology/test/test_exit.py
index 11125b937..e8c22bf20 100644
--- a/teuthology/test/test_exit.py
+++ b/teuthology/test/test_exit.py
@@ -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):
@@ -40,13 +38,6 @@ class TestExiter(object):
         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()