Skip to content

Commit

Permalink
Remove outdated argument
Browse files Browse the repository at this point in the history
  • Loading branch information
Nashtare committed Jan 27, 2024
1 parent 20ee126 commit 9d432b3
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ Then launch the runner pointing it at the parsed tests directory:

```sh
cd ../evm_test_runner
cargo run --release -- -r summary ../generation_inputs/BlockchainTests # For a high-level summary report
cargo run --release -- -r test ../generation_inputs/BlockchainTests # For detailed information per test (likely want to use a filter with `-f`)
cargo run --release -- -r summary # For a high-level summary report
cargo run --release -- -r test # For detailed information per test (likely want to use a filter with `-f`)
```

The test runner supports secondary arguments to customize the testing flow. While they are all displayed by calling `cargo run -- --help`,
Expand Down
3 changes: 0 additions & 3 deletions evm_test_runner/src/arg_parsing.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,6 @@ pub(crate) enum ReportType {
#[derive(Debug, Parser)]
#[clap(author, version, about)]
pub(crate) struct ProgArgs {
/// The path to the parsed tests directory.
pub(crate) parsed_tests_path: Option<PathBuf>,

/// An optional path to a blacklist file containing test variants to prevent
/// from running. This can be used to skip particularly heavy or badly
/// configured tests.
Expand Down
5 changes: 1 addition & 4 deletions evm_test_runner/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ async fn run() -> anyhow::Result<bool> {
skip_passed,
witness_only,
test_timeout,
parsed_tests_path,
blacklist_path,
simple_progress_indicator,
update_persistent_state_from_upstream,
Expand Down Expand Up @@ -99,9 +98,7 @@ async fn run() -> anyhow::Result<bool> {
}
};

let parsed_tests_path = parsed_tests_path
.map(Ok)
.unwrap_or_else(get_default_parsed_tests_path)?;
let parsed_tests_path = get_default_parsed_tests_path()?;

let parsed_tests = Rc::new(
read_in_all_parsed_tests(
Expand Down

0 comments on commit 9d432b3

Please sign in to comment.