diff options
author | Sam Davis | 2014-05-08 00:03:29 +0000 |
---|---|---|
committer | Sam Davis | 2014-05-08 16:44:35 +0000 |
commit | 5fa632424ac4a18d6ccf0cbfa806127d36e902a3 (patch) | |
tree | 16bcebd3b7101096a8c2451bc011f359f8364918 /org.eclipse.mylyn.tasks.tests/src/org | |
parent | 155dcd67532d50d9b35249be09a5ea3788cf2d37 (diff) | |
download | org.eclipse.mylyn.tasks-5fa632424ac4a18d6ccf0cbfa806127d36e902a3.tar.gz org.eclipse.mylyn.tasks-5fa632424ac4a18d6ccf0cbfa806127d36e902a3.tar.xz org.eclipse.mylyn.tasks-5fa632424ac4a18d6ccf0cbfa806127d36e902a3.zip |
remove unnecessary uses of AtomicReference
Change-Id: Ic260ff94a59e9879397c8b552b3e09ab24bf5b0d
Diffstat (limited to 'org.eclipse.mylyn.tasks.tests/src/org')
-rw-r--r-- | org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/SynchronizeTasksJobTest.java | 44 |
1 files changed, 20 insertions, 24 deletions
diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/SynchronizeTasksJobTest.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/SynchronizeTasksJobTest.java index 41aadcd08..001d68d58 100644 --- a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/SynchronizeTasksJobTest.java +++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/SynchronizeTasksJobTest.java @@ -475,13 +475,12 @@ public class SynchronizeTasksJobTest extends TestCase { } public void testGetSingleTaskDataWithRelations() throws Exception { - final AtomicReference<List<String>> requestedTaskIds = new AtomicReference<List<String>>(); - requestedTaskIds.set(new ArrayList<String>()); + final List<String> requestedTaskIds = new ArrayList<String>(); AbstractRepositoryConnector connector = new MockRepositoryConnectorWithTaskDataHandler() { @Override public TaskData getTaskData(TaskRepository taskRepository, String taskId, IProgressMonitor monitor) throws CoreException { - requestedTaskIds.get().add(taskId); + requestedTaskIds.add(taskId); return createTaskData(taskId); } @@ -503,10 +502,10 @@ public class SynchronizeTasksJobTest extends TestCase { taskList.addTask(task); SynchronizeTasksJob job = createSyncJob(connector, Collections.singleton(task)); job.run(new NullProgressMonitor()); - assertEquals(3, requestedTaskIds.get().size()); - assertTrue(requestedTaskIds.get().contains("1")); - assertTrue(requestedTaskIds.get().contains("1.sub")); - assertTrue(requestedTaskIds.get().contains("1.sub2")); + assertEquals(3, requestedTaskIds.size()); + assertTrue(requestedTaskIds.contains("1")); + assertTrue(requestedTaskIds.contains("1.sub")); + assertTrue(requestedTaskIds.contains("1.sub2")); ITask sub1 = taskList.getTask(MockRepositoryConnector.REPOSITORY_URL, "1.sub"); assertNotNull(sub1); @@ -516,21 +515,20 @@ public class SynchronizeTasksJobTest extends TestCase { assertEquals(SynchronizationState.INCOMING_NEW, ((AbstractTask) sub2).getSynchronizationState()); // same again but this time we already got them in the task list - requestedTaskIds.get().clear(); + requestedTaskIds.clear(); job = createSyncJob(connector, Collections.singleton(task)); job.run(new NullProgressMonitor()); - assertEquals(1, requestedTaskIds.get().size()); - assertTrue(requestedTaskIds.get().contains("1")); + assertEquals(1, requestedTaskIds.size()); + assertTrue(requestedTaskIds.contains("1")); } public void testGetSingleTaskDataWithRelationsAndRemoveRelation() throws Exception { - final AtomicReference<List<String>> requestedTaskIds = new AtomicReference<List<String>>(); - requestedTaskIds.set(new ArrayList<String>()); + final List<String> requestedTaskIds = new ArrayList<String>(); AbstractRepositoryConnector connector = new MockRepositoryConnectorWithTaskDataHandler() { @Override public TaskData getTaskData(TaskRepository taskRepository, String taskId, IProgressMonitor monitor) throws CoreException { - requestedTaskIds.get().add(taskId); + requestedTaskIds.add(taskId); return createTaskData(taskId); } @@ -553,9 +551,9 @@ public class SynchronizeTasksJobTest extends TestCase { taskList.addTask(subtaskToStay, ((AbstractTaskContainer) task)); SynchronizeTasksJob job = createSyncJob(connector, Collections.singleton(task)); job.run(new NullProgressMonitor()); - assertEquals(2, requestedTaskIds.get().size()); - assertTrue(requestedTaskIds.get().contains("1")); - assertTrue(requestedTaskIds.get().contains("1.sub")); + assertEquals(2, requestedTaskIds.size()); + assertTrue(requestedTaskIds.contains("1")); + assertTrue(requestedTaskIds.contains("1.sub")); ITask sub1 = taskList.getTask(MockRepositoryConnector.REPOSITORY_URL, "1.sub"); ITask sub2 = taskList.getTask(MockRepositoryConnector.REPOSITORY_URL, "1.sub2"); @@ -571,13 +569,12 @@ public class SynchronizeTasksJobTest extends TestCase { } public void testErrorOnRelationRetrieval() throws Exception { - final AtomicReference<List<String>> requestedTaskIds = new AtomicReference<List<String>>(); - requestedTaskIds.set(new ArrayList<String>()); + final List<String> requestedTaskIds = new ArrayList<String>(); AbstractRepositoryConnector connector = new MockRepositoryConnectorWithTaskDataHandler() { @Override public TaskData getTaskData(TaskRepository taskRepository, String taskId, IProgressMonitor monitor) throws CoreException { - requestedTaskIds.get().add(taskId); + requestedTaskIds.add(taskId); if (taskId.equals("1.sub")) { throw new CoreException(new Status(IStatus.ERROR, "bundle", "log me")); } @@ -613,13 +610,12 @@ public class SynchronizeTasksJobTest extends TestCase { } public void testTasksForSeveralRepositories() throws Exception { - final AtomicReference<List<String>> requestedTaskIds = new AtomicReference<List<String>>(); - requestedTaskIds.set(new ArrayList<String>()); + final List<String> requestedTaskIds = new ArrayList<String>(); AbstractRepositoryConnector connector = new MockRepositoryConnectorWithTaskDataHandler() { @Override public TaskData getTaskData(TaskRepository taskRepository, String taskId, IProgressMonitor monitor) throws CoreException { - requestedTaskIds.get().add(taskId + " on " + taskRepository.getRepositoryUrl()); + requestedTaskIds.add(taskId + " on " + taskRepository.getRepositoryUrl()); return createTaskData(taskId); } @@ -662,8 +658,8 @@ public class SynchronizeTasksJobTest extends TestCase { }); assertEquals("beginTask|subTask|subTask|subTask|subTask|done", progressLog.toString()); - assertTrue(requestedTaskIds.get().contains("1 on " + MockRepositoryConnector.REPOSITORY_URL)); - assertTrue(requestedTaskIds.get().contains("5 on " + MockRepositoryConnector.REPOSITORY_URL + "2")); + assertTrue(requestedTaskIds.contains("1 on " + MockRepositoryConnector.REPOSITORY_URL)); + assertTrue(requestedTaskIds.contains("5 on " + MockRepositoryConnector.REPOSITORY_URL + "2")); ((TaskRepositoryManager) TasksUi.getRepositoryManager()).removeRepository(secondRepository); ((TaskRepositoryManager) TasksUi.getRepositoryManager()).removeRepository(firstRepository); |