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