fixes networknt#1629 Make error message clear if serviceId is missing…

… from the values.yml (networknt#1630)
younggwon1 · Feb 24, 2023 · ff364f2 · ff364f2
1 parent 0117978
commit ff364f2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/handler/src/main/java/com/networknt/handler/config/PathChain.java b/handler/src/main/java/com/networknt/handler/config/PathChain.java
@@ -76,6 +76,7 @@ public void validate(String origin) {
         if(source == null) {
             if(path == null) {
                 problems.add("You must specify either path or source");
+                problems.add("It is possible that serviceId is missing from the values.yml and it is mandatory.");
             } else if(method == null) {
                 problems.add("You must specify method along with path: " + path);
             }

diff --git a/handler/src/test/java/com/networknt/handler/PathChainTest.java b/handler/src/test/java/com/networknt/handler/PathChainTest.java
@@ -61,7 +61,7 @@ public void validate_BadMethod() {
             Assert.fail("Expected exception");
         } catch (Exception e) {
             System.out.println(e.toString());
-            String ex_message = "Bad paths element in unit test config [ You must specify either path or source ]";
+            String ex_message = "Bad paths element in unit test config [ You must specify either path or source | It is possible that serviceId is missing from the values.yml and it is mandatory. ]";
             Assert.assertEquals(ex_message, e.getMessage());
         }
     }