diff options
author | Jon Bratseth <bratseth@gmail.com> | 2023-01-19 19:01:11 +0100 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2023-01-19 19:01:11 +0100 |
commit | b4de61b832dbb50ad8d7a1100e384869538bcd83 (patch) | |
tree | 4b115ae8d6aca844ac24fbcbaae06829446153dc /indexinglanguage | |
parent | 8ce1d5213208bc434582f664ea671cf08b75c31d (diff) |
Expect the correction exceptions
Diffstat (limited to 'indexinglanguage')
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) { } } |