Skip to content

Commit

Permalink
Disable wallet tests (#18498)
Browse files Browse the repository at this point in the history
## Description 

Describe the changes or additions included in this PR.

## Test plan 

How did you test the new or updated feature?

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] Indexer: 
- [ ] JSON-RPC: 
- [ ] GraphQL: 
- [ ] CLI: 
- [ ] Rust SDK:
  • Loading branch information
Jordan-Mysten authored Jul 2, 2024
1 parent 79f22bc commit 8d0562b
Showing 1 changed file with 20 additions and 22 deletions.
42 changes: 20 additions & 22 deletions .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,16 +50,16 @@ jobs:
- uses: pnpm/action-setup@a3252b78c470c02df07e9d59298aecedc3ccdd6d # [email protected]
with:
version: 9.1.1
- run: cargo build --bin sui --features indexer --profile dev
- run: cargo build --bin sui --features indexer --profile dev
- name: Install Nodejs
uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # [email protected]
with:
node-version: '18'
cache: 'pnpm'
- name: Install dependencies
run: pnpm install --frozen-lockfile
- name: Install Playwright Browsers
run: pnpm --filter sui-wallet playwright install --with-deps chromium
# - name: Install Playwright Browsers
# run: pnpm --filter sui-wallet playwright install --with-deps chromium

- name: Set env
run: |
Expand All @@ -74,29 +74,28 @@ jobs:
if: ${{ needs.diff.outputs.isGraphQlTransport == 'true' || needs.diff.outputs.isRust == 'true'}}
run: pnpm dlx concurrently --kill-others --success command-1 "$E2E_RUN_LOCAL_NET_CMD" 'pnpm --filter @mysten/graphql-transport test:e2e'

- name: Run Local net
run: cargo run --bin sui -- start --with-faucet --force-regenesis --epoch-duration-ms 10000 &
- name: Set Wallet env
run: echo "API_ENV=local" > "$PWD/apps/wallet/configs/environment/.env"
- name: Build Wallet
# need to run Wallet e2e when its upstream(TS SDK and Rust) or itself is changed
if: ${{ needs.diff.outputs.isWallet == 'true' || needs.diff.outputs.isRust == 'true' || needs.diff.outputs.isTypescriptSDK == 'true'}}
run: pnpm wallet build
- name: Run Wallet e2e tests
if: ${{ needs.diff.outputs.isWallet == 'true' || needs.diff.outputs.isRust == 'true' || needs.diff.outputs.isTypescriptSDK == 'true'}}
run: xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- pnpm --filter sui-wallet playwright test
- uses: actions/upload-artifact@v3
if: always()
with:
name: playwright-report-wallet
path: apps/wallet/playwright-report/
retention-days: 30
# - name: Run Local net
# run: cargo run --bin sui -- start --with-faucet --force-regenesis --epoch-duration-ms 10000 &
# - name: Set Wallet env
# run: echo "API_ENV=local" > "$PWD/apps/wallet/configs/environment/.env"
# - name: Build Wallet
# # need to run Wallet e2e when its upstream(TS SDK and Rust) or itself is changed
# if: ${{ needs.diff.outputs.isWallet == 'true' || needs.diff.outputs.isRust == 'true' || needs.diff.outputs.isTypescriptSDK == 'true'}}
# run: pnpm wallet build
# - name: Run Wallet e2e tests
# if: ${{ needs.diff.outputs.isWallet == 'true' || needs.diff.outputs.isRust == 'true' || needs.diff.outputs.isTypescriptSDK == 'true'}}
# run: xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- pnpm --filter sui-wallet playwright test
# - uses: actions/upload-artifact@v3
# if: always()
# with:
# name: playwright-report-wallet
# path: apps/wallet/playwright-report/
# retention-days: 30

# Run e2e test against localnet built on the devnet branch for backward compatibility check
local_devnet_branch:
name: Local Network Built on devnet branch
needs: diff
# TODO: add wallet e2e to the `if` condition when available
if: needs.diff.outputs.isTypescriptSDK == 'true'
runs-on: ubuntu-ghcloud
services:
Expand Down Expand Up @@ -125,7 +124,6 @@ jobs:
local_testnet_branch:
name: Local Network Built on testnet branch
needs: diff
# TODO: add wallet e2e to the `if` condition when available
if: needs.diff.outputs.isTypescriptSDK == 'true'
runs-on: ubuntu-ghcloud
services:
Expand Down

0 comments on commit 8d0562b

Please sign in to comment.