spec icon indicating copy to clipboard operation
spec copied to clipboard

build.py doesn't build tests in subdirectories

Open foolip opened this issue 2 years ago • 5 comments

The tests in https://github.com/WebAssembly/gc/tree/main/test/core/gc aren't build by the build.py, which only looks for *.wast files in test/core/:

https://github.com/WebAssembly/gc/blob/ec57b2a506a7e220a1cdb980adc7fdcca8b31d05/test/build.py#L70

Are these tests not being run yet, or should they be moved to the parent directory when they are complete?

foolip avatar Oct 06 '23 14:10 foolip

This can be worked around by updating WAST_TESTS_DIR. Looking for tests recursively and generating a directory tree should be straightforward, if that is the desired behavior.

foolip avatar Oct 06 '23 14:10 foolip

That seems to be an oversight. It should probably have been changed in the same way as test/run.py back when SIMD tests where written or merged. I don't know who owns or uses this script.

@eqrion, @Ms2ger, @gahaas, you seem to have touched it last. Any ideas?

[Transferring issue to upstream repo.]

rossberg avatar Oct 06 '23 14:10 rossberg

That seems to be an oversight. It should probably have been changed in the same way as test/run.py back when SIMD tests where written or merged. I don't know who owns or uses this script.

@eqrion, @Ms2ger, @gahaas, you seem to have touched it last. Any ideas?

rossberg avatar Oct 06 '23 14:10 rossberg

I think I touched it just to tweak error reporting. We don't use this script anymore though and have a custom way we run the spec tests in SM.

eqrion avatar Oct 06 '23 15:10 eqrion

We are also not using this script in V8. The script should be easy to fix, though, and I think there is value in having this script. I put it on my TODO list to fix it eventually.

gahaas avatar Oct 10 '23 08:10 gahaas