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

Joe Hershberger
2015-05-03T20:16:49Z

Hi Simon,

On Thu, Apr 30, 2015 at 10:46 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi Joe,
>
> On 28 April 2015 at 23:51, Joe Hershberger <joe.hershberger@ni.com> wrote:
>> Add a test of the env_attr_lookup() function.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger@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@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