From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id C13072BDE for ; Wed, 3 Aug 2016 03:44:17 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga101.jf.intel.com with ESMTP; 02 Aug 2016 18:44:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,463,1464678000"; d="scan'208";a="743392180" Received: from stv-crb-56.sh.intel.com (HELO [10.239.128.116]) ([10.239.128.116]) by FMSMGA003.fm.intel.com with ESMTP; 02 Aug 2016 18:44:13 -0700 Message-ID: <57A14D2A.5020300@intel.com> Date: Wed, 03 Aug 2016 09:47:22 +0800 From: "Liu, Yong" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Lijuan Tu , dts@dpdk.org References: <1470026474-7217-1-git-send-email-lijuanx.a.tu@intel.com> <1470027062-7342-1-git-send-email-lijuanx.a.tu@intel.com> In-Reply-To: <1470027062-7342-1-git-send-email-lijuanx.a.tu@intel.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dts] [PATCH V1 2/2] add unit_tests_distributor test script X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Aug 2016 01:44:18 -0000 Lijuan, some comments below. On 08/01/2016 12:51 PM, Lijuan Tu wrote: > Signed-off-by: Lijuan Tu > --- > tests/TestSuite_unit_tests_distributor.py | 129 ++++++++++++++++++++++++++++++ > 1 file changed, 129 insertions(+) > create mode 100644 tests/TestSuite_unit_tests_distributor.py > > diff --git a/tests/TestSuite_unit_tests_distributor.py b/tests/TestSuite_unit_tests_distributor.py > new file mode 100644 > index 0000000..55f5a80 > --- /dev/null > +++ b/tests/TestSuite_unit_tests_distributor.py > @@ -0,0 +1,129 @@ > +# BSD LICENSE > +# > +# Copyright(c) 2010-2014 Intel Corporation. All rights reserved. > +# All rights reserved. Copyright should update. > +# > +# Redistribution and use in source and binary forms, with or without > +# modification, are permitted provided that the following conditions > +# are met: > +# > +# * Redistributions of source code must retain the above copyright > +# notice, this list of conditions and the following disclaimer. > +# * Redistributions in binary form must reproduce the above copyright > +# notice, this list of conditions and the following disclaimer in > +# the documentation and/or other materials provided with the > +# distribution. > +# * Neither the name of Intel Corporation nor the names of its > +# contributors may be used to endorse or promote products derived > +# from this software without specific prior written permission. > +# > +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS > +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT > +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR > +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT > +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, > +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT > +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, > +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY > +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT > +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE > +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > + > +""" > +DPDK Test suite. > + > +This TestSuite runs the unit tests included in DPDK for power feature. > +""" > + > +from test_case import TestCase > + > +# > +# > +# Test class. > +# > + > + > +class TestUnitTestsDistributor(TestCase): > + > + # > + # > + # > + # Test cases. > + # > + > + def set_up_all(self): > + """ > + Run at the start of each test suite. > + > + Power Prerequisites Since this suite required lot of lcores, need check in set_up_all function. > + """ > + out = self.dut.build_dpdk_apps('./app/test/') > + self.verify('make: Leaving directory' in out, "Compilation failed") > + > + def set_up(self): > + """ > + Run before each test case. > + """ > + pass > + > + def test_1_distributor_1_worker(self): > + """ > + Run power autotest. Test case description tope. > + """ > + Core should be gotten by dut.get_core_list() function and mask generate by create_mask function. Please add description in test plan, couldn't know the difference between each case. > + self.dut.send_expect("./app/test/test -n 4 -c 0xC", "R.*T.*E.*>.*>", 60) > + out = self.dut.send_expect("distributor_autotest", "RTE>>", 60) > + self.dut.send_expect("quit", "# ") > + self.verify("Test OK" in out, "Test failed") > + > + def test_1_distributor_4_worker(self): > + """ > + Run power autotest. > + """ > + > + self.dut.send_expect("./app/test/test -n 4 -c 0x3E", "R.*T.*E.*>.*>", 60) > + out = self.dut.send_expect("distributor_autotest", "RTE>>", 60) > + self.dut.send_expect("quit", "# ") > + self.verify("Test OK" in out, "Test failed") > + > + def test_1_distributor_8_worker(self): > + """ > + Run power autotest. > + """ > + > + self.dut.send_expect("./app/test/test -n 4 -c 0x3FE", "R.*T.*E.*>.*>", 60) > + out = self.dut.send_expect("distributor_autotest", "RTE>>", 60) > + self.dut.send_expect("quit", "# ") > + self.verify("Test OK" in out, "Test failed") > + > + def test_1_distributor_10_worker(self): > + """ > + Run power autotest. > + """ > + > + self.dut.send_expect("./app/test/test -n 4 -c 0xFFE", "R.*T.*E.*>.*>", 60) > + out = self.dut.send_expect("distributor_autotest", "RTE>>", 60) > + self.dut.send_expect("quit", "# ") > + self.verify("Test OK" in out, "Test failed") > + > + def test_1_distributor_5_worker(self): > + """ > + Run power autotest. > + """ > + > + self.dut.send_expect("./app/test/test -n 4 -c 0x100F", "R.*T.*E.*>.*>", 60) > + out = self.dut.send_expect("distributor_autotest", "RTE>>", 60) > + self.dut.send_expect("quit", "# ") > + self.verify("Test OK" in out, "Test failed") > + > + def tear_down(self): > + """ > + Run after each test case. > + """ > + pass > + > + def tear_down_all(self): > + """ > + Run after each test suite. > + """ > + pass