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

Introduce a testing framework for Wasmer Pack #112

Merged
merged 14 commits into from
Dec 12, 2022

Conversation

Michael-F-Bryan
Copy link
Contributor

@Michael-F-Bryan Michael-F-Bryan commented Dec 8, 2022

Description

This PR introduces a wasmer-pack-testing crate that people can use when writing integration tests for bindings generated by wasmer-pack. At the moment, support for Python is implemented, and we'll implement the JavaScript tests in a future PR.

CC #87.

Context

At the moment, creating and running integration tests for Wasmer Pack is quite cumbersome because it relies on several external tools (cargo wapm, yarn, python, pip, etc.).

Wasmer Pack also has integration tests which run the real wasmer/wasmer-pack and wasmer/wabt packages. This is great because we're testing real-world packages, but it's also unhelpful because those integration tests are testing real-world packages which don't necessarily include every little feature of WAI/Wasmer Pack that we'd like to test.

Checklist

@Michael-F-Bryan Michael-F-Bryan added category: feature New feature or request priority: high area: javascript JavaScript code generation area: python Python code generation labels Dec 8, 2022
@Michael-F-Bryan Michael-F-Bryan marked this pull request as ready for review December 12, 2022 08:49
@Michael-F-Bryan Michael-F-Bryan merged commit 8c20b86 into master Dec 12, 2022
@Michael-F-Bryan Michael-F-Bryan deleted the integration-test-framework branch December 12, 2022 09:18
@github-actions
Copy link

⏱ Workflow Timer ⏱

Make sure you keep an eye on build times!

One of this project's goals is to keep CI runs under 5 minutes so developers can maintain fast edit-compile-test cycles.

Run Compile and Test
master 3mins 3s
4ee2ea8 7mins 21s

🤖 Beep. Boop. I'm a bot. If you find any issues, please report them to https://github.com/Michael-F-Bryan/workflow-timer.

@Michael-F-Bryan
Copy link
Contributor Author

After talking with @dynamite-bud, we're thinking we might merge this PR as-is and follow up with PRs that implement JavaScript testing and refactor the workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: javascript JavaScript code generation area: python Python code generation category: feature New feature or request priority: high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants