You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ch...@apache.org on 2017/01/05 08:28:11 UTC
svn commit: r1777432 -
/sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java
Author: chetanm
Date: Thu Jan 5 08:28:11 2017
New Revision: 1777432
URL: http://svn.apache.org/viewvc?rev=1777432&view=rev
Log:
SLING-6437 - TestDescriptionRule in testing rules has incorrect has wrong order of method calls
Modified:
sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java
Modified: sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java
URL: http://svn.apache.org/viewvc/sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java?rev=1777432&r1=1777431&r2=1777432&view=diff
==============================================================================
--- sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java (original)
+++ sling/trunk/testing/junit/rules/src/main/java/org/apache/sling/testing/junit/rules/TestDescriptionRule.java Thu Jan 5 08:28:11 2017
@@ -25,17 +25,16 @@ import org.junit.runner.Description;
*/
public class TestDescriptionRule extends TestWatcher {
-
@Override
protected void finished(Description description) {
- TestDescriptionHolder.setClassName(description.getClassName());
- TestDescriptionHolder.setMethodName(description.getMethodName());
+ TestDescriptionHolder.removeMethodName();
+ TestDescriptionHolder.removeClassName();
}
@Override
protected void starting (Description description) {
- TestDescriptionHolder.removeMethodName();
- TestDescriptionHolder.removeClassName();
+ TestDescriptionHolder.setClassName(description.getClassName());
+ TestDescriptionHolder.setMethodName(description.getMethodName());
}
}