From 1e364eccf842c5da46e8b0f53d6e0c8868ece3d5 Mon Sep 17 00:00:00 2001 From: gregw Date: Wed, 17 Feb 2021 12:41:55 +0100 Subject: [PATCH] Re re Fix rewrite tests for #5954 Re re Fix rewrite tests for #5954. Restore setPathQuery behaviour to preserve queries if none are set. --- .../src/test/java/org/eclipse/jetty/http/HttpURITest.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/jetty-http/src/test/java/org/eclipse/jetty/http/HttpURITest.java b/jetty-http/src/test/java/org/eclipse/jetty/http/HttpURITest.java index 98f78ac896eb..53e385f2942c 100644 --- a/jetty-http/src/test/java/org/eclipse/jetty/http/HttpURITest.java +++ b/jetty-http/src/test/java/org/eclipse/jetty/http/HttpURITest.java @@ -256,6 +256,9 @@ public void testSetters() throws Exception assertThat(uri.getPathQuery(), is("/other;xxx/path;ppp?query")); uri.setPathQuery(null); + assertEquals("http://host:8080?query", uri.toString()); // Yes silly result! + + uri.setQuery(null); assertEquals("http://host:8080", uri.toString()); uri.setPathQuery("/other;xxx/path;ppp?query");