Skip to content

Commit

Permalink
feat: run the unit-style tests in parallel (#4985)
Browse files Browse the repository at this point in the history
Run go, node (jest) and tap tests all in their own jobs. This will allow us to
more easily upload test results, or parallelize the tap tests (which take the
most time).
  • Loading branch information
jbowes committed Jan 5, 2024
1 parent 0156d5b commit f3b705f
Showing 1 changed file with 35 additions and 5 deletions.
40 changes: 35 additions & 5 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,27 @@ workflows:
branches:
ignore: master

- test-code:
- test-node:
context:
- nodejs-install
- team_hammerhead-cli
requires:
- prepare-build
filters:
branches:
ignore: master

- test-go:
context:
- nodejs-install
- team_hammerhead-cli
requires:
- prepare-build
filters:
branches:
ignore: master

- test-legacy-tap:
context:
- nodejs-install
- team_hammerhead-cli
Expand Down Expand Up @@ -786,20 +806,30 @@ jobs:
monitor-on-build: true
severity-threshold: high

test-code:
test-node:
executor: docker-amd64
steps:
- prepare-workspace
- run:
name: Configuring artifact
command: node ./bin/snyk config set "api=${TEST_SNYK_TOKEN}" # many tests require the token to be in the config
- run:
name: Running TS unit tests
command: npm run test:unit

test-go:
executor: docker-amd64
steps:
- prepare-workspace
- run:
name: Running Go unit tests
working_directory: ./cliv2
command: make openboxtest

test-legacy-tap:
executor: docker-amd64
steps:
- prepare-workspace
- run:
name: Configuring artifact
command: node ./bin/snyk config set "api=${TEST_SNYK_TOKEN}" # many tests require the token to be in the config
- run:
name: Running Tap tests
command:
Expand Down

0 comments on commit f3b705f

Please sign in to comment.