Skip to content

Commit

Permalink
Fix start_test not suppressing timeout errors (#25200)
Browse files Browse the repository at this point in the history
Fixes an issue were `start_test` would not suppress timeout errors
properly. Adjusted `sub_test` to add the appropriate future line.

[Review by @jeremiah-corrado]
  • Loading branch information
jabraham17 authored Jun 10, 2024
2 parents 324cb1a + 365a6b3 commit e266bd5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion util/test/sub_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -2481,7 +2481,7 @@ def run_compileline(flag, lookingfor):
sys.stdout.write(']\n')
# only notify for a failed execution if launching the test was successful
elif (not launcher_error):
sys.stdout.write('[Error execution failed for %s]\n'%(test_name))
sys.stdout.write('%s[Error execution failed for %s]\n'%(futuretest,test_name))

if exectimeout or status != 0 or exec_status != 0:
break
Expand Down

0 comments on commit e266bd5

Please sign in to comment.