Merge pull request #2443 from Accenture/bug/2439-createdDate-and-modifiedDate-saved-as-0001-01-01T00-00-00-after-update

bug/2439 refresh createdDate and modifiedDate after updating scripts

Author: JoernBerkefeld

@types/lib/metadataTypes/Script.d.ts

@@ -83,6 +83,22 @@ class Script extends MetadataType {
         return super.updateREST(script, '/automation/v1/scripts/' + script.ssjsActivityId);
     }
 
+    /**
+     * helper for {@link MetadataType.updateREST} and {@link MetadataType.updateSOAP}
+     *
+     * @param {MetadataTypeItem} metadataEntry a single metadata Entry
+     * @param {object} apiResponse varies depending on the API call
+     * @returns {Promise.<object>} apiResponse, potentially modified
+     */
+    static async postUpdateTasks(metadataEntry, apiResponse) {
+        // script update endpoint returns wrong values for createdDate and modifiedDate - need to re-retrieve to get correct values
+        const ssjsActivityId = apiResponse?.ssjsActivityId;
+        if (ssjsActivityId) {
+            apiResponse = await this.client.rest.get('/automation/v1/scripts/' + ssjsActivityId);
+        }
+        return apiResponse;
+    }
+
     /**
      * Creates a single Script
      *

@types/lib/metadataTypes/Script.d.ts.map

@@ -0,0 +1,10 @@
+{
+    "ssjsActivityId": "39f6a488-20eb-4ba0-b0b9-023725b574e4",
+    "name": "testExisting_script",
+    "key": "testExisting_script",
+    "description": "updated on deploy",
+    "script": "<script runat=\"server\">\n// dummy updated\n</script>",
+    "categoryId": 304,
+    "createdDate": "2022-10-20T00:41:26.163",
+    "modifiedDate": "2022-10-20T00:41:26.163"
+}

lib/metadataTypes/Script.js

@@ -5,6 +5,6 @@
     "description": "updated on deploy",
     "script": "<script runat=\"server\">\n// dummy updated\n</script>",
     "categoryId": 304,
-    "createdDate": "2022-10-20T00:41:26.163",
-    "modifiedDate": "2022-10-20T00:41:26.163"
+    "createdDate": "0001-01-01T00:00:00",
+    "modifiedDate": "0001-01-01T00:00:00"
 }

test/resources/9999999/automation/v1/scripts/39f6a488-20eb-4ba0-b0b9-023725b574e4/get-response.json

@@ -261,7 +261,7 @@ describe('type: script', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                4,
+                5,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;