std.experimental.testing formal review

[Atila Neves via Digitalmars-d]

On Wednesday, 9 September 2015 at 18:54:30 UTC, Brian Schott 
wrote:
> On Wednesday, 9 September 2015 at 15:20:41 UTC, Robert burner 
> Schadek wrote:
>> This post marks the start of the two week review process of 
>> std.experimental.testing.
>>
>> PR: https://github.com/D-Programming-Language/phobos/pull/3207
>> Dub: http://code.dlang.org/packages/unit-threaded
>> Doc: See CyberShadow/DAutoTest for up-to-date documentation 
>> build
>>
>> Previous Thread: 
>> http://forum.dlang.org/post/uzocokshugchescbawlj@forum.dlang.org
>
> Package-level documentation seems to be missing from the 
> auto-generated documentation.

I think I generated the docs before there were any, let me go see.

> The gen_ut_main link on the side bar is also a 404.

Good catch, I'll take a look. I'm not looking forward to trying 
to recreate the site... it really should be easier.

>
> std.experimental.testing.options.Options and 
> std.experimental.testing.reflection.TestData fields have no 
> DDoc, so they don't show up in the generated documentation.

I'll add DDoc.


> Is there going to be a shouldEqual that's specialized for 
> floating point, or should shouldBeTrue(approxEqual(...)) be 
> used instead? (If so, this should be documented)

Good question.

> std.experimental.testing.testcase.TestCase.numTestsRun should 
> be @property?

Sure.


Atila