Merge pull request #378 from code-dot-org/use-logger-utils

Refactor Logger.getLogger references to use LoggerUtils

Author: Madelyn Kasula

org-code-javabuilder/lib/src/main/java/dev/javabuilder/WebSocketServer.java

@@ -133,7 +133,7 @@ public void onOpen(Session session) {
 
   @OnClose
   public void myOnClose() {
-    Logger.getLogger(MAIN_LOGGER).info("WebSocket closed.");
+    LoggerUtils.logInfo("WebSocket closed.");
     PerformanceTracker performanceTracker =
         (PerformanceTracker) JavabuilderContext.getInstance().get(PerformanceTracker.class);
     performanceTracker.logPerformance();

org-code-javabuilder/lib/src/main/java/org/code/javabuilder/PerformanceTracker.java

@@ -1,10 +1,7 @@
 package org.code.javabuilder;
 
-import static org.code.protocol.LoggerNames.MAIN_LOGGER;
-
 import java.time.Clock;
 import java.time.Instant;
-import java.util.logging.Logger;
 import org.code.protocol.*;
 import org.json.JSONObject;
 
@@ -103,6 +100,6 @@ public void logPerformance() {
       metricClient.publishCleanupTime(cleanupTime);
     }
 
-    Logger.getLogger(MAIN_LOGGER).info(logs.toString());
+    LoggerUtils.logInfo(logs.toString());
   }
 }

org-code-javabuilder/validation/src/main/java/org/code/validation/support/NeighborhoodTracker.java

@@ -1,11 +1,9 @@
 package org.code.validation.support;
 
 import static org.code.protocol.ClientMessageDetailKeys.*;
-import static org.code.protocol.LoggerNames.MAIN_LOGGER;
 
 import java.util.*;
 import java.util.function.Function;
-import java.util.logging.Logger;
 import java.util.stream.Collectors;
 import org.code.neighborhood.support.NeighborhoodSignalKey;
 import org.code.neighborhood.support.World;
@@ -58,11 +56,6 @@ public void trackEvent(ClientMessage message) {
     }
 
     if (!this.painterTrackers.containsKey(id) || !this.isInitialized) {
-      Logger.getLogger(MAIN_LOGGER)
-          .severe(
-              String.format(
-                  "Error: received a painter event for an uninitialized painter: %s, %s\n",
-                  message.getValue(), message.getDetail()));
       return;
     }