test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case
@ 2020-08-27 14:13 Nannan Lu
  2020-09-02  3:14 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Nannan Lu @ 2020-08-27 14:13 UTC (permalink / raw)
  To: dts; +Cc: Nannan Lu

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(-)

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)
-- 
2.7.4


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dts] [PATCH V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case
  2020-08-27 14:13 [dts] [PATCH V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case Nannan Lu
@ 2020-09-02  3:14 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2020-09-02  3:14 UTC (permalink / raw)
  To: Lu, Nannan, dts; +Cc: Lu, Nannan

> 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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-09-02  3:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-27 14:13 [dts] [PATCH V1] tests/TestSuite_cvl_switch_filter.py: add validate check for unsupported_pattern_in_OS_default_package case Nannan Lu
2020-09-02  3:14 ` Tu, Lijuan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).