Skip to content

Commit ee113c7

Browse files
authored
[test] Fix wasm2js stale check failure (#6168)
I tried to exclude wasm2js asserts tests from `check_for_stale_files` in #6164, but ended up doing it incorrectly. The file I checked for was `wasm2js.wast.asserts`, while the output I should have excluded was `wasm2js.asserts.js`. This fixes the code so we now check the prefix and not the filename.
1 parent 5db8eb2 commit ee113c7

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

scripts/test/wasm2js.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,9 @@ def check_for_stale_files():
4141

4242
all_files = os.listdir(shared.get_test_dir('wasm2js'))
4343
for f in all_files:
44-
if f in assert_tests:
45-
continue
4644
prefix = f.split('.')[0]
45+
if prefix in [t.split('.')[0] for t in assert_tests]:
46+
continue
4747
if prefix not in all_tests:
4848
shared.fail_with_error('orphan test output: %s' % f)
4949

0 commit comments

Comments
 (0)