test suite reviews and discussions
 help / color / mirror / Atom feed
From: Yang Leweix <leweix.yang@intel.com>
To: dts@dpdk.org
Cc: leweix.yang@intel.com
Subject: [dts] [PATCH V1] tests:fix some suites impacted by Strip prompt in send_expect
Date: Thu,  3 Jun 2021 17:33:28 +0800	[thread overview]
Message-ID: <20210603093328.2253-1-leweix.yang@intel.com> (raw)

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


             reply	other threads:[~2021-06-03  9:28 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-03  9:33 Yang Leweix [this message]
  -- strict thread matches above, loose matches on Subject: below --
2021-06-03  9:21 Yang Leweix
2021-06-09  7:43 ` Tu, Lijuan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210603093328.2253-1-leweix.yang@intel.com \
    --to=leweix.yang@intel.com \
    --cc=dts@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).