Skip to content

Commit

Permalink
Run turbopack integration tests for next.js changes too (#47608)
Browse files Browse the repository at this point in the history
### What?

Turbopack tests need to run on any change

### Why?

Next.js changes might break turbopack and we want to avoid that.

### How?

Change CI job

---------

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
  • Loading branch information
sokra and kodiakhq[bot] committed Mar 30, 2023
1 parent cbb8486 commit ebe0686
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions .github/workflows/build_test_deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1167,13 +1167,14 @@ jobs:
path: ./*
key: ${{ github.sha }}-${{ github.run_number }}

- run: echo "SWC_CHANGE<<EOF" >> $GITHUB_OUTPUT; echo "$(node scripts/run-for-change.js --type next-swc --exec echo 'yup')" >> $GITHUB_OUTPUT; echo 'EOF' >> $GITHUB_OUTPUT
id: swc-change
- name: Check non-docs only change
run: echo "DOCS_CHANGE<<EOF" >> $GITHUB_OUTPUT; echo "$(node scripts/run-for-change.js --type docs --exec echo 'yep')" >> $GITHUB_OUTPUT; echo 'EOF' >> $GITHUB_OUTPUT
id: docs-change

- run: echo "${{ steps.swc-change.outputs.SWC_CHANGE }}"
- run: echo "${{ steps.docs-chang.outputs.DOCS_CHANGE }}"

- name: Install
if: ${{ steps.swc-change.outputs.SWC_CHANGE == 'yup' }}
if: ${{ steps.docs-chang.outputs.DOCS_CHANGE != 'yep' }}
uses: actions-rs/toolchain@v1
with:
toolchain: ${{ env.RUST_TOOLCHAIN }}
Expand All @@ -1185,12 +1186,12 @@ jobs:
- name: Build tests
timeout-minutes: 60
run: cd packages/next-swc && cargo nextest run -p next-dev-tests --release --no-run
if: ${{ steps.swc-change.outputs.SWC_CHANGE == 'yup' }}
if: ${{ steps.docs-chang.outputs.DOCS_CHANGE != 'yep' }}

- name: Run tests
timeout-minutes: 20
run: cd packages/next-swc && cargo nextest run -p next-dev-tests --release --no-fail-fast
if: ${{ steps.swc-change.outputs.SWC_CHANGE == 'yup' }}
if: ${{ steps.docs-chang.outputs.DOCS_CHANGE != 'yep' }}

test-wasm:
name: Test the wasm build
Expand Down

0 comments on commit ebe0686

Please sign in to comment.