fix linting and unit tests #1075

ral-facilities · Jan 14, 2025 · 0cc1b3a · 0cc1b3a
1 parent 3f4e8db
commit 0cc1b3a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/common/editFileDialog.component.test.tsx b/src/common/editFileDialog.component.test.tsx
@@ -98,7 +98,7 @@ describe('Edit file dialog', () => {
       await user.click(saveButton);
 
       expect(axiosPatchSpy).toHaveBeenCalledWith('/images/1', {
-        file_name: 'test_file_name.jpeg',
+        file_name: 'test_file_name.jpeg.png',
         title: 'Test Title',
         description: 'Test Description',
       });

diff --git a/src/common/editFileDialog.component.tsx b/src/common/editFileDialog.component.tsx
@@ -53,7 +53,7 @@ const EditFileDialog = (props: FileDialogProps) => {
 
   const selectedFileCopy: ObjectFilePatch = React.useMemo(
     () => (selectedFile ? { ...selectedFile, file_name: initialName } : {}),
-    [selectedFile]
+    [selectedFile, initialName]
   );
 
   const initialFile: ObjectFilePatch = React.useMemo(
@@ -109,7 +109,7 @@ const EditFileDialog = (props: FileDialogProps) => {
 
         const isTitleUpdated = fileData.title !== selectedFileCopy.title;
 
-        let fileToEdit: ObjectFilePatch = {};
+        const fileToEdit: ObjectFilePatch = {};
 
         if (isFileNameUpdated)
           fileToEdit.file_name = fileData.file_name + extension;

diff --git a/src/mocks/handlers.ts b/src/mocks/handlers.ts
@@ -1084,7 +1084,7 @@ export const handlers = [
 
       const fullBody = { ...obj, ...body };
 
-      if (fullBody.file_name === 'Error 500') {
+      if (fullBody.file_name === 'Error 500.png') {
         return HttpResponse.json(
           { detail: 'Something went wrong' },
           { status: 500 }