aboutsummaryrefslogtreecommitdiffstats
path: root/indexinglanguage
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@gmail.com>2023-01-19 19:01:11 +0100
committerJon Bratseth <bratseth@gmail.com>2023-01-19 19:01:11 +0100
commitb4de61b832dbb50ad8d7a1100e384869538bcd83 (patch)
tree4b115ae8d6aca844ac24fbcbaae06829446153dc /indexinglanguage
parent8ce1d5213208bc434582f664ea671cf08b75c31d (diff)
Expect the correction exceptions
Diffstat (limited to 'indexinglanguage')
-rw-r--r--indexinglanguage/src/main/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolver.java2
-rw-r--r--indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolverTestCase.java4
2 files changed, 3 insertions, 3 deletions
diff --git a/indexinglanguage/src/main/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolver.java b/indexinglanguage/src/main/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolver.java
index 4cf5b24941e..47d4e789602 100644
--- a/indexinglanguage/src/main/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolver.java
+++ b/indexinglanguage/src/main/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolver.java
@@ -15,7 +15,7 @@ public class MathResolver {
public void push(ArithmeticExpression.Operator op, Expression exp) {
if (items.isEmpty() && op != ArithmeticExpression.Operator.ADD)
- throw new IllegalArgumentException("First item in an arithmetic operation must be an addition.");
+ throw new IllegalArgumentException("First item in an arithmetic operation must be an addition, not " + op);
items.add(new Item(op, exp));
}
diff --git a/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolverTestCase.java b/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolverTestCase.java
index decd0d7670e..1f9afdea44d 100644
--- a/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolverTestCase.java
+++ b/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/MathResolverTestCase.java
@@ -71,7 +71,7 @@ public class MathResolverTestCase {
try {
resolver.push(type, newInteger(69));
} catch (IllegalArgumentException e) {
- assertEquals("First item in an arithmetic operation must be an addition.", e.getMessage());
+ assertEquals("First item in an arithmetic operation must be an addition, not " + type, e.getMessage());
}
}
}
@@ -81,7 +81,7 @@ public class MathResolverTestCase {
try {
new MathResolver().push(null, newInteger(69));
fail();
- } catch (NullPointerException e) {
+ } catch (IllegalArgumentException e) {
}
}