Merge pull request #77 from cburroughs/fq-logging

qualify log statements with plugin name
uber-archive · Aug 17, 2015 · 3b38f5c · 3b38f5c
2 parents eaf4068 + de92e73
commit 3b38f5c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/uber/jenkins/phabricator/utils/Logger.java b/src/main/java/com/uber/jenkins/phabricator/utils/Logger.java
@@ -26,7 +26,7 @@
  * Logger utility.
  */
 public class Logger {
-    private static final String LOG_FORMAT = "[%s] %s";
+    private static final String LOG_FORMAT = "[phabricator:%s] %s";
 
     private final PrintStream stream;
 

diff --git a/src/test/java/com/uber/jenkins/phabricator/utils/LoggerTest.java b/src/test/java/com/uber/jenkins/phabricator/utils/LoggerTest.java
@@ -47,7 +47,7 @@ public void testInfo() {
         String message = "This is a great plugin!";
 
         logger.info(tag, message);
-        assertEquals("[phabricator-jenkins] This is a great plugin!\n", byteArrayOutputStream.toString());
+        assertEquals("[phabricator:phabricator-jenkins] This is a great plugin!\n", byteArrayOutputStream.toString());
     }
 
     @Test
@@ -56,6 +56,6 @@ public void testWarn() {
         String message = "This is a great comic";
 
         logger.info(tag, message);
-        assertEquals("[a-softer-world] This is a great comic\n", byteArrayOutputStream.toString());
+        assertEquals("[phabricator:a-softer-world] This is a great comic\n", byteArrayOutputStream.toString());
     }
 }