Author: tilman
Date: Sun Jan  4 08:29:11 2026
New Revision: 1931086

Log:
PDFBOX-5660: improve exception message, adjust test

Modified:
   
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
   
pdfbox/branches/2.0/xmpbox/src/test/java/org/apache/xmpbox/xml/DomXmpParserTest.java

Modified: 
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
==============================================================================
--- 
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
    Sun Jan  4 08:29:06 2026        (r1931085)
+++ 
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
    Sun Jan  4 08:29:11 2026        (r1931086)
@@ -246,31 +246,31 @@ public final class TypeMapping
         }
         catch (NoSuchMethodError e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (IllegalArgumentException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (InstantiationException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (IllegalAccessException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (InvocationTargetException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (SecurityException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
         catch (NoSuchMethodException e)
         {
-            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value " + value, e);
+            throw new IllegalArgumentException("Failed to instantiate " + 
clz.getSimpleName() + " property with value '" + value + "'", e);
         }
     }
 

Modified: 
pdfbox/branches/2.0/xmpbox/src/test/java/org/apache/xmpbox/xml/DomXmpParserTest.java
==============================================================================
--- 
pdfbox/branches/2.0/xmpbox/src/test/java/org/apache/xmpbox/xml/DomXmpParserTest.java
        Sun Jan  4 08:29:06 2026        (r1931085)
+++ 
pdfbox/branches/2.0/xmpbox/src/test/java/org/apache/xmpbox/xml/DomXmpParserTest.java
        Sun Jan  4 08:29:11 2026        (r1931086)
@@ -1505,7 +1505,7 @@ public class DomXmpParserTest
         }
         catch (XmpParsingException ex)
         {
-            assertEquals("Failed to instantiate DateType property with value 
2019-05-02T22:03:5Z in xmp:CreateDate", ex.getMessage());
+            assertEquals("Failed to instantiate DateType property with value 
'2019-05-02T22:03:5Z' in xmp:CreateDate", ex.getMessage());
         }
     }
 
@@ -1529,7 +1529,7 @@ public class DomXmpParserTest
         }
         catch (XmpParsingException ex)
         {
-            assertEquals("Failed to instantiate DateType property with value 
2016-03-09T19:47:1Z in xap:CreateDate", ex.getMessage());
+            assertEquals("Failed to instantiate DateType property with value 
'2016-03-09T19:47:1Z' in xap:CreateDate", ex.getMessage());
         }
     }
 

Reply via email to