don't always print module header when only one testset

When only one module was tested and explicitly passed to `retest`,
`moduleheader` was false, but we still printed the header when
there was only one toplevel testset, so that the module name was
printed at least once. But this was not very consistent: depending
on the filtering pattern, the module name could be printed either
at the beginning or at the end. When the module is passed explicitly,
it doesn't seem necessary to print the module name at all, so we only
print it at the end when a summary is needed, i.e. more than one
toplevel testset was tested.

Author: rfourquet

docs/src/index.md

@@ -81,9 +81,8 @@ run the tests):
 julia> using MyPackage, ReTest
 
 julia> MyPackage.runtests()
-               Pass
-MyPackage:
-  greet    |      1
+             Pass
+greet    |      1
 ```
 
 Suppose now that we organize `MyPackage` as a standard package, with
@@ -149,9 +148,8 @@ Main.MyPackageTests:
 Overall              |      2
 
 julia> MyPackageTests.runtests(3) # run only testset with ID 3 in MyPackageTests
-                            Pass
-   Main.MyPackageTests:
-3|   stuff              |      1
+                          Pass
+3| stuff              |      1
 ```
 
 Here it is for basic usage!

src/ReTest.jl

@@ -870,11 +870,9 @@ function retest(@nospecialize(args::ArgType...);
 
             ndone = 0
 
-            if moduleheader | !many
+            if moduleheader
                 # + if moduleheader, we print the module as a header, to know where the currently
                 #   printed testsets belong
-                # + if !many, we won't print the overall afterwads, which would be redundant
-                #   with the only one printed top-level testset
                 ntests += 1
                 put!(outchan, module_ts) # printer task will take care of feeding computechan
             else
@@ -1281,7 +1279,7 @@ function fetchtests((mod, pat), verbose, moduleheader, maxidw; static, strict, d
     tests, descwidth, hasbroken
 end
 
-isindented(verbose, moduleheader, many) = (verbose > 0) & (moduleheader | !many)
+isindented(verbose, moduleheader, many) = (verbose > 0) & moduleheader
 
 function dryrun(mod::Module, ts::TestsetExpr, pat::Pattern, align::Int=0, parentsubj=""
                 ; maxidw::Int, # external calls

test/runtests.jl

@@ -789,10 +789,9 @@ end # MV
 
 @chapter MiscVerbose begin
     check(MV, [], output = """
-               Pass
-Main.MV:
-  outer    |      2
-    inner  |      1
+             Pass
+outer    |      2
+  inner  |      1
 """)
 end