NEW - bug 162094: [upstream] query by milestone returns no result when

milestone name includes ampersand 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=162094

2 files changed, 20 insertions, 4 deletions

diff --git a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracClientTest.java b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracClientTest.java
index b87ef64e5..fb11a4c3f 100644
--- a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracClientTest.java
+++ b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracClientTest.java
@@ -168,10 +168,6 @@ public class TracClientTest extends TestCase {
 	}
 
 	public void testSearchMilestoneAmpersand() throws Exception {
-		// XXX re-enable for XML-RPC when bug 162094 is resolved
-		if (client.getAccessMode() == Version.XML_RPC) {
-			return;
-		}
 		TracSearch search = new TracSearch();
 		search.addFilter("milestone", "mile&stone");
 		search.setOrderBy("id");
diff --git a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracSearchTest.java b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracSearchTest.java
index ced5053a0..7371f3d7c 100644
--- a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracSearchTest.java
+++ b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/client/TracSearchTest.java
@@ -133,4 +133,24 @@ public class TracSearchTest extends TestCase {
 		assertEquals("&order=summary" + QUERY1, search1.toQuery());
 	}
 
+	public void testToQueryAmpersand() {
+		TracSearch search = new TracSearch();
+		search.addFilter("milestone", "mile&stone&");
+		assertEquals("&milestone=mile\\&stone\\&", search.toQuery());
+	}
+
+	public void testToQueryPipe() {
+		TracSearch search = new TracSearch();
+		search.addFilter("field", "|pipes||");
+		assertEquals("&field=\\|pipes\\|\\|", search.toQuery());
+	}
+
+	public void testToQueryPipeAmpersand() {
+		TracSearch search = new TracSearch();
+		search.addFilter("pipe", "|");
+		search.addFilter("amp", "&");
+		search.addFilter("amppipe", "|&");
+		assertEquals("&pipe=\\|&amp=\\&&amppipe=\\|\\&", search.toQuery());
+	}
+
 }