Skip to content

Commit

Permalink
Fix tools/run-tests.py --test262-es2015 (#3803)
Browse files Browse the repository at this point in the history
The output of the test262 runner changed between ES5.1 and ES6,
our script should respect it to signal regressions in return code too.

JerryScript-DCO-1.0-Signed-off-by: Csaba Osztrogonác oszi@inf.u-szeged.hu
  • Loading branch information
ossy-szeged authored May 28, 2020
1 parent d4fe48f commit ab3461b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tools/runners/run-test-suite-test262.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ def main(args):
universal_newlines=True,
stdout=subprocess.PIPE)

return_code = 0
return_code = 1
with open(os.path.join(os.path.dirname(args.engine), 'test262.report'), 'w') as output_file:
counter = 0
summary_found = False
Expand All @@ -132,8 +132,8 @@ def main(args):

if summary_found:
print(output, end='')
if ('Failed tests' in output) or ('Expected to fail but passed' in output):
return_code = 1
if 'All tests succeeded' in output:
return_code = 0

proc.wait()

Expand Down

0 comments on commit ab3461b

Please sign in to comment.