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

["Mo, YufengX" <yufengx.mo@intel.com>]

Missing `self` in using this_timeout.

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Thinh Tran
> Sent: Friday, July 19, 2019 2:39 AM
> To: dts@dpdk.org
> Cc: thinhtr@linux.vnet.ibm.com
> Subject: [dts] [PATCH] tests/unit_tests_timer increase timeout
> 
> 
> - fix the TIMEOUT Failure for some systems have more than 16 cores
> 
> 
> Signed-off-by: Thinh Tran <thinhtr@linux.vnet.ibm.com>
> ---
>  tests/TestSuite_unit_tests_timer.py | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/TestSuite_unit_tests_timer.py b/tests/TestSuite_unit_tests_timer.py
> index 717dab8..f3bb02d 100644
> --- a/tests/TestSuite_unit_tests_timer.py
> +++ b/tests/TestSuite_unit_tests_timer.py
> @@ -59,6 +59,13 @@ class TestUnitTestsTimer(TestCase):
>          """
>          cores = self.dut.get_core_list("all")
>          self.coremask = utils.create_mask(cores)
> +        #
> +        # change timeout base number of cores on the system
> +        # default 60 secs
> +        #
> +        self.this_timeout = 60
> +        if (len(cores) > 16) :
> +            self.this_timeout = self.this_timeout * len(cores)/16
> 
>      def set_up(self):
>          """
> @@ -71,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>>", 60)
> +        out = self.dut.send_expect("timer_autotest", "RTE>>", this_timeout)
>          self.dut.send_expect("quit", "# ")
>          self.verify("Test OK" in out, "Test failed")
> 
> @@ -80,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>>", 60)
> +        out = self.dut.send_expect("timer_perf_autotest", "RTE>>", this_timeout)
>          self.dut.send_expect("quit", "# ")
>          self.verify("Test OK" in out, "Test failed")
> 
> --
> 2.17.1