You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by mr...@apache.org on 2016/04/12 09:47:24 UTC

svn commit: r1738720 - /jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java

Author: mreutegg
Date: Tue Apr 12 07:47:24 2016
New Revision: 1738720

URL: http://svn.apache.org/viewvc?rev=1738720&view=rev
Log:
JCR-3964: Skip tests marked as known issues

Modified:
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java?rev=1738720&r1=1738719&r2=1738720&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/JCRTestResult.java Tue Apr 12 07:47:24 2016
@@ -64,6 +64,18 @@ public class JCRTestResult extends TestR
         this.knownIssuesOverride = JCRTestResult.tokenize("known.issues.override");
     }
 
+    @Override
+    protected void run(TestCase test) {
+        if (isKnownIssue(test)) {
+            // notify listeners but not actually run the test
+            startTest(test);
+            log.println("Known issue: " + test + " (skipped)");
+            endTest(test);
+        } else {
+            super.run(test);
+        }
+    }
+
     /**
      * Only add an error if <code>throwable</code> is not of type
      * {@link NotExecutableException} and the test case is not a known issue.