svn commit: r1446196 - in /chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck: impl/AbstractSessionTest.java tests/crud/CreateAndDeleteDocumentTest.java

[fm...@apache.org]

Author: fmui
Date: Thu Feb 14 14:18:03 2013
New Revision: 1446196

URL: http://svn.apache.org/r1446196
Log:
TCK improvements

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/crud/CreateAndDeleteDocumentTest.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=1446196&r1=1446195&r2=1446196&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 Thu Feb 14 14:18:03 2013
@@ -926,6 +926,21 @@ public abstract class AbstractSessionTes
             if (object.getRenditions() != null) {
                 addResult(results, checkRenditions(session, object, "Rendition check"));
             }
+
+            // check path
+            if (object instanceof FileableCmisObject) {
+                List<String> paths = ((FileableCmisObject) object).getPaths();
+                if (object instanceof Folder) {
+                    f = createResult(FAILURE, "Folder does not have excatly one path! This is an OpenCMIS bug!");
+                    addResult(results, assertEquals(1, paths.size(), null, f));
+                } else {
+                    if (Boolean.FALSE.equals(session.getRepositoryInfo().getCapabilities().isMultifilingSupported())) {
+                        f = createResult(FAILURE,
+                                "Repository does not support multi-filing, but the object has more than one parent!");
+                        addResult(results, assertIsTrue(paths.size() < 2, null, f));
+                    }
+                }
+            }
         }
 
         CmisTestResultImpl result = createResult(getWorst(results), message);

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/CreateAndDeleteDocumentTest.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/crud/CreateAndDeleteDocumentTest.java?rev=1446196&r1=1446195&r2=1446196&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/CreateAndDeleteDocumentTest.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/CreateAndDeleteDocumentTest.java Thu Feb 14 14:18:03 2013
@@ -24,8 +24,10 @@ import static org.apache.chemistry.openc
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 
 import org.apache.chemistry.opencmis.client.api.CmisObject;
 import org.apache.chemistry.opencmis.client.api.Document;
@@ -61,11 +63,13 @@ public class CreateAndDeleteDocumentTest
 
         try {
             Map<String, Document> documents = new HashMap<String, Document>();
+            Set<String> versionSeriesIds = new HashSet<String>();
 
             // create documents
             for (int i = 0; i < numOfDocuments; i++) {
                 Document newDocument = createDocument(session, testFolder, "doc" + i, CONTENT);
                 documents.put(newDocument.getId(), newDocument);
+                versionSeriesIds.add(newDocument.getVersionSeriesId());
             }
 
             // simple children test
@@ -84,7 +88,7 @@ public class CreateAndDeleteDocumentTest
                 }
             }
 
-            f = createResult(FAILURE, "Number of created folders does not match the number of existing folders!");
+            f = createResult(FAILURE, "Number of created documents does not match the number of existing documents!");
             addResult(assertEquals(numOfDocuments, childrenIds.size(), null, f));
 
             for (Document document : documents.values()) {
@@ -94,6 +98,10 @@ public class CreateAndDeleteDocumentTest
                 }
             }
 
+            // check version series ids
+            f = createResult(FAILURE, "Although the created documents are independent, some share a Version Series Id!");
+            addResult(assertEquals(numOfDocuments, versionSeriesIds.size(), null, f));
+
             // check paging
             int pageSize = 5;
             CmisObject lastObject = null;