tests: skip tests that require HOSTNAME != 'localhost' (#501)

Inspired by arkamar/gentoo@1d69b11
cea-hpc · Nov 26, 2022 · aae7bc7 · aae7bc7
1 parent e341732
commit aae7bc7
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 0 deletions.
diff --git a/tests/CLIClushTest.py b/tests/CLIClushTest.py
@@ -215,12 +215,14 @@ def test_009_file_copy_tty(self):
         finally:
             delattr(ClusterShell.CLI.Clush, '_f_user_interaction')
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_010_diff(self):
         """test clush (diff)"""
         self._clush_t(["-w", HOSTNAME, "--diff", "echo", "ok"], None, b"")
         self._clush_t(["-w", "%s,localhost" % HOSTNAME, "--diff", "echo",
                        "ok"], None, b"")
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_011_diff_tty(self):
         """test clush (diff) [tty]"""
         setattr(ClusterShell.CLI.Clush, '_f_user_interaction', True)
@@ -229,6 +231,7 @@ def test_011_diff_tty(self):
         finally:
             delattr(ClusterShell.CLI.Clush, '_f_user_interaction')
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_012_diff_null(self):
         """test clush (diff w/o output)"""
         rxs = r"^--- %s\n\+\+\+ localhost\n@@ -1(,1)? \+[01],0 @@\n-ok\n$" % HOSTNAME
@@ -268,6 +271,7 @@ def test_016_stderr_tty(self):
         finally:
             delattr(ClusterShell.CLI.Clush, '_f_user_interaction')
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_017_retcodes(self):
         """test clush (retcodes)"""
         s = "clush: %s: exited with exit code 1\n" % HOSTNAME
@@ -301,6 +305,7 @@ def test_017_retcodes(self):
                       s.encode())
         self._clush_t(["-w", duo, "-S", "-b", "-q", "/bin/false"], None, b"", 1)
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_018_retcodes_tty(self):
         """test clush (retcodes) [tty]"""
         setattr(ClusterShell.CLI.Clush, '_f_user_interaction', True)
@@ -468,6 +473,7 @@ def test_030_config_options(self):
                        "color=never", "-w", HOSTNAME, "echo", "ok"], None,
                       self.output_ok)
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_031_progress(self):
         """test clush -P/--progress"""
         self._clush_t(["-w", HOSTNAME, "--progress", "echo", "ok"], None,
@@ -518,6 +524,7 @@ def test_033_worker_pdsh_tty(self):
         finally:
             delattr(ClusterShell.CLI.Clush, '_f_user_interaction')
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_034_pick(self):
         """test clush --pick"""
         rxs = r"^(localhost|%s): foo\n$" % HOSTNAME

diff --git a/tests/TaskDistantMixin.py b/tests/TaskDistantMixin.py
@@ -5,6 +5,7 @@
 
 import pwd
 import shutil
+import unittest
 import warnings
 
 from TLib import HOSTNAME, make_temp_filename, make_temp_dir
@@ -722,6 +723,7 @@ def ev_hup(self, worker, node, rc):
         def ev_close(self, worker, timedout):
             self.close_count += 1
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def testWorkerEventCount(self):
         test_eh = self.__class__.TEventHandlerEvCountChecker()
         nodes = "localhost,%s" % HOSTNAME

diff --git a/tests/TaskDistantPdshMixin.py b/tests/TaskDistantPdshMixin.py
@@ -13,6 +13,7 @@
 from ClusterShell.Worker.EngineClient import *
 
 import socket
+import unittest
 
 # TEventHandlerChecker 'received event' flags
 EV_START = 0x01
@@ -507,6 +508,7 @@ def ev_hup(self, worker, node, rc):
         def ev_close(self, worker, timedout):
             self.close_count += 1
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def testWorkerEventCount(self):
         test_eh = self.__class__.TEventHandlerEvCountChecker()
         nodes = "localhost,%s" % HOSTNAME

diff --git a/tests/TreeWorkerTest.py b/tests/TreeWorkerTest.py
@@ -100,6 +100,7 @@ def ev_close(self, worker, timedout):
             self.ev_timedout_cnt += 1
 
 
+@unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
 class TreeWorkerTest(unittest.TestCase):
     """
     TreeWorkerTest: test TreeWorker

diff --git a/tests/WorkerExecTest.py b/tests/WorkerExecTest.py
@@ -45,6 +45,7 @@ def test_one_node_error(self):
         self.assertEqual(task_self().max_retcode(), 1)
         self.assertEqual(task_self().node_buffer('localhost'), b'')
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_timeout(self):
         """test ExecWorker with a timeout"""
         nodes = "localhost,%s" % HOSTNAME
@@ -67,6 +68,7 @@ def test_bad_placeholder(self):
         self.assertRaises(WorkerError, self.execw,
                           nodes="localhost", handler=None, command="echo %")
 
+    @unittest.skipIf(HOSTNAME == 'localhost', "does not work with hostname set to 'localhost'")
     def test_rank_placeholder(self):
         """test ExecWorker with several nodes and %n (rank)"""
         nodes = "localhost,%s" % HOSTNAME