You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2008/12/15 10:10:57 UTC

svn commit: r726658 - /incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java

Author: fmeschbe
Date: Mon Dec 15 01:10:56 2008
New Revision: 726658

URL: http://svn.apache.org/viewvc?rev=726658&view=rev
Log:
JCR Items should be compared with Item.isSame() and not with Object.equals()

Modified:
    incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java

Modified: incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java?rev=726658&r1=726657&r2=726658&view=diff
==============================================================================
--- incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java (original)
+++ incubator/sling/trunk/jcr/resource/src/test/java/org/apache/sling/jcr/resource/JcrPropertyMapTest.java Mon Dec 15 01:10:56 2008
@@ -132,12 +132,12 @@
         
         // explicite type
         Property result = map.get(PROP_NAME, Property.class);
-        assertEquals(prop, result);
+        assertTrue(prop.isSame(result));
 
         // type by default value
         Property defaultValue = rootNode.getProperty("jcr:primaryType");
         result = map.get(PROP_NAME, defaultValue);
-        assertEquals(prop, result);
+        assertTrue(prop.isSame(result));
         
         // default value
         result = map.get(PROP_NAME_NIL, defaultValue);