Skip to content

Commit

Permalink
chore: Separated running versioned tests between local and external (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
bizob2828 committed Oct 3, 2023
1 parent a748b84 commit 6ffe2f6
Show file tree
Hide file tree
Showing 5 changed files with 189 additions and 134 deletions.
29 changes: 26 additions & 3 deletions .github/workflows/ci-workflow.yml
Expand Up @@ -112,7 +112,7 @@ jobs:
name: integration-tests-${{ matrix.node-version }}
path: ./coverage/integration/lcov.info

versioned:
versioned-internal:
needs: skip_if_release
if: needs.skip_if_release.outputs.should_skip != 'true'
runs-on: ubuntu-latest
Expand All @@ -132,7 +132,7 @@ jobs:
- name: Run Docker Services
run: npm run services
- name: Run Versioned Tests
run: TEST_CHILD_TIMEOUT=600000 npm run versioned
run: TEST_CHILD_TIMEOUT=600000 npm run versioned:internal
env:
VERSIONED_MODE: ${{ github.ref == 'refs/heads/main' && '--minor' || '--major' }}
JOBS: 4 # 2 per CPU seems to be the sweet spot in GHA (July 2022)
Expand All @@ -142,9 +142,32 @@ jobs:
with:
name: versioned-tests-${{ matrix.node-version }}
path: ./coverage/versioned/lcov.info

# There is no coverage for external as that's tracked in their respective repos
versioned-external:
needs: skip_if_release
if: needs.skip_if_release.outputs.should_skip != 'true'
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [16.x, 18.x, 20.x]

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
- name: Install Dependencies
run: npm ci
- name: Run Versioned Tests
run: TEST_CHILD_TIMEOUT=600000 npm run versioned:external
env:
VERSIONED_MODE: ${{ github.ref == 'refs/heads/main' && '--minor' || '--major' }}
JOBS: 4 # 2 per CPU seems to be the sweet spot in GHA (July 2022)
codecov:
needs: [unit, integration, versioned]
needs: [unit, integration, versioned-internal]
runs-on: ubuntu-latest

strategy:
Expand Down

0 comments on commit 6ffe2f6

Please sign in to comment.