Commit 8981a85e authored by Pedro Tammela's avatar Pedro Tammela Committed by Jakub Kicinski
Browse files

selftests: tc-testing: return fail if a test fails in setup/teardown 



As of today tests throwing exceptions in setup/teardown phase are
treated as skipped but they should really be failures.

Signed-off-by: default avatarPedro Tammela <pctammela@mojatatu.com>
Reviewed-by: default avatarDavide Caratti <dcaratti@redhat.com>
Acked-by: default avatarJamal Hadi Salim <jhs@mojatatu.com>
Reviewed-by: default avatarDavide Caratti <dcaratti@redhat.com>
Link: https://lore.kernel.org/r/20240124181933.75724-6-pctammela@mojatatu.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent d17d0e33

tools/testing/selftests/tc-testing/tdc.py

+1 −1
Original line number Diff line number Diff line
@@ -541,7 +541,7 @@ def test_runner(pm, args, filtered_tests): 
            message = pmtf.message

            output = pmtf.output

            res = TestResult(tidx['id'], tidx['name'])

            res.set_result(ResultState.skip)

            res.set_result(ResultState.fail)

            res.set_errormsg(pmtf.message)

            res.set_failmsg(pmtf.output)

            tsr.add_resultdata(res)