From c4981af83f49c56f88b5433b68e5d0acfbfc481c Mon Sep 17 00:00:00 2001
From: Ryan Ng <ryan.ng@norconex.com>
Date: Mon, 16 Sep 2024 14:36:29 -0600
Subject: [PATCH] Fix unit test ConfigurationException throw

---
 .../commons/lang/config/vlt/CustomIncludeDirective.java   | 3 ++-
 .../commons/lang/config/ConfigurationLoaderTest.java      | 8 --------
 2 files changed, 2 insertions(+), 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
index fd9a6c26..d424167a 100644
--- 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
index 425cab41..14db4256 100644
--- 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) {