test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Tu, LijuanX A" <lijuanx.a.tu@intel.com>
To: "Xu, HuilongX" <huilongx.xu@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Subject: Re: [dts] [03/12] add test start time and exec cmd time for dump unit	test
Date: Tue, 27 Oct 2015 08:33:21 +0000	[thread overview]
Message-ID: <38D041F150D4184C8114E499040E62340723594E@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <1445931412-86352-4-git-send-email-huilongx.xu@intel.com>

Acked by tu,lijuan <lijuanx.a.tu@intel.com>

> Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> ---
>  tests/TestSuite_unit_tests_dump.py | 34 ++++++++++++++++++----------------
>  1 file changed, 18 insertions(+), 16 deletions(-)
> 
> diff --git a/tests/TestSuite_unit_tests_dump.py b/tests/TestSuite_unit_tests_dump.py
> index ce04439..2052cb4 100644
> --- a/tests/TestSuite_unit_tests_dump.py
> +++ b/tests/TestSuite_unit_tests_dump.py
> @@ -62,6 +62,8 @@ class TestUnitTestsDump(TestCase):
>          # Based on h/w type, choose how many ports to use
>          self.dut_ports = self.dut.get_ports(self.nic)
>          self.verify(len(self.dut_ports) >= 1, "Insufficient ports for testing")
> +        self.start_test_time = 60
> +        self.run_cmd_time = 60
>  
>      def set_up(self):
>          """
> @@ -74,8 +76,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run history log dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_log_history", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_log_history", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>          self.verify("EAL" in out, "Test failed")
>  
> @@ -83,8 +85,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run history log dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_ring", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_ring", "RTE>>", 
> + self.run_cmd_time)
>          self.dut.send_expect("quit", "# ")
>          elements = ['ring', 'address', 'flags', 'size', 'ct', 'ch', 'pt', 'ph', 'used', 'avail', 'watermark']
>          match_regex = "ring <(.*?)>@0x(.*)\r\n"
> @@ -100,8 +102,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run mempool dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_mempool", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_mempool", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>          elements = ['mempool', 'address', 'flags', 'ring', 'phys_addr', 'size', 'header_size', 'elt_size',
>                      'trailer_size', 'total_obj_size', 'private_data_size', 'pg_num', 'pg_shift', 'pg_mask', @@ -120,8 +122,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run physical memory dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_physmem", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_physmem", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>          elements = ['Segment', 'phys', 'len', 'virt', 'socket_id', 'hugepage_sz', 'nchannel', 'nrank']
>          match_regex = "Segment (\d)+:"
> @@ -140,8 +142,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run memzone dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_memzone", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_memzone", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>  
>          elements = ['Zone', 'name', 'phys', 'len', 'virt', 'socket_id', 'flags'] @@ -162,8 +164,8 @@ class TestUnitTestsDump(TestCase):
>          """
>          Run struct size dump test case.
>          """
> -        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_struct_sizes", "RTE>>", 120)
> +        self.dut.send_expect("./%s/app/test -n 1 -c ffff" % (self.target), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_struct_sizes", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>  
>          elements = ['struct rte_mbuf', 'struct rte_mempool', 'struct rte_ring'] @@ -181,15 +183,15 @@ class TestUnitTestsDump(TestCase):
>          """
>          test_port = self.dut_ports[0]
>          self.dut.send_expect("./%s/app/test -n 1 -c ffff -b 0000:%s"
> -                             % (self.target, self.dut.ports_info[test_port]['pci']), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_devargs", "RTE>>", 120)
> +                             % (self.target, self.dut.ports_info[test_port]['pci']), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_devargs", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>          black_str = "PCI blacklist 0000:%s" % self.dut.ports_info[test_port]['pci']
>          self.verify(black_str in out, "Dump black list failed")
>  
>          self.dut.send_expect("./%s/app/test -n 1 -c ffff -w 0000:%s"
> -                             % (self.target, self.dut.ports_info[test_port]['pci']), "R.*T.*E.*>.*>", 10)
> -        out = self.dut.send_expect("dump_devargs", "RTE>>", 120)
> +                             % (self.target, self.dut.ports_info[test_port]['pci']), "R.*T.*E.*>.*>", self.start_test_time)
> +        out = self.dut.send_expect("dump_devargs", "RTE>>", 
> + self.run_cmd_time * 2)
>          self.dut.send_expect("quit", "# ")
>  
>          white_str = "PCI whitelist 0000:%s" % self.dut.ports_info[test_port]['pci']
> --
> 1.9.3
> 
>

  reply	other threads:[~2015-10-27  8:33 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-27  7:36 [dts] [00/12] add test start time xu,huilong
2015-10-27  7:36 ` [dts] [01/12] add test start time and exec cmd time for cmdline unit test xu,huilong
2015-10-27  8:28   ` Tu, LijuanX A
2015-10-27  8:29   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [02/12] add test start time and exec cmd time for eal " xu,huilong
2015-10-27  8:31   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [03/12] add test start time and exec cmd time for dump " xu,huilong
2015-10-27  8:33   ` Tu, LijuanX A [this message]
2015-10-27  7:36 ` [dts] [04/12] add test start time for ivshmem " xu,huilong
2015-10-27  8:34   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [05/12] add test start time for kni " xu,huilong
2015-10-27  8:35   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [06/12] add test start time for lpm " xu,huilong
2015-10-27  8:42   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [07/12] add test start time for mbuf " xu,huilong
2015-10-27  8:43   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [08/12] add test start time for mempool " xu,huilong
2015-10-27  8:42   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [09/12] add test start time and exec cmd time for power " xu,huilong
2015-10-27  8:38   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [10/12] add test start time and exec cmd time for pmd_perf " xu,huilong
2015-10-27  8:39   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [11/12] add test start time for ring " xu,huilong
2015-10-27  8:40   ` Tu, LijuanX A
2015-10-27  7:36 ` [dts] [12/12] add test start time for timer " xu,huilong
2015-10-27  8:21 ` [dts] [00/12] add test start time Tu, LijuanX A
2015-10-27  8:43   ` Liu, Yong
2015-10-27  8:54 ` Liu, Yong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=38D041F150D4184C8114E499040E62340723594E@shsmsx102.ccr.corp.intel.com \
    --to=lijuanx.a.tu@intel.com \
    --cc=dts@dpdk.org \
    --cc=huilongx.xu@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).