aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArne Juul <arnej@vespa.ai>2023-11-06 13:28:13 +0000
committerArne Juul <arnej@vespa.ai>2023-11-06 13:28:13 +0000
commit634608cfe7088bb106a5e63e2008e9f91c80b0ff (patch)
tree092bf6cd05dd6c83e7124471e8d3ae261b4dd632
parent0949449969d922211bfec5e6dd27899c46d3d455 (diff)
stop using NameNode for tests
-rw-r--r--searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/ReferenceTestCase.java2
-rw-r--r--searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ArgumentsTestCase.java10
-rw-r--r--searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ReferenceNodeTestCase.java14
3 files changed, 12 insertions, 14 deletions
diff --git a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/ReferenceTestCase.java b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/ReferenceTestCase.java
index 5899d4fc07e..fd361d4aaa3 100644
--- a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/ReferenceTestCase.java
+++ b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/ReferenceTestCase.java
@@ -19,7 +19,6 @@ public class ReferenceTestCase {
public void testSimple() {
assertTrue(new Reference("foo", new Arguments(new ReferenceNode("arg")), null).isSimple());
assertTrue(new Reference("foo", new Arguments(new ReferenceNode("arg")), "out").isSimple());
- assertTrue(new Reference("foo", new Arguments(new NameNode("arg")), "out").isSimple());
assertFalse(new Reference("foo", new Arguments(), null).isSimple());
}
@@ -27,7 +26,6 @@ public class ReferenceTestCase {
public void testToString() {
assertEquals("foo(arg_1)", new Reference("foo", new Arguments(new ReferenceNode("arg_1")), null).toString());
assertEquals("foo(arg_1).out", new Reference("foo", new Arguments(new ReferenceNode("arg_1")), "out").toString());
- assertEquals("foo(arg_1).out", new Reference("foo", new Arguments(new NameNode("arg_1")), "out").toString());
assertEquals("foo", new Reference("foo", new Arguments(), null).toString());
}
diff --git a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ArgumentsTestCase.java b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ArgumentsTestCase.java
index 2d38c58703c..df91c78c1b4 100644
--- a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ArgumentsTestCase.java
+++ b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ArgumentsTestCase.java
@@ -21,8 +21,8 @@ public class ArgumentsTestCase {
args = new Arguments(Collections.<ExpressionNode>emptyList());
assertTrue(args.expressions().isEmpty());
- NameNode foo = new NameNode("foo");
- NameNode bar = new NameNode("bar");
+ var foo = new ReferenceNode("foo");
+ var bar = new ReferenceNode("bar");
args = new Arguments(Arrays.asList(foo, bar));
assertEquals(2, args.expressions().size());
assertSame(foo, args.expressions().get(0));
@@ -31,9 +31,9 @@ public class ArgumentsTestCase {
@Test
public void requireThatHashCodeAndEqualsWork() {
- Arguments arg1 = new Arguments(Arrays.asList(new NameNode("foo"), new NameNode("bar")));
- Arguments arg2 = new Arguments(Arrays.asList(new NameNode("foo"), new NameNode("bar")));
- Arguments arg3 = new Arguments(Arrays.asList(new NameNode("foo")));
+ Arguments arg1 = new Arguments(Arrays.asList(new ReferenceNode("foo"), new ReferenceNode("bar")));
+ Arguments arg2 = new Arguments(Arrays.asList(new ReferenceNode("foo"), new ReferenceNode("bar")));
+ Arguments arg3 = new Arguments(Arrays.asList(new ReferenceNode("foo")));
assertEquals(arg1.hashCode(), arg2.hashCode());
assertTrue(arg1.equals(arg2));
diff --git a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ReferenceNodeTestCase.java b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ReferenceNodeTestCase.java
index 225748bbc2e..7d0e0d6da84 100644
--- a/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ReferenceNodeTestCase.java
+++ b/searchlib/src/test/java/com/yahoo/searchlib/rankingexpression/rule/ReferenceNodeTestCase.java
@@ -15,19 +15,19 @@ public class ReferenceNodeTestCase {
@Test
public void requireThatAccessorsWork() {
- ReferenceNode node = new ReferenceNode("foo", Arrays.asList(new NameNode("bar"), new NameNode("baz")), "cox");
+ ReferenceNode node = new ReferenceNode("foo", Arrays.asList(new ReferenceNode("bar"), new ReferenceNode("baz")), "cox");
assertEquals("foo", node.getName());
List<ExpressionNode> args = node.getArguments().expressions();
assertEquals(2, args.size());
- assertEquals(new NameNode("bar"), args.get(0));
- assertEquals(new NameNode("baz"), args.get(1));
+ assertEquals(new ReferenceNode("bar"), args.get(0));
+ assertEquals(new ReferenceNode("baz"), args.get(1));
assertEquals("cox", node.getOutput());
- node = node.setArguments(Arrays.<ExpressionNode>asList(new NameNode("bar'")));
- assertEquals(new NameNode("bar'"), node.getArguments().expressions().get(0));
+ node = node.setArguments(Arrays.<ExpressionNode>asList(new ReferenceNode("bar@")));
+ assertEquals(new ReferenceNode("bar@"), node.getArguments().expressions().get(0));
- node = node.setArguments(Arrays.<ExpressionNode>asList(new NameNode("baz'")));
- assertEquals(new NameNode("baz'"), node.getArguments().expressions().get(0));
+ node = node.setArguments(Arrays.<ExpressionNode>asList(new ReferenceNode("baz$")));
+ assertEquals(new ReferenceNode("baz$"), node.getArguments().expressions().get(0));
node = node.setOutput("cox'");
assertEquals("cox'", node.getOutput());