Re: [dts] [PATCH] tests unit_test_eal: fix test_pci unit test

[Jianbo Liu <jianbo.liu@linaro.org>]

On 29 November 2016 at 09:39, Xu, HuilongX <huilongx.xu@intel.com> wrote:
> Hi jianbo,
> If we run unit pci test case, we must re-compile test app and enable CONFIG_RTE_APP_TEST_RESOURCE_TAR.
> We will submit a patch for compile with enable CONFIG_RTE_APP_TEST_RESOURCE_TAR, and change test app name to test_pci.
> Thanks  a lot
>

Yes. But it depends on the user's requirement.
If he want to enable CONFIG_RTE_APP_TEST_RESOURCE_TAR, he can change
it by specifying a configuration patch with "dts --patch " argument.


>> -----Original Message-----
>> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Jianbo Liu
>> Sent: Friday, November 25, 2016 2:36 PM
>> To: dts@dpdk.org
>> Cc: Jianbo Liu
>> Subject: [dts] [PATCH] tests unit_test_eal: fix test_pci unit test
>>
>> There is no app/test_pci, uses app/test instead.
>>
>> Signed-off-by: Jianbo Liu <jianbo.liu@linaro.org>
>> ---
>>  tests/TestSuite_unit_tests_eal.py | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/TestSuite_unit_tests_eal.py
>> b/tests/TestSuite_unit_tests_eal.py
>> index d27ba39..07d423c 100644
>> --- a/tests/TestSuite_unit_tests_eal.py
>> +++ b/tests/TestSuite_unit_tests_eal.py
>> @@ -158,7 +158,7 @@ class TestUnitTestsEal(TestCase):
>>          Run pci autotest.
>>          """
>>          # should re-compile test app, this is only a demo. will
>> implementation after framework support
>> -        self.dut.send_expect("./%s/app/test_pci -n 1 -c f" %
>> self.target, "R.*T.*E.*>.*>" , self.start_test_time)
>> +        self.dut.send_expect("./%s/app/test -n 1 -c f" % self.target,
>> "R.*T.*E.*>.*>" , self.start_test_time)
>>          out = self.dut.send_expect("pci_autotest", "RTE>>",
>> self.run_cmd_time)
>>          self.dut.send_expect("quit", "# ")
>>          self.verify("Test OK" in out, "Test failed")
>> --
>> 2.5.5
>