diff mbox series

[V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case

Message ID 1598537597-771420-1-git-send-email-nannan.lu@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case |

Commit Message

Lu, Nannan Aug. 27, 2020, 2:13 p.m. UTC 
  tests/TestSuite_cvl_switch_filter.py: add validate check for
unsupported_pattern_in_OS_default_package case

Signed-off-by: Nannan Lu <nannan.lu@intel.com>
---
 tests/TestSuite_cvl_switch_filter.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Tu, Lijuan Sept. 2, 2020, 3:14 a.m. UTC | #1 
> Subject: [dts] [PATCH V1] tests/TestSuite_cvl_switch_filter.py: add validate
> check for unsupported_pattern_in_OS_default_package case
> 
> tests/TestSuite_cvl_switch_filter.py: add validate check for
> unsupported_pattern_in_OS_default_package case
> 
> Signed-off-by: Nannan Lu <nannan.lu@intel.com>

Applied
diff mbox series

Patch

diff --git a/tests/TestSuite_cvl_switch_filter.py b/tests/TestSuite_cvl_switch_filter.py
index 339dae6..27815d5 100644
--- a/tests/TestSuite_cvl_switch_filter.py
+++ b/tests/TestSuite_cvl_switch_filter.py
@@ -5588,10 +5588,10 @@  class CVLSwitchFilterTest(TestCase):
         self.bind_nics_driver(self.dut_ports, driver="vfio-pci")
         self.launch_testpmd(False)
 
-        # rule_list = self.validate_switch_filter_rule(rule, check_stats=False)
-        # self.verify(not any(rule_list), "all rules should validate failed, result %s" % rule_list)
-        # result = self.check_switch_filter_rule_list(self.dut_ports[0], is_non_pipeline=False, is_need_rss_rule=False, check_stats=False)
-        # self.verify(result == [], "the rule list is not the same. expect %s, result %s" % ([], result))
+        rule_list = self.validate_switch_filter_rule(rule, check_stats=False)
+        self.verify(not any(rule_list), "all rules should validate failed, result %s" % rule_list)
+        result = self.check_switch_filter_rule_list(self.dut_ports[0], is_non_pipeline=False, is_need_rss_rule=False, check_stats=False)
+        self.verify(result == [], "the rule list is not the same. expect %s, result %s" % ([], result))
         rule_list = self.create_switch_filter_rule(rule, check_stats=False)
         self.verify(not any(rule_list), "all rules should create failed, result %s" % rule_list)
         result = self.check_switch_filter_rule_list(self.dut_ports[0], is_non_pipeline=False, is_need_rss_rule=False, check_stats=False)