disable logging during tests via system variable, configurable in pom.xml

Author: PhilippSalvisberg

standalone/pom.xml

@@ -19,6 +19,7 @@
         <skip.native>true</skip.native>
         <!-- relevant when skip.native is false, valid: "no", "force" -->
         <native.image.fallback>force</native.image.fallback>
+        <disable.logging>true</disable.logging>
     </properties>
     <dependencies>
         <!-- GraalVM JavaScript engine used for testing format.js as if executed via SQLcl -->
@@ -267,6 +268,9 @@
                     <includes>
                         <include>**/*.java</include>
                     </includes>
+                    <systemPropertyVariables>
+                        <disable.logging>${disable.logging}</disable.logging>
+                    </systemPropertyVariables>
                 </configuration>
             </plugin>
         </plugins>

standalone/src/test/java/com/trivadis/plsql/formatter/settings/ConfiguredTestFormatter.java

@@ -28,11 +28,15 @@ private void setArboriHome() {
     }
 
     private void loadLoggingConf() {
+        var disableLogging = System.getProperty("disable.logging");
         var manager = LogManager.getLogManager();
-        try {
-            manager.readConfiguration(Thread.currentThread().getContextClassLoader().getResourceAsStream("logging.conf"));
-        } catch (SecurityException | IOException e) {
-            e.printStackTrace();
+        manager.reset();
+        if (disableLogging != null && !disableLogging.trim().equalsIgnoreCase("true")) {
+            try {
+                manager.readConfiguration(Thread.currentThread().getContextClassLoader().getResourceAsStream("logging.conf"));
+            } catch (SecurityException | IOException e) {
+                e.printStackTrace();
+            }
         }
     }
 

standalone/src/test/java/com/trivadis/plsql/formatter/sqlcl/tests/AbstractSqlclTest.java

@@ -31,11 +31,15 @@ public abstract class AbstractSqlclTest {
     }
 
     private void loadLoggingConf() {
+        var disableLogging = System.getProperty("disable.logging");
         var manager = LogManager.getLogManager();
-        try {
-            manager.readConfiguration(Thread.currentThread().getContextClassLoader().getResourceAsStream("logging.conf"));
-        } catch (SecurityException | IOException e) {
-            e.printStackTrace();
+        manager.reset();
+        if (disableLogging != null && !disableLogging.trim().equalsIgnoreCase("true")) {
+            try {
+                manager.readConfiguration(Thread.currentThread().getContextClassLoader().getResourceAsStream("logging.conf"));
+            } catch (SecurityException | IOException e) {
+                e.printStackTrace();
+            }
         }
     }