diff options
| author | Alon Zakai <azakai@google.com> | 2024-11-26 14:15:02 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-11-26 14:15:02 -0800 |
| commit | ffc3f2219b18c2a2ddb160c0d81518234faa2cd1 (patch) | |
| tree | c43749c3c04d9edc43bd1f6dab5d715c7d53bae9 | |
| parent | acb12584c277d9bfb0b1c78e65101fa54e9a7361 (diff) | |
| download | binaryen-ffc3f2219b18c2a2ddb160c0d81518234faa2cd1.tar.gz binaryen-ffc3f2219b18c2a2ddb160c0d81518234faa2cd1.tar.bz2 binaryen-ffc3f2219b18c2a2ddb160c0d81518234faa2cd1.zip | |
Error on missing wasm2js test expectations (#7119)
| -rw-r--r-- | scripts/test/wasm2js.py | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/scripts/test/wasm2js.py b/scripts/test/wasm2js.py index 224dc01cb..95fe4fdac 100644 --- a/scripts/test/wasm2js.py +++ b/scripts/test/wasm2js.py @@ -65,7 +65,15 @@ def test_wasm2js_output(): expected_file += '.opt' if not os.path.exists(expected_file): - continue + # It is ok to skip tests from other test suites that we also + # test on wasm2js (the basic and spec tests). When such files + # pass in wasm2js, we add expected files for them, so lacking + # such a file just means we should ignore it. But lacking an + # expected file for an explicit wasm2js test is an error. + if t in basic_tests or t in spec_tests: + continue + else: + raise Exception(f'missing expected file {expected_file}') print('..', os.path.basename(t)) |