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());
}
}