Fix issue 20910: fix misleading unittest runner messages.

[quickfur]

The number returned by runModuleUnitTests is not the number of unittests, but the number of modules tested.

[dlang-bot]

Thanks for your pull request, @quickfur!

Bugzilla references

Auto-close	Bugzilla	Severity	Description
✓	20910	normal	Default unittest runner reports wrong unittest count

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + druntime#3129"

[thewilsonator]

Please rebase this PR onto dlang/master's HEAD and the errors should do away.

[thewilsonator]

Also please prepend Fix to the front of the commit message so the bot closes the issue. Feel free to add auto merge yourself once this passes.

[quickfur]

I'm already working off HEAD. No idea what the problem might be.

[wilzbach]

No idea what the problem might be.

Druntime's testsuite is failing. Is it passing for you locally?
There are tests which grep for the previous output, e.g.

test -z "FAILED" || grep -q "unittests FAILED" generated/linux/release/32/badn.done
/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/dmd/generated/linux/release/32/dmd -m32  -w -I../../src -I../../import -Isrc -defaultlib= -debuglib= -dip1000 -L-lpthread -L-lm -L/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/druntime/generated/linux/release/32/libdruntime.a -O -release -version=_GLIBCXX_USE_CXX98_ABI -main -unittest -version=CoreUnittest -c -of=./generated/linux/release/32/string_old_d.o src/string_test.d
/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/dmd/generated/linux/release/32/dmd -m32  -w -I../../src -I../../import -Isrc -defaultlib= -debuglib= -dip1000 -L-lpthread -L-lm -L/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/druntime/generated/linux/release/32/libdruntime.a -O -release -unittest -version=CoreUnittest -gx -ofgenerated/linux/release/32/precisegc ../../src/gc/impl/conservative/gc.d ../../src/rt/lifetime.d precisegc.d
Makefile:36: recipe for target 'generated/linux/release/32/badn.done' failed
make[2]: *** [generated/linux/release/32/badn.done] Error 1
/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/dmd/generated/linux/release/32/dmd -m32  -w -I../../src -I../../import -Isrc -defaultlib= -debuglib= -dip1000 -L-lpthread -L-lm -L/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/druntime/generated/linux/release/32/libdruntime.a -O -release -g -ofgenerated/linux/release/32/rt_trap_exceptions_drt src/rt_trap_exceptions_drt.d
test 1 -eq 1 || ! grep -q main generated/linux/release/32/na.done
test -z "" || grep -q "unittests " generated/linux/release/32/na.done
test -n "" || ! grep -q "unittests" generated/linux/release/32/na.done
make[2]: Leaving directory '/media/disk1/braddr/sandbox/at-client/pull-4078093-Linux_32/druntime/test/unittest'
posix.mak:339: recipe for target 'test/unittest/.run' failed

[quickfur]

@thewilsonator BTW, I don't think I have commit access to druntime.