diff options
author | Jon Marius Venstad <jonmv@gmail.com> | 2022-04-07 16:36:31 +0200 |
---|---|---|
committer | Jon Marius Venstad <jonmv@gmail.com> | 2022-04-07 16:36:31 +0200 |
commit | 6c4304d75f3db1494c1059339c07f68e620f5d65 (patch) | |
tree | 50f64b14d96d0e05ef85070652fbb03154eb7c01 /vespajlib | |
parent | 75114c6c4b96f507071a2caf145b24a22ef0c5ff (diff) |
Some more unit test coverage of query manipulation
Diffstat (limited to 'vespajlib')
-rw-r--r-- | vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java | 15 |
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)); |