Skip to content

Commit 943780e

Browse files
committed
Fixes #28.
1 parent 7819d05 commit 943780e

File tree

2 files changed

+22
-4
lines changed

2 files changed

+22
-4
lines changed

compiler/src/main/java/org/hisrc/jsonix/configuration/AmbiguousPackageMappingException.java

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package org.hisrc.jsonix.configuration;
22

33
import java.text.MessageFormat;
4+
import java.util.Arrays;
45

56
import org.apache.commons.lang3.Validate;
67

@@ -12,10 +13,13 @@ public class AmbiguousPackageMappingException extends ConfigurationException {
1213

1314
public AmbiguousPackageMappingException(String packageName,
1415
String... mappingNames) {
15-
super(MessageFormat.format(
16-
"Package [{0}] is mapped using different mapping names [{1}].",
17-
Validate.notNull(packageName),
18-
Validate.noNullElements(mappingNames).toString()));
16+
super(
17+
MessageFormat
18+
.format("Package [{0}] is mapped using different mapping names [{1}].",
19+
Validate.notNull(packageName),
20+
Arrays.asList(
21+
Validate.noNullElements(mappingNames))
22+
.toString()));
1923
this.packageName = packageName;
2024
this.mappingNames = mappingNames;
2125
}
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
package org.hisrc.jsonix.configuration.tests;
2+
3+
import org.hisrc.jsonix.configuration.AmbiguousPackageMappingException;
4+
import org.junit.Assert;
5+
import org.junit.Test;
6+
7+
public class AmbiguousPackageMappingExceptionTest {
8+
9+
@Test
10+
public void producesReasableMessage() {
11+
final Exception ex = new AmbiguousPackageMappingException("a", "b", "c");
12+
Assert.assertTrue(ex.getMessage().contains("b, c"));
13+
}
14+
}

0 commit comments

Comments
 (0)