svn commit: rev 6621 - incubator/directory/naming/trunk/core/src/test/org/apache/naming/resources

[ps...@apache.org]

Author: psteitz
Date: Wed Feb 11 06:11:03 2004
New Revision: 6621

Modified:
   incubator/directory/naming/trunk/core/src/test/org/apache/naming/resources/AbstractDirContextTest.java
Log:
Removed verifyLookup override, added some comments.

Modified: incubator/directory/naming/trunk/core/src/test/org/apache/naming/resources/AbstractDirContextTest.java
==============================================================================
--- incubator/directory/naming/trunk/core/src/test/org/apache/naming/resources/AbstractDirContextTest.java	(original)
+++ incubator/directory/naming/trunk/core/src/test/org/apache/naming/resources/AbstractDirContextTest.java	Wed Feb 11 06:11:03 2004
@@ -105,10 +105,10 @@
         return new Resource(new ByteArrayInputStream(bytes));
     }
     
-    protected void verifyLookup(Object boundObject, Object returnedObject) {
-        assertTrue(returnedObject instanceof Resource);
-    }
-    
+    /**
+     * Verify that the listed bound names match expectation and
+     * that the class names of bound objects are not empty
+     */
     protected void verifyList(Map expected, Map returned) {
         assertEquals(expected.keySet(), returned.keySet());
         Iterator iterator = returned.values().iterator();
@@ -117,11 +117,13 @@
         }
     }
     
+    /**
+     * Verify that the listed bound names match expectation
+     */
     protected void verifyListBindings(Map expected, Map returned) {
         assertEquals(expected.keySet(), returned.keySet());
-        Iterator iterator = returned.values().iterator();
-        while (iterator.hasNext()) {
-            assertTrue(iterator.next() instanceof Resource);
-        }
     }
+    
+    // todo: Add tests for DirContext 
+    
 }