[PATCH] libtest: Change expected fail state string

[Sebastian Huber]

On 02/12/2019 23:32, Chris Johns wrote:
> On 2/12/19 6:24 pm, Sebastian Huber wrote:
>> Use the value expected by the RTEMS Tester.  There are two separator
>> characters used ('-' and '_') for the states.  This is a bit
>> inconsistent.
> Sorry about this. I had not noticed.
> 
>> ---
>>   cpukit/libtest/testbeginend.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/cpukit/libtest/testbeginend.c b/cpukit/libtest/testbeginend.c
>> index b739e47b2e..9bdc86780c 100644
>> --- a/cpukit/libtest/testbeginend.c
>> +++ b/cpukit/libtest/testbeginend.c
>> @@ -29,7 +29,7 @@ rtems_printer rtems_test_printer = {
>>   static const char* const test_state_strings[] =
>>   {
>>     "EXPECTED-PASS",
> Should this change or should `USER_INPUT` be changed and we update ...
> 
> https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n159
> https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n161
> 
> ?
> 
> It would be a simpler change because this change means we move to having 2 with
> `_` and a bunch with `-`. There are internal states in the tester such as
> `WRONG-VERSION` and changing all of those is a bigger change in the tester.

If we update the tester, then it should accept '-' and '_' versions so 
that it can be used for test runs before and after a change of 
cpukit/libtest/testbeginend.c.

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.