You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chemistry.apache.org by fm...@apache.org on 2011/10/04 18:43:25 UTC
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
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) {