From 6c4304d75f3db1494c1059339c07f68e620f5d65 Mon Sep 17 00:00:00 2001 From: Jon Marius Venstad Date: Thu, 7 Apr 2022 16:36:31 +0200 Subject: Some more unit test coverage of query manipulation --- vespajlib/src/test/java/ai/vespa/http/HttpURLTest.java | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'vespajlib/src') 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)); -- cgit v1.2.3