Page MenuHomePhabricator

Added test namespace field to `ArcanistUnitTestResult`
ClosedPublic

Authored by joshuaspence on Jan 13 2014, 8:31 PM.
Tags
None
Referenced Files
F15410898: D7952.diff
Wed, Mar 19, 8:40 AM
F15410859: D7952.diff
Wed, Mar 19, 8:35 AM
F15410842: D7952.diff
Wed, Mar 19, 8:32 AM
F15410820: D7952.diff
Wed, Mar 19, 8:29 AM
F15410799: D7952.diff
Wed, Mar 19, 8:26 AM
F15410755: D7952.diff
Wed, Mar 19, 8:21 AM
F15410736: D7952.diff
Wed, Mar 19, 8:19 AM
F15410715: D7952.diff
Wed, Mar 19, 8:17 AM

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Thanks!

src/unit/renderer/ArcanistUnitConsoleRenderer.php
25

If the namespace is empty, I think we should probably omit the ::, since otherwise everyone's stuff will become ::testname. I'll tweak that in the pull.