Removed try/catch to retain the stacktrace of where the NPE occurs

egonw · Mar 27, 2010 · ce11b52 · ce11b52
1 parent 0aa632a
commit ce11b52
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/src/test/org/openscience/cdk/interfaces/AbstractChemObjectTest.java b/src/test/org/openscience/cdk/interfaces/AbstractChemObjectTest.java
@@ -224,13 +224,9 @@ public static IChemObject newChemObject() {
     	IChemObject chemObject = newChemObject();
     	final String key = "NullProperty";
     	chemObject.setProperty( key, null);
-    	try {
-    		IChemObject chemObject_clone = (IChemObject) chemObject.clone();
-    		Assert.assertFalse(chemObject_clone.getProperties().containsKey(key));
-    		Assert.assertNull(chemObject_clone.getProperty(null));
-    	} catch (NullPointerException e) {
-			Assert.fail();
-		}
+        IChemObject chemObject_clone = (IChemObject) chemObject.clone();
+        Assert.assertFalse(chemObject_clone.getProperties().containsKey(key));
+        Assert.assertNull(chemObject_clone.getProperty(null));
     }
 
     @Test public void testClone_ChemObjectListeners() throws Exception {