Since assert is a reserved keyword, and is now part of the language,
junit moved away from using it for test assertions, opting instead for
things like assertTrue or assertEquals

Here is the patch for BaseDirContextTestCase, to bring it up to par with
junit's and Java's recommended practices.

Jeff Tulley  ([EMAIL PROTECTED])
(801)861-5322
Novell, Inc., The Leading Provider of Net Business Solutions
http://www.novell.com
Index: BaseDirContextTestCase.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-tomcat-4.0/catalina/src/test/org/apache/naming/resources/BaseDirContextTestCase.java,v
retrieving revision 1.3
diff -u -r1.3 BaseDirContextTestCase.java
--- BaseDirContextTestCase.java 21 Oct 2001 22:03:34 -0000      1.3
+++ BaseDirContextTestCase.java 23 Oct 2003 22:16:03 -0000
@@ -247,7 +247,7 @@
             // Look up the WEB-INF entry
             Object webInfEntry = context.lookup("WEB-INF");
             assertNotNull("Found WEB-INF entry", webInfEntry);
-            assert("WEB-INF entry is a DirContext",
+            assertTrue("WEB-INF entry is a DirContext",
                    webInfEntry instanceof DirContext);
             DirContext webInfContext = (DirContext) webInfEntry;
 
@@ -303,7 +303,7 @@
             // Look up the WEB-INF entry
             Object webInfEntry = context.lookup("WEB-INF");
             assertNotNull("Found WEB-INF entry", webInfEntry);
-            assert("WEB-INF entry is a DirContext",
+            assertTrue("WEB-INF entry is a DirContext",
                    webInfEntry instanceof DirContext);
             DirContext webInfContext = (DirContext) webInfEntry;
 
@@ -353,20 +353,20 @@
         while (ne.hasMore()) {
 
             Object next = ne.next();
-            assert("list() returns NameClassPair instances",
+            assertTrue("list() returns NameClassPair instances",
                    next instanceof NameClassPair);
             NameClassPair ncp = (NameClassPair) next;
 
-            assert("Name '" + ncp.getName() + "' is expected",
+            assertTrue("Name '" + ncp.getName() + "' is expected",
                    isListed(ncp.getName(), list));
 
             if (isDirContext(ncp.getName())) {
-                assert("Class '" + ncp.getClassName() + "' is '" +
+                assertTrue("Class '" + ncp.getClassName() + "' is '" +
                        contextClassName + "'",
                        contextClassName.equals(ncp.getClassName()));
             }
 
-            assert("Relative is 'true'", ncp.isRelative());
+            assertTrue("Relative is 'true'", ncp.isRelative());
 
         }
 
@@ -391,29 +391,29 @@
         while (ne.hasMore()) {
 
             Object next = ne.next();
-            assert("listBindings() returns Binding instances",
+            assertTrue("listBindings() returns Binding instances",
                    next instanceof Binding);
             Binding b = (Binding) next;
 
-            assert("Name '" + b.getName() + "' is expected",
+            assertTrue("Name '" + b.getName() + "' is expected",
                    isListed(b.getName(), list));
 
             if (isDirContext(b.getName())) {
-                assert("Class '" + b.getClassName() + "' is '" +
+                assertTrue("Class '" + b.getClassName() + "' is '" +
                        contextClassName + "'",
                        contextClassName.equals(b.getClassName()));
             }
 
-            assert("Relative is 'true'", b.isRelative());
+            assertTrue("Relative is 'true'", b.isRelative());
 
             Object object = b.getObject();
             assertNotNull("Name '" + b.getName() + "' has a non-null object",
                           object);
             if (b.getName().equals("web.xml")) {
-                assert("Entry '" + b.getName() + "' is a Resource",
+                assertTrue("Entry '" + b.getName() + "' is a Resource",
                        object instanceof Resource);
             } else {
-                assert("Entry '" + b.getName() + "' is a DirContext",
+                assertTrue("Entry '" + b.getName() + "' is a DirContext",
                        object instanceof DirContext);
             }
 
@@ -445,35 +445,35 @@
         while (ne.hasMore()) {
 
             Object next = ne.next();
-            assert("getAll() returns Attribute instances",
+            assertTrue("getAll() returns Attribute instances",
                    next instanceof Attribute);
             Attribute attr = (Attribute) next;
             String name = attr.getID();
             int index = getIndex(name, webInfAttrs);
-            assert("WEB-INF attribute '" + name + "' is expected",
+            assertTrue("WEB-INF attribute '" + name + "' is expected",
                    index >= 0);
             Object value = attr.get();
             assertNotNull("get() returned non-null", value);
 
             if (name.equals("creationdate")) {
-                assert("Creation date is a date",
+                assertTrue("Creation date is a date",
                        value instanceof Date);
-                assert("Creation date equals " + creationDate,
+                assertTrue("Creation date equals " + creationDate,
                        creationDate.equals((Date) value));
             } else if (name.equals("displayname")) {
-                assert("Display name is a string",
+                assertTrue("Display name is a string",
                        value instanceof String);
-                assert("Display name equals " + displayName,
+                assertTrue("Display name equals " + displayName,
                        displayName.equals((String) value));
             } else if (name.equals("getcontentlength")) {
-                assert("Content length is a long",
+                assertTrue("Content length is a long",
                        value instanceof Long);
-                assert("Content length equals " + contentLength,
+                assertTrue("Content length equals " + contentLength,
                        contentLength == ((Long) value).longValue());
             } else if (name.equals("getlastmodified")) {
-                assert("Last modified date is a date",
+                assertTrue("Last modified date is a date",
                        value instanceof Date);
-                assert("Last modified date is " + lastModifiedDate,
+                assertTrue("Last modified date is " + lastModifiedDate,
                        lastModifiedDate.equals((Date) value));
             }
 
@@ -505,35 +505,35 @@
         while (ne.hasMore()) {
 
             Object next = ne.next();
-            assert("getAll() returns Attribute instances",
+            assertTrue("getAll() returns Attribute instances",
                    next instanceof Attribute);
             Attribute attr = (Attribute) next;
             String name = attr.getID();
             int index = getIndex(name, webXmlAttrs);
-            assert("WEB-INF/web.xml attribute '" + name + "' is expected",
+            assertTrue("WEB-INF/web.xml attribute '" + name + "' is expected",
                    index >= 0);
             Object value = attr.get();
             assertNotNull("get() returned non-null", value);
 
             if (name.equals("creationdate")) {
-                assert("Creation date is a date",
+                assertTrue("Creation date is a date",
                        value instanceof Date);
-                assert("Creation date equals " + creationDate,
+                assertTrue("Creation date equals " + creationDate,
                        creationDate.equals((Date) value));
             } else if (name.equals("displayname")) {
-                assert("Display name is a string",
+                assertTrue("Display name is a string",
                        value instanceof String);
-                assert("Display name equals " + displayName,
+                assertTrue("Display name equals " + displayName,
                        displayName.equals((String) value));
             } else if (name.equals("getcontentlength")) {
-                assert("Content length is a long",
+                assertTrue("Content length is a long",
                        value instanceof Long);
-                assert("Content length equals " + contentLength,
+                assertTrue("Content length equals " + contentLength,
                        contentLength == ((Long) value).longValue());
             } else if (name.equals("getlastmodified")) {
-                assert("Last modified date is a date",
+                assertTrue("Last modified date is a date",
                        value instanceof Date);
-                assert("Last modified date is " + lastModifiedDate,
+                assertTrue("Last modified date is " + lastModifiedDate,
                        lastModifiedDate.equals((Date) value));
             }
 

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to