Fix logging test counting bug when more workers than tests (#219)

* Fix test counting bug when more workers than tests

* Bump version

Author: nickrobinson251

Project.toml

@@ -1,6 +1,6 @@
 name = "ReTestItems"
 uuid = "817f1d60-ba6b-4fd5-9520-3cf149f6a823"
-version = "1.33.0"
+version = "1.33.1"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

src/testcontext.jl

@@ -163,7 +163,7 @@ function get_starting_testitems(ti::TestItems, n)
         # mark eval_number
         t.eval_number[] = i
     end
-    @atomic ti.count += n
+    @atomic ti.count += length(testitems)
     return [testitems; fill(nothing, n - length(testitems))]
 end
 

test/integrationtests.jl

@@ -1496,4 +1496,17 @@ end
     @test_throws exc runtests(joinpath(TEST_FILES_DIR, "_happy_tests.jl"); tags=[:blahahahaha_nope])
 end
 
+@testset "bugfix: don't overcount when nworkers > ntestitems" begin
+    using IOCapture
+    nworkers = 4
+    c = IOCapture.capture() do
+        encased_testset(() -> runtests(joinpath(TEST_FILES_DIR, "_happy_tests.jl"); nworkers))
+    end
+    results = c.value
+    @assert nworkers > n_tests(results)
+    @assert n_tests(results) == 3
+    # the bug was printing `4/3`
+    @test contains(c.output, "3/3 test items were run.")
+end
+
 end # integrationtests.jl testset