shuffle modules even when none is passed

Author: rfourquet

src/ReTest.jl

@@ -703,8 +703,6 @@ function process_args(args, verbose, shuffle)
 end
 
 function computemodules!(modules::Vector{Module}, shuffle)
-    unique!(modules)
-    shuffle && shuffle!(modules)
     if isempty(modules)
         # TESTED_MODULES is not up-to-date w.r.t. package modules which have
         # precompilation, so we have to also look in Base.loaded_modules
@@ -728,7 +726,10 @@ function computemodules!(modules::Vector{Module}, shuffle)
         unique!(modules)
         # will automatically skip ReTest and ReTest.ReTestTest
         filter!(m -> isdefined(m, INLINE_TEST[]) && m ∉ (ReTest, ReTestTest),  modules)
+    else
+        unique!(modules)
     end
+    shuffle && shuffle!(modules)
     modules
 end