You are viewing a plain text version of this content. The canonical link for it is here.
Posted to graffito-commits@incubator.apache.org by ap...@apache.org on 2006/02/09 00:44:56 UTC

svn commit: r376132 - /incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java

Author: apopescu
Date: Wed Feb  8 16:44:55 2006
New Revision: 376132

URL: http://svn.apache.org/viewcvs?rev=376132&view=rev
Log:
user assert

Modified:
    incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java

Modified: incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java
URL: http://svn.apache.org/viewcvs/incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java?rev=376132&r1=376131&r2=376132&view=diff
==============================================================================
--- incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java (original)
+++ incubator/graffito/trunk/jcr/jcr-mapping/src/test/org/apache/portals/graffito/jcr/persistence/collectionconverter/impl/DefaultCollectionConverterImplTest.java Wed Feb  8 16:44:55 2006
@@ -107,7 +107,7 @@
             // --------------------------------------------------------------------------------           
             a = (A) persistenceManager.getObject(A.class, "/test");
             assertNotNull("a.collection is null", a.getCollection());
-            assertTrue("Incorrect a.collection size", a.getCollection().size() == 3);
+            assertEquals("Incorrect a.collection size", 3, a.getCollection().size());
             assertTrue("Incorrect a.collection", ((C) a.getCollection().iterator().next()).getId().equals("first"));
             
             // --------------------------------------------------------------------------------
@@ -191,23 +191,12 @@
             // --------------------------------------------------------------------------------           
             a = (A) persistenceManager.getObject(A.class, "/test");
             assertNotNull("a.collection is null", a.getCollection());
-            assertTrue("Incorrect a.collection size", a.getCollection().size() == 3);
-            assertTrue("Incorrect a.collection", ((C) a.getCollection().iterator().next()).getId().equals("first"));
+            assertEquals("Incorrect a.collection size", 3, a.getCollection().size());
+            assertEquals("Incorrect a.collection", "first", ((C) a.getCollection().iterator().next()).getId());
             
             // --------------------------------------------------------------------------------
             // Update the object
             // --------------------------------------------------------------------------------
-            c1 = new C();
-            c1.setId("first");
-            c1.setName("First Element");
-            c2 = new C();
-            c2.setId("second");
-            c2.setName("Second Element");
-            
-            c3 = new C();
-            c3.setId("third");
-            c3.setName("Third Element");
-            
             C c4 = new C();
             c4.setId("Fourth");
             c4.setName("Fourth Element");
@@ -228,8 +217,8 @@
             a = (A) persistenceManager.getObject(A.class, "/test");
             assertNotNull("a is null", a);
             assertNotNull("a.collection is null", a.getCollection());
-            assertTrue("Incorrect collection size", a.getCollection().size() == 4);
-            assertTrue("Incorrect a.collection", ((C) a.getCollection().iterator().next()).getId().equals("first"));
+            assertEquals("Incorrect collection size", 4, a.getCollection().size());
+            assertEquals("Incorrect a.collection", "first", ((C) a.getCollection().iterator().next()).getId());
             
         }
         catch (Exception e)