From 3bf947d7ef8f1b2855d7dca91156a61694792bf5 Mon Sep 17 00:00:00 2001 From: Eugene Boguslavsky Date: Thu, 24 Oct 2024 14:41:48 -0700 Subject: [PATCH] Move IDE tests into it's own workflow file (#20019) ## Description Move IDE tests into it's own workflow file ## Test plan https://github.com/MystenLabs/sui/actions/runs/11507121245/job/32032582303?pr=20019 --- .github/workflows/e2e.yml | 2 +- .github/workflows/ide-tests.yml | 91 +++++++++++++++++++++++++++++++++ .github/workflows/rust.yml | 62 ---------------------- 3 files changed, 92 insertions(+), 63 deletions(-) create mode 100644 .github/workflows/ide-tests.yml diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml index 57b43d7609204..7879882dd7400 100644 --- a/.github/workflows/e2e.yml +++ b/.github/workflows/e2e.yml @@ -70,7 +70,7 @@ jobs: if: ${{ needs.diff.outputs.isTypescriptSDK == 'true' || needs.diff.outputs.isRust == 'true'}} run: pnpm dlx concurrently --kill-others --success command-1 "$E2E_RUN_LOCAL_NET_CMD" 'pnpm --filter @mysten/sui test:e2e' - - name: Run RPC/GrpahQL comaptability e2e tests + - name: Run RPC/GraphQL compatibility e2e tests 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' diff --git a/.github/workflows/ide-tests.yml b/.github/workflows/ide-tests.yml new file mode 100644 index 0000000000000..ecb0e2a7435e8 --- /dev/null +++ b/.github/workflows/ide-tests.yml @@ -0,0 +1,91 @@ +name: IDE Tests + +on: + push: + branches: main + pull_request: + types: [ opened, synchronize, reopened, ready_for_review ] + workflow_dispatch: + inputs: + sui_repo_ref: + description: "Branch / commit to test" + type: string + required: false + default: '' + +jobs: + diff: + runs-on: [ubuntu-latest] + outputs: + isMoveAutoFormatter: ${{ steps.diff.outputs.isMoveAutoFormatter }} + isMoveAnalyzerTraceAdapter: ${{ steps.diff.outputs.isMoveAnalyzerTraceAdapter }} + steps: + - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 + - name: Detect Changes + uses: './.github/actions/diffs' + id: diff + + move-auto-formatter-ci-test: + name: Move Auto-formatter Test + needs: diff + if: needs.diff.outputs.isMoveAutoFormatter == 'true' + runs-on: [ ubuntu-latest ] + + steps: + - name: Checkout + uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 + with: + ref: ${{ github.event.inputs.sui_repo_ref || github.ref }} + + - name: pnpm setup + uses: pnpm/action-setup@a3252b78c470c02df07e9d59298aecedc3ccdd6d # pin@v3.0.0 + with: + version: 9.1.1 + + - name: Setup Node + uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # pin@v4.0.2 + with: + node-version: '20' + cache: 'pnpm' + + - name: Install dependencies + working-directory: ./external-crates/move/crates/move-analyzer/prettier-plugin + run: npm install && npm i web-tree-sitter + + - name: Run npm test + working-directory: ./external-crates/move/crates/move-analyzer/prettier-plugin + shell: bash + run: npm run test + + move-analyzer-trace-adapter-ci-test: + name: Trace Adapter Test + needs: diff + if: needs.diff.outputs.isMoveAnalyzerTraceAdapter == 'true' + runs-on: [ ubuntu-latest ] + + steps: + - name: Checkout + uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 + with: + ref: ${{ github.event.inputs.sui_repo_ref || github.ref }} + + - name: pnpm setup + uses: pnpm/action-setup@a3252b78c470c02df07e9d59298aecedc3ccdd6d # pin@v3.0.0 + with: + version: 9.1.1 + + - name: Setup Node + uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # pin@v4.0.2 + with: + node-version: '20' + cache: 'pnpm' + + - name: Install dependencies + working-directory: ./external-crates/move/crates/move-analyzer/trace-adapter + shell: bash + run: npm install + + - name: Run npm test + working-directory: ./external-crates/move/crates/move-analyzer/trace-adapter + shell: bash + run: npm run test diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index 2f85c9eb0fa5c..79e66e95a7e2a 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -55,8 +55,6 @@ jobs: outputs: isRust: ${{ steps.diff.outputs.isRust }} isMove: ${{ steps.diff.outputs.isMove }} - isMoveAutoFormatter: ${{ steps.diff.outputs.isMoveAutoFormatter }} - isMoveAnalyzerTraceAdapter: ${{ steps.diff.outputs.isMoveAnalyzerTraceAdapter }} steps: - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 with: @@ -74,66 +72,6 @@ jobs: ref: ${{ github.event.inputs.sui_repo_ref || github.ref }} - run: cargo xlint - move-auto-formatter-ci-test: - needs: diff - if: needs.diff.outputs.isMoveAutoFormatter == 'true' - runs-on: [ ubuntu-latest ] - strategy: - matrix: - node-version: [ 20 ] - steps: - - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 - with: - ref: ${{ github.event.inputs.sui_repo_ref || github.ref }} - - uses: pnpm/action-setup@a3252b78c470c02df07e9d59298aecedc3ccdd6d # pin@v3.0.0 - with: - version: 9.1.1 - - name: Setup Node - uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # pin@v4.0.2 - with: - node-version: '20.x' - cache: 'pnpm' - - name: Install dependencies - working-directory: ./external-crates/move/crates/move-analyzer/prettier-plugin - run: npm install && npm i web-tree-sitter - - name: Run npm test - working-directory: ./external-crates/move/crates/move-analyzer/prettier-plugin - shell: bash - run: npm run test - - move-analyzer-trace-adapter-ci-test: - name: Move Analyzer Trace Adapter Test - needs: diff - if: needs.diff.outputs.isMoveAnalyzerTraceAdapter == 'true' - runs-on: [ ubuntu-latest ] - strategy: - matrix: - node-version: [ 20 ] - steps: - - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # Pin v4.1.1 - with: - ref: ${{ github.event.inputs.sui_repo_ref || github.ref }} - - - uses: pnpm/action-setup@a3252b78c470c02df07e9d59298aecedc3ccdd6d # pin@v3.0.0 - with: - version: 9.1.1 - - - name: Setup Node - uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # pin@v4.0.2 - with: - node-version: '20.x' - cache: 'pnpm' - - - name: Install dependencies - working-directory: ./external-crates/move/crates/move-analyzer/trace-adapter - shell: bash - run: npm install - - - name: Run npm test - working-directory: ./external-crates/move/crates/move-analyzer/trace-adapter - shell: bash - run: npm run test - test: needs: diff if: needs.diff.outputs.isRust == 'true'