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

Wasm doc tests #2651

Merged
merged 12 commits into from
May 2, 2022
Merged

Wasm doc tests #2651

merged 12 commits into from
May 2, 2022

Conversation

ranile
Copy link
Member

@ranile ranile commented Apr 30, 2022

Description

Run doc tests in CI on wasm32-unknown-unknown target and on nightly toolchain

Fixes #2503

Checklist

  • I have reviewed my own code
  • I have added tests

@ranile ranile added the A-ci Area: The continuous integration label Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
@ranile
Copy link
Member Author

ranile commented Apr 30, 2022

I've added ci/wasm-bindgen-test-runner until rustwasm/wasm-bindgen#2881 is merged and released

See #2651 (comment)

@github-actions
Copy link

github-actions bot commented Apr 30, 2022

Visit the preview URL for this PR (updated for commit 57004dc):

https://yew-rs--pr2651-wasm-doc-tests-1oxkqqrv.web.app

(expires Sun, 08 May 2022 18:56:42 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
Comment on lines 85 to 101
with:
path: "./wasm-bindgen"
ref: respect-rustdoc-tmp-paths
repository: hamza1311/wasm-bindgen

- uses: Swatinem/rust-cache@v1

- uses: actions-rs/toolchain@v1
with:
toolchain: stable
target: wasm32-unknown-unknown
override: true
profile: minimal

- uses: Swatinem/rust-cache@v1
- run: |
cd wasm-bindgen
cargo build -p wasm-bindgen-cli --bin wasm-bindgen-test-runner --release
cp target/release/wasm-bindgen-test-runner ~/.cargo/bin/wasm-bindgen-test-runner
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the best solution I have until rustwasm/wasm-bindgen#2881 is merged and published. I tried building wasm-bindgen-test-runner on my local computer and putting it in ci directory but it won't run because it's not built with musl and I can't get a musl build to work locally

github-actions[bot]
github-actions bot previously approved these changes Apr 30, 2022
github-actions[bot]
github-actions bot previously approved these changes May 1, 2022
github-actions[bot]
github-actions bot previously approved these changes May 1, 2022
github-actions[bot]
github-actions bot previously approved these changes May 1, 2022
@ranile
Copy link
Member Author

ranile commented May 1, 2022

Why is the benchmark check failing?

@ranile ranile requested a review from WorldSEnder May 1, 2022 13:51
@WorldSEnder
Copy link
Member

Why is the benchmark check failing?

It seems that some tests are not being in a headless version of chrome. Could it have something to do with krausest/js-framework-benchmark@a92354a? At least I think that (re)enabled the tests we see failing.

github-actions[bot]
github-actions bot previously approved these changes May 1, 2022
@ranile
Copy link
Member Author

ranile commented May 1, 2022

Why is the benchmark check failing?

It seems that some tests are not being in a headless version of chrome. Could it have something to do with krausest/js-framework-benchmark@a92354a? At least I think that (re)enabled the tests we see failing.

If that's the case then it should be fixed when this PR is merged. I made it pull a specific commit until krausest/js-framework-benchmark#1032 is resolved

@ranile ranile mentioned this pull request May 1, 2022
@ranile ranile merged commit 8d9ecf2 into yewstack:master May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make website-test run with wasm32-unknown-unkown target
2 participants