Add tests to copyS3File to handle files that have already been persisted

Author: davidjamesstone

src/services/file-service.test.js

@@ -801,7 +801,7 @@ describe('Files service', () => {
       })
     })
 
-    it('should not allow a previously extended file to be extended again', async () => {
+    it('should allow a previously extended file to be extended again', async () => {
       /** @type {FormFileUploadStatus} */
       const dummyData = {
         ...successfulFile,
@@ -812,20 +812,25 @@ describe('Files service', () => {
       jest.mocked(verify).mockResolvedValueOnce(true)
       jest.mocked(repository.getByFileId).mockResolvedValueOnce(dummyData)
 
-      await expect(
-        persistFiles(
-          [
-            {
-              fileId: dummyData.fileId,
-              initiatedRetrievalKey: dummyData.retrievalKey
-            }
-          ],
-          dummyData.retrievalKey
-        )
-      ).rejects.toThrow(
-        Boom.badRequest(
-          `File ID ${dummyData.fileId} has already been persisted`
-        )
+      jest.mocked(hash).mockResolvedValueOnce('caseSensitiveHash')
+      jest.mocked(verify).mockResolvedValueOnce(true)
+
+      await persistFiles(
+        [
+          {
+            fileId: dummyData.fileId,
+            initiatedRetrievalKey: dummyData.retrievalKey
+          }
+        ],
+        dummyData.retrievalKey
+      )
+
+      expect(hash).toHaveBeenCalledWith(dummyData.retrievalKey.toLowerCase())
+      expect(repository.updateRetrievalKeys).toHaveBeenCalledWith(
+        [dummyData.fileId],
+        'caseSensitiveHash',
+        false,
+        expect.any(Object)
       )
     })