summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahooinc.com>2022-03-09 14:08:38 +0000
committerArne H Juul <arnej@yahooinc.com>2022-03-09 14:08:38 +0000
commitaf75acd826dd5a6b6894bb16dfb12ecb266a2525 (patch)
tree2cf38fc5ac6f8229ee400c3223dca845386b6197
parent7ce318cae722f791eade72bc3106143d251f8bbc (diff)
more informative exception message
-rw-r--r--config-model/src/main/java/com/yahoo/searchdefinition/SDDocumentTypeOrderer.java5
-rw-r--r--config-model/src/test/java/com/yahoo/searchdefinition/derived/InheritanceTestCase.java2
2 files changed, 5 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/SDDocumentTypeOrderer.java b/config-model/src/main/java/com/yahoo/searchdefinition/SDDocumentTypeOrderer.java
index ba34045e7de..848631f1739 100644
--- a/config-model/src/main/java/com/yahoo/searchdefinition/SDDocumentTypeOrderer.java
+++ b/config-model/src/main/java/com/yahoo/searchdefinition/SDDocumentTypeOrderer.java
@@ -81,7 +81,10 @@ public class SDDocumentTypeOrderer {
}
else {
inherited = createdSDTypes.get(name);
- if (inherited == null) throw new IllegalArgumentException("Document type '" + name + "' not found");
+ if (inherited == null) {
+ throw new IllegalArgumentException("document " + type.getName() +
+ " inherits from unavailable document " + name);
+ }
process(inherited, inherited);
}
type.inherit(inherited);
diff --git a/config-model/src/test/java/com/yahoo/searchdefinition/derived/InheritanceTestCase.java b/config-model/src/test/java/com/yahoo/searchdefinition/derived/InheritanceTestCase.java
index bcf68387294..3bd1e568400 100644
--- a/config-model/src/test/java/com/yahoo/searchdefinition/derived/InheritanceTestCase.java
+++ b/config-model/src/test/java/com/yahoo/searchdefinition/derived/InheritanceTestCase.java
@@ -54,7 +54,7 @@ public class InheritanceTestCase extends AbstractExportingTestCase {
try {
assertCorrectDeriving("inheritfromnull");
} catch (IllegalArgumentException e) {
- assertEquals("Document type 'foo' not found", e.getMessage());
+ assertEquals("document inheritfromnull inherits from unavailable document foo", e.getMessage());
}
}