[PATCH] libtest: Change expected fail state string

Sebastian Huber sebastian.huber at embedded-brains.de
Thu Dec 5 06:07:07 UTC 2019


On 05/12/2019 01:24, Chris Johns wrote:
> 
> 
> On 4/12/19 11:11 pm, Sebastian Huber wrote:
>> On 04/12/2019 12:59, Joel Sherrill wrote:
>>>
>>> On Wed, Dec 4, 2019, 3:57 AM Sebastian Huber
>>> <sebastian.huber at embedded-brains.de
>>> <mailto:sebastian.huber at embedded-brains.de>> wrote:
>>>
>>>      On 04/12/2019 09:49, Chris Johns wrote:
>>>       > On 4/12/19 5:43 pm, Sebastian Huber wrote:
>>>       >> On 04/12/2019 05:14, Chris Johns wrote:
>>>       >>> On 3/12/19 9:59 pm, Sebastian Huber wrote:
>>>       >>>> 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.
>>>       >>> Yes this is a good point. Shame it has to be this way but I
>>>      cannot see an
>>>       >>> alternative.
>>>       >> I checked in the patch as is. Using only one separator character
>>>      for all states
>>>       >> is just a beautification.
>>>       > That is not what I was meaning. I would like RTEMS master to use
>>>      only '-' and
>>>       > the tester has backward compatibility.
>>>
>>>      I would change everything to '_' separators since we have also the
>>>      RTEMS
>>>      options:
>>>
>>>
>>> Dashes are often used as separators so can be questionable to use in strings
>>> intended to be a single token.
>>
>> Ok, so change "EXPECTED-PASS" to "EXPECTED_PASS", all "-" to "_" in report.py,
>> and accept [-_] in report.py.
> 
> Please make sure
> https://docs.rtems.org/branches/master/user/testing/tests.html#test-controls is
> also updated?

I had a look at it. It already uses EXPECTED_FAIL and USER_INPUT. I have 
to update the "Test Configuration" for the new build system.

-- 
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.


More information about the devel mailing list