It would be nice if we could see unit test results on the home view, e.g. with a color coding alongside the CR color coding. This would discourage people from landing in the situation where a diff has passed CR but hasn't passed tests.
Description
Description
Event Timeline
Comment Actions
It sounds like the problem you want to solve is that people sometimes land code before tests have finished running, correct?
I just want to clarify the root problem, as opposed to the one specific solution you're suggesting, as per https://secure.phabricator.com/book/phabcontrib/article/feature_requests/#describe-problems.
I think there are probably other solutions to this problem -- such as properly integrating our unit test suite with arc unit, because then you (I believe) get a warning on the command-line if you try to land when tests are failing/not yet reported.
Comment Actions
This would discourage people from landing in the situation where a diff has passed CR but hasn't passed tests.
You should already get an explicit prompt from arc land in this case, requiring you to confirm that you want to land code with failing or still-running tests. Are you not seeing this?
Comment Actions
(I believe) get a warning on the command-line if you try to land when tests are failing/not yet reported.
Ah, yeah. This might be an integration thing, then.