392688: TaskAttribute support for Unset

Change-Id: Ia4b44d557afcda8ae30c8b165a28e4af10b4d47d
Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=392688

3 files changed, 54 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 ea22db338..d1ffa1e9e 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
@@ -48,6 +48,7 @@ import org.eclipse.mylyn.tasks.tests.ui.editor.TaskUrlHyperlinkDetectorTest;
  * @author Shawn Minto
  * @author Steffen Pingel
  * @author Benjamin Muskalla
+ * @author Miles Parker
  */
 public class AllTasksTests {
 
@@ -129,6 +130,7 @@ public class AllTasksTests {
 		suite.addTestSuite(MultipleTaskHyperlinkDetectorTest.class);
 		suite.addTestSuite(RegionComparatorTest.class);
 		suite.addTestSuite(PriorityLevelTest.class);
+		suite.addTestSuite(TaskAttributeTest.class);
 		suite.addTestSuite(TaskAttributeMapperTest.class);
 		suite.addTestSuite(SupportHandlerManagerTest.class);
 		suite.addTestSuite(RepositoryClientManagerTest.class);
diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/TaskDataStoreTest.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/TaskDataStoreTest.java
index 0ad07bd3b..a506fd4cc 100644
--- a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/TaskDataStoreTest.java
+++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/TaskDataStoreTest.java
@@ -257,4 +257,35 @@ public class TaskDataStoreTest extends TestCase {
 		}
 	}
 
+	public void testReadWriteSetValue() throws Exception {
+		setupData();
+		TaskAttribute attribute = data.getRoot().createAttribute("attribute");
+		assertFalse(attribute.hasValue());
+		attribute.setValue("foo");
+		assertTrue(attribute.hasValue());
+
+		File file = File.createTempFile("mylyn", null);
+		file.deleteOnExit();
+		storage.putTaskData(file, state);
+
+		TaskDataState state2 = storage.getTaskDataState(file);
+		assertTrue(state2.getRepositoryData().getRoot().getAttribute("attribute").hasValue());
+	}
+
+	public void testReadWriteUnsetValue() throws Exception {
+		setupData();
+		TaskAttribute attribute = data.getRoot().createAttribute("attribute");
+		assertFalse(attribute.hasValue());
+		attribute.setValue("foo");
+		assertTrue(attribute.hasValue());
+		attribute.clearValues();
+		assertFalse(attribute.hasValue());
+
+		File file = File.createTempFile("mylyn", null);
+		file.deleteOnExit();
+		storage.putTaskData(file, state);
+
+		TaskDataState state2 = storage.getTaskDataState(file);
+		assertFalse(state2.getRepositoryData().getRoot().getAttribute("attribute").hasValue());
+	}
 }
diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/data/TaskAttributeTest.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/data/TaskAttributeTest.java
index b9a3a0a2e..abf71052f 100644
--- a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/data/TaskAttributeTest.java
+++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/data/TaskAttributeTest.java
@@ -23,6 +23,7 @@ import org.eclipse.mylyn.tasks.core.data.TaskData;
 
 /**
  * @author Benjamin Muskalla
+ * @author Miles Parker
  */
 public class TaskAttributeTest extends TestCase {
 
@@ -36,6 +37,13 @@ public class TaskAttributeTest extends TestCase {
 		attribute = new TaskAttribute(data.getRoot(), "test");
 	}
 
+	public void testGetValue() {
+		attribute.setValue("baz");
+		assertEquals("baz", attribute.getValue());
+		attribute.setValue("bee");
+		assertEquals("bee", attribute.getValue());
+	}
+
 	public void testRegularValue() throws Exception {
 		attribute.setValue("foo");
 		assertEquals("foo", attribute.getValue());
@@ -46,6 +54,19 @@ public class TaskAttributeTest extends TestCase {
 		assertEquals("foo", attribute.getValue());
 	}
 
+	public void testHasValue() {
+		assertFalse(attribute.hasValue());
+
+		attribute.setValue("description");
+		assertTrue(attribute.hasValue());
+
+		attribute.clearValues();
+		assertFalse(attribute.hasValue());
+
+		attribute.setValue("description");
+		assertTrue(attribute.hasValue());
+	}
+
 	public void testNullValue() throws Exception {
 		try {
 			attribute.setValue(null);