morgand     2003/01/24 15:23:47

  Modified:    jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit
                        AssertEqualsTag.java AssertTag.java
                        AssertThrowsTag.java
  Log:
  converting junit tags from Exceptions to JellyTagExceptions
  
  Revision  Changes    Path
  1.2       +7 -6      
jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertEqualsTag.java
  
  Index: AssertEqualsTag.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertEqualsTag.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- AssertEqualsTag.java      19 Jan 2003 06:17:10 -0000      1.1
  +++ AssertEqualsTag.java      24 Jan 2003 23:23:47 -0000      1.2
  @@ -61,6 +61,7 @@
    */
   package org.apache.commons.jelly.tags.junit;
   
  +import org.apache.commons.jelly.JellyTagException;
   import org.apache.commons.jelly.XMLOutput;
   import org.apache.commons.jelly.expression.Expression;
   
  @@ -79,7 +80,7 @@
   
       // Tag interface
       //------------------------------------------------------------------------- 
  -    public void doTag(XMLOutput output) throws Exception {
  +    public void doTag(XMLOutput output) throws JellyTagException {
           String message = getBodyText();
   
           Object expectedValue = expected.evaluate(context);                    
  
  
  
  1.2       +16 -10    
jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertTag.java
  
  Index: AssertTag.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertTag.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- AssertTag.java    19 Jan 2003 06:17:10 -0000      1.1
  +++ AssertTag.java    24 Jan 2003 23:23:47 -0000      1.2
  @@ -61,14 +61,15 @@
    */
   package org.apache.commons.jelly.tags.junit;
   
  -import org.apache.commons.jelly.XMLOutput;
  +import org.apache.commons.jelly.JellyTagException;
   import org.apache.commons.jelly.MissingAttributeException;
  +import org.apache.commons.jelly.XMLOutput;
   import org.apache.commons.jelly.expression.Expression;
   
   import org.apache.commons.logging.Log;
   import org.apache.commons.logging.LogFactory;
   
  -
  +import org.jaxen.JaxenException;
   import org.jaxen.XPath;
   
   /** 
  @@ -94,7 +95,7 @@
   
       // Tag interface
       //------------------------------------------------------------------------- 
  -    public void doTag(XMLOutput output) throws Exception {
  +    public void doTag(XMLOutput output) throws MissingAttributeException, 
JellyTagException {
           if (test == null && xpath == null) {
               throw new MissingAttributeException( "test" );
           }
  @@ -105,8 +106,13 @@
           }
           else {
               Object xpathContext = getXPathContext();
  -            if (! xpath.booleanValueOf(xpathContext)) {
  -                fail( getBodyText(), "evaluating xpath: "+ xpath );
  +            try {
  +                if (! xpath.booleanValueOf(xpathContext)) {
  +                    fail( getBodyText(), "evaluating xpath: "+ xpath );
  +                }
  +            }
  +            catch (JaxenException e) {
  +                throw new JellyTagException(e);
               }
           }
   
  
  
  
  1.2       +21 -13    
jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertThrowsTag.java
  
  Index: AssertThrowsTag.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/jelly/jelly-tags/junit/src/java/org/apache/commons/jelly/tags/junit/AssertThrowsTag.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- AssertThrowsTag.java      19 Jan 2003 06:17:10 -0000      1.1
  +++ AssertThrowsTag.java      24 Jan 2003 23:23:47 -0000      1.2
  @@ -56,6 +56,7 @@
   package org.apache.commons.jelly.tags.junit;
   
   import org.apache.commons.jelly.JellyException;
  +import org.apache.commons.jelly.JellyTagException;
   import org.apache.commons.jelly.XMLOutput;
   import org.apache.commons.logging.Log;
   import org.apache.commons.logging.LogFactory;
  @@ -86,17 +87,23 @@
        private String expected;
   
       /**
  -     * Sets the ClassLoader to be used when loading an exception class
     */
  +     * Sets the ClassLoader to be used when loading an exception class
  +     */
       private ClassLoader classLoader;
       
  -     // Tag interface
  -     //-------------------------------------------------------------------------
  -     public void doTag(XMLOutput output) throws Exception {
  -             Class throwableClass = getThrowableClass();
  -
  -             try {
  -                     invokeBody(output);
  -             } 
  +    // Tag interface
  +    //-------------------------------------------------------------------------
  +    public void doTag(XMLOutput output) throws JellyTagException {
  +        Class throwableClass = null;
  +        try {
  +             throwableClass = getThrowableClass();
  +        } catch (ClassNotFoundException e) {
  +            throw new JellyTagException(e);
  +        }
  +
  +        try {
  +            invokeBody(output);
  +        } 
           catch (Throwable t) {
               if (t instanceof JellyException) {
                   // unwrap Jelly exceptions which wrap other exceptions
  @@ -110,10 +117,10 @@
                        }
                        if (throwableClass != null && 
!throwableClass.isAssignableFrom(t.getClass())) {
                                fail("Unexpected exception: " + t);
  -                     } 
  +            } 
               else {
  -                             return;
  -                     }
  +                return;
  +            }
                }
                fail("No exception was thrown.");
        }
  @@ -138,7 +145,8 @@
        }
   
       /**
  -     * Sets the class loader to be used to load the exception type
     */
  +     * Sets the class loader to be used to load the exception type
  +     */
       public void setClassLoader(ClassLoader classLoader) {
           this.classLoader = classLoader;
       }
  
  
  

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

Reply via email to