Commit f7410440 authored by Joshua Gutow's avatar Joshua Gutow Committed by GitHub

Disable parallel op-e2e tests in CI (#3979)

Running them sequentially actually turns out to be faster. In addition it
turns out to be very hard to split the tests properly. `go list ./...` which
was the previous approach splits them by package, but we need to split them
by test name. Using `circleci tests glob "**/*.go"` did not work either
becaue it was combining files from different packages.
parent 3ee0a6cc
......@@ -426,7 +426,6 @@ jobs:
type: string
docker:
- image: cimg/go:1.19
parallelism: 6
resource_class: xlarge
steps:
- checkout
......@@ -436,9 +435,10 @@ jobs:
- run:
name: run tests
command: |
# Note: We don't use circle CI test splits because we need to split by test name, not by package. There is an additional
# constraint that gotestsum does not currently (nor likely will) accept files from different pacakges when building.
OP_TESTLOG_DISABLE_COLOR=true OP_E2E_DISABLE_PARALLEL=true OP_E2E_USE_HTTP=<<parameters.use_http>> gotestsum \
--format=standard-verbose --junitfile=/tmp/test-results/<<parameters.module>>_http_<<parameters.use_http>>.xml \
$(go list ./... | circleci tests split --split-by=timings)
--format=standard-verbose --junitfile=/tmp/test-results/<<parameters.module>>_http_<<parameters.use_http>>.xml ./...
working_directory: <<parameters.module>>
- store_test_results:
path: /tmp/test-results
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment