From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Thu, 26 Apr 2012 01:31:26 +0000 (-0700)
Subject: CSPACE-3917,CSPACE-5049: Additional check of server-side variable expansion in collec... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=7a077aaca42710e30b95bb620c282e2ac9b4789a;p=tmp%2Fjakarta-migration.git

CSPACE-3917,CSPACE-5049: Additional check of server-side variable expansion in collectionspace_core part.
---

diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/import-objectexit-docid.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/import-objectexit-varexpansion.xml
similarity index 100%
rename from services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/import-objectexit-docid.xml
rename to services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/import-objectexit-varexpansion.xml
diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/imports.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/imports.xml
index 45f8e63ac..0d829b1f3 100644
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/imports.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/imports.xml
@@ -77,15 +77,21 @@
         
         <!--
             Import a record with a client-provided CSID.
-            Then also insert the value of that CSID into a field via the ${docID} variable. 
+            Then also exercise the Imports services's variable expansion:
+            * Insert the value of that CSID into a field via expansion
+              of the ${docID} variable.
+            * Verify that variable has been expanded in the imported record.
+            * Verify that a server-side variable has also been expanded in the
+              wrapper template for imported records; specifically, that an
+              expected value is is present in the collectionspace_core part.
             (See "Variables supported in expansion of request" in
             http://wiki.collectionspace.org/display/collectionspace/Imports+Service+Home)
         -->
-        <test ID="importObjectExitWithDocID">
+        <test ID="importObjectExitWithVarExpansion">
             <expectedCodes>200</expectedCodes>
             <method>POST</method>
             <uri>/cspace-services/imports</uri>
-            <filename>imports/import-objectexit-docid.xml</filename>
+            <filename>imports/import-objectexit-varexpansion.xml</filename>
             <vars>
                 <var ID="recordCSID">920c578f-e5d6-45da-adab-6f26f6a491ff</var>
             </vars>
@@ -94,20 +100,20 @@
                 <filename>imports/res/import-objectexit.res.xml</filename>
             </response>
         </test>
-        <test ID="verifyObjectExitWithDocID">
+        <test ID="verifyObjectExitWithVarExpansion">
             <expectedCodes>200</expectedCodes>
             <method>GET</method>
-            <uri>/cspace-services/objectexit/${importObjectExitWithDocID.recordCSID}</uri>
+            <uri>/cspace-services/objectexit/${importObjectExitWithVarExpansion.recordCSID}</uri>
             <response>
                 <expected level="ADDOK" />
-                <filename>imports/res/import-objectexit-docID.res.xml</filename>
+                <filename>imports/res/import-objectexit-varexpansion.res.xml</filename>
                 <label>objectexit_common</label>
             </response>
         </test>
-        <test ID="deleteObjectExitWithDocID">
+        <test ID="deleteObjectExitWithVarExpansion">
             <expectedCodes>200</expectedCodes>
             <method>DELETE</method>
-            <uri>/cspace-services/objectexit/${importObjectExitWithDocID.recordCSID}</uri>
+            <uri>/cspace-services/objectexit/${importObjectExitWithVarExpansion.recordCSID}</uri>
         </test>
         
         <!--
@@ -152,6 +158,17 @@
     -->
     <testGroup ID="importsTemporaryTestGroup" autoDeletePOSTS="false">
         
+        <!-- Add new or changed tests here -->
+        
+    </testGroup>
+    
+    
+    <!--
+      Test group holding notes and markup fragments for possible future
+      standard tests of the Imports service.
+    -->
+    <testGroup ID="importsTestNotesGroup" autoDeletePOSTS="false">
+        
         <!-- The following tests are not currently working, and thus are commented out. -->
         <!--
             Import a single ObjectExit record, at least one of whose fields
@@ -190,7 +207,8 @@
         </test>
         -->
         
-        <!-- FIXME: Add forms-based import test -->
+        <!-- FIXME: Add a test of the forms-based import pathway. -->
+        <!-- This may require support for the requisite MIME type in XmlReplay -->
         <!--
         <test ID="import-post-form">
             <expectedCodes>200</expectedCodes>
@@ -204,10 +222,11 @@
         </test>
         -->
         
-        <!-- FIXME: Add tests with invalid payloads, via each imports path -->
-        
-        <!-- Add new or changed tests here -->
+        <!-- FIXME: Add import of records with schema expansion part(s). -->
         
+        <!-- FIXME: Add tests with invalid payloads, via each imports path, -->
+        <!-- and verify failure results. -->
+                
     </testGroup>
 
 </xmlReplay>
diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-docid.res.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-varexpansion.res.xml
similarity index 54%
rename from services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-docid.res.xml
rename to services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-varexpansion.res.xml
index cde41b302..d007a8b73 100644
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-docid.res.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/imports/res/import-objectexit-varexpansion.res.xml
@@ -1,8 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <document name="objectexit">
     <ns2:objectexit_common xmlns:ns2="http://collectionspace.org/services/objectexit" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
-        <exitNote>The value of a 'docID' variable in the original import document should be inserted here: ${importObjectExitWithDocID.recordCSID}</exitNote>
+        <exitNote>The value of a 'docID' variable in the original import document should be inserted here: ${importObjectExitWithVarExpansion.recordCSID}</exitNote>
         <exitNumber>OE-IMPORT-TEST-1999.8</exitNumber>
     </ns2:objectexit_common>
+    <ns2:collectionspace_core xmlns:ns2="http://collectionspace.org/collectionspace_core/"
+                              xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+        <!-- Note: This test may fail if the identifier for the core tenant changes. -->
+        <tenantId>1</tenantId>
+    </ns2:collectionspace_core>
 </document>