Improve message on mapping generation failure (#1175)

smallrye · Jun 13, 2024 · 6399d0a · 6399d0a
1 parent 16f10da
commit 6399d0a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/implementation/src/main/java/io/smallrye/config/ConfigMappingContext.java b/implementation/src/main/java/io/smallrye/config/ConfigMappingContext.java
@@ -148,7 +148,6 @@ public StringBuilder getNameBuilder() {
 
     @SuppressWarnings("unused")
     public void reportProblem(RuntimeException problem) {
-        problem.printStackTrace();
         problems.add(new Problem(problem.toString()));
     }
 

diff --git a/implementation/src/main/java/io/smallrye/config/ConfigMappingInterface.java b/implementation/src/main/java/io/smallrye/config/ConfigMappingInterface.java
@@ -167,7 +167,11 @@ List<ConfigMappingInterface> getNested() {
     }
 
     public byte[] getClassBytes() {
-        return ConfigMappingGenerator.generate(this);
+        try {
+            return ConfigMappingGenerator.generate(this);
+        } catch (Throwable e) {
+            throw ConfigMessages.msg.couldNotGenerateMapping(e);
+        }
     }
 
     public static abstract class Property {

diff --git a/implementation/src/main/java/io/smallrye/config/_private/ConfigMessages.java b/implementation/src/main/java/io/smallrye/config/_private/ConfigMessages.java
@@ -169,4 +169,7 @@ IllegalArgumentException converterException(@Cause Throwable converterException,
 
     @Message(id = 50, value = "%s in %s does not map to any root")
     IllegalStateException propertyDoesNotMapToAnyRoot(String name, String location);
+
+    @Message(id = 51, value = "Could not generate ConfigMapping")
+    IllegalStateException couldNotGenerateMapping(@Cause Throwable throwable);
 }