Skip to content

Commit ffc3f22

Browse files
authored
Error on missing wasm2js test expectations (#7119)
1 parent acb1258 commit ffc3f22

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

scripts/test/wasm2js.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,15 @@ def test_wasm2js_output():
6565
expected_file += '.opt'
6666

6767
if not os.path.exists(expected_file):
68-
continue
68+
# It is ok to skip tests from other test suites that we also
69+
# test on wasm2js (the basic and spec tests). When such files
70+
# pass in wasm2js, we add expected files for them, so lacking
71+
# such a file just means we should ignore it. But lacking an
72+
# expected file for an explicit wasm2js test is an error.
73+
if t in basic_tests or t in spec_tests:
74+
continue
75+
else:
76+
raise Exception(f'missing expected file {expected_file}')
6977

7078
print('..', os.path.basename(t))
7179

0 commit comments

Comments
 (0)