update FileNode tests

Author: raclim

client/modules/IDE/components/FileNode.unit.test.jsx

@@ -2,6 +2,7 @@ import React from 'react';
 import { Provider } from 'react-redux';
 import configureStore from 'redux-mock-store';
 import { useDispatch } from 'react-redux';
+import * as FileActions from '../actions/files';
 
 import {
   fireEvent,
@@ -10,19 +11,25 @@ import {
   waitFor,
   within
 } from '../../../test-utils';
-import { FileNode } from './FileNode';
+import FileNode from './FileNode';
 
 jest.mock('react-redux', () => ({
   ...jest.requireActual('react-redux'),
   useDispatch: jest.fn()
 }));
 
+jest.mock('../actions/files', () => ({
+  updateFileName: jest.fn()
+}));
+
+const mockStore = configureStore([]);
+
 describe('<FileNode />', () => {
   const mockDispatch = jest.fn();
-  const mockStore = configureStore([]);
 
   beforeEach(() => {
     useDispatch.mockReturnValue(mockDispatch);
+    jest.clearAllMocks();
   });
 
   const changeName = (newFileName) => {
@@ -39,138 +46,123 @@ describe('<FileNode />', () => {
     await waitFor(() => within(name).queryByText(expectedName));
   };
 
-  const renderFileNode = (fileType, extraProps = {}) => {
+  const renderFileNode = (fileType, extraState = {}) => {
     const initialState = {
       files: [
         {
           id: '0',
           name: fileType === 'folder' ? 'afolder' : 'test.jsx',
-          fileType
+          fileType,
+          parentId: 'root',
+          children: [],
+          isSelectedFile: false,
+          isFolderClosed: false
         }
       ],
-      user: { authenticated: false }
+      user: { authenticated: false },
+      ...extraState
     };
 
     const store = mockStore(initialState);
 
-    const props = {
-      ...extraProps,
-      id: '0',
-      name: fileType === 'folder' ? 'afolder' : 'test.jsx',
-      fileType,
-      canEdit: true,
-      children: [],
-      authenticated: false,
-      setSelectedFile: jest.fn(),
-      deleteFile: jest.fn(),
-      updateFileName: jest.fn(),
-      resetSelectedFile: jest.fn(),
-      newFile: jest.fn(),
-      newFolder: jest.fn(),
-      showFolderChildren: jest.fn(),
-      hideFolderChildren: jest.fn(),
-      openUploadFileModal: jest.fn(),
-      setProjectName: jest.fn()
-    };
-
     render(
       <Provider store={store}>
-        <FileNode {...props} />
+        <FileNode id="0" canEdit />
       </Provider>
     );
 
-    return { store, props };
+    return { store };
   };
 
   describe('fileType: file', () => {
     it('cannot change to an empty name', async () => {
-      const { props } = renderFileNode('file');
+      renderFileNode('file');
 
       changeName('');
 
       await waitFor(() => expect(mockDispatch).not.toHaveBeenCalled());
-      await expectFileNameToBe(props.name);
+      await expectFileNameToBe('test.jsx');
     });
 
     it('can change to a valid filename', async () => {
       const newName = 'newname.jsx';
-      const { props } = renderFileNode('file');
+      renderFileNode('file');
 
       changeName(newName);
 
       await waitFor(() =>
-        expect(props.updateFileName).toHaveBeenCalledWith(props.id, newName)
+        expect(FileActions.updateFileName).toHaveBeenCalledWith('0', newName)
       );
       await expectFileNameToBe(newName);
     });
 
     it('must have an extension', async () => {
       const newName = 'newname';
-      const { props } = renderFileNode('file');
+      renderFileNode('file');
 
       changeName(newName);
 
       await waitFor(() => expect(mockDispatch).not.toHaveBeenCalled());
-      await expectFileNameToBe(props.name);
+      await expectFileNameToBe('test.jsx');
     });
 
     it('can change to a different extension', async () => {
       const mockConfirm = jest.fn(() => true);
       window.confirm = mockConfirm;
 
       const newName = 'newname.gif';
-      const { props } = renderFileNode('file');
+      renderFileNode('file');
 
       changeName(newName);
 
       expect(mockConfirm).toHaveBeenCalled();
       await waitFor(() =>
-        expect(props.updateFileName).toHaveBeenCalledWith(props.id, newName)
+        expect(FileActions.updateFileName).toHaveBeenCalledWith('0', newName)
       );
       await expectFileNameToBe(newName);
     });
 
     it('cannot be just an extension', async () => {
       const newName = '.jsx';
-      const { props } = renderFileNode('file');
+      renderFileNode('file');
 
       changeName(newName);
 
       await waitFor(() => expect(mockDispatch).not.toHaveBeenCalled());
-      await expectFileNameToBe(props.name);
+      await expectFileNameToBe('test.jsx');
     });
   });
 
   describe('fileType: folder', () => {
     it('cannot change to an empty name', async () => {
-      const { props } = renderFileNode('folder');
+      renderFileNode('folder');
 
       changeName('');
 
       await waitFor(() => expect(mockDispatch).not.toHaveBeenCalled());
-      await expectFileNameToBe(props.name);
+      await expectFileNameToBe('afolder');
     });
 
     it('can change to another name', async () => {
       const newName = 'foldername';
-      const { props } = renderFileNode('folder');
+      renderFileNode('folder');
 
       changeName(newName);
 
       await waitFor(() =>
-        expect(props.updateFileName).toHaveBeenCalledWith(props.id, newName)
+        expect(FileActions.updateFileName).toHaveBeenCalledWith('0', newName)
       );
       await expectFileNameToBe(newName);
     });
 
     it('cannot have a file extension', async () => {
       const newName = 'foldername.jsx';
-      const { props } = renderFileNode('folder');
+      renderFileNode('folder');
 
       changeName(newName);
 
       await waitFor(() => expect(mockDispatch).not.toHaveBeenCalled());
-      await expectFileNameToBe(props.name);
+      await expectFileNameToBe('afolder');
     });
   });
 });