oops... let's get more tests!

Author: rfourquet

src/ReTest.jl

@@ -1336,7 +1336,7 @@ function process_args(@nospecialize(args);
     # if a parent module with no tests is explicitly passed, which contains a submodule
     # and recursive==true, then the submodule name is printed (and similarly for
     # printing `MyPackageTests` when `MyPackage` is passed and `load==true`)
-    module_header = length(modules) > 1 | implicitmodules
+    module_header = (length(modules) > 1) | implicitmodules
 
     # remove modules which don't have tests, which can happen when a parent module without
     # tests is passed to retest in order to run tests in its submodules

test/runtests.jl

@@ -58,23 +58,35 @@ end
 
 innertestsets = ["d", "e1", "e2", "g", "h1", "h2"]
 
-function check(rx, list)
+function check(rx, list; implicit=false)
     empty!(RUN)
-    retest(M, Regex(rx))
+    if implicit
+        retest(Regex(rx))
+    else
+        retest(M, Regex(rx))
+    end
     @test RUN == list
     mktemp() do path, io
         redirect_stdout(io) do
-            retest(M, Regex(rx), dry=true, id=false) # TODO: test with id=true
+            if implicit
+                retest(Regex(rx), dry=true, id=false)
+            else
+                retest(M, Regex(rx), dry=true, id=false) # TODO: test with id=true
+            end
         end
         seekstart(io)
         expected = map(list) do t
-            if t in innertestsets
-                "  " * t * " ✔"
-            else
-                t * " ✔"
-            end
+            "  "^implicit *
+                if t in innertestsets
+                    "  " * t * " ✔"
+                else
+                    t * " ✔"
+                end
         end
         expected = join(expected, '\n')
+        if implicit
+            expected = "Main.M\n" * expected
+        end
         actual = readchomp(io)
         if isempty(expected)
             @test startswith(actual, "No matching tests for module")
@@ -85,6 +97,10 @@ function check(rx, list)
 end
 end #
 
+@chapter OnlyOneModule begin
+    M.check("a", ["a"], implicit=true)
+end
+
 @chapter M begin
     # we don't put these checks in a @testset, as this would modify filtering logic
     # (e.g. with `@testset "filtering" ...`, testset subjects would start with "/filtering/...")