From aa21c11f2a40b4b65421f61b9603b8ec0e07ae40 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Fri, 27 Jul 2018 10:53:53 +0100 Subject: [PATCH] AbstractQueryChangesTest: Add a test with project name that requires URL encoding This test is passing on both Lucene and Elasticsearch, but exposes the problem in the staleness checker as reported in issue 9482. When running the test, the RepositoryNotFoundException can be seen in the logs. Bug: Issue 9482 Change-Id: I3dfe66e0f2051a6743963c1660ba435b1f8694d3 --- .../server/query/change/AbstractQueryChangesTest.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java b/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java index 6ce0188ce2..0e0387113e 100644 --- a/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java +++ b/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java @@ -2245,6 +2245,13 @@ public abstract class AbstractQueryChangesTest extends GerritServerTests { assertQuery(query); } + @Test + public void byUrlEncodedProject() throws Exception { + TestRepository repo = createProject("repo+foo"); + Change change = insert(repo, newChange(repo)); + assertQuery("project:repo+foo", change); + } + protected ChangeInserter newChange(TestRepository repo) throws Exception { return newChange(repo, null, null, null, null); }