From 747ff0b67ad2ef0472cdb7a7a38b9a9a5c334bc5 Mon Sep 17 00:00:00 2001 From: Christian Guinard <28689358+christiangnrd@users.noreply.github.com> Date: Wed, 7 Oct 2026 13:00:10 -0300 Subject: [PATCH 1/2] Revert "Allow skipping passing tests (#207)" This reverts commit 0416785979b0af4ff08a2522cd2db37a0fad8616. --- src/ParallelTestRunner.jl | 14 ++------------ src/args.jl | 4 +--- 2 files changed, 3 insertions(+), 15 deletions(-) diff --git a/src/ParallelTestRunner.jl b/src/ParallelTestRunner.jl index 0dee6a65..4f6105ca 100644 --- a/src/ParallelTestRunner.jl +++ b/src/ParallelTestRunner.jl @@ -242,11 +242,9 @@ function runtests(mod::Module, args::ParsedArgs; # set-up # - - historical_durations, historical_failures = load_test_history(mod, history_key) - # list tests, if requested if args.list !== nothing + historical_durations, historical_failures = load_test_history(mod, history_key) sorted_tests = sort(collect(keys(testsuite))) name_align = isempty(sorted_tests) ? 0 : maximum(textwidth, sorted_tests) duration_strs = Dict( @@ -272,21 +270,13 @@ function runtests(mod::Module, args::ParsedArgs; # filter tests filter_tests!(testsuite, args) - # remove tests that previously passed - if args.skippassed !== nothing - for test in keys(testsuite) - if test in keys(historical_durations) && test ∉ historical_failures - delete!(testsuite, test) - end - end - end - # filter serial list to only include tests that survived filtering serial = filter(t -> haskey(testsuite, t), serial) # determine test order tests = collect(keys(testsuite)) Random.shuffle!(tests) + historical_durations, historical_failures = load_test_history(mod, history_key) get_historical_duration(test) = TestHistoryEntry(get(historical_durations, test, Inf), test in historical_failures) sort!(tests, by = x -> get_historical_duration(x), rev = true) diff --git a/src/args.jl b/src/args.jl index 24bc768a..5f20c4be 100644 --- a/src/args.jl +++ b/src/args.jl @@ -63,7 +63,6 @@ struct ParsedArgs verbose::Union{Some{Nothing}, Nothing} quickfail::Union{Some{Nothing}, Nothing} list::Union{Some{Nothing}, Nothing} - skippassed::Union{Some{Nothing}, Nothing} custom::Dict{String,Any} @@ -146,7 +145,6 @@ function parse_args(args; custom::Array{String} = String[]) verbose = extract_flag!(args, "--verbose") quickfail = extract_flag!(args, "--quickfail") list = extract_flag!(args, "--list") - skippassed = extract_flag!(args, "--skippassed") # Experimental # boolean flags don't take values for (flag, val) in (("--verbose", verbose), ("--quickfail", quickfail), ("--list", list)) @@ -166,7 +164,7 @@ function parse_args(args; custom::Array{String} = String[]) error("Unknown test options `$(join(optlike_args, " "))` (try `--help` for usage instructions)") end - return ParsedArgs(jobs, verbose, quickfail, list, skippassed, custom_args, args) + return ParsedArgs(jobs, verbose, quickfail, list, custom_args, args) end """ From 8a7f61c3b813b61b103e090ca4bb052bbfa2bb45 Mon Sep 17 00:00:00 2001 From: Christian Guinard <28689358+christiangnrd@users.noreply.github.com> Date: Wed, 7 Oct 2026 13:01:01 -0300 Subject: [PATCH 2/2] Bump version to 2.9.3 --- Project.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Project.toml b/Project.toml index c9fafbdc..124083de 100644 --- a/Project.toml +++ b/Project.toml @@ -1,6 +1,6 @@ name = "ParallelTestRunner" uuid = "d3525ed8-44d0-4b2c-a655-542cee43accc" -version = "2.9.2" +version = "2.9.3" authors = ["Valentin Churavy "] [deps]