On 1/31/19 9:49 AM, Uwe Kleine-König wrote: > This simplifies finding the offending test when the build ended with > > KO: out of 584 tests, 527 passed, 57 failed > 56 of them are known to fail > > Signed-off-by: Uwe Kleine-König <uwe@xxxxxxxxxxxxxxxxx> > --- > validation/test-suite | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/validation/test-suite b/validation/test-suite > index f79a9023d95a..a5572d5e6965 100755 > --- a/validation/test-suite > +++ b/validation/test-suite > @@ -403,9 +403,16 @@ do_test() > if [ "$must_fail" -eq "1" ]; then > if [ "$test_failed" -eq "1" ]; then > [ -z "$vquiet" ] && \ > - echo "info: test '$file' is known to fail" > + echo "info: XFAIL: test '$file' is known to fail" > else > - echo "error: test '$file' is known to fail but succeed!" > + echo "error: XPASS: test '$file' is known to fail but succeed!" > + fi > + else > + if [ "$test_failed" -eq "1" ]; then > + echo "error: FAIL: test '$file' is failed" When reading my own patch after it returned to me in my sparse folder I noticed a fault here. We want the "is" dropped in the above line. Should I resend for that? Best regards Uwe
Attachment:
signature.asc
Description: OpenPGP digital signature
