diff options
author | Heejin Ahn <aheejin@gmail.com> | 2023-12-11 17:40:22 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-11 17:40:22 -0800 |
commit | 9e07c8266edcc77624748032ae1186953f6242df (patch) | |
tree | 58c27e64fb2dad131fba1f15a59f242f17962e42 /test/wasm2js | |
parent | cfb2a6e4ae7d1aefa53963b9249abadb6f756784 (diff) | |
download | binaryen-9e07c8266edcc77624748032ae1186953f6242df.tar.gz binaryen-9e07c8266edcc77624748032ae1186953f6242df.tar.bz2 binaryen-9e07c8266edcc77624748032ae1186953f6242df.zip |
[test] Make get_tests return only files (#6164)
Currently `get_tests` returns files and directories, especially when
the extension is not given. This makes `get_tests` return a directory
like `test/wasm2js/` as a test.
`wasm2js.py`'s `check_for_stale_files` errors out when there are files
within `test/wasm2js/` whose basenames don't match any files within any
of `test/`, `test/spec/`, `test/wasm2js/`.
https://github.com/WebAssembly/binaryen/blob/1d615b38dd4152494d2f4d3520c8b1d917624a30/scripts/test/wasm2js.py#L33-L46
`wasm2js.wast.asserts` is apparently a special case for asserts test:
https://github.com/WebAssembly/binaryen/blob/1d615b38dd4152494d2f4d3520c8b1d917624a30/scripts/test/wasm2js.py#L28
and this doesn't seem to have the matching `wast` tests in the three
test directories. But it just happened to not error out because
`get_tests` returns directory names too and one of them was `wasm2js`
(`test/wasm2js/` directory).
This makes `get_tests` return only files, and make files in
`assert_tests` not error out additionally.
Diffstat (limited to 'test/wasm2js')
0 files changed, 0 insertions, 0 deletions