test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] veb_switch: remove the unnecessary comparison
@ 2017-11-16  7:53 Peng Yuan
  2017-11-20 13:16 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: Peng Yuan @ 2017-11-16  7:53 UTC (permalink / raw)
  To: dts; +Cc: Peng Yuan

Signed-off-by: Peng Yuan <yuan.peng@intel.com>

diff --git a/tests/TestSuite_veb_switch.py b/tests/TestSuite_veb_switch.py
index 843c4f0..c1953e2 100644
--- a/tests/TestSuite_veb_switch.py
+++ b/tests/TestSuite_veb_switch.py
@@ -395,14 +395,11 @@ class TestVEBSwitching(TestCase):
         time.sleep(2)
 
         vf0_start_rx_stats = self.veb_get_pmd_stats("second", 0, "rx")
-        pf_start_tx_stats = self.veb_get_pmd_stats("first", 0, "tx")
         self.send_packet(self.vf0_mac, self.tester_itf)
         time.sleep(2)
         self.session_secondary.send_expect("stop", "testpmd>", 2)
         self.dut.send_expect("stop", "testpmd>", 2)
         vf0_end_rx_stats = self.veb_get_pmd_stats("second", 0, "rx")
-        pf_end_tx_stats = self.veb_get_pmd_stats("first", 0, "tx")
-        self.verify(pf_end_tx_stats[0] - pf_start_tx_stats[0] == 0, "PF received unexpected packet")
         self.verify(vf0_end_rx_stats[0] - vf0_start_rx_stats[0] == 1, "no packet was received by VF0")
  
         self.dut.send_expect("start", "testpmd>")
@@ -419,8 +416,8 @@ class TestVEBSwitching(TestCase):
         vf0_end_rx_stats = self.veb_get_pmd_stats("second", 0, "rx")
         pf_end_tx_stats = self.veb_get_pmd_stats("first", 0, "tx")
         pf_end_rx_stats = self.veb_get_pmd_stats("first", 0, "rx")
-        self.verify(pf_end_tx_stats[0] - pf_start_tx_stats[0] == 1, "no packet was sent by VF0")
-        self.verify(pf_end_rx_stats[0] - pf_start_rx_stats[0] == 1, "no packet was received by VF0")
+        self.verify(pf_end_tx_stats[0] - pf_start_tx_stats[0] == 1, "no packet was sent by PF")
+        self.verify(pf_end_rx_stats[0] - pf_start_rx_stats[0] == 1, "no packet was received by PF")
         self.verify(vf0_end_rx_stats[0] - vf0_start_rx_stats[0] == 0, "VF0 received unexpected packet")
         self.session_secondary.send_expect("quit", "# ")
         time.sleep(2)
-- 
2.5.0

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

* Re: [dts] [PATCH] veb_switch: remove the unnecessary comparison
  2017-11-16  7:53 [dts] [PATCH] veb_switch: remove the unnecessary comparison Peng Yuan
@ 2017-11-20 13:16 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-11-20 13:16 UTC (permalink / raw)
  To: Peng Yuan, dts

Thanks, Yuan. Applied.

On 11/16/2017 03:53 PM, Peng Yuan wrote:
> Signed-off-by: Peng Yuan<yuan.peng@intel.com>
>
> diff --git a/tests/TestSuite_veb_switch.py b/tests/TestSuite_veb_switch.py

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

end of thread, other threads:[~2017-11-20  5:23 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-16  7:53 [dts] [PATCH] veb_switch: remove the unnecessary comparison Peng Yuan
2017-11-20 13:16 ` Liu, Yong

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