Fix unit test ConfigurationException throw

Norconex · Sep 16, 2024 · c4981af · c4981af
1 parent b8d93f7
commit c4981af
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/src/main/java/com/norconex/commons/lang/config/vlt/CustomIncludeDirective.java b/src/main/java/com/norconex/commons/lang/config/vlt/CustomIncludeDirective.java
@@ -1,5 +1,6 @@
 package com.norconex.commons.lang.config.vlt;
 
+import com.norconex.commons.lang.config.ConfigurationException;
 import org.apache.velocity.app.event.EventHandlerUtil;
 import org.apache.velocity.context.InternalContextAdapter;
 import org.apache.velocity.exception.MethodInvocationException;
@@ -52,7 +53,7 @@ public boolean render(InternalContextAdapter context,
                 }
             }
         } catch (Exception e) {
-            e.printStackTrace();
+            throw new ConfigurationException(e);
         }
         return true;
     }

diff --git a/src/test/java/com/norconex/commons/lang/config/ConfigurationLoaderTest.java b/src/test/java/com/norconex/commons/lang/config/ConfigurationLoaderTest.java
@@ -183,14 +183,6 @@ void testIndentToString() throws Exception {
                           Age: 50
                           Feels: very old
                             """);
-
-        // null path
-        assertThrows(ConfigurationException.class,
-                () -> loader.toString(null));
-
-
-        // invalid path
-        assertThat(loader.toString(cfgPath("doesntExist"))).isNull();
     }
 
     private Path cfgPath(String path) {