You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2010/10/18 17:09:02 UTC

svn commit: r1023835 - /commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java

Author: sebb
Date: Mon Oct 18 15:09:02 2010
New Revision: 1023835

URL: http://svn.apache.org/viewvc?rev=1023835&view=rev
Log:
Avoid NPE warning by using assertNotNull

Modified:
    commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java

Modified: commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java?rev=1023835&r1=1023834&r2=1023835&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java (original)
+++ commons/proper/collections/trunk/src/test/org/apache/commons/collections/map/AbstractTestMap.java Mon Oct 18 15:09:02 2010
@@ -482,11 +482,11 @@ public abstract class AbstractTestMap<K,
         Object[] values = getSampleValues();
         Object[] newValues = getNewSampleValues();
 
-        assertTrue("failure in test: Must have keys returned from " +
-                 "getSampleKeys.", keys != null);
+        assertNotNull("failure in test: Must have keys returned from " +
+                 "getSampleKeys.", keys);
 
-        assertTrue("failure in test: Must have values returned from " +
-                 "getSampleValues.", values != null);
+        assertNotNull("failure in test: Must have values returned from " +
+                 "getSampleValues.", values);
 
         // verify keys and values have equivalent lengths (in case getSampleX are
         // overridden)