Skip to content

Commit

Permalink
feature to exclude time consuming tests (but not enabled)
Browse files Browse the repository at this point in the history
  • Loading branch information
debjit-bw committed Dec 4, 2024
1 parent 5491013 commit 29dc893
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions src/testing/cases/blockchain_test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -234,9 +234,18 @@ impl Case for BlockchainTestCase {
/// The reason should be documented in a comment above the file name(s).
pub fn should_skip(path: &Path) -> bool {
let path_str = path.to_str().expect("Path is not valid UTF-8");
let name = path.file_name().unwrap().to_str().unwrap();
let test_name = path.file_name().unwrap().to_str().unwrap();
let test_folder = path
.parent()
.unwrap()
.file_name()
.unwrap()
.to_str()
.unwrap();

// executnig all tests for now
false && matches!(
name,
test_name,
// funky test with `bigint 0x00` value in json :) not possible to happen on mainnet and require
// custom json parser. https://github.com/ethereum/tests/issues/971
| "ValueOverflow.json"
Expand Down Expand Up @@ -271,6 +280,11 @@ pub fn should_skip(path: &Path) -> bool {
| "CALLBlake2f_MaxRounds.json"
| "shiftCombinations.json"
)
|| matches!(
test_folder,
// These tests are passing, but they take a lot of time to execute
| "stTimeConsuming"
)
// Ignore outdated EOF tests that haven't been updated for Cancun yet.
|| path_contains(path_str, &["EIPTests", "stEOF"])
}
Expand Down

0 comments on commit 29dc893

Please sign in to comment.