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);