added back display level static default

Author: scottf

build.gradle

@@ -8,7 +8,7 @@ plugins {
     id 'signing'
 }
 
-def jarVersion = "1.1.0"
+def jarVersion = "1.1.1"
 group = 'io.nats'
 
 def isMerge = System.getenv("BUILD_EVENT") == "push"

src/main/java/nats/io/NatsServerRunner.java

@@ -39,6 +39,12 @@ public class NatsServerRunner implements AutoCloseable {
     public static long DEFAULT_RUN_CHECK_WAIT = 100;
     public static int DEFAULT_RUN_CHECK_TRIES = 3;
 
+    private static Level DefaultDisplayLevel = Level.ALL;
+
+    public static void setDefaultDisplayLevel(Level defaultDisplayLevel) {
+        DefaultDisplayLevel = defaultDisplayLevel;
+    }
+
     private final String _executablePath;
     private final Logger _displayOut;
     private final int _port;
@@ -307,7 +313,10 @@ public NatsServerRunner(int port, boolean debug, boolean jetstream, String confi
         _port = b.port == null || b.port <= 0 ? nextPort() : b.port;
 
         _displayOut = b.displayOut == null ? Logger.getLogger(NatsServerRunner.class.getName()) : b.displayOut;
-        if (b.displayOutLevel != null) {
+        if (b.displayOutLevel == null) {
+            _displayOut.setLevel(DefaultDisplayLevel);
+        }
+        else {
             _displayOut.setLevel(b.displayOutLevel);
         }
 

src/test/java/nats/io/DisplayLevelTest.java

@@ -0,0 +1,99 @@
+// Copyright 2022 The NATS Authors
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at:
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+package nats.io;
+
+import org.junit.jupiter.api.Test;
+
+import java.io.IOException;
+import java.util.logging.Level;
+
+public class DisplayLevelTest extends TestBase {
+
+    // RUN THIS TEST MANUALLY TO SEE THE PROPER SERVER OUTPUT
+    @Test
+    public void testDisplayOutLevel() throws IOException, InterruptedException {
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD SHOW EVERYTHING. Level:ALL (Default), No Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD ALSO SHOW EVERYTHING. Level:INFO, No Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .displayOutLevel(Level.INFO)
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        Thread.sleep(100);
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD SHOW THE ERROR ONLY. Level:SEVERE, Yes Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .configFilePath("not-a-real-path")
+            .displayOutLevel(Level.SEVERE)
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        Thread.sleep(100);
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD ALSO SHOW THE ERROR ONLY. Level:INFO, Yes Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .configFilePath("not-a-real-path")
+            .displayOutLevel(Level.INFO)
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        Thread.sleep(100);
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD SHOW NOTHING. Level:OFF, Yes Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .configFilePath("not-a-real-path")
+            .displayOutLevel(Level.OFF)
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        Thread.sleep(100);
+        System.out.println("------------------------------------------------------------------------------");
+        System.out.println("THIS SHOULD ALSO SHOW NOTHING. Level:SEVERE, No Error.");
+        System.out.println("------------------------------------------------------------------------------");
+        try (NatsServerRunner runner = NatsServerRunner.builder()
+            .displayOutLevel(Level.SEVERE)
+            .build())
+        {
+            connect(runner);
+        }
+        catch (Exception ignore) {}
+
+        System.out.println("------------------------------------------------------------------------------");
+    }
+}

src/test/java/nats/io/NatsServerRunnerTest.java

@@ -19,7 +19,6 @@
 
 import java.io.IOException;
 import java.util.Arrays;
-import java.util.logging.Level;
 import java.util.stream.Stream;
 
 public class NatsServerRunnerTest extends TestBase {
@@ -134,15 +133,4 @@ public void testWithConfigBuilder(String configFile, boolean checkConnect) throw
             _testWithConfig(configFile, checkConnect, configInserts, runner);
         }
     }
-
-    // RUN THIS TEST MANUALLY TO SEE THAT THERE IS NO SERVER OUTPUT TO THE CONSOLE
-    // COMMENT OUT THE .displayOutLevel(Level.SEVERE) TO SEE THE OUTPUT APPEAR.
-    @Test
-    public void testDisplayOutLevel() throws IOException, InterruptedException {
-        try (NatsServerRunner runner = NatsServerRunner.builder()
-            .displayOutLevel(Level.SEVERE)
-            .build()) {
-            connect(runner);
-        }
-    }
 }