CSPACE-5677: Updated comments, test group name for clarity.

diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/versioning/versioning.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/versioning/versioning.xml
index 9ab52ab0fde2a114c73c666eb4ec5343800f85e4..c8edb53f11c0a9f9447f235630f89a282e78d937 100644 (file)
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/versioning/versioning.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/versioning/versioning.xml
@@ -1,12 +1,17 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <xmlReplay>
-  <!-- Authenticate into the lifesci tenant for extension testing -->
+  <!-- Authenticate into the lifesci tenant. By default, versioning -->
+  <!-- support is exposed in the Location/Movement/Inventory service -->
+  <!-- in that tenant. -->
   <auths default="admin@lifesci.collectionspace.org">
     <auth ID="admin@lifesci.collectionspace.org"
       >YWRtaW5AbGlmZXNjaS5jb2xsZWN0aW9uc3BhY2Uub3JnOkFkbWluaXN0cmF0b3I=</auth>
   </auths>
 
-  <testGroup ID="Versioning" autoDeletePOSTS="true">
+  <!-- Verifies that, by default: -->
+  <!--   * Versioned records are excluded (filtered out) from list results. -->
+  <!--   * Only the current version of a record is retrieved on a read request. -->
+  <testGroup ID="VersionedRecordFiltering" autoDeletePOSTS="true">
   
     <test ID="CreateMovement" auth="admin@lifesci.collectionspace.org">
       <method>POST</method>
@@ -41,8 +46,8 @@
     </test>
     
     <!-- The total number of Movement records reported should not change following -->
-    <!-- the update, even though a new version of one Movement document was created -->
-    <!-- as a result of that update. -->
+    <!-- the update, even though a past version of one Movement document was created -->
+    <!-- automatically as a result of that update. -->
     <test ID="ListAllMovementRecordsFollowingUpdate">
       <method>GET</method>
       <uri>/cspace-services/movements</uri>
@@ -58,7 +63,7 @@
     <!-- A keyword search on the (intended to be unique) currentLocation value -->
     <!-- should retrieve only one document record, even though there are now two -->
     <!-- documents extant containing that value: the current version and one past -->
-    <!-- version, which was created as a result of the update. -->
+    <!-- version, the latter created automatically as a result of the update. -->
     <test ID="SearchReturnsOnlyCurrentVersion">
       <method>GET</method>
       <uri>/cspace-services/movements?kw=${CreateMovement.currentLocationValue}</uri>
@@ -71,9 +76,8 @@
       </response>
     </test>
     
-    <!-- Retrieving the document via a read (GET) request should also verify -->
-    <!-- that only the current version, containing the most recently updated -->
-    <!-- value(s), is returned. -->
+    <!-- Retrieving the document via a read request should return only -->
+    <!-- the current version, containing the most recently updated value(s). -->
     <test ID="ReadReturnsOnlyCurrentVersion">
       <method>GET</method>
       <uri>/cspace-services/movements/${CreateMovement.CSID}</uri>

diff --git a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/xml-replay-master.xml b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/xml-replay-master.xml
index e5b14a767e7037a93160373212b70a69003601e0..6de5391ce54db01f5256c6fa8e5624814073207c 100644 (file)
--- a/services/IntegrationTests/src/test/resources/test-data/xmlreplay/xml-replay-master.xml
+++ b/services/IntegrationTests/src/test/resources/test-data/xmlreplay/xml-replay-master.xml
@@ -41,7 +41,7 @@
     <run controlFile="collectionobject/collectionobject-hierarchy-refname.xml" testGroup="CreateUpdateReadStructuredObjects" />\r
     <run controlFile="collectionobject/collectionobject-displayname-update.xml" testGroup="DisplayNameChangesReflectedInRefNames" />\r
     \r
-    <run controlFile="versioning/versioning.xml" testGroup="Versioning" />\r
+    <run controlFile="versioning/versioning.xml" testGroup="VersionedRecordFiltering" />\r
 \r
     \r
     <!-- Broken tests -->\r