Don't perform useless unlocks.

Author: maleadt

src/ParallelTestRunner.jl

@@ -1285,7 +1285,8 @@ function runtests(mod::Module, args::ParsedArgs;
     end
 
     # print the output generated by each testset
-    for (testname, result, output, _start, _stop) in @lock(results, results[])
+    # (`@sync` above joined all writers, so `results` is quiescent from here on)
+    for (testname, result, output, _start, _stop) in results.value
         if !isempty(output)
             print(io_ctx.stdout, "\nOutput generated during execution of '")
             if result isa Exception || anynonpass(result[])
@@ -1343,7 +1344,7 @@ function runtests(mod::Module, args::ParsedArgs;
     function collect_results()
         with_testset(o_ts) do
             completed_tests = Set{String}()
-            for (testname, result, _output, start, stop) in @lock(results, results[])
+            for (testname, result, _output, start, stop) in results.value
                 push!(completed_tests, testname)
 
                 if result isa AbstractTestRecord