Suppress Jetty logger in tests (#611)

* Suppress Jetty logger in tests

Author: SamCarlberg

core/src/test/java/edu/wpi/grip/core/http/GripServerTest.java

@@ -38,15 +38,9 @@ public class GripServerTest {
 
   public static class TestServerFactory implements GripServer.JettyServerFactory {
 
-    private int port;
-
-    public int getPort() {
-      return port;
-    }
-
     @Override
     public Server create(int port) {
-      this.port = 0;
+      org.eclipse.jetty.util.log.Log.setLog(new NoLogger());
       return new Server(0); // 0 -> some random open port, we don't care which
     }
 

core/src/test/java/edu/wpi/grip/core/http/NoLogger.java

@@ -0,0 +1,85 @@
+package edu.wpi.grip.core.http;
+
+import org.eclipse.jetty.util.log.Logger;
+
+/**
+ * Jetty logger implementation that ignores all logging calls.
+ */
+@SuppressWarnings("PMD.UncommentedEmptyMethodBody")
+public class NoLogger implements Logger {
+
+  @Override
+  public String getName() {
+    return "NoLogger";
+  }
+
+  @Override
+  public void warn(String msg, Object... args) {
+
+  }
+
+  @Override
+  public void warn(Throwable thrown) {
+
+  }
+
+  @Override
+  public void warn(String msg, Throwable thrown) {
+
+  }
+
+  @Override
+  public void info(String msg, Object... args) {
+
+  }
+
+  @Override
+  public void info(Throwable thrown) {
+
+  }
+
+  @Override
+  public void info(String msg, Throwable thrown) {
+
+  }
+
+  @Override
+  public boolean isDebugEnabled() {
+    return false;
+  }
+
+  @Override
+  public void setDebugEnabled(boolean enabled) {
+
+  }
+
+  @Override
+  public void debug(String msg, Object... args) {
+
+  }
+
+  @Override
+  public void debug(String msg, long value) {
+
+  }
+
+  @Override
+  public void debug(Throwable thrown) {
+
+  }
+
+  @Override
+  public void debug(String msg, Throwable thrown) {
+
+  }
+
+  @Override
+  public Logger getLogger(String name) {
+    return this;
+  }
+
+  @Override
+  public void ignore(Throwable ignored) {
+
+  }
+}