Skip to content

Commit

Permalink
test: basic run, with index update
Browse files Browse the repository at this point in the history
  • Loading branch information
erikburt committed Nov 28, 2024
1 parent ea15888 commit 1de5dfc
Showing 1 changed file with 32 additions and 7 deletions.
39 changes: 32 additions & 7 deletions .github/workflows/ci-core-partial.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,40 @@
name: Core Unit Tests

# Run on key branches to make sure integration is good, otherwise run on all PR's
on:
push:
branches:
- develop
- main
- "release/*"
merge_group:
pull_request:

jobs:

filter:
name: Detect Changes
permissions:
pull-requests: read
outputs:
should-run-all-tests: ${{ steps.match-some.outputs.test-data == 'true' }}
runs-on: ubuntu-latest
steps:
- name: Checkout the repo
uses: actions/[email protected]
with:
repository: smartcontractkit/chainlink
- uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
id: match-some
with:
# "if any changed file matches one or more of the conditions" (https://github.com/dorny/paths-filter/issues/225)
predicate-quantifier: some
# test-data - any changes to any testdata files/paths
filters: |
test-data:
- '**/testdata/**'
run-unit-tests:
name: Tests (${{ matrix.type.test-suite }})
needs: filter
runs-on: ubuntu22.04-32cores-128GB

Check failure on line 38 in .github/workflows/ci-core-partial.yml

View workflow job for this annotation

GitHub Actions / Validate Workflow Changes

1. This Ubuntu runner is 8-16 more expensive than a base Ubuntu runner. Consider using a smaller Ubuntu runner. (runner-ubuntu / error)
permissions:
id-token: write
Expand All @@ -30,6 +52,7 @@ jobs:
# We explicitly have this env var not be "CL_DATABASE_URL" to avoid having it be used by core related tests
# when they should not be using it, while still allowing us to DRY up the setup
DB_URL: postgresql://postgres:postgres@localhost:5432/chainlink_test?sslmode=disable
collect-coverage: ${{ github.event_name == 'pull_request' }}
steps:
- name: Checkout the repo
uses: actions/[email protected]
Expand All @@ -44,25 +67,26 @@ jobs:

- name: Build Tests
uses: smartcontractkit/.github/apps/go-test-caching@feat/go-test-binary-comparison
timeout-minutes: 20
timeout-minutes: 10
with:
pipeline-step: "build"
build-concurrency: "32"
collect-coverage: "true"
collect-coverage: ${{ env.collect-coverage }}
test-suite: ${{ matrix.type.test-suite }}
module-directory: ${{ matrix.type.module-directory }}
github-token: ${{ secrets.GITHUB_TOKEN }}
build-flags: ${{ matrix.type.build-flags }}

- name: Run Tests
uses: smartcontractkit/.github/apps/go-test-caching@feat/go-test-binary-comparison
timeout-minutes: 20
timeout-minutes: 15
env:
CL_DATABASE_URL: ${{ env.DB_URL }}
with:
pipeline-step: "run"
run-concurrency: "16"
collect-coverage: "true"
run-concurrency: "24"
run-all-tests: ${{ needs.filter.outputs.should-run-all-tests }}
collect-coverage: ${{ env.collect-coverage }}
test-suite: ${{ matrix.type.test-suite }}
module-directory: ${{ matrix.type.module-directory }}
github-token: ${{ secrets.GITHUB_TOKEN }}
Expand All @@ -72,6 +96,7 @@ jobs:
with:
pipeline-step: "update"
force-update-index: "true"
collect-coverage: ${{ env.collect-coverage }}
test-suite: ${{ matrix.type.test-suite }}
github-token: ${{ secrets.GITHUB_TOKEN }}

Expand Down

0 comments on commit 1de5dfc

Please sign in to comment.