Re: [dts] [PATCH V1] tests/tx_preparation: optimize script

["Tu, Lijuan" <lijuan.tu@intel.com>]

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Wenjie Li
> Sent: Saturday, August 24, 2019 1:03 AM
> To: dts@dpdk.org
> Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> Subject: [dts] [PATCH V1] tests/tx_preparation: optimize script
> 
> move stop/quit testpmd into tear_down; otherwise, if one case is failed, the
> testpmd is still running, it will affect the following cases.
> 
> Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
> ---
>  tests/TestSuite_tx_preparation.py | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/tests/TestSuite_tx_preparation.py
> b/tests/TestSuite_tx_preparation.py
> index eea95c1..ba0aac7 100644
> --- a/tests/TestSuite_tx_preparation.py
> +++ b/tests/TestSuite_tx_preparation.py
> @@ -173,8 +173,6 @@ class TestTX_preparation(TestCase):
>          self.dut_testpmd.execute_cmd('start')
> 
>          self.send_packet_verify()
> -        self.dut_testpmd.execute_cmd('stop')
> -        self.dut_testpmd.quit()
> 
>      def test_tx_preparation_TSO(self):
>          """
> @@ -185,16 +183,13 @@ class TestTX_preparation(TestCase):
>          self.dut_testpmd.execute_cmd('start')
> 
>          self.send_packet_verify(1)
> -        self.dut_testpmd.execute_cmd('stop')
> -        self.dut_testpmd.quit()
> -
> 
>      def tear_down(self):
>          """
>          Run after each test case.
> -        """
> -        pass
> -
> +        """
> +        self.dut_testpmd.execute_cmd('stop')
> +        self.dut_testpmd.quit()
> 
>      def tear_down_all(self):
>          """
> --
> 2.17.1