Stabilize the copp test case (sonic-net#20595)

The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Guy Shemesh <[email protected]>

Author: nhe-NV

tests/copp/copp_utils.py

@@ -540,6 +540,18 @@ def is_trap_installed(duthost, trap_id):
     return output[trap_id]["hw_status"] == "installed"
 
 
+def is_trap_uninstalled(duthost, trap_id):
+    """
+    Checks if a specific trap is uninstalled by parsing the output of `show copp configuration`.
+    Args:
+        dut (SonicHost): The target device
+        trap_id: The trap ID to check.
+    Returns:
+        bool: True if the trap is uninstalled, False otherwise.
+    """
+    return not is_trap_installed(duthost, trap_id)
+
+
 def get_trap_hw_status(duthost):
     """
     Retrieves the hw_status for traps from the STATE_DB.

tests/copp/test_copp.py

@@ -240,9 +240,8 @@ def test_remove_trap(self, duthosts, enum_rand_one_per_hwsku_frontend_hostname,
             copp_utils.disable_feature_entry(duthost, self.feature_name)
 
         logging.info("Verify {} trap is uninstalled through CLI".format(self.trap_id))
-        pytest_assert(not copp_utils.is_trap_installed(duthost, self.trap_id),
+        pytest_assert(wait_until(30, 2, 0, copp_utils.is_trap_uninstalled, duthost, self.trap_id),
                       "Trap {} is not uninstalled".format(self.trap_id))
-
         logger.info("Verify {} trap status is uninstalled by sending traffic".format(self.trap_id))
         pytest_assert(
             wait_until(100, 20, 0, _copp_runner, duthost, ptfhost, self.trap_id.upper(),