Re: [dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before cp

Re: [dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before cp

[Ling, WeiX]

[-- Attachment #1: Type: text/plain, Size: 322 bytes --]

Tested-by: Wei Ling <weix.ling@intel.com>

Regards,
Ling Wei

> -----Original Message-----
> From: Ling Wei <weix.ling@intel.com>
> Sent: Friday, December 18, 2020 07:37 PM
> To: dts@dpdk.org
> Cc: Ling, WeiX <weix.ling@intel.com>
> Subject: [dts][PATCH V1] tests/unit_tests_loopback:rm exists file before cp


[-- Attachment #2: TestUnitTestsLoopback.log --]
[-- Type: application/octet-stream, Size: 796997 bytes --]

Re: [dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before cp

[Zhao, HaiyangX]

Acked-by: Haiyang Zhao <haiyangx.zhao@intel.com>

Best Regards,
Zhao Haiyang

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Ling, WeiX
> Sent: Friday, December 18, 2020 11:42
> To: Ling, WeiX <weix.ling@intel.com>; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before
> cp
> 
> Tested-by: Wei Ling <weix.ling@intel.com>
> 
> Regards,
> Ling Wei
> 
> > -----Original Message-----
> > From: Ling Wei <weix.ling@intel.com>
> > Sent: Friday, December 18, 2020 07:37 PM
> > To: dts@dpdk.org
> > Cc: Ling, WeiX <weix.ling@intel.com>
> > Subject: [dts][PATCH V1] tests/unit_tests_loopback:rm exists file before cp

[dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before cp

[Ling Wei]

rm exists file before cp

Signed-off-by: Ling Wei <weix.ling@intel.com>
---
 tests/TestSuite_unit_tests_loopback.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_unit_tests_loopback.py b/tests/TestSuite_unit_tests_loopback.py
index 6babc2a1..0198a913 100644
--- a/tests/TestSuite_unit_tests_loopback.py
+++ b/tests/TestSuite_unit_tests_loopback.py
@@ -71,6 +71,7 @@ class TestUnitTestsLoopback(TestCase):
         self.max_traffic_burst = self.get_max_traffic_burst()
         self.dut.send_expect("sed -i -e 's/#define MAX_TRAFFIC_BURST              %s/#define MAX_TRAFFIC_BURST              32/' app/test/test_pmd_perf.c" % self.max_traffic_burst, "# ", 30)
         self.tmp_path = '/tmp/test_pmd_perf.c'
+        self.dut.send_expect("rm -fr %s" % self.tmp_path, "# ")
         self.dut.send_expect("cp app/test/test_pmd_perf.c %s" % self.tmp_path, "# ")
 
     def set_up(self):
@@ -133,6 +134,7 @@ class TestUnitTestsLoopback(TestCase):
         """
         Run after each test case.
         """
+        self.dut.send_expect("rm -fr app/test/test_pmd_perf.c", "# ")
         self.dut.send_expect("cp %s app/test/test_pmd_perf.c" % self.tmp_path, "# ")
         self.dut.kill_all()
 
@@ -140,7 +142,6 @@ class TestUnitTestsLoopback(TestCase):
         """
         Run after each test suite.
         """
-        self.dut.send_expect("cp %s app/test/test_pmd_perf.c" % self.tmp_path, "# ")
         self.dut.send_expect("sed -i -e 's/#define MAX_TRAFFIC_BURST              32/#define MAX_TRAFFIC_BURST              %s/' app/test/test_pmd_perf.c" % self.max_traffic_burst, "# ", 30)
         self.dut.build_install_dpdk(self.target)        
         self.dut.kill_all()
-- 
2.25.1

Re: [dts] [PATCH V1] tests/unit_tests_loopback:rm exists file before cp

[Tu, Lijuan]

> rm exists file before cp
> 
> Signed-off-by: Ling Wei <weix.ling@intel.com>

Applied with commit message changed