diff options
author | Asher Gomez <ashersaupingomez@gmail.com> | 2024-04-17 20:40:47 +1000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-17 10:40:47 +0000 |
commit | 9acbf90b06bf79dd6e4cf2428b3566da009bed65 (patch) | |
tree | fbc7e2deb36fffefab39998606e42b2a3cbf543c /tests/wpt | |
parent | ebc22d9d7ed53031428dfecde1b0a57121c91685 (diff) |
fix: allow WPT to successfully exit using `--exit-zero` (#23418)
I went with `--exit-zero`. Happy to change to `--no-exit` if feelings
are strong.
Supercedes #23417
Diffstat (limited to 'tests/wpt')
-rwxr-xr-x | tests/wpt/wpt.ts | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/wpt/wpt.ts b/tests/wpt/wpt.ts index 056db5dc0..675ef3ae0 100755 --- a/tests/wpt/wpt.ts +++ b/tests/wpt/wpt.ts @@ -550,6 +550,12 @@ function reportFinal( }. ${finalPassedCount} passed; ${finalFailedCount} failed; ${finalExpectedFailedAndFailedCount} expected failure; total ${finalTotalCount} (${duration}ms)\n`, ); + // We ignore the exit code of the test run because the CI job reports the + // results to WPT.fyi, and we still want to report failure. + if (Deno.args.includes("--exit-zero")) { + return 0; + } + return failed ? 1 : 0; } |