chemistry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From f...@apache.org
Subject svn commit: r1531916 - in /chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl: AbstractCmisTest.java AbstractCmisTestGroup.java JUnitHelper.java
Date Mon, 14 Oct 2013 14:40:10 GMT
Author: fmui
Date: Mon Oct 14 14:40:10 2013
New Revision: 1531916

URL: http://svn.apache.org/r1531916
Log:
CMIS-728: TCK JUnit tests: let exceptions pass through

Modified:
    chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTest.java
    chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTestGroup.java
    chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/JUnitHelper.java

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTest.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTest.java?rev=1531916&r1=1531915&r2=1531916&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTest.java
(original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTest.java
Mon Oct 14 14:40:10 2013
@@ -80,7 +80,7 @@ public abstract class AbstractCmisTest i
     public abstract void run() throws Exception;
 
     @Test
-    public void junit() {
+    public void junit() throws Exception {
         JUnitHelper.run(this);
     }
 

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTestGroup.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTestGroup.java?rev=1531916&r1=1531915&r2=1531916&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTestGroup.java
(original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractCmisTestGroup.java
Mon Oct 14 14:40:10 2013
@@ -134,7 +134,7 @@ public abstract class AbstractCmisTestGr
     }
 
     @Test
-    public void junit() {
+    public void junit() throws Exception {
         JUnitHelper.run(this);
     }
 

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/JUnitHelper.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/JUnitHelper.java?rev=1531916&r1=1531915&r2=1531916&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/JUnitHelper.java
(original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/JUnitHelper.java
Mon Oct 14 14:40:10 2013
@@ -38,35 +38,27 @@ public final class JUnitHelper {
     private JUnitHelper() {
     }
 
-    public static void run(CmisTest test) {
-        try {
-            run(new WrapperCmisTestGroup(test));
-        } catch (Exception e) {
-            Assert.fail(e.getMessage());
-        }
+    public static void run(CmisTest test) throws Exception {
+        run(new WrapperCmisTestGroup(test));
     }
 
-    public static void run(CmisTestGroup group) {
-        try {
-            JUnitRunner runner = new JUnitRunner();
-
-            String parametersFile = System.getProperty(JUNIT_PARAMETERS);
-            if (parametersFile == null) {
-                runner.setParameters(null);
-            } else {
-                runner.loadParameters(new File(parametersFile));
-            }
+    public static void run(CmisTestGroup group) throws Exception {
+        JUnitRunner runner = new JUnitRunner();
+
+        String parametersFile = System.getProperty(JUNIT_PARAMETERS);
+        if (parametersFile == null) {
+            runner.setParameters(null);
+        } else {
+            runner.loadParameters(new File(parametersFile));
+        }
 
-            runner.addGroup(group);
-            runner.run(new JUnitProgressMonitor());
+        runner.addGroup(group);
+        runner.run(new JUnitProgressMonitor());
 
-            CmisTestReport report = new TextReport();
-            report.createReport(runner.getParameters(), runner.getGroups(), new PrintWriter(System.out));
+        CmisTestReport report = new TextReport();
+        report.createReport(runner.getParameters(), runner.getGroups(), new PrintWriter(System.out));
 
-            checkForFailures(runner);
-        } catch (Exception e) {
-            Assert.fail(e.getMessage());
-        }
+        checkForFailures(runner);
     }
 
     private static void checkForFailures(JUnitRunner runner) {
@@ -85,7 +77,7 @@ public final class JUnitHelper {
     }
 
     private static class JUnitProgressMonitor implements CmisTestProgressMonitor {
-        
+
         @SuppressWarnings("PMD.SystemPrintln")
         public void startGroup(CmisTestGroup group) {
             System.out.println(group.getName() + " (" + group.getTests().size() + " tests)");



Mime
View raw message