Fix return code of restricted trace tests.

The prior script was incorrectly comparing a boolean result with a number and producing the opposite result: True would be reported as a failure, and False would be reported as success. Bug: angleproject:4090 Bug: b/168049670 Change-Id: I47ec684a57c455c6a833232d0a49774b265f7b39 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2430667Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>

Fix return code of restricted trace tests.
962e3473 · Jamie Madill · Commit Bot · 69da0b92 · 962e3473
Commit 962e3473 authored Sep 25, 2020 by Jamie Madill Committed by Commit Bot Sep 25, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

restricted_trace_gold_tests.py src/tests/restricted_traces/restricted_trace_gold_tests.py +4 -2

No files found.
--- a/src/tests/restricted_traces/restricted_trace_gold_tests.py
+++ b/src/tests/restricted_traces/restricted_trace_gold_tests.py
@@ -131,10 +131,12 @@ def main():
            tests = json.load(fp)
        if args.render_test_output_dir:
-            rc = run_tests(args, tests, extra_flags, env, args.render_test_output_dir)
+            if not run_tests(args, tests, extra_flags, env, args.render_test_output_dir):
+                rc = 1
        else:
            with temporary_dir('angle_trace_') as temp_dir:
-                rc = run_tests(args, tests, extra_flags, env, temp_dir)
+                if not run_tests(args, tests, extra_flags, env, temp_dir):
+                    rc = 1
    except Exception:
        traceback.print_exc()