From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Wed, 9 Jan 2013 23:39:10 +0000 (-0800)
Subject: CSPACE-5728: Initial steps toward supporting list-mode invocation of the batch job... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=7217c927b40f4b35c5d2cbcfcc2ac7e8d99e1605;p=tmp%2Fjakarta-migration.git

CSPACE-5728: Initial steps toward supporting list-mode invocation of the batch job to update computedCurrentLocation values. List CSIDs aren't currently appearing in the invocation context; to be investigated.
---

diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-list.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-list.xml
new file mode 100644
index 000000000..0d36993fe
--- /dev/null
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-list.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<ns2:invocationContext
+    xmlns:ns2="http://collectionspace.org/services/common/invocable"
+    xmlns:ns3="http://collectionspace.org/services/jaxb">
+    <mode>list</mode>
+    <docType>CollectionObject</docType>
+    <singleCSID></singleCSID>
+    <groupCSID></groupCSID>
+    <listCsids>
+        <csid>${collectionObject1CSID}</csid>
+        <csid>${collectionObject2CSID}</csid>
+    </listCsids>
+    <params>
+        <param>
+            <key>some key</key>
+            <value>some value</value>
+        </param>
+    </params>
+</ns2:invocationContext>
+
+
diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-single.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-single.xml
index a4a1d6af5..755f5917d 100644
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-single.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-invoke-updateobjloc-single.xml
@@ -5,6 +5,9 @@
     <mode>single</mode>
     <docType>CollectionObject</docType>
     <singleCSID>${collectionObjectCSID}</singleCSID>
+    <groupCSID></groupCSID>
+    <listCsids></listCsids>
+    <params></params>
 </ns2:invocationContext>
 
 
diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-update-object-loc.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-update-object-loc.xml
index 84088950e..43747c2cd 100644
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-update-object-loc.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/batch-update-object-loc.xml
@@ -7,7 +7,7 @@
         
     <!-- This tests the UpdateObjectLocationBatchJob -->
         
-    <testGroup ID="testSingleRecordBatchUpdate" autoDeletePOSTS="true">
+    <testGroup ID="testBatchUpdateComputedCurrentLocationInvModeSingle" autoDeletePOSTS="true">
         
         <!-- Tests with CollectionObject->Movement relations -->
             
@@ -103,7 +103,7 @@
         
         <!-- This post of a batch job invocation request doesn't create a new record, -->
         <!-- so we need to exclude it from the post-run auto-deletion of records. -->
-        <test ID="invokeBatch1" auth="test" autoDeletePOSTS="false">
+        <test ID="invokeBatch" auth="test" autoDeletePOSTS="false">
             <method>POST</method>
             <uri>/cspace-services/batch/${createBatchRecord.CSID}</uri>
             <filename>batch/batch-invoke-updateobjloc-single.xml</filename>
@@ -190,5 +190,145 @@
         
     </testGroup>
     
+    <testGroup ID="testBatchUpdateComputedCurrentLocationInvModeList" autoDeletePOSTS="true">
+        
+        <test ID="createBatchRecord">
+            <method>POST</method>
+            <uri>/cspace-services/batch</uri>
+            <filename>batch/updateobjloc.xml</filename>
+            <expectedCodes>201</expectedCodes>
+        </test>
+                
+        <test ID="createCollectionObject1">
+            <method>POST</method>
+            <uri>/cspace-services/collectionobjects</uri>
+            <filename>batch/collObj1.xml</filename>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="createCollectionObject2">
+            <method>POST</method>
+            <uri>/cspace-services/collectionobjects</uri>
+            <filename>batch/collObj1.xml</filename>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="createMovement1">
+            <method>POST</method>
+            <uri>/cspace-services/movements</uri>
+            <filename>batch/movement.xml</filename>
+            <vars>
+                <var ID="currentLocation">location-1</var>
+                <var ID="locationDate">1900-01-01</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="createMovement2">
+            <method>POST</method>
+            <uri>/cspace-services/movements</uri>
+            <filename>batch/movement.xml</filename>
+            <vars>
+                <var ID="currentLocation">location-2</var>
+                <var ID="locationDate">2000-01-01</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="createMovement3">
+            <method>POST</method>
+            <uri>/cspace-services/movements</uri>
+            <filename>batch/movement.xml</filename>
+            <vars>
+                <var ID="currentLocation">location-3</var>
+                <var ID="locationDate">1800-01-01</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="relateCollectionObject1ToMovement1">
+            <method>POST</method>
+            <uri>/cspace-services/relations</uri>
+            <filename>batch/relation.xml</filename>
+            <vars>
+                <var ID="subjectCsid">${createCollectionObject1.CSID}</var>
+                <var ID="subjectDocumentType">CollectionObject</var>
+                <var ID="objectCsid">${createMovement1.CSID}</var>
+                <var ID="objectDocumentType">Movement</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="relateCollection1ObjectToMovement2">
+            <method>POST</method>
+            <uri>/cspace-services/relations</uri>
+            <filename>batch/relation.xml</filename>
+            <vars>
+                <var ID="subjectCsid">${createCollectionObject1.CSID}</var>
+                <var ID="subjectDocumentType">CollectionObject</var>
+                <var ID="objectCsid">${createMovement2.CSID}</var>
+                <var ID="objectDocumentType">Movement</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="relateCollectionObject2ToMovement3">
+            <method>POST</method>
+            <uri>/cspace-services/relations</uri>
+            <filename>batch/relation.xml</filename>
+            <vars>
+                <var ID="subjectCsid">${createCollectionObject2.CSID}</var>
+                <var ID="subjectDocumentType">CollectionObject</var>
+                <var ID="objectCsid">${createMovement3.CSID}</var>
+                <var ID="objectDocumentType">Movement</var>
+            </vars>
+            <expectedCodes>201</expectedCodes>
+        </test>
+        
+        <test ID="invokeBatchWithList" auth="test" autoDeletePOSTS="false">
+            <method>POST</method>
+            <uri>/cspace-services/batch/${createBatchRecord.CSID}</uri>
+            <filename>batch/batch-invoke-updateobjloc-list.xml</filename>
+            <vars>
+                <var ID="collectionObject1CSID">${createCollectionObject1.CSID}</var>
+                <var ID="collectionObject2CSID">${createCollectionObject2.CSID}</var>
+            </vars>
+            <expectedCodes>200</expectedCodes>
+        </test>
+        
+        <!-- This CollectionObject record's computedCurrentLocation field should -->
+        <!-- have been updated with the currentLocation value from Movement record 2 -->
+        <test ID="readUpdatedCollectionObjectRecord1">
+            <method>GET</method>
+            <uri>/cspace-services/collectionobjects/${createCollectionObject1.CSID}</uri>
+            <filename>batch/updateobjloc.xml</filename>
+            <response>
+                <expected level="ADDOK" />
+                <filename>batch/res/collectionobject.res.xml</filename>
+                <vars>
+                    <var ID="computedCurrentLocationValue">${createMovement2.currentLocation}</var>
+                </vars>
+            </response>
+            <expectedCodes>200</expectedCodes>
+        </test>
+        
+        <!-- This CollectionObject record's computedCurrentLocation field should -->
+        <!-- have been updated with the currentLocation value from Movement record 3 -->
+        <test ID="readUpdatedCollectionObjectRecord2">
+            <method>GET</method>
+            <uri>/cspace-services/collectionobjects/${createCollectionObject2.CSID}</uri>
+            <filename>batch/updateobjloc.xml</filename>
+            <response>
+                <expected level="ADDOK" />
+                <filename>batch/res/collectionobject.res.xml</filename>
+                <vars>
+                    <var ID="computedCurrentLocationValue">${createMovement3.currentLocation}</var>
+                </vars>
+            </response>
+            <expectedCodes>200</expectedCodes>
+        </test>
+
+    </testGroup>
+    
 </xmlReplay>
 
diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/updateobjloc.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/updateobjloc.xml
index c83b22db1..1129a2a77 100644
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/updateobjloc.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/batch/updateobjloc.xml
@@ -8,7 +8,7 @@
             <forDocType>CollectionObject</forDocType>
         </forDocTypes>
         <supportsSingleDoc>true</supportsSingleDoc>
-        <supportsDocList>false</supportsDocList>
+        <supportsDocList>true</supportsDocList>
         <createsNewFocus>false</createsNewFocus>
         <className>org.collectionspace.services.batch.nuxeo.UpdateObjectLocationBatchJob</className>
     </ns2:batch_common>
diff --git a/services/batch/service/src/main/java/org/collectionspace/services/batch/AbstractBatchInvocable.java b/services/batch/service/src/main/java/org/collectionspace/services/batch/AbstractBatchInvocable.java
index 1dc9f5001..02c5ddb2a 100644
--- a/services/batch/service/src/main/java/org/collectionspace/services/batch/AbstractBatchInvocable.java
+++ b/services/batch/service/src/main/java/org/collectionspace/services/batch/AbstractBatchInvocable.java
@@ -53,6 +53,38 @@ public abstract class AbstractBatchInvocable implements BatchInvocable {
         this.errorInfo = null;
     }
 
+    protected void logInvocationContext() {
+        if (logger.isInfoEnabled()) {
+            logger.info("Invocation mode=" + this.getInvocationContext().getMode());
+            logger.info("Invocation doc type=" + this.getInvocationContext().getDocType());
+            logger.info("Invocation single CSID=" + this.getInvocationContext().getSingleCSID());
+            logger.info("Invocation group CSID=" + this.getInvocationContext().getGroupCSID());
+            InvocationContext.ListCSIDs lcsids = this.getInvocationContext().getListCSIDs();
+            if (lcsids == null) {
+                logger.info("Invocation list CSIDs are null.");
+            } else {
+                List<String> csidsList = lcsids.getCsid();
+                if (csidsList != null) {
+                    for (String listcsid : csidsList) {
+                        logger.info("List CSID=" + listcsid);
+                    }
+                }
+            }
+            InvocationContext.Params params = this.getInvocationContext().getParams();
+            if (params == null) {
+                logger.info("Invocation list Params are null.");
+            } else {
+                List<InvocationContext.Params.Param> paramsList = params.getParam();
+                if (paramsList != null) {
+                    for (InvocationContext.Params.Param param : paramsList) {
+                        logger.info("Param key=" + param.getKey());
+                        logger.info("Param value=" + param.getValue());
+                    }
+                }
+            }
+        }
+    }
+
     @Override
     public List<String> getSupportedInvocationModes() {
         return invocationModes;
diff --git a/services/batch/service/src/main/java/org/collectionspace/services/batch/nuxeo/UpdateObjectLocationBatchJob.java b/services/batch/service/src/main/java/org/collectionspace/services/batch/nuxeo/UpdateObjectLocationBatchJob.java
index 455f4ffae..cf9cbf427 100644
--- a/services/batch/service/src/main/java/org/collectionspace/services/batch/nuxeo/UpdateObjectLocationBatchJob.java
+++ b/services/batch/service/src/main/java/org/collectionspace/services/batch/nuxeo/UpdateObjectLocationBatchJob.java
@@ -19,6 +19,7 @@ import org.collectionspace.services.client.PoxPayloadOut;
 import org.collectionspace.services.common.ResourceBase;
 import org.collectionspace.services.common.ResourceMap;
 import org.collectionspace.services.common.api.Tools;
+import org.collectionspace.services.common.invocable.InvocationContext;
 import org.collectionspace.services.common.invocable.InvocationResults;
 import org.collectionspace.services.jaxb.AbstractCommonList;
 import org.dom4j.DocumentException;
@@ -62,10 +63,8 @@ public class UpdateObjectLocationBatchJob extends AbstractBatchInvocable {
         setCompletionStatus(STATUS_MIN_PROGRESS);
 
         try {
-            if (logger.isTraceEnabled()) {
-                logger.trace("Invoking " + CLASSNAME + " ...");
-                logger.trace("Invocation context is: " + getInvocationContext().getMode());
-            }
+
+            logInvocationContext();
 
             List<String> csids = new ArrayList<String>();
             if (requestIsForInvocationModeSingle()) {
@@ -74,7 +73,11 @@ public class UpdateObjectLocationBatchJob extends AbstractBatchInvocable {
                     csids.add(singleCsid);
                 }
             } else if (requestIsForInvocationModeList()) {
-                List<String> listCsids = (getInvocationContext().getListCSIDs().getCsid());
+                InvocationContext.ListCSIDs invListCsids = getInvocationContext().getListCSIDs();
+                if (invListCsids == null) {
+                    throw new Exception(CSID_VALUES_NOT_PROVIDED_IN_INVOCATION_CONTEXT);
+                }
+                List<String> listCsids = invListCsids.getCsid();
                 if (listCsids == null || listCsids.isEmpty()) {
                     throw new Exception(CSID_VALUES_NOT_PROVIDED_IN_INVOCATION_CONTEXT);
                 }
@@ -296,4 +299,5 @@ public class UpdateObjectLocationBatchJob extends AbstractBatchInvocable {
         getResults().setNumAffected(numAffected);
         return getResults();
     }
+
 }