test: Adjust log messages for consistency

zanata · Mar 3, 2016 · e5c1978 · e5c1978
1 parent dd00d25
commit e5c1978
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/functional-test/src/test/java/org/zanata/feature/testharness/ZanataTestCase.java b/functional-test/src/test/java/org/zanata/feature/testharness/ZanataTestCase.java
@@ -75,7 +75,7 @@ private String getTestDescription() {
 
     @Before
     public final void testEntry() {
-        log.info("Starting ".concat(getTestDescription()));
+        log.info("Test starting: {}", getTestDescription());
         testFunctionStart = new DateTime();
         WebDriverFactory.INSTANCE.testEntry();
         zanataRestCaller.signalBeforeTest(getClass().getName(), testName.getMethodName());
@@ -87,8 +87,8 @@ public final void testExit() {
         zanataRestCaller.signalAfterTest(getClass().getName(), testName.getMethodName());
         Duration duration = new Duration(testFunctionStart, new DateTime());
         PeriodFormatter periodFormatter = new PeriodFormatterBuilder()
-                .appendLiteral("Finished "
-                        .concat(getTestDescription()).concat(" in "))
+                .appendLiteral("Test finished: "
+                        .concat(getTestDescription()).concat(": in "))
                 .printZeroAlways()
                 .appendMinutes()
                 .appendSuffix(" minutes, ")

diff --git a/functional-test/src/test/java/org/zanata/util/TestLogger.java b/functional-test/src/test/java/org/zanata/util/TestLogger.java
@@ -34,28 +34,28 @@
 public class TestLogger extends RunListener {
     @Override
     public void testStarted(Description description) throws Exception {
-        log.info("Test {} starting", description);
+        log.info("Test starting: {}", description);
     }
 
     @Override
     public void testFinished(Description description) throws Exception {
-        log.info("Test {} finished", description);
+        log.info("Test finished: {}", description);
     }
 
     @Override
-    public void testFailure(Failure failure) throws Exception {
-        log.error("FAILED test " + failure, failure.getException());
+    public void testIgnored(Description description) throws Exception {
+        log.error("Test IGNORED: {}", description);
     }
 
     @Override
-    public void testAssumptionFailure(Failure failure) {
-        log.error("FAILED ASSUMPTION in test " + failure,
-                failure.getException());
+    public void testFailure(Failure failure) throws Exception {
+        log.error("Test FAILED: " + failure, failure.getException());
     }
 
     @Override
-    public void testIgnored(Description description) throws Exception {
-        log.error("Test {} IGNORED", description);
+    public void testAssumptionFailure(Failure failure) {
+        log.error("Test FAILED ASSUMPTION: " + failure,
+                failure.getException());
     }
 
 }