Page MenuHomePhabricator

D13821.diff
No OneTemporary

D13821.diff

diff --git a/src/applications/differential/customfield/DifferentialUnitField.php b/src/applications/differential/customfield/DifferentialUnitField.php
--- a/src/applications/differential/customfield/DifferentialUnitField.php
+++ b/src/applications/differential/customfield/DifferentialUnitField.php
@@ -56,8 +56,14 @@
protected function newHarbormasterMessageView(array $messages) {
foreach ($messages as $key => $message) {
- if ($message->getResult() == ArcanistUnitTestResult::RESULT_PASS) {
- unset($messages[$key]);
+ switch ($message->getResult()) {
+ case ArcanistUnitTestResult::RESULT_PASS:
+ case ArcanistUnitTestResult::RESULT_SKIP:
+ // Don't show "Pass" or "Skip" in the UI since they aren't very
+ // interesting. The user can click through to the full results if
+ // they want details.
+ unset($messages[$key]);
+ break;
}
}

File Metadata

Mime Type
text/plain
Expires
Sep 21 2025, 11:17 PM (5 w, 6 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
8455182
Default Alt Text
D13821.diff (942 B)

Event Timeline