diff options
author | Klaus Aehlig <aehlig@google.com> | 2016-06-02 16:04:26 +0000 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2016-06-03 12:45:21 +0000 |
commit | 07122cf2f623cf50f9fe7ecf7171d6efbc23bed2 (patch) | |
tree | d38ab8d8287bf8a49367b299cba8eec44a79c081 /src/main/java/com/google/devtools | |
parent | c728a631ac6ad87599270bf93a81c8d8a8c23437 (diff) |
experimental UI: when showing the last test, color-code its state
The experimental UI also keeps track, in the progress bar, of the last test
that completed. When using curses, use colors to indicate whether the test
passed or not.
--
Change-Id: Iaa01a773c3bbf534692ed21dd420596cb63e2585
Reviewed-on: https://bazel-review.googlesource.com/#/c/3752
MOS_MIGRATED_REVID=123871492
Diffstat (limited to 'src/main/java/com/google/devtools')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/ExperimentalStateTracker.java | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/ExperimentalStateTracker.java b/src/main/java/com/google/devtools/build/lib/runtime/ExperimentalStateTracker.java index 44e38f08b4..eee5e0fd37 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/ExperimentalStateTracker.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/ExperimentalStateTracker.java @@ -410,10 +410,15 @@ class ExperimentalStateTracker { final String prefix = "; last test: "; if (!shortVersion && mostRecentTest != null) { if (terminalWriter != null) { - terminalWriter - .normal() - .append(prefix + shortenedLabelString( - mostRecentTest.getTarget().getLabel(), width - prefix.length())); + terminalWriter.normal().append(prefix); + if (mostRecentTest.getStatus() == BlazeTestStatus.PASSED) { + terminalWriter.okStatus(); + } else { + terminalWriter.failStatus(); + } + terminalWriter.append( + shortenedLabelString(mostRecentTest.getTarget().getLabel(), width - prefix.length())); + terminalWriter.normal(); } return true; } else { |