You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by js...@apache.org on 2002/12/10 09:59:43 UTC
cvs commit: jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/tags/junit JellyAssertionFailedError.java
jstrachan 2002/12/10 00:59:43
Modified: jelly/src/test/org/apache/commons/jelly/junit suite.jelly
jelly/src/java/org/apache/commons/jelly/tags/junit
JellyAssertionFailedError.java
Log:
Patched JellyAssertionFailureError to actually extend AssertionFailureError, like it was always supposed to but wasn't.
So now JellyUnit throws real JUnit exceptions (which are Error instances)
Revision Changes Path
1.10 +1 -1 jakarta-commons-sandbox/jelly/src/test/org/apache/commons/jelly/junit/suite.jelly
Index: suite.jelly
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/jelly/src/test/org/apache/commons/jelly/junit/suite.jelly,v
retrieving revision 1.9
retrieving revision 1.10
diff -u -r1.9 -r1.10
--- suite.jelly 4 Dec 2002 16:09:12 -0000 1.9
+++ suite.jelly 10 Dec 2002 08:59:42 -0000 1.10
@@ -61,7 +61,7 @@
<test:assert test="${ex != null}">No exception exported</test:assert>
<!-- test using superclass of expected throwable -->
- <test:assertThrows expected="java.lang.Exception">
+ <test:assertThrows expected="java.lang.Error">
<test:fail message="This should always fail"/>
</test:assertThrows>
1.2 +1 -1 jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/tags/junit/JellyAssertionFailedError.java
Index: JellyAssertionFailedError.java
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/tags/junit/JellyAssertionFailedError.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- JellyAssertionFailedError.java 28 Nov 2002 08:35:48 -0000 1.1
+++ JellyAssertionFailedError.java 10 Dec 2002 08:59:43 -0000 1.2
@@ -78,7 +78,7 @@
* @version $Revision$
*/
-public class JellyAssertionFailedError extends Exception implements LocationAware {
+public class JellyAssertionFailedError extends AssertionFailedError implements LocationAware {
/** the underlying cause of the exception */
private Throwable cause;
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>