Thread overview
[Issue 20910] Default unittest runner reports wrong unittest count
[Issue 20910] Unittest runner reports wrong unittest count
Jun 09, 2020
Dlang Bot
Jun 16, 2020
Dlang Bot
June 08, 2020
https://issues.dlang.org/show_bug.cgi?id=20910

hsteoh@quickfur.ath.cx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |bootcamp

--
June 08, 2020
https://issues.dlang.org/show_bug.cgi?id=20910

hsteoh@quickfur.ath.cx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Unittest runner reports     |Default unittest runner
                   |wrong unittest count        |reports wrong unittest
                   |                            |count

--
June 09, 2020
https://issues.dlang.org/show_bug.cgi?id=20910

moonlightsentinel@disroot.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |moonlightsentinel@disroot.o
                   |                            |rg

--- Comment #1 from moonlightsentinel@disroot.org ---
The message is wrong, it shows how many modules were tested, not the number of unittests.

That number is probably not available in the current implementation which
relies on ModuleInfo, see
https://github.com/dlang/druntime/blob/435ae8a21034ef50fa4769e3aaba5778dbc76557/src/core/runtime.d#L602-L617

--
June 09, 2020
https://issues.dlang.org/show_bug.cgi?id=20910

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
@quickfur created dlang/druntime pull request #3129 "Fix issue 20910: fix misleading unittest runner messages." mentioning this issue:

- Issue 20910: fix misleading unittest runner messages.

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

https://github.com/dlang/druntime/pull/3129

--
June 16, 2020
https://issues.dlang.org/show_bug.cgi?id=20910

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/druntime pull request #3129 "Fix issue 20910: fix misleading unittest runner messages." was merged into master:

- 209ce9b0c18ba27519f900ca31947ee3bf1e0f15 by H. S. Teoh:
  Fix issue 20910: fix misleading unittest runner messages.

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

https://github.com/dlang/druntime/pull/3129

--