Thread overview
[Issue 20911] Documentation for test/unit is non-existant
Jun 08, 2020
Seb
Jun 19, 2020
Walter Bright
Aug 05, 2020
Dlang Bot
Aug 05, 2020
Dlang Bot
June 08, 2020
https://issues.dlang.org/show_bug.cgi?id=20911

Seb <greeenify@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |greeenify@gmail.com

--- Comment #1 from Seb <greeenify@gmail.com> ---
(answering from my phone - please excuse the brevity)

Basically all your points are answered by the test's readme:

https://github.com/dlang/dmd/blob/master/test/README.md

1-3) You just need to scroll down long enough. In short:
1) test suite which allows to call DMD and thus allowing more advanced tests
that just compile or error
2) every file contains test cases like the other test suite
3) ./test/run.d unit (it's also part of the all target if no arguments get
passed)
4) Frontend.d does have named unit tests. That's a lot more than most of all
test files have no comments.
5) It doesn't need a main function as it is a test file

We don't have a header in every test file with full build and explanations instructions either.

Hence, I'm sorry, but I fail to see actionables here at the moment.

Btw as this is a prime example of the lack of good communication tools. I guess all your questions could have been answered quickly in the #dmd or #ci slack groups. Their content to noise ratios are pretty decent and quick chats there might help you save a lot of time.

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

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |TestSuite

--
August 05, 2020
https://issues.dlang.org/show_bug.cgi?id=20911

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
@CyberShadow created dlang/dmd pull request #11513 "Fix Issue 20911 - Documentation for test/unit is non-existant" fixing this issue:

- Fix Issue 20911 - Documentation for test/unit is non-existant

  Add a short comment at the top of each file describing where to find
  the documentation on how to author and run DMD unit tests.

https://github.com/dlang/dmd/pull/11513

--
August 05, 2020
https://issues.dlang.org/show_bug.cgi?id=20911

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/dmd pull request #11513 "Fix Issue 20911 - Documentation for test/unit is non-existant" was merged into master:

- e9d0524ff21c224cd6d6b8354bf74c2fa6f2b6aa by Vladimir Panteleev:
  Fix Issue 20911 - Documentation for test/unit is non-existant

  Add a short comment at the top of each file describing where to find
  the documentation on how to author and run DMD unit tests.

https://github.com/dlang/dmd/pull/11513

--