Media Upload failure provides a reason.

samagra-comms · Oct 6, 2023 · a6ccd0f · a6ccd0f
1 parent a490eaf
commit a6ccd0f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/modules/form/form.controller.ts b/src/modules/form/form.controller.ts
@@ -66,7 +66,7 @@ export class FormController {
   async single(
     @UploadedFiles() files: {form: Express.Multer.File[], mediaFiles: Express.Multer.File[]},
   ) {
-    if (!files.form[0]) {
+    if (!files.form || !files.form[0]) {
       throw new BadRequestException('Form file is required!');
     }
     const response = await this.formService.uploadForm(files.form[0], files.mediaFiles);

diff --git a/src/modules/form/form.service.spec.ts b/src/modules/form/form.service.spec.ts
@@ -127,7 +127,7 @@ describe('FormService', () => {
 
     expect(formService.uploadForm(mockXmlFile, mockMediaFiles))
     .rejects
-    .toThrowError(new ServiceUnavailableException('Media upload failed!'));
+    .toThrowError(new ServiceUnavailableException('Media upload failed! Reason: Error'));
   });
 
   it('form upload throws service unavailable exception when ODK server is unavailable', async () => {

diff --git a/src/modules/form/form.service.ts b/src/modules/form/form.service.ts
@@ -81,7 +81,7 @@ export class FormService {
       const mediaUploadResult = await this.uploadFormMediaFiles(mediaFiles);
       if (mediaUploadResult.error || !mediaUploadResult.data) {
         this.logger.error(`FormService::uploadForm: Media Files upload failed!`);
-        throw new ServiceUnavailableException('Media upload failed!');
+        throw new ServiceUnavailableException(`Media upload failed! Reason: ${mediaUploadResult.error}`);
       }
       const xmlModificationError = this.replaceMediaFileName(formFile, mediaUploadResult.data);
       if (xmlModificationError != '') {