476383: add an attribute editor for label fields

Add a text attribute editor that supports single and multi values and
has content assist

Change-Id: I9e53666e713abb1affbb7b6376c5af7192174ddb
Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=476383
Signed-off-by: chris.poon <chris.poon@tasktop.com>

2 files changed, 177 insertions, 0 deletions

diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/AllTasksTests.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/AllTasksTests.java
index 4aab2e9ca..a819eb752 100644
--- a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/AllTasksTests.java
+++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/AllTasksTests.java
@@ -108,6 +108,7 @@ public class AllTasksTests {
 		suite.addTestSuite(TaskDataStoreTest.class);
 		suite.addTestSuite(TaskExportImportTest.class);
 		suite.addTestSuite(PersonProposalProviderTest.class);
+		suite.addTestSuite(OptionsProposalProviderTest.class);
 		suite.addTestSuite(TaskRepositoryLocationTest.class);
 		suite.addTestSuite(TaskRepositoryTest.class);
 		suite.addTestSuite(AttachmentSizeFormatterTest.class);
diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/OptionsProposalProviderTest.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/OptionsProposalProviderTest.java
new file mode 100644
index 000000000..04a24cb18
--- /dev/null
+++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/OptionsProposalProviderTest.java
@@ -0,0 +1,176 @@
+/*******************************************************************************
+ * Copyright (c) 2015 Tasktop Technologies and others.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     Tasktop Technologies - initial API and implementation
+ *******************************************************************************/
+
+package org.eclipse.mylyn.tasks.tests;
+
+import java.util.Arrays;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
+import junit.framework.TestCase;
+
+import org.eclipse.jface.fieldassist.IContentProposal;
+import org.eclipse.mylyn.internal.tasks.ui.OptionsProposalProvider;
+import org.junit.Test;
+
+import com.google.common.collect.ImmutableMap;
+
+public class OptionsProposalProviderTest extends TestCase {
+
+	@Override
+	protected void setUp() throws Exception {
+		TaskTestUtil.resetTaskListAndRepositories();
+	}
+
+	@Override
+	protected void tearDown() throws Exception {
+		TaskTestUtil.resetTaskListAndRepositories();
+	}
+
+	@Test
+	public void testEmptyProposals() {
+		OptionsProposalProvider provider = new OptionsProposalProvider(new HashMap<String, String>(), true);
+		assertEquals(0, provider.getProposals("", 0).length);
+		assertEquals(0, provider.getProposals("", 10).length);
+		assertEquals(0, provider.getProposals("test", 0).length);
+	}
+
+	@Test
+	public void testSingleProposal() {
+		final Map<String, String> proposalMap = ImmutableMap.of("aTest", "1");
+		OptionsProposalProvider provider = new OptionsProposalProvider(proposalMap, true);
+
+		List<IContentProposal> proposals = Arrays.asList(provider.getProposals("", 0));
+		assertEquals(1, proposals.size());
+		assertProposal("aTest", "aTest", proposals.get(0));
+	}
+
+	@Test
+	public void testMultipleProposals() {
+		final Map<String, String> proposalMap = ImmutableMap.of("aTest", "1", "bTest", "2", "cTest", "3");
+		OptionsProposalProvider provider = new OptionsProposalProvider(proposalMap, true);
+
+		List<IContentProposal> proposals = Arrays.asList(provider.getProposals("", 0));
+		assertEquals(3, proposals.size());
+		assertProposal("aTest", "aTest", proposals.get(0));
+		assertProposal("bTest", "bTest", proposals.get(1));
+		assertProposal("cTest", "cTest", proposals.get(2));
+	}
+
+	@Test
+	public void testMultipleProposalsSorted() {
+		final Map<String, String> proposalMap = ImmutableMap.of("oneTest", "1", "twoTest", "2", "threeTest", "3");
+		OptionsProposalProvider provider = new OptionsProposalProvider(proposalMap, true);
+
+		List<IContentProposal> proposals = Arrays.asList(provider.getProposals("", 0));
+		assertEquals(3, proposals.size());
+		// NOTE: Sorted alphabetically by contents
+		assertProposal("oneTest", "oneTest", proposals.get(0));
+		assertProposal("threeTest", "threeTest", proposals.get(1));
+		assertProposal("twoTest", "twoTest", proposals.get(2));
+	}
+
+	@Test
+	public void testProposalFilterMultiSelect() {
+		final Map<String, String> proposalMap = ImmutableMap.of("OneTest", "1", "TwoTest", "2", "ThreeTest", "3");
+		OptionsProposalProvider provider = new OptionsProposalProvider(proposalMap, true);
+
+		assertEquals(0, provider.getProposals("ThreeTest", 0).length);
+
+		List<IContentProposal> proposals = Arrays.asList(provider.getProposals("ThreeTest,", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("ThreeTest,OneTest", "OneTest", proposals.get(0));
+		assertProposal("ThreeTest,TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("ThreeTest, ", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("ThreeTest, OneTest", "OneTest", proposals.get(0));
+		assertProposal("ThreeTest, TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("o", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+		assertProposal("TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("O", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+		assertProposal("TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("one", 0));
+		assertEquals(1, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+
+		assertEquals(0, provider.getProposals("four", 0).length);
+
+		proposals = Arrays.asList(provider.getProposals("four,", 0));
+		assertEquals(3, proposals.size());
+		assertProposal("four,OneTest", "OneTest", proposals.get(0));
+		assertProposal("four,ThreeTest", "ThreeTest", proposals.get(1));
+		assertProposal("four,TwoTest", "TwoTest", proposals.get(2));
+
+		proposals = Arrays.asList(provider.getProposals("four,   ", 0));
+		assertEquals(3, proposals.size());
+		assertProposal("four,   OneTest", "OneTest", proposals.get(0));
+		assertProposal("four,   ThreeTest", "ThreeTest", proposals.get(1));
+		assertProposal("four,   TwoTest", "TwoTest", proposals.get(2));
+
+		proposals = Arrays.asList(provider.getProposals(",,           ,four,        five     ,   ", 0));
+		assertEquals(3, proposals.size());
+		assertProposal(",,           ,four,        five     ,   OneTest", "OneTest", proposals.get(0));
+		assertProposal(",,           ,four,        five     ,   ThreeTest", "ThreeTest", proposals.get(1));
+		assertProposal(",,           ,four,        five     ,   TwoTest", "TwoTest", proposals.get(2));
+
+		proposals = Arrays.asList(provider.getProposals(",,           ,four,        five     ,   one", 0));
+		assertEquals(1, proposals.size());
+		assertProposal(",,           ,four,        five     ,   OneTest", "OneTest", proposals.get(0));
+	}
+
+	@Test
+	public void testProposalFilterSingleSelect() {
+		final Map<String, String> proposalMap = ImmutableMap.of("OneTest", "1", "TwoTest", "2", "ThreeTest", "3");
+		OptionsProposalProvider provider = new OptionsProposalProvider(proposalMap, false);
+
+		List<IContentProposal> proposals = Arrays.asList(provider.getProposals("ThreeTest", 0));
+		assertEquals(1, proposals.size());
+		assertProposal("ThreeTest", "ThreeTest", proposals.get(0));
+
+		assertEquals(0, provider.getProposals("ThreeTest,", 0).length);
+		assertEquals(0, provider.getProposals("ThreeTest, ", 0).length);
+
+		proposals = Arrays.asList(provider.getProposals("o", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+		assertProposal("TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("O", 0));
+		assertEquals(2, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+		assertProposal("TwoTest", "TwoTest", proposals.get(1));
+
+		proposals = Arrays.asList(provider.getProposals("one", 0));
+		assertEquals(1, proposals.size());
+		assertProposal("OneTest", "OneTest", proposals.get(0));
+
+		assertEquals(0, provider.getProposals("four", 0).length);
+		assertEquals(0, provider.getProposals("four,", 0).length);
+		assertEquals(0, provider.getProposals("four,   ", 0).length);
+		assertEquals(0, provider.getProposals(",,           ,four,        five     ,   ", 0).length);
+		assertEquals(0, provider.getProposals(",,           ,four,        five     ,   one", 0).length);
+	}
+
+	private void assertProposal(String content, String label, IContentProposal proposal) {
+		assertEquals(content, proposal.getContent());
+		assertEquals(label, proposal.getLabel());
+		assertNull(proposal.getDescription());
+	}
+}