You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by gs...@apache.org on 2009/01/18 20:23:30 UTC
svn commit: r735525 -
/ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java
Author: gscokart
Date: Sun Jan 18 11:23:30 2009
New Revision: 735525
URL: http://svn.apache.org/viewvc?rev=735525&view=rev
Log:
refactor duplicated code
Modified:
ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java
Modified: ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java
URL: http://svn.apache.org/viewvc/ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java?rev=735525&r1=735524&r2=735525&view=diff
==============================================================================
--- ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java (original)
+++ ant/antlibs/antunit/trunk/src/main/org/apache/ant/antunit/AntUnit.java Sun Jan 18 11:23:30 2009
@@ -221,6 +221,8 @@
}
}
+
+
/**
* Processes a single build file.
*/
@@ -251,25 +253,8 @@
fireStartTest(SUITESETUP);
fireFail(SUITESETUP, e);
} catch (BuildException e) {
- boolean failed = false;
fireStartTest(SUITESETUP);
-
- // try to see whether the BuildException masks
- // an AssertionFailedException. If so, treat
- // it as failure instead of error.
- Throwable t = e.getCause();
- while (t != null && t instanceof BuildException) {
- if (t instanceof AssertionFailedException) {
- failed = true;
- fireFail(SUITESETUP, (AssertionFailedException) t);
- break;
- }
- t = ((BuildException) t).getCause();
- }
-
- if (!failed) {
- fireError(SUITESETUP, e);
- }
+ fireFailOrError(SUITESETUP, e);
}
if (!success) {
return;
@@ -292,24 +277,7 @@
} catch (AssertionFailedException e) {
fireFail(name, e);
} catch (BuildException e) {
- boolean failed = false;
-
- // try to see whether the BuildException masks
- // an AssertionFailedException. If so, treat
- // it as failure instead of error.
- Throwable t = e.getCause();
- while (t != null && t instanceof BuildException) {
- if (t instanceof AssertionFailedException) {
- failed = true;
- fireFail(name, (AssertionFailedException) t);
- break;
- }
- t = ((BuildException) t).getCause();
- }
-
- if (!failed) {
- fireError(name, e);
- }
+ fireFailOrError(name, e);
} finally {
// fire endTest here instead of the endTarget
// event, otherwise an error would be
@@ -324,24 +292,7 @@
} catch (final AssertionFailedException e) {
fireFail(name, e);
} catch (final BuildException e) {
- boolean failed = false;
-
- // try to see whether the BuildException masks
- // an AssertionFailedException. If so, treat
- // it as failure instead of error.
- Throwable t = e.getCause();
- while (t != null && t instanceof BuildException) {
- if (t instanceof AssertionFailedException) {
- failed = true;
- fireFail(name, (AssertionFailedException) t);
- break;
- }
- t = ((BuildException) t).getCause();
- }
-
- if (!failed) {
- fireError(name, e);
- }
+ fireFailOrError(name, e);
}
}
if (iter.hasNext()) {
@@ -360,25 +311,8 @@
fireStartTest(SUITETEARDOWN);
fireFail(SUITETEARDOWN, e);
} catch (BuildException e) {
- boolean failed = false;
- fireStartTest(SUITETEARDOWN);
-
- // try to see whether the BuildException masks
- // an AssertionFailedException. If so, treat
- // it as failure instead of error.
- Throwable t = e.getCause();
- while (t != null && t instanceof BuildException) {
- if (t instanceof AssertionFailedException) {
- failed = true;
- fireFail(SUITETEARDOWN, (AssertionFailedException) t);
- break;
- }
- t = ((BuildException) t).getCause();
- }
-
- if (!failed) {
- fireError(SUITETEARDOWN, e);
- }
+ fireStartTest(SUITETEARDOWN);
+ fireFailOrError(SUITETEARDOWN, e);
}
}
@@ -387,6 +321,28 @@
}
}
+
+ /** Report a failure or an exception for the test target name */
+ private void fireFailOrError(String name, BuildException e) {
+ boolean failed = false;
+ // try to see whether the BuildException masks
+ // an AssertionFailedException. If so, treat
+ // it as failure instead of error.
+ Throwable t = e.getCause();
+ while (t != null && t instanceof BuildException) {
+ if (t instanceof AssertionFailedException) {
+ failed = true;
+ fireFail(name, (AssertionFailedException) t);
+ break;
+ }
+ t = ((BuildException) t).getCause();
+ }
+
+ if (!failed) {
+ fireError(name, e);
+ }
+ }
+
/**
* Redirect output to new project instance.
* @param outputToHandle the output to handle.