Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial checkin of test scripts, copied from wasi-sdk. #397

Merged
merged 3 commits into from
Mar 25, 2021

Conversation

sunfishcode
Copy link
Member

This just copies in the test scripts from https://github.com/WebAssembly/wasi-sdk/tree/main/tests. We can evolve them from this point on.

@sunfishcode
Copy link
Member Author

This is a first draft at a concrete setup for the testing process described in #373. I copied in the same test script that #373 describes and set up a basic directory structure, which hopefully makes it clear where things will go in the repository.

@caspervonb
Copy link
Contributor

caspervonb commented Feb 24, 2021

Got some things I want to change here (filters being bash scripts in particular) but I'll follow up with that.

LGTM, good to get the ball finally rolling on this.

@sbc100
Copy link
Member

sbc100 commented Feb 24, 2021

Is the idea that wasi-sdk will then get its test runner from the WASI submodule (and we can then delete its copy).?

@sunfishcode
Copy link
Member Author

The wasi-sdk version of these scripts has support for running the tests with different optimization levels, and also compile-only tests, but I think what we can do is build those on top of the scripts here.

@sunfishcode
Copy link
Member Author

I'm going to take the liberty of merging this, so we can proceed to iterate on top of it. If there are further comments, please file issues in the issue tracker!

@sunfishcode sunfishcode merged commit 713727f into main Mar 25, 2021
@sunfishcode sunfishcode deleted the sunfishcode/test-harness branch March 25, 2021 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants