[build] improve test setup

- add logback-test.xml to stop logcaptor to spam to terminal during test runs
- remove maven-failsafe-plugin configuration as it is not used in this module
- cleanup of RangeIndexConfigTest (try-with resources, no exceptions are thrown)

Author: line-o

extensions/indexes/range/pom.xml

@@ -145,16 +145,6 @@
                     </classpathDependencyExcludes>
                 </configuration>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-failsafe-plugin</artifactId>
-                <configuration>
-                    <classpathDependencyExcludes>
-                        <classpathDependencyExclude>org.apache.logging.log4j:log4j-slf4j-impl</classpathDependencyExclude>
-                        <classpathDependencyExclude>org.apache.logging.log4j:log4j-slf4j2-impl</classpathDependencyExclude>
-                    </classpathDependencyExcludes>
-                </configuration>
-            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-dependency-plugin</artifactId>

extensions/indexes/range/src/test/java/org/exist/indexing/range/RangeIndexConfigTest.java

@@ -45,7 +45,7 @@ public class RangeIndexConfigTest {
      * {@see https://github.com/eXist-db/exist/issues/1339}
      */
     @Test
-    public void errorsHaveSourceContext() throws NoSuchFieldException, IllegalAccessException {
+    public void errorsHaveSourceContext() {
         final String badCreateQName = "tei:persName "; // Note the trailing
         final String mockCollectionXConfUri = "/db/system/conf/db/mock/" + DEFAULT_COLLECTION_CONFIG_FILE;
 
@@ -89,15 +89,16 @@ public void errorsHaveSourceContext() throws NoSuchFieldException, IllegalAccess
         final Map<String, String> namespaces = new HashMap<>();
         namespaces.put("tei", "http://www.tei-c.org/ns/1.0");
 
-        LogCaptor logCaptor = LogCaptor.forClass(RangeIndexConfig.class);
+        try (LogCaptor logCaptor = LogCaptor.forClass(RangeIndexConfig.class)) {
 
-        final RangeIndexConfig config = new RangeIndexConfig(mockConfigNodes, namespaces);
+            final RangeIndexConfig config = new RangeIndexConfig(mockConfigNodes, namespaces);
 
-        assertTrue(logCaptor.getLogs().getFirst()
-                .contains("Illegal QName: '" + badCreateQName + "'.. QName is invalid: INVALID_LOCAL_PART"));
+            assertTrue(logCaptor.getLogs().getFirst()
+                    .contains("Illegal QName: '" + badCreateQName + "'.. QName is invalid: INVALID_LOCAL_PART"));
 
-        assertTrue(logCaptor.getLogs().getFirst()
-                .contains("(" + mockCollectionXConfUri + ")"));
+            assertTrue(logCaptor.getLogs().getFirst()
+                    .contains("(" + mockCollectionXConfUri + ")"));
+        }
 
         verify(mockConfigNodes, mockConfigNode, mockCreates, mockCreateDocument, mockCreate, mockEmptyNodeList);
     }

extensions/indexes/range/src/test/resources/logback-test.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+    eXist-db Open Source Native XML Database
+    Copyright (C) 2001 The eXist-db Authors
+
+    [email protected]
+    http://www.exist-db.org
+
+    This library is free software; you can redistribute it and/or
+    modify it under the terms of the GNU Lesser General Public
+    License as published by the Free Software Foundation; either
+    version 2.1 of the License, or (at your option) any later version.
+
+    This library is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+    Lesser General Public License for more details.
+
+    You should have received a copy of the GNU Lesser General Public
+    License along with this library; if not, write to the Free Software
+    Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+
+-->
+<configuration>
+    <statusListener class="ch.qos.logback.core.status.NopStatusListener" />
+</configuration>