Re: [dts] [PATCH v2] tests/unit_tests_timer increase timeout

[Thinh Tran <thinhtr@linux.vnet.ibm.com>]

Sorry. v2 was on top off v1. will resend it shortly.

On 7/21/2019 9:01 PM, Mo, YufengX wrote:
> self.this_timeout is not defined. I think you haven't rebase your source code.
> 
> 
>> -----Original Message-----
>> From: Thinh Tran [mailto:thinhtr@linux.vnet.ibm.com]
>> Sent: Friday, July 19, 2019 9:00 PM
>> To: Mo, YufengX <yufengx.mo@intel.com>
>> Cc: dts@dpdk.org; thinhtr@linux.vnet.ibm.com
>> Subject: [dts][PATCH v2] tests/unit_tests_timer increase timeout
>>
>> Signed-off-by: Thinh Tran <thinhtr@linux.vnet.ibm.com>
>> ---
>>   changes v1 -> v2:
>>     - add missing self to this_timeout
>>
>>   tests/TestSuite_unit_tests_timer.py | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/TestSuite_unit_tests_timer.py b/tests/TestSuite_unit_tests_timer.py
>> index f3bb02d..365c5f6 100644
>> --- a/tests/TestSuite_unit_tests_timer.py
>> +++ b/tests/TestSuite_unit_tests_timer.py
>> @@ -78,7 +78,7 @@ class TestUnitTestsTimer(TestCase):
>>           Run timer autotest.
>>           """
>>           self.dut.send_expect("./%s/app/test -n 1 -c %s" % (self.target, self.coremask), "R.*T.*E.*>.*>", 60)
>> -        out = self.dut.send_expect("timer_autotest", "RTE>>", this_timeout)
>> +        out = self.dut.send_expect("timer_autotest", "RTE>>", self.this_timeout)
>>           self.dut.send_expect("quit", "# ")
>>           self.verify("Test OK" in out, "Test failed")
>>
>> @@ -87,7 +87,7 @@ class TestUnitTestsTimer(TestCase):
>>           Run timer autotest.
>>           """
>>           self.dut.send_expect("./%s/app/test -n 1 -c %s" % (self.target, self.coremask), "R.*T.*E.*>.*>", 60)
>> -        out = self.dut.send_expect("timer_perf_autotest", "RTE>>", this_timeout)
>> +        out = self.dut.send_expect("timer_perf_autotest", "RTE>>", self.this_timeout)
>>           self.dut.send_expect("quit", "# ")
>>           self.verify("Test OK" in out, "Test failed")
>>
>> --
>> 2.17.1
>