Change in asterixdb[master]: [NO ISSUE][TEST] Use log4j for TestMethodTracer tracing

[Michael Blow (Code Review)]

Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/2374

Change subject: [NO ISSUE][TEST] Use log4j for TestMethodTracer tracing
......................................................................

[NO ISSUE][TEST] Use log4j for TestMethodTracer tracing

Change-Id: I9d1ebc072e9d0d5d3930a0eb3cb6d884af4d5b06
---
M 
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
1 file changed, 6 insertions(+), 15 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/74/2374/1

diff --git 
a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
 
b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
index 692805f..34f7adf 100644
--- 
a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
+++ 
b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
@@ -18,8 +18,8 @@
  */
 package org.apache.asterix.test.base;
 
-import java.io.PrintStream;
-
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.junit.rules.TestWatcher;
 import org.junit.runner.Description;
 
@@ -34,29 +34,20 @@
  */
 
 public class TestMethodTracer extends TestWatcher {
-
-    private final PrintStream out;
-
-    public TestMethodTracer(PrintStream out) {
-        this.out = out;
-    }
-
-    public TestMethodTracer() {
-        this(System.out);
-    }
+    private static final Logger LOGGER = LogManager.getLogger();
 
     @Override
     protected void starting(Description description) {
-        out.println("## " + description.getMethodName() + " START");
+        LOGGER.info("### {} START", description.getMethodName());
     }
 
     @Override
     protected void failed(Throwable e, Description description) {
-        out.println("## " + description.getMethodName() + " FAILED (" + 
e.getClass().getName() + ")");
+        LOGGER.info("### {} FAILED ({})", description.getMethodName(), 
e.getClass().getName());
     }
 
     @Override
     protected void succeeded(Description description) {
-        out.println("## " + description.getMethodName() + " SUCCEEDED");
+        LOGGER.info("### {} SUCCEEDED", description.getMethodName());
     }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2374
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9d1ebc072e9d0d5d3930a0eb3cb6d884af4d5b06
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>