summaryrefslogtreecommitdiffstats
path: root/vespajlib/src
diff options
context:
space:
mode:
authorJon Marius Venstad <jonmv@gmail.com>2022-04-07 16:36:31 +0200
committerJon Marius Venstad <jonmv@gmail.com>2022-04-07 16:36:31 +0200
commit6c4304d75f3db1494c1059339c07f68e620f5d65 (patch)
tree50f64b14d96d0e05ef85070652fbb03154eb7c01 /vespajlib/src
parent75114c6c4b96f507071a2caf145b24a22ef0c5ff (diff)
Some more unit test coverage of query manipulation
Diffstat (limited to 'vespajlib/src')
-rw-r--r--vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java15
1 files changed, 14 insertions, 1 deletions
diff --git a/vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java b/vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java
index 85e78c0b7af..3dbb022148b 100644
--- a/vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java
+++ b/vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java
@@ -181,8 +181,21 @@ class HttpURLTest {
assertEquals("query '?foo=bar&baz=bax&quu=fez&moo'",
query.set("baz", "bax").set(Map.of("quu", "fez")).set("moo").toString());
+ Query bloated = query.add("baz", "bax").add(Map.of("quu", List.of("fez", "pop"))).add("moo").add("moo").add("foo", "bar");
assertEquals("query '?foo=bar&baz&baz=bax&quu=fez&quu=pop&moo&moo&foo=bar'",
- query.add("baz", "bax").add(Map.of("quu", List.of("fez", "pop"))).add("moo").add("moo").add("foo", "bar").toString());
+ bloated.toString());
+
+ assertEquals("query '?foo=bar&quu=fez&quu=pop&moo&moo&foo=bar'",
+ bloated.remove("baz").toString());
+
+ assertEquals("query '?baz&baz=bax&quu=fez&quu=pop&moo&moo'",
+ bloated.remove("foo").toString());
+
+ assertEquals("query '?foo=bar&baz&baz=bax&quu=fez&quu=pop&foo=bar&moo'",
+ bloated.set("moo").toString());
+
+ assertEquals("query ''",
+ bloated.remove("foo").remove("baz").remove("quu").remove("moo").toString());
assertThrows(NullPointerException.class,
() -> query.remove(null));