test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in send_expect
@ 2021-06-03  9:33 Yang Leweix
  0 siblings, 0 replies; 3+ messages in thread
From: Yang Leweix @ 2021-06-03  9:33 UTC (permalink / raw)
  To: dts; +Cc: leweix.yang

fixs following suite impacted by commit f498f50a62e30f6f8fb9c4e1a759e3d35861b978:
    * flow_classify_softnic
	* test_ipv4_acl_jump
	* test_ipv4_acl_table
	* test_ipv6_hash_jump
    * unit_tests_loopback
	* test_link_mode
	* test_loopback_mode
    * distributor
	* maximum_workers  

Signed-off-by: Yang Leweix <leweix.yang@intel.com>
---
 tests/TestSuite_distributor.py           | 2 +-
 tests/TestSuite_flow_classify_softnic.py | 2 +-
 tests/TestSuite_unit_tests_loopback.py   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/TestSuite_distributor.py b/tests/TestSuite_distributor.py
index 0f61cc71..cd88977d 100644
--- a/tests/TestSuite_distributor.py
+++ b/tests/TestSuite_distributor.py
@@ -165,7 +165,7 @@ class TestDistributor(TestCase):
         self.verify(len(self.dut_ports) >= 1, "Not enough ports")
 
         cmd_fmt = "%s %s -- -p 0x1"
-        out = self.dut.send_expect("sed -n '/#define RTE_DISTRIB_MAX_WORKERS/p' lib/distributor/distributor_private.h", "# ")
+        out = self.dut.send_expect("sed -n '/#define RTE_DISTRIB_MAX_WORKERS/p' lib/distributor/distributor_private.h", "# ", trim_whitespace=False)
         reg_match = r"#define RTE_DISTRIB_MAX_WORKERS (.*)"
         m = re.match(reg_match, out)
         self.verify(m, "Can't find maximum worker number")
diff --git a/tests/TestSuite_flow_classify_softnic.py b/tests/TestSuite_flow_classify_softnic.py
index db7a81dd..e5937289 100644
--- a/tests/TestSuite_flow_classify_softnic.py
+++ b/tests/TestSuite_flow_classify_softnic.py
@@ -129,7 +129,7 @@ class TestFlowClassifySoftnic(TestCase):
         """
         param = ""
         direct_param = r"(\s+)\[ (\S+) in\|out\|inout \]"
-        out = self.tester.send_expect('tcpdump -h', '# ')
+        out = self.tester.send_expect('tcpdump -h', '# ', trim_whitespace=False)
         for line in out.split('\n'):
             m = re.match(direct_param, line)
             if m:
diff --git a/tests/TestSuite_unit_tests_loopback.py b/tests/TestSuite_unit_tests_loopback.py
index 0198a913..a0467378 100644
--- a/tests/TestSuite_unit_tests_loopback.py
+++ b/tests/TestSuite_unit_tests_loopback.py
@@ -81,7 +81,7 @@ class TestUnitTestsLoopback(TestCase):
         pass
 
     def get_max_traffic_burst(self):
-        pmd_file = self.dut.send_expect("cat app/test/test_pmd_perf.c", "# ", 30)
+        pmd_file = self.dut.send_expect("cat app/test/test_pmd_perf.c", "# ", 30, trim_whitespace=False)
         result_scanner = r"#define MAX_TRAFFIC_BURST\s+([0-9]+)"
         scanner = re.compile(result_scanner, re.DOTALL)
         m = scanner.search(pmd_file)
-- 
2.17.1


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

* Re: [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in send_expect
  2021-06-03  9:21 Yang Leweix
@ 2021-06-09  7:43 ` Tu, Lijuan
  0 siblings, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2021-06-09  7:43 UTC (permalink / raw)
  To: Yang, LeweiX, dts; +Cc: Yang, LeweiX

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Yang Leweix
> Sent: 2021年6月3日 17:22
> To: dts@dpdk.org
> Cc: Yang, LeweiX <leweix.yang@intel.com>
> Subject: [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in
> send_expect
> 
> fixs following suite impacted by commit f498f5:
>     * flow_classify_softnic
> 	* test_ipv4_acl_jump
> 	* test_ipv4_acl_table
> 	* test_ipv6_hash_jump
>     * unit_tests_loopback
> 	* test_link_mode
> 	* test_loopback_mode
>     * distributor
> 	* maximum_workers
> 
> Signed-off-by: Yang Leweix <leweix.yang@intel.com>

Applied, thanks

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

* [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in send_expect
@ 2021-06-03  9:21 Yang Leweix
  2021-06-09  7:43 ` Tu, Lijuan
  0 siblings, 1 reply; 3+ messages in thread
From: Yang Leweix @ 2021-06-03  9:21 UTC (permalink / raw)
  To: dts; +Cc: leweix.yang

fixs following suite impacted by commit f498f5:
    * flow_classify_softnic
	* test_ipv4_acl_jump
	* test_ipv4_acl_table
	* test_ipv6_hash_jump
    * unit_tests_loopback
	* test_link_mode
	* test_loopback_mode
    * distributor
	* maximum_workers  

Signed-off-by: Yang Leweix <leweix.yang@intel.com>
---
 tests/TestSuite_distributor.py           | 2 +-
 tests/TestSuite_flow_classify_softnic.py | 2 +-
 tests/TestSuite_unit_tests_loopback.py   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/TestSuite_distributor.py b/tests/TestSuite_distributor.py
index 0f61cc71..cd88977d 100644
--- a/tests/TestSuite_distributor.py
+++ b/tests/TestSuite_distributor.py
@@ -165,7 +165,7 @@ class TestDistributor(TestCase):
         self.verify(len(self.dut_ports) >= 1, "Not enough ports")
 
         cmd_fmt = "%s %s -- -p 0x1"
-        out = self.dut.send_expect("sed -n '/#define RTE_DISTRIB_MAX_WORKERS/p' lib/distributor/distributor_private.h", "# ")
+        out = self.dut.send_expect("sed -n '/#define RTE_DISTRIB_MAX_WORKERS/p' lib/distributor/distributor_private.h", "# ", trim_whitespace=False)
         reg_match = r"#define RTE_DISTRIB_MAX_WORKERS (.*)"
         m = re.match(reg_match, out)
         self.verify(m, "Can't find maximum worker number")
diff --git a/tests/TestSuite_flow_classify_softnic.py b/tests/TestSuite_flow_classify_softnic.py
index db7a81dd..e5937289 100644
--- a/tests/TestSuite_flow_classify_softnic.py
+++ b/tests/TestSuite_flow_classify_softnic.py
@@ -129,7 +129,7 @@ class TestFlowClassifySoftnic(TestCase):
         """
         param = ""
         direct_param = r"(\s+)\[ (\S+) in\|out\|inout \]"
-        out = self.tester.send_expect('tcpdump -h', '# ')
+        out = self.tester.send_expect('tcpdump -h', '# ', trim_whitespace=False)
         for line in out.split('\n'):
             m = re.match(direct_param, line)
             if m:
diff --git a/tests/TestSuite_unit_tests_loopback.py b/tests/TestSuite_unit_tests_loopback.py
index 0198a913..a0467378 100644
--- a/tests/TestSuite_unit_tests_loopback.py
+++ b/tests/TestSuite_unit_tests_loopback.py
@@ -81,7 +81,7 @@ class TestUnitTestsLoopback(TestCase):
         pass
 
     def get_max_traffic_burst(self):
-        pmd_file = self.dut.send_expect("cat app/test/test_pmd_perf.c", "# ", 30)
+        pmd_file = self.dut.send_expect("cat app/test/test_pmd_perf.c", "# ", 30, trim_whitespace=False)
         result_scanner = r"#define MAX_TRAFFIC_BURST\s+([0-9]+)"
         scanner = re.compile(result_scanner, re.DOTALL)
         m = scanner.search(pmd_file)
-- 
2.17.1


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

end of thread, other threads:[~2021-06-09  7:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-03  9:33 [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in send_expect Yang Leweix
  -- strict thread matches above, loose matches on Subject: below --
2021-06-03  9:21 Yang Leweix
2021-06-09  7:43 ` 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).