Add a lock to protect pushes to results

Author: giordano

src/ParallelTestRunner.jl

@@ -834,6 +834,7 @@ function runtests(mod::Module, args::ParsedArgs;
     results = []
     running_tests = Dict{String, Float64}()  # test => start_time
     test_lock = ReentrantLock() # to protect crucial access to tests and running_tests
+    results_lock = ReentrantLock() # to protect concurrent access to results
 
     worker_tasks = Task[]
 
@@ -1063,8 +1064,8 @@ function runtests(mod::Module, args::ParsedArgs;
                     ex
                 end
                 test_t1 = time()
-                output = @lock wrkr.io_lock String(take!(wrkr.io))
-                push!(results, (test, result, output, test_t0, test_t1))
+                output = Base.@lock wrkr.io_lock String(take!(wrkr.io))
+                Base.@lock results_lock push!(results, (test, result, output, test_t0, test_t1))
 
                 # act on the results
                 if result isa AbstractTestRecord