tests: remove decorator tag_fixme_ubuntu2204

Type: test

There is no need to use the tag_fixme_ubuntu2204 decorator.
All tests that previously used it are now stable. The neighbor test
has been moved to the extended suite because it is not related to the
type of operating system.

Change-Id: Ifa0a30cbeeeaabb2bbe4d213fa61e6fbde8789ca
Signed-off-by: Ivan Ivanets <iivanets@cisco.com>
This commit is contained in:
Ivan Ivanets 2024-12-02 20:38:42 +00:00 committed by Dave Wallace
parent 63b34c8d22
commit 19e2c7ac01
5 changed files with 7 additions and 35 deletions

View File

@ -155,17 +155,6 @@ def _is_platform_aarch64():
is_platform_aarch64 = _is_platform_aarch64()
def _is_distro_ubuntu2204():
with open("/etc/os-release") as f:
for line in f.readlines():
if "jammy" in line:
return True
return False
is_distro_ubuntu2204 = _is_distro_ubuntu2204()
def _is_distro_ubuntu2404():
with open("/etc/os-release") as f:
for line in f.readlines():
@ -233,14 +222,12 @@ class TestCaseTag(Enum):
FIXME_VPP_WORKERS = 2
# marks the suites broken when ASan is enabled
FIXME_ASAN = 3
# marks suites broken on Ubuntu-22.04
FIXME_UBUNTU2204 = 4
# marks suites broken on Debian-11
FIXME_DEBIAN11 = 5
FIXME_DEBIAN11 = 4
# marks suites broken on debug vpp image
FIXME_VPP_DEBUG = 6
FIXME_VPP_DEBUG = 5
# marks suites broken on Ubuntu-24.04
FIXME_UBUNTU2404 = 7
FIXME_UBUNTU2404 = 6
def create_tag_decorator(e):
@ -257,7 +244,6 @@ def create_tag_decorator(e):
tag_run_solo = create_tag_decorator(TestCaseTag.RUN_SOLO)
tag_fixme_vpp_workers = create_tag_decorator(TestCaseTag.FIXME_VPP_WORKERS)
tag_fixme_asan = create_tag_decorator(TestCaseTag.FIXME_ASAN)
tag_fixme_ubuntu2204 = create_tag_decorator(TestCaseTag.FIXME_UBUNTU2204)
tag_fixme_debian11 = create_tag_decorator(TestCaseTag.FIXME_DEBIAN11)
tag_fixme_vpp_debug = create_tag_decorator(TestCaseTag.FIXME_VPP_DEBUG)
tag_fixme_ubuntu2404 = create_tag_decorator(TestCaseTag.FIXME_UBUNTU2404)
@ -322,12 +308,6 @@ class VppAsfTestCase(CPUInterface, unittest.TestCase):
if "DVPP_ENABLE_SANITIZE_ADDR=ON" in vpp_extra_cmake_args:
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 skip_fixme_ubuntu2404(cls):
"""if @tag_fixme_ubuntu2404 & is Ubuntu24.04 - mark for skip"""
@ -1381,13 +1361,6 @@ 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 (
test.has_tag(TestCaseTag.FIXME_UBUNTU2404)
and is_distro_ubuntu2404 == True

View File

@ -18,7 +18,6 @@ from asfframework import (
tag_fixme_vpp_workers,
tag_fixme_debian11,
tag_run_solo,
is_distro_ubuntu2204,
is_distro_debian11,
VppTestRunner,
)

View File

@ -7,7 +7,7 @@ from random import randint, choice
import re
import scapy.compat
from framework import VppTestCase, VppLoInterface
from asfframework import VppTestRunner, is_distro_ubuntu2204
from asfframework import VppTestRunner
from scapy.data import IP_PROTOS
from scapy.layers.inet import IP, TCP, UDP, ICMP, GRE
from scapy.layers.inet import IPerror, TCPerror

View File

@ -12,7 +12,6 @@ from config import config
from framework import VppTestCase
from asfframework import (
tag_fixme_vpp_workers,
is_distro_ubuntu2204,
VppTestRunner,
)
from ipfix import IPFIX, Set, Template, Data, IPFIXDecoder

View File

@ -3,8 +3,9 @@
import unittest
import os
from config import config
from framework import VppTestCase
from asfframework import VppTestRunner, tag_fixme_vpp_workers, tag_fixme_ubuntu2204
from asfframework import VppTestRunner, tag_fixme_vpp_workers
from vpp_neighbor import VppNeighbor, find_nbr
from vpp_ip_route import (
VppIpRoute,
@ -2330,7 +2331,7 @@ class NeighborStatsTestCase(VppTestCase):
self.assertEqual(NUM_PKTS + 16, nd1.get_stats()["packets"])
@tag_fixme_ubuntu2204
@unittest.skipUnless(config.extended, "part of extended tests")
class NeighborAgeTestCase(VppTestCase):
"""ARP/ND Aging"""