From de32d9a9e19e5fd8d0993cd2c1bab1f9706c92df Mon Sep 17 00:00:00 2001
From: Sam Davis
Date: Tue, 6 Sep 2016 10:46:19 -0700
Subject: GerritSynchronizationTest should synchronize specific query instead
 of repository

Change-Id: I10cf29c5ade59aa4688f711775b0485bea60e555
---
 .../tests/core/GerritSynchronizationTest.java      | 37 ++++++++++++----------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/gerrit/tests/core/GerritSynchronizationTest.java b/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/gerrit/tests/core/GerritSynchronizationTest.java
index 4a6b4bff7..472ccebde 100644
--- a/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/gerrit/tests/core/GerritSynchronizationTest.java
+++ b/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/gerrit/tests/core/GerritSynchronizationTest.java
@@ -21,8 +21,6 @@ import java.io.File;
 import java.util.Collections;
 import java.util.List;
 
-import junit.framework.TestCase;
-
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.core.runtime.NullProgressMonitor;
 import org.eclipse.core.runtime.jobs.Job;
@@ -64,6 +62,8 @@ import org.junit.Test;
 import com.google.gerrit.reviewdb.ApprovalCategoryValue;
 import com.google.gerrit.reviewdb.Project;
 
+import junit.framework.TestCase;
+
 /**
  * @author Steffen Pingel
  */
@@ -100,7 +100,8 @@ public class GerritSynchronizationTest extends TestCase {
 				.getConnector()
 				.setFactoryProviderConfigurer(new RemoteUiFactoryProviderConfigurer());
 		client = GerritCorePlugin.getDefault().getConnector().getClient(repository);
-		AbstractRemoteEditFactoryProvider abstractRemoteEditFactoryProvider = (AbstractRemoteEditFactoryProvider) client.getFactoryProvider();
+		AbstractRemoteEditFactoryProvider abstractRemoteEditFactoryProvider = (AbstractRemoteEditFactoryProvider) client
+				.getFactoryProvider();
 		GerritCorePlugin.getDefault()
 				.getConnector()
 				.getFactoryProviderConfigurer()
@@ -125,7 +126,8 @@ public class GerritSynchronizationTest extends TestCase {
 	public void testSynchronizeBackgroundQueryTaskUpdated() throws Exception {
 		ITask task = createAndSynchronizeQuery(false);
 		String message = addComment(task);
-		synchronizeRepository(false);
+		RepositoryQuery query = taskList.getQueries().iterator().next();
+		synchronizeQuery(query, false);
 		assertHasNewComment(task, message);
 	}
 
@@ -146,7 +148,8 @@ public class GerritSynchronizationTest extends TestCase {
 	public void testSynchronizeQueryTaskUpdated() throws Exception {
 		ITask task = createAndSynchronizeQuery(true);
 		String message = addComment(task);
-		synchronizeRepository(true);
+		RepositoryQuery query = taskList.getQueries().iterator().next();
+		synchronizeQuery(query, true);
 		assertHasNewComment(task, message);
 	}
 
@@ -163,10 +166,8 @@ public class GerritSynchronizationTest extends TestCase {
 		ITask task = createAndSynchronizeQuery(true);
 		AbstractRepositoryConnector connector = TasksUi.getRepositoryConnector(repository.getConnectorKind());
 		GerritClient client = ((GerritConnector) connector).getClient(repository);
-		Project.NameKey project = new Project.NameKey(taskDataManager.getTaskData(task)
-				.getRoot()
-				.getMappedAttribute(TaskAttribute.PRODUCT)
-				.getValue());
+		Project.NameKey project = new Project.NameKey(
+				taskDataManager.getTaskData(task).getRoot().getMappedAttribute(TaskAttribute.PRODUCT).getValue());
 		client.clearCachedBranches(project);
 		assertNull(client.getCachedBranches(project));
 
@@ -174,8 +175,8 @@ public class GerritSynchronizationTest extends TestCase {
 		assertEquals(task.getTaskId(), taskData.getTaskId());
 		assertNotNull(client.getCachedBranches(project));
 
-		TaskAttribute changeIdAttribute = taskData.getRoot().getAttribute(
-				GerritTaskSchema.getDefault().CHANGE_ID.getKey());
+		TaskAttribute changeIdAttribute = taskData.getRoot()
+				.getAttribute(GerritTaskSchema.getDefault().CHANGE_ID.getKey());
 		assertNotNull(changeIdAttribute);
 		assertThat(task.getTaskId(), not(equalTo(changeIdAttribute.getValue())));
 		TaskData taskDataFromChangeId = connector.getTaskData(repository, changeIdAttribute.getValue(),
@@ -200,8 +201,8 @@ public class GerritSynchronizationTest extends TestCase {
 		List<TaskAttribute> comments = taskData.getAttributeMapper().getAttributesByType(taskData,
 				TaskAttribute.TYPE_COMMENT);
 		TaskCommentMapper lastComment = TaskCommentMapper.createFrom(comments.get(comments.size() - 1));
-		assertEquals(
-				"Failure on " + GerritFixture.current().getRepositoryUrl() + "/" + task.getTaskId(), "Patch Set 1:\n\n" + message, lastComment.getText()); //$NON-NLS-1$
+		assertEquals("Failure on " + GerritFixture.current().getRepositoryUrl() + "/" + task.getTaskId(), //$NON-NLS-1$
+				"Patch Set 1:\n\n" + message, lastComment.getText());
 	}
 
 	private ITask assertTaskListHasOneTask() throws CoreException {
@@ -218,7 +219,7 @@ public class GerritSynchronizationTest extends TestCase {
 		query.setAttribute(GerritQuery.QUERY_STRING, harness.defaultQuery());
 		taskList.addQuery((RepositoryQuery) query);
 
-		synchronizeRepository(user);
+		synchronizeQuery((RepositoryQuery) query, user);
 
 		ITask task = assertTaskListHasOneTask();
 		assertEquals(SynchronizationState.INCOMING_NEW, task.getSynchronizationState());
@@ -232,9 +233,11 @@ public class GerritSynchronizationTest extends TestCase {
 		return task;
 	}
 
-	private void synchronizeRepository(boolean user) throws InterruptedException {
-		SynchronizationJob job = TasksUiPlugin.getTaskJobFactory().createSynchronizeRepositoriesJob(
-				Collections.singleton(repository));
+	private void synchronizeQuery(RepositoryQuery query, boolean user) throws InterruptedException {
+		AbstractRepositoryConnector connector = TasksUiPlugin.getRepositoryManager()
+				.getRepositoryConnector(repository.getConnectorKind());
+		SynchronizationJob job = TasksUiPlugin.getTaskJobFactory().createSynchronizeQueriesJob(connector, repository,
+				Collections.singleton(query));
 		job.setUser(user);
 		job.schedule();
 		job.join();
-- 
cgit v1.2.3