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

Fallible Wasm Module for testing #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fallible Wasm Module for testing #7

wants to merge 2 commits into from

Conversation

jmwample
Copy link
Member

Adding a wasm module that takes in a config indicating one of several internal failures that an implemented web assembly binary might encounter. This will be useful in integration tests for all different client languages as the internal failure modes that the client must gracefully handle are consistent.

At the same time this PR may make changes to the internal interfaces and layout to make the use of sync / async WebAssembly Modules more modular and the independent errors clearer.

In general we need to start drawing a more clear line around what we provide for in the wasm library and what developers should write. This can include varying degrees of involvement from Encode / Decode /Config all the way down to the raw WATER API, and we can provide trait boundaries and wrappers at the various levels of abstraction.

@jmwample jmwample added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 24, 2023
@jmwample
Copy link
Member Author

This is likely dependent on #8 to allow for configurable failures in the places that I would like them to be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant