386764: return null instead of throwing exception for getTaskData()

Change-Id: Iad409821c9fd799e704b831f8006a136724cf144
Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=386764
Also-By: Frank Becker <frank@frank-becker.de>

1 files changed, 10 insertions, 54 deletions

diff --git a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/TaskInitializationDataTest.java b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/TaskInitializationDataTest.java
index 3da37a270..1fbe25c6b 100644
--- a/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/TaskInitializationDataTest.java
+++ b/org.eclipse.mylyn.tasks.tests/src/org/eclipse/mylyn/tasks/tests/core/TaskInitializationDataTest.java
@@ -7,6 +7,7 @@
  *
  * Contributors:
  *     Tasktop Technologies - initial API and implementation
+ *     Frank Becker - improvements
  *******************************************************************************/
 
 package org.eclipse.mylyn.tasks.tests.core;
@@ -35,60 +36,15 @@ public class TaskInitializationDataTest extends TestCase {
 		} catch (UnsupportedOperationException e) {
 			// expected
 		}
-		try {
-			data.getTaskStatus();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getTaskData();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getPriorityLevel();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getModificationDate();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getKeywords();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getDueDate();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getCreationDate();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getCompletionDate();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
-		try {
-			data.getCc();
-			fail("Should not be supported");
-		} catch (UnsupportedOperationException e) {
-			// expected
-		}
+		assertNull(data.getTaskStatus());
+		assertNull(data.getTaskData());
+		assertNull(data.getPriorityLevel());
+		assertNull(data.getModificationDate());
+		assertNull(data.getKeywords());
+		assertNull(data.getDueDate());
+		assertNull(data.getCreationDate());
+		assertNull(data.getCompletionDate());
+		assertNull(data.getCc());
 	}
 
 	public void testTaskKind() throws Exception {