Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: fix evm tests CI #2887

Merged
merged 1 commit into from
Feb 11, 2025
Merged

Conversation

buddh0
Copy link
Collaborator

@buddh0 buddh0 commented Feb 11, 2025

Description

tests: fix evm tests CI

Rationale

when run run-evm-tests.sh locally, there are 518641 cases are run
but in CI https://github.com/bnb-chain/bsc/actions/runs/13255495909/job/37001551107
even it shows that it succeed, but only 212204 case are run
image
the reason is that a timeout is trigged
image

in this PR, the flag short is added following to geth,
for one json test file, one of the four modes [path/snap, path/trie, hash/snap, hash/trie] will be selected randomly.

	if testing.Short() {
		snapshotConf = []bool{snapshotConf[rand.Int()%2]}
		dbschemeConf = []string{dbschemeConf[rand.Int()%2]}
	}

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

@brilliant-lx brilliant-lx merged commit 0e03696 into bnb-chain:develop Feb 11, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants