chemistry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From f...@apache.org
Subject svn commit: r1178857 - in /chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck: impl/AbstractSessionTest.java tests/versioning/CheckedOutTest.java
Date Tue, 04 Oct 2011 16:43:25 GMT
Author: fmui
Date: Tue Oct  4 16:43:25 2011
New Revision: 1178857

URL: http://svn.apache.org/viewvc?rev=1178857&view=rev
Log:
TCK: CMIS-431: changed PWC latest version check from FAILURE to WARNING

Modified:
    chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractSessionTest.java
    chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractSessionTest.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/AbstractSessionTest.java?rev=1178857&r1=1178856&r2=1178857&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractSessionTest.java
(original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/impl/AbstractSessionTest.java
Tue Oct  4 16:43:25 2011
@@ -1241,9 +1241,21 @@ public abstract class AbstractSessionTes
 
                 // check first entry
                 if (i == 0) {
-                    f = createResult(FAILURE,
-                            "First version history entry is not the latest version! Id: "
+ version.getId());
-                    addResult(results, assertIsTrue(version.isLatestVersion(), null, f));
+                    if (version.isVersionSeriesCheckedOut()) {
+                        f = createResult(
+                                WARNING,
+                                "Version series is checked-out and the PWC is not the latest
version! Id: "
+                                        + version.getId()
+                                        + " (Note: The words of the CMIS specification define
that the PWC is the latest version."
+                                        + " But that is not the intention of the spec and
will be changed in CMIS 1.1."
+                                        + " Thus this a warning, not an error.)");
+                        addResult(results, assertIsTrue(version.isLatestVersion(), null,
f));
+                    } else {
+                        f = createResult(FAILURE,
+                                "Version series is not checked-out and first version history
entry is not the latest version! Id: "
+                                        + version.getId());
+                        addResult(results, assertIsTrue(version.isLatestVersion(), null,
f));
+                    }
                 }
 
                 // check version id

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.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/tests/versioning/CheckedOutTest.java?rev=1178857&r1=1178856&r2=1178857&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java
(original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java
Tue Oct  4 16:43:25 2011
@@ -18,7 +18,6 @@
  */
 package org.apache.chemistry.opencmis.tck.tests.versioning;
 
-import static org.apache.chemistry.opencmis.tck.CmisTestResultStatus.FAILURE;
 import static org.apache.chemistry.opencmis.tck.CmisTestResultStatus.WARNING;
 
 import java.util.Map;
@@ -69,7 +68,10 @@ public class CheckedOutTest extends Abst
             addResult(checkObject(session, pwc, propertiesToCheck, "PWC check: " + pwc.getId()));
 
             if (pwc != null) {
-                f = createResult(FAILURE, "PWC is not latest version! Id: " + pwc.getId());
+                f = createResult(WARNING, "PWC is not the latest version! Id: " + pwc.getId()
+                        + " (Note: The words of the CMIS specification define that the PWC
is the latest version."
+                        + " But that is not the intention of the spec and will be changed
in CMIS 1.1."
+                        + " Thus this a warning, not an error.)");
                 addResult(assertIsTrue(pwc.isLatestVersion(), null, f));
 
                 if (lastName != null && pwc.getName() != null) {



Mime
View raw message