Skip to content

Commit

Permalink
Merge pull request #5141 from onflow/petera/cleanup-integration-make-…
Browse files Browse the repository at this point in the history
…targets

[CI] Cleanup integration make targets
  • Loading branch information
peterargue authored Dec 13, 2023
2 parents ba6f0ff + 2c5c8e7 commit 88e8600
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ ci: install-tools test
# Runs integration tests
.PHONY: ci-integration
ci-integration:
$(MAKE) -C integration ci-integration-test
$(MAKE) -C integration integration-test

# Runs benchmark tests
# NOTE: we do not need `docker-build-flow` as this is run as a separate step
Expand Down
12 changes: 7 additions & 5 deletions integration/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,16 @@ CGO_FLAG := CGO_CFLAGS=$(CRYPTO_FLAG)

# Run the integration test suite
.PHONY: integration-test
integration-test: access-tests ghost-tests mvp-tests execution-tests verification-tests upgrades-tests collection-tests epochs-cohort1-tests epochs-cohort2-tests network-tests consensus-tests

.PHONY: ci-integration-test
ci-integration-test: access-tests ghost-tests mvp-tests epochs-cohort1-tests epochs-cohort2-tests consensus-tests execution-tests verification-tests upgrades-tests network-tests collection-tests
integration-test: access-tests ghost-tests mvp-tests execution-tests verification-tests upgrades-tests collection-tests epochs-tests network-tests consensus-tests

# Run unit tests for test utilities in this module
.PHONY: test
test:
$(CGO_FLAG) go test $(if $(VERBOSE),-v,) -coverprofile=$(COVER_PROFILE) $(RACE_FLAG) $(if $(JSON_OUTPUT),-json,) $(if $(NUM_RUNS),-count $(NUM_RUNS),) `go list ./... | grep -v -e integration/tests`

.PHONY: access-tests
access-tests: access-cohort1-tests access-cohort2-tests access-cohort3-tests

.PHONY: access-cohort1-tests
access-cohort1-tests:
$(CGO_FLAG) go test $(if $(VERBOSE),-v,) $(RACE_FLAG) $(if $(JSON_OUTPUT),-json,) $(if $(NUM_RUNS),-count $(NUM_RUNS),) ./tests/access/cohort1/...
Expand All @@ -36,7 +36,6 @@ access-cohort2-tests:
access-cohort3-tests:
$(CGO_FLAG) go test $(if $(VERBOSE),-v,) $(RACE_FLAG) $(if $(JSON_OUTPUT),-json,) $(if $(NUM_RUNS),-count $(NUM_RUNS),) ./tests/access/cohort3/...


.PHONY: collection-tests
collection-tests:
$(CGO_FLAG) go test $(if $(VERBOSE),-v,) $(RACE_FLAG) $(if $(JSON_OUTPUT),-json,) $(if $(NUM_RUNS),-count $(NUM_RUNS),) ./tests/collection/...
Expand All @@ -45,6 +44,9 @@ collection-tests:
consensus-tests:
$(CGO_FLAG) go test $(if $(VERBOSE),-v,) $(RACE_FLAG) $(if $(JSON_OUTPUT),-json,) $(if $(NUM_RUNS),-count $(NUM_RUNS),) ./tests/consensus/...

.PHONY: epochs-tests
epochs-tests: epochs-cohort1-tests epochs-cohort2-tests

.PHONY: epochs-cohort1-tests
epochs-cohort1-tests:
# Use a higher timeout of 20m for the suite of tests which span full epochs
Expand Down

0 comments on commit 88e8600

Please sign in to comment.