-
Notifications
You must be signed in to change notification settings - Fork 49
feat: wrappers test cases removed from toolchain #1476
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
Merged
Merged
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
c6c10d1
feat: wrappers fetched from wasm harness first iteration
cbrzn 6ab3586
Merge branch 'origin-dev' into feat/new-test-cases
cbrzn 47b6a82
chore(cli): workflow job runner tests updated
cbrzn e298bc7
chore(js/client): almost all core tests updated
cbrzn 96b914e
chore: update wasm and core client tests
cbrzn 73de2ea
chore: remove wrappers folder & ignore it in git
cbrzn 704c35b
chore: uri resolution & error structure tests updated
cbrzn 66e918e
chore: client test restructured
cbrzn 86fdae6
chore: job runner tests updated
cbrzn 7f246db
chore: fix rust ci
cbrzn 68db1ad
chore: solved conflicts with origin-dev
cbrzn 380b7df
chore: validate test fixed
cbrzn c325aa1
chore: add generate wrappers script in ci
cbrzn c3cef10
chore(wip): use releases and unzip to get wrappers instead of tag & c…
cbrzn 7e26561
chore: fetch and unzip wrappers implemented
cbrzn 75ea0e8
chore: update fetch wrappers script to pull the latest
cbrzn 290d5a9
chore: remove type module from test case package json
cbrzn c3ecaba
Merge branch 'origin-dev' into feat/new-test-cases
cbrzn 4125781
chore: update dev guidelines & remove unncessary changes from rs-ci
cbrzn dc1525d
update with latest from origin-dev
cbrzn 9c9566c
js/test-env: move deploy to one function and use it from different tests
cbrzn 492dce6
chore: fix lint & fs resolver test
cbrzn 514b356
Merge branch 'origin-dev' into feat/new-test-cases
cbrzn 1e74e05
chore: fix http test
cbrzn 943449c
chore: update with latest & update readmes of uri resolver plugins
cbrzn 07e5e7f
js/test-env: fix lint
cbrzn fd59032
ci: updated with generate wrappers reusable workflow & update dev gui…
cbrzn 5b008af
ci: add generation of modified tests wrappers reusable workflow
cbrzn 026067f
ci: add dependency of generate-test-wrappers in rust
cbrzn aac2d63
test-cases: make fetch wrappers async & fix typo in dev guidelines
cbrzn 4fd6c54
chore: fix conflicts
cbrzn 1ce344c
client/tests: invoke wrapper of compatible version almost works
cbrzn 0d6b424
client/tests: incompatible version wrapper invocation
cbrzn 5744fe2
client/tests: incompatible version plugin invocation
cbrzn b7bf152
merge: origin-dev
dOrgJelli 30e911b
update with origin-dev
cbrzn a6945a7
test-cases/fetch-wrappers: update with latest asset release
cbrzn 8b63ea7
ci(wrapper-generation-script): use head branch instead of base
cbrzn 91cca19
chore: fix conflicts & test
cbrzn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
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
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
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 |
|---|---|---|
| @@ -0,0 +1,91 @@ | ||
| name: generate-wrappers | ||
|
|
||
| on: | ||
| workflow_call: | ||
| inputs: | ||
| cli-path: | ||
| required: false | ||
| type: string | ||
| wasm-path: | ||
| required: false | ||
| type: string | ||
|
|
||
| jobs: | ||
| generate_wrappers: | ||
| name: Generate wrappers | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout Repository | ||
| uses: actions/checkout@v2 | ||
| with: | ||
| repository: polywrap/wasm-test-harness | ||
| ref: ${{ github.event.pull_request.head.ref }} | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Checkout Repository | ||
| uses: actions/checkout@v2 | ||
| with: | ||
| ref: ${{ github.event.pull_request.head.ref }} | ||
| path: toolchain | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Read .nvmrc | ||
| run: echo ::set-output name=NVMRC::$(cat .nvmrc) | ||
| id: nvm | ||
| working-directory: ./toolchain | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@master | ||
| with: | ||
| node-version: '${{ steps.nvm.outputs.NVMRC }}' | ||
|
|
||
| - name: Get yarn cache directory path | ||
| id: yarn-cache-dir-path | ||
| run: echo "::set-output name=dir::$(yarn cache dir)" | ||
|
|
||
| - uses: actions/cache@v2 | ||
| with: | ||
| path: ${{ steps.yarn-cache-dir-path.outputs.dir }} | ||
| key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-yarn- | ||
| - name: Install dependencies | ||
| run: yarn install --nonInteractive --frozen-lockfile --prefer-offline | ||
| working-directory: ./toolchain | ||
|
|
||
| - name: Build toolchain | ||
| run: yarn build | ||
| working-directory: ./toolchain | ||
|
|
||
| - name: Install Rust | ||
| uses: actions-rs/toolchain@v1 | ||
| with: | ||
| toolchain: nightly | ||
| override: true | ||
|
|
||
| - uses: actions/cache@v2 | ||
| with: | ||
| path: | | ||
| ~/.cargo/registry/index/ | ||
| ~/.cargo/registry/cache/ | ||
| ~/.cargo/git/ | ||
| ./target/ | ||
| key: cargo-${{ runner.os }}-${{ github.job }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-keys: | | ||
| cargo-${{ runner.os }}-${{ github.job }} | ||
|
|
||
| - name: Set environment variable | ||
| if: ${{ inputs.cli-path }} | ||
| run: echo "POLYWRAP_CLI_PATH=${{ github.workspace }}/${{ inputs.cli-path }}" >> $GITHUB_ENV | ||
|
|
||
| - name: Set environment variable | ||
| if: ${{ inputs.wasm-path }} | ||
| run: echo "POLYWRAP_WASM_PATH=${{ github.workspace }}/${{ inputs.wasm-path }}" >> $GITHUB_ENV | ||
|
|
||
| - name: Run wrappers build | ||
| run: cargo run -- -w | ||
|
|
||
| - uses: actions/upload-artifact@v3 | ||
| with: | ||
| name: wrappers | ||
| path: ./wrappers | ||
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 |
|---|---|---|
|
|
@@ -17,3 +17,4 @@ bin | |
| pkg | ||
| wasm-pack.log | ||
| .env | ||
| packages/test-cases/cases/wrappers | ||
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
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
Oops, something went wrong.
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.