fix: patch to name duplicate

Author: leangseu-edx

package-lock.json

@@ -24,7 +24,7 @@ export const FetchSubmissionFilesException = () => ({
  * @return {string} - manifest text file content.
  */
 export const genManifest = (files) => files.map(
-  (file) => `Filename: ${file.name}\nDescription: ${file.description}\nSize: ${file.size}`,
+  (file, i) => `Filename: ${i}-${file.name}\nDescription: ${file.description}\nSize: ${file.size}`,
 ).join('\n\n');
 
 /**
@@ -41,7 +41,7 @@ export const zipFiles = async (files, blobs, username) => {
   // promise or async function just to circumvent that.
   for (let i = 0; i < blobs.length; i++) {
     // eslint-disable-next-line no-await-in-loop
-    await zipWriter.add(files[i].name, new zip.BlobReader(blobs[i]), {
+    await zipWriter.add(`${i}-${files[i].name}`, new zip.BlobReader(blobs[i]), {
       bufferedWrite: true,
     });
   }

src/data/redux/thunkActions/download.js

@@ -66,8 +66,8 @@ describe('download thunkActions', () => {
     test('returns a list of strings with filename and description for each file', () => {
       expect(download.genManifest(files)).toEqual(
         [
-          `Filename: ${files[0].name}\nDescription: ${files[0].description}\nSize: ${files[0].size}`,
-          `Filename: ${files[1].name}\nDescription: ${files[1].description}\nSize: ${files[0].size}`,
+          `Filename: 0-${files[0].name}\nDescription: ${files[0].description}\nSize: ${files[0].size}`,
+          `Filename: 1-${files[1].name}\nDescription: ${files[1].description}\nSize: ${files[0].size}`,
         ].join('\n\n'),
       );
     });
@@ -78,8 +78,8 @@ describe('download thunkActions', () => {
       return download.zipFiles(files, blobs, username).then(() => {
         expect(mockZipWriter.files).toEqual([
           ['manifest.txt', mockTextReader],
-          [files[0].name, mockBlobReader],
-          [files[1].name, mockBlobReader],
+          [`0-${files[0].name}`, mockBlobReader],
+          [`1-${files[1].name}`, mockBlobReader],
         ]);
         expect(mockZipAdd).toHaveBeenCalledTimes(mockZipWriter.files.length);
         expect(mockZipClose).toHaveBeenCalledTimes(1);