Merge pull request #226 from adhocteam/js-fix-be-tests

File upload test shouldn't error out
HHS · Mar 11, 2021 · 9369bd5 · 9369bd5
2 parents 0c44259 + de255ce
commit 9369bd5
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/routes/files/handlers.test.js b/src/routes/files/handlers.test.js
@@ -80,11 +80,9 @@ describe('File Upload', () => {
   });
 
   describe('File Upload Handlers happy path', () => {
-    beforeEach(() => {
+    beforeEach(async () => {
       uploadFile.mockReset();
       getPresignedURL.mockReset();
-    });
-    it('tests a file upload', async () => {
       ActivityReportPolicy.mockImplementation(() => ({
         canUpdate: () => true,
       }));
@@ -96,8 +94,10 @@ describe('File Upload', () => {
         .expect(200)
         .then((res) => {
           fileId = res.body.id;
-          expect(uploadFile).toHaveBeenCalled();
         });
+    });
+    it('tests a file upload', async () => {
+      expect(uploadFile).toHaveBeenCalled();
       expect(mockAddToScanQueue).toHaveBeenCalled();
     });
     it('checks the metadata was uploaded to the database', async () => {