Author: markt
Date: Sun Mar  7 11:39:39 2010
New Revision: 919973

URL: http://svn.apache.org/viewvc?rev=919973&view=rev
Log:
Ensure result of getVersion() is consistent with getMajorVersion() and 
getMinorVersion()
With the current code this fails. Patch to fix this to follow.

Modified:
    tomcat/trunk/test/org/apache/catalina/deploy/TestWebXml.java

Modified: tomcat/trunk/test/org/apache/catalina/deploy/TestWebXml.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/deploy/TestWebXml.java?rev=919973&r1=919972&r2=919973&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/deploy/TestWebXml.java (original)
+++ tomcat/trunk/test/org/apache/catalina/deploy/TestWebXml.java Sun Mar  7 
11:39:39 2010
@@ -79,6 +79,7 @@
                 org.apache.catalina.startup.Constants.WebDtdPublicId_22);
         assertEquals(2, webxml.getMajorVersion());
         assertEquals(2, webxml.getMinorVersion());
+        assertEquals("2.2", webxml.getVersion());
     }
 
     public void testParsePublicIdVersion23() {
@@ -89,6 +90,7 @@
                 org.apache.catalina.startup.Constants.WebDtdPublicId_23);
         assertEquals(2, webxml.getMajorVersion());
         assertEquals(3, webxml.getMinorVersion());
+        assertEquals("2.3", webxml.getVersion());
     }
 
     public void testParsePublicIdVersion24() {
@@ -99,6 +101,7 @@
                 org.apache.catalina.startup.Constants.WebSchemaPublicId_24);
         assertEquals(2, webxml.getMajorVersion());
         assertEquals(4, webxml.getMinorVersion());
+        assertEquals("2.4", webxml.getVersion());
     }
 
     public void testParsePublicIdVersion25() {
@@ -109,6 +112,7 @@
                 org.apache.catalina.startup.Constants.WebSchemaPublicId_25);
         assertEquals(2, webxml.getMajorVersion());
         assertEquals(5, webxml.getMinorVersion());
+        assertEquals("2.5", webxml.getVersion());
     }
 
     public void testParsePublicIdVersion30() {
@@ -119,5 +123,6 @@
                 org.apache.catalina.startup.Constants.WebSchemaPublicId_30);
         assertEquals(3, webxml.getMajorVersion());
         assertEquals(0, webxml.getMinorVersion());
+        assertEquals("3.0", webxml.getVersion());
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to