Skip to content

Commit

Permalink
Remove option happy-path-only, short-happy-path
Browse files Browse the repository at this point in the history
  • Loading branch information
bermuell committed Aug 9, 2023
1 parent 2f9abd0 commit b28e1d5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 21 deletions.
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ test-diff:

# run only happy path E2E tests
test-e2e-short:
go run ./tests/e2e/... --happy-path-only
go run ./tests/e2e/... --tc happy-path

# run full E2E tests in sequence (including multiconsumer)
test-e2e-multi-consumer:
Expand All @@ -47,7 +47,7 @@ test-gaia-e2e:

# run only happy path E2E tests using latest tagged gaia
test-gaia-e2e-short:
go run ./tests/e2e/... --happy-path-only --use-gaia
go run ./tests/e2e/... --tc happy-path --use-gaia

# run full E2E tests in parallel (including multiconsumer) using latest tagged gaia
test-gaia-e2e-parallel:
Expand All @@ -61,7 +61,7 @@ test-gaia-e2e-tagged:
# run only happy path E2E tests using latest tagged gaia
# usage: GAIA_TAG=v9.0.0 make test-gaia-e2e-short-tagged
test-gaia-e2e-short-tagged:
go run ./tests/e2e/... --happy-path-only --use-gaia --gaia-tag $(GAIA_TAG)
go run ./tests/e2e/... --tc happy-path --use-gaia --gaia-tag $(GAIA_TAG)

# run full E2E tests in parallel (including multiconsumer) using specific tagged version of gaia
# usage: GAIA_TAG=v9.0.0 make test-gaia-e2e-parallel-tagged
Expand Down
19 changes: 1 addition & 18 deletions tests/e2e/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,7 @@ func (t *TestSet) String() string {
}

var (
verbose = flag.Bool("verbose", false, "turn verbose logging on/off")
happyPathOnly = flag.Bool("happy-path-only", false, "run happy path tests only")
shortHappyPathOnly = flag.Bool("short-happy-path", false, `run abridged happy path tests only.
This is like the happy path, but skips steps
that involve starting or stopping nodes for the same chain outside of the chain setup or teardown.
In particular, this skips steps related to downtime and double signing.
This is suited for CometMock+Gorelayer testing`)
verbose = flag.Bool("verbose", false, "turn verbose logging on/off")
includeMultiConsumer = flag.Bool("include-multi-consumer", false, "include multiconsumer tests in run")
parallel = flag.Bool("parallel", false, "run all tests in parallel")
localSdkPath = flag.String("local-sdk-path", "",
Expand Down Expand Up @@ -122,17 +116,6 @@ func parseArguments() (err error) {
}

func getTestCases(selection TestSet) (tests []testRunWithSteps) {

if shortHappyPathOnly != nil && *shortHappyPathOnly {
tests = append(tests, *testMap["happy-path-short"])
return
}

if happyPathOnly != nil && *happyPathOnly {
tests = append(tests, *testMap["happy-path"])
return
}

// Run default tests if no test cases were selected
if len(selection) == 0 {
selection = TestSet{"changeover", "happy-path",
Expand Down

0 comments on commit b28e1d5

Please sign in to comment.