diff options
| author | Heejin Ahn <aheejin@gmail.com> | 2023-12-11 19:16:19 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-11 19:16:19 -0800 |
| commit | ee113c7e4cdff655c395598ee9f61512c3a5b3eb (patch) | |
| tree | 5b13d09ff4ddfd0c21a7d91d260f91ac3a585fa4 /scripts | |
| parent | 5db8eb2131bebfed4e0748056d9fafaca60ce83e (diff) | |
| download | binaryen-ee113c7e4cdff655c395598ee9f61512c3a5b3eb.tar.gz binaryen-ee113c7e4cdff655c395598ee9f61512c3a5b3eb.tar.bz2 binaryen-ee113c7e4cdff655c395598ee9f61512c3a5b3eb.zip | |
[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.
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/test/wasm2js.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/test/wasm2js.py b/scripts/test/wasm2js.py index 6463fa3a8..cbae2216a 100644 --- a/scripts/test/wasm2js.py +++ b/scripts/test/wasm2js.py @@ -41,9 +41,9 @@ def check_for_stale_files(): all_files = os.listdir(shared.get_test_dir('wasm2js')) for f in all_files: - if f in assert_tests: - continue prefix = f.split('.')[0] + if prefix in [t.split('.')[0] for t in assert_tests]: + continue if prefix not in all_tests: shared.fail_with_error('orphan test output: %s' % f) |