This repository was archived by the owner on Apr 9, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 17
chore: add ci for acvmjs rust test #516
Merged
Merged
Changes from 19 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
c04d4ab
Add script for cargo test
93c7438
Add test workflow
9bec54e
Fix ci
ede7291
Fix ci
ba42dcd
Add wasm32 target
0681b14
Fix ci
f04a67b
Add cargo test on nix flake
4d894c2
Remove test from flake, remove comment
4051c88
Replace rust set up
25bbc28
Comment out rust test
7723809
Edit matrix
273b73c
Update toolchain
c357700
Add wasm32 target
7c06d9d
Remove hardcoded wasm-bindgen-cli version
4adeca1
Readd cargo install
d544348
Run test in parallel with build-acvm-js-package
a72cd16
Merge branch 'master' into jb/ci-test
5924ca6
Update .github/workflows/test.yml
TomAFrench d00e0ed
Update acvm_js/package.json
TomAFrench dfc622a
Update .github/workflows/test.yml
TomAFrench 7fcc496
Update .github/workflows/test.yml
TomAFrench File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -91,3 +91,29 @@ jobs: | |
| - name: Run browser tests | ||
| working-directory: ./acvm_js | ||
| run: yarn test:browser | ||
|
|
||
| test-acvm_js-rust: | ||
| name: Rust Tests | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: | ||
| matrix: | ||
| include: | ||
| - os: ubuntu | ||
| runner: ubuntu-latest | ||
| target: wasm32-unknown-unknown | ||
|
|
||
| steps: | ||
| - name: Checkout sources | ||
| uses: actions/checkout@v3 | ||
|
|
||
| - name: Setup toolchain | ||
| uses: dtolnay/[email protected] | ||
| with: | ||
| targets: ${{ matrix.target }} | ||
|
|
||
| - name: Add wasm32 target | ||
TomAFrench marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| run: cargo install wasm-bindgen-cli --vers "0.2.87" --locked | ||
|
|
||
| - name: Run rust tests | ||
| run: cargo test | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.