Fixes tests creating multiple log files in home directory

The problem was we were setting up the logger multiple times
durring the test's lifecycle.
We should only have one global logger setup durring the tests.

Closes #309

Author: JLLeitschuh

core/src/main/java/edu/wpi/grip/core/GRIPCoreModule.java

@@ -23,14 +23,17 @@
  * A Guice {@link com.google.inject.Module} for GRIP's core package.  This is where instances of {@link Pipeline},
  * {@link Palette}, {@link Project}, etc... are created.
  */
+@SuppressWarnings("PMD.MoreThanOneLogger")
 public class GRIPCoreModule extends AbstractModule {
     private final Logger logger = Logger.getLogger(GRIPCoreModule.class.getName());
 
     private final EventBus eventBus = new EventBus(this::onSubscriberException);
 
-    public GRIPCoreModule() {
+    // This is in a static initialization block so that we don't create a ton of
+    // log files when running tests
+    static {
         //Set up the global level logger. This handles IO for all loggers.
-        final Logger globalLogger = LogManager.getLogManager().getLogger("");//This is our global logger
+        final Logger globalLogger = LogManager.getLogManager().getLogger("");
 
         try {
             // Remove the default handlers that stream to System.err
@@ -69,7 +72,10 @@ public synchronized void publish(final LogRecord record) {
         } catch (IOException exception) {//Something happened setting up file IO
             throw new IllegalStateException("Failed to configure the Logger", exception);
         }
+    }
 
+    public GRIPCoreModule() {
+        // TODO: HACK! Don't assign the global thread handler to an instance method. Creates global state.
         Thread.setDefaultUncaughtExceptionHandler(this::onThreadException);
     }