diff options
author | Andrew Yourtchenko <ayourtch@gmail.com> | 2024-07-02 18:02:09 +0200 |
---|---|---|
committer | Dave Wallace <dwallacelf@gmail.com> | 2024-07-10 01:16:10 +0000 |
commit | a095e37d2a39c1194111938ecb862838f224c172 (patch) | |
tree | ebad09591be697879b2ea8af6e09e2400f1fce9a | |
parent | 6e8b350a0154caee6e9f037ae4938b83fb5dcf20 (diff) |
tests: disable failing tests on Ubuntu 22.04
Also rework the logic so the skipping of marked Ubuntu 22.04 occurs at framework level
Leave debian11 special cases as-is.
Type: fix
Change-Id: I481eb32cd1a0860935482e9f930ced409da653c9
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
(cherry picked from commit 9987d470a6a502dd64266204cb006f30d7245cdd)
-rw-r--r-- | test/asf/asfframework.py | 13 | ||||
-rw-r--r-- | test/test_flowprobe.py | 4 | ||||
-rw-r--r-- | test/test_ikev2.py | 10 | ||||
-rw-r--r-- | test/test_nat44_ed.py | 2 | ||||
-rw-r--r-- | test/test_nat64.py | 2 |
5 files changed, 21 insertions, 10 deletions
diff --git a/test/asf/asfframework.py b/test/asf/asfframework.py index 24880044cec..4cd4d94ab7b 100644 --- a/test/asf/asfframework.py +++ b/test/asf/asfframework.py @@ -317,6 +317,12 @@ class VppAsfTestCase(CPUInterface, unittest.TestCase): cls = unittest.skip("Skipping @tag_fixme_asan tests")(cls) @classmethod + def skip_fixme_ubuntu2204(cls): + """if @tag_fixme_ubuntu2204 & is Ubuntu22.04 - mark for skip""" + if cls.has_tag(TestCaseTag.FIXME_UBUNTU2204) and is_distro_ubuntu2204 == True: + cls = unittest.skip("Skipping @tag_fixme_ubuntu2204 tests")(cls) + + @classmethod def instance(cls): """Return the instance of this testcase""" return cls.test_instance @@ -1361,6 +1367,13 @@ class VppTestResult(unittest.TestResult): test_title = colorize(f"FIXME with ASAN: {test_title}", RED) test.skip_fixme_asan() + if ( + test.has_tag(TestCaseTag.FIXME_UBUNTU2204) + and is_distro_ubuntu2204 == True + ): + test_title = colorize(f"FIXME with Ubuntu 22.04: {test_title}", RED) + test.skip_fixme_ubuntu2204() + if hasattr(test, "vpp_worker_count"): if test.vpp_worker_count == 0: test_title += " [main thread only]" diff --git a/test/test_flowprobe.py b/test/test_flowprobe.py index 8e3fecfd7b4..89ac97ea88c 100644 --- a/test/test_flowprobe.py +++ b/test/test_flowprobe.py @@ -183,9 +183,7 @@ class MethodHolder(VppTestCase): variables and configure VPP. """ super(MethodHolder, cls).setUpClass() - if (is_distro_ubuntu2204 == True or is_distro_debian11 == True) and not hasattr( - cls, "vpp" - ): + if (is_distro_debian11 == True) and not hasattr(cls, "vpp"): return try: # Create pg interfaces diff --git a/test/test_ikev2.py b/test/test_ikev2.py index fd065b47c98..ea425e2e489 100644 --- a/test/test_ikev2.py +++ b/test/test_ikev2.py @@ -2036,6 +2036,7 @@ class TestResponderBehindNAT(TemplateResponder, Ikev2Params): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorNATT(TemplateInitiator, Ikev2Params): """test ikev2 initiator - NAT traversal (intitiator behind NAT)""" @@ -2068,6 +2069,7 @@ class TestInitiatorNATT(TemplateInitiator, Ikev2Params): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorPsk(TemplateInitiator, Ikev2Params): """test ikev2 initiator - pre shared key auth""" @@ -2099,6 +2101,7 @@ class TestInitiatorPsk(TemplateInitiator, Ikev2Params): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorRequestWindowSize(TestInitiatorPsk): """test initiator - request window size (1)""" @@ -2148,6 +2151,7 @@ class TestInitiatorRequestWindowSize(TestInitiatorPsk): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorRekey(TestInitiatorPsk): """test ikev2 initiator - rekey""" @@ -2193,6 +2197,7 @@ class TestInitiatorRekey(TestInitiatorPsk): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorDelSAFromResponder(TemplateInitiator, Ikev2Params): """test ikev2 initiator - delete IKE SA from responder""" @@ -2413,9 +2418,7 @@ class TestResponderVrf(TestResponderPsk, Ikev2Params): globals()["ikev2"] = _ikev2 super(IkePeer, cls).setUpClass() - if (is_distro_ubuntu2204 == True or is_distro_debian11 == True) and not hasattr( - cls, "vpp" - ): + if (is_distro_debian11 == True) and not hasattr(cls, "vpp"): return cls.create_pg_interfaces(range(1)) cls.vapi.cli("ip table add 1") @@ -2525,6 +2528,7 @@ class Test_IKE_AES_GCM_16_256(TemplateResponder, Ikev2Params): @tag_fixme_vpp_workers +@tag_fixme_ubuntu2204 class TestInitiatorKeepaliveMsg(TestInitiatorPsk): """ Test for keep alive messages diff --git a/test/test_nat44_ed.py b/test/test_nat44_ed.py index eed89f1a399..d3d6d07457b 100644 --- a/test/test_nat44_ed.py +++ b/test/test_nat44_ed.py @@ -162,8 +162,6 @@ class TestNAT44ED(VppTestCase): @classmethod def setUpClass(cls): super().setUpClass() - if is_distro_ubuntu2204 == True and not hasattr(cls, "vpp"): - return cls.create_pg_interfaces(range(12)) cls.interfaces = list(cls.pg_interfaces[:4]) diff --git a/test/test_nat64.py b/test/test_nat64.py index f650b8d5f43..7333181e7f0 100644 --- a/test/test_nat64.py +++ b/test/test_nat64.py @@ -55,8 +55,6 @@ class TestNAT64(VppTestCase): def setUpClass(cls): super(TestNAT64, cls).setUpClass() - if is_distro_ubuntu2204 == True and not hasattr(cls, "vpp"): - return cls.tcp_port_in = 6303 cls.tcp_port_out = 6303 cls.udp_port_in = 6304 |