test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/queue_start_stop: change the verify way
@ 2019-06-24  4:09 Wenjie Li
  2019-06-24  8:52 ` Ma, LihongX
  0 siblings, 1 reply; 2+ messages in thread
From: Wenjie Li @ 2019-06-24  4:09 UTC (permalink / raw)
  To: dts; +Cc: Wenjie Li

In dts d9bc72a3, Phanendra Vukkisala changed the packet number to 4
and the verify message to "ports 0 queue 0 receive 4 packages", but in
test, the out is:
"
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
"
So change the verify way.

Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
---
 tests/TestSuite_queue_start_stop.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_queue_start_stop.py b/tests/TestSuite_queue_start_stop.py
index 34a2f41..7fdef30 100644
--- a/tests/TestSuite_queue_start_stop.py
+++ b/tests/TestSuite_queue_start_stop.py
@@ -157,7 +157,12 @@ class TestQueueStartStop(TestCase):
         except Exception, e:
             raise IOError("queue start/stop forward failure: %s" % e)
 
-        self.verify("ports 0 queue 0 receive 4 packages" in out, "start queue revice package failed, out = %s"%out)
+        lines = out.split("\r\n")
+        print lines
+        if len(lines) == 5:
+            lines.pop()
+        for line in lines:
+            self.verify("ports 0 queue 0 receive 1 packages" in line, "start queue revice package failed, out = %s" % out)
 
         try:
             # start tx queue test
-- 
2.17.2


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

* Re: [dts] [PATCH V1] tests/queue_start_stop: change the verify way
  2019-06-24  4:09 [dts] [PATCH V1] tests/queue_start_stop: change the verify way Wenjie Li
@ 2019-06-24  8:52 ` Ma, LihongX
  0 siblings, 0 replies; 2+ messages in thread
From: Ma, LihongX @ 2019-06-24  8:52 UTC (permalink / raw)
  To: Li, WenjieX A, dts; +Cc: Li, WenjieX A

Hi, wenjie
As feedback from Phanendra Vukkisala, on cavium_a063 platform , it will output "ports 0 queue 0 receive 4 packages"
So I think you will change you code base on different env.

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Wenjie Li
Sent: Monday, June 24, 2019 12:09 PM
To: dts@dpdk.org
Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
Subject: [dts] [PATCH V1] tests/queue_start_stop: change the verify way

In dts d9bc72a3, Phanendra Vukkisala changed the packet number to 4 and the verify message to "ports 0 queue 0 receive 4 packages", but in test, the out is:
"
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
ports 0 queue 0 receive 1 packages
"
So change the verify way.

Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
---
 tests/TestSuite_queue_start_stop.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_queue_start_stop.py b/tests/TestSuite_queue_start_stop.py
index 34a2f41..7fdef30 100644
--- a/tests/TestSuite_queue_start_stop.py
+++ b/tests/TestSuite_queue_start_stop.py
@@ -157,7 +157,12 @@ class TestQueueStartStop(TestCase):
         except Exception, e:
             raise IOError("queue start/stop forward failure: %s" % e)
 
-        self.verify("ports 0 queue 0 receive 4 packages" in out, "start queue revice package failed, out = %s"%out)
+        lines = out.split("\r\n")
+        print lines
+        if len(lines) == 5:
+            lines.pop()
+        for line in lines:
+            self.verify("ports 0 queue 0 receive 1 packages" in line, 
+ "start queue revice package failed, out = %s" % out)
 
         try:
             # start tx queue test
--
2.17.2


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

end of thread, other threads:[~2019-06-24  8:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-24  4:09 [dts] [PATCH V1] tests/queue_start_stop: change the verify way Wenjie Li
2019-06-24  8:52 ` Ma, LihongX

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