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

Add tests for assert, abort, and environ. #105

Merged
merged 1 commit into from
Mar 11, 2020
Merged

Add tests for assert, abort, and environ. #105

merged 1 commit into from
Mar 11, 2020

Conversation

sunfishcode
Copy link
Member

To support this, add a mechanism for filtering test output to filter
out uninteresting diffs.

This adds some coverage for the code being changed in
WebAssembly/wasi-libc#184.

To support this, add a mechanism for filtering test output to filter
out uninteresting diffs.

This adds some coverage for the code being changed in
WebAssembly/wasi-libc#184.
Copy link
Collaborator

@pchickey pchickey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful, thank you for adding these

@sunfishcode sunfishcode merged commit b80771e into master Mar 11, 2020
@sunfishcode sunfishcode deleted the more-tests branch March 11, 2020 20:13
Caused by:
0: failed to invoke `_start`
1: wasm trap: unreachable, source location: @----
wasm backtrace:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this output coming from? Is it coming from some wasmtime-internal place?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. If other engines have a different format, we can adjust the filter script as needed.

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.

None yet

3 participants