[U-Boot] [PATCH v2 16/19] test: env: Add test for verifying env attrs

Joe Hershberger joe.hershberger at gmail.com
Sun May 3 22:16:49 CEST 2015


Hi Simon,

On Thu, Apr 30, 2015 at 10:46 PM, Simon Glass <sjg at chromium.org> wrote:
> Hi Joe,
>
> On 28 April 2015 at 23:51, Joe Hershberger <joe.hershberger at ni.com> wrote:
>> Add a test of the env_attr_lookup() function.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
>> ---
>>
>> Changes in v2:
>> -New for version 2
>>
>>  test/env/Makefile |  1 +
>>  test/env/attr.c   | 62 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 63 insertions(+)
>>  create mode 100644 test/env/attr.c
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> I don't know how much it matters, but I was trying to make test
> commands start with 'ut_' (unit test) so they would all appear in the
> same place when you type 'help'.

I took it one step further by creating a ut command that can have
sub-commands that are test suites. This also should help address Tom's
concerns about ease of running.

Cheers,
-Joe


More information about the U-Boot mailing list