test(app): remove commented out tests (#6169)

* test(app): remove commented out tests remove commented out tests to get rid of linter warning * add todos
Opentrons · Aug 4, 2020 · 1bdc118 · 1bdc118
1 parent 0e089a7
commit 1bdc118
Showing 1 changed file with 5 additions and 45 deletions.
diff --git a/app/src/components/ModuleItem/__tests__/ModuleUpdate.test.js b/app/src/components/ModuleItem/__tests__/ModuleUpdate.test.js
@@ -26,51 +26,6 @@ describe('ModuleUpdate', () => {
     jest.resetAllMocks()
   })
 
-  // TODO(mc, 2020-03-16): these shallow snapshots don't test anything
-  // remove commented out tests when actual tests have been written
-  // it('component renders', () => {
-  //   const treeAvailableCanControl = shallow(
-  //     <Provider store={store}>
-  //       <ModuleUpdate
-  //         controlDisabledReason={null}
-  //         moduleId="FAKEMODULE1234"
-  //         hasAvailableUpdate={true}
-  //       />
-  //     </Provider>,
-  //   )
-  //   const treeAvailableNoControl = shallow(
-  //     <Provider store={store}>
-  //       <ModuleUpdate
-  //         controlDisabledReason={"Can't touch this"}
-  //         moduleId="FAKEMODULE1234"
-  //         hasAvailableUpdate={true}
-  //       />
-  //     </Provider>
-  //   )
-  //   const treeNotAvailableCanControl = shallow(
-  //     <Provider store={store}>
-  //       <ModuleUpdate
-  //         controlDisabledReason={null}
-  //         moduleId="FAKEMODULE1234"
-  //         hasAvailableUpdate={false}
-  //       />
-  //     </Provider>
-  //   )
-  //   const treeNotAvailableNoControl = shallow(
-  //     <Provider store={store}>
-  //       <ModuleUpdate
-  //         controlDisabledReason={"Can't touch this"}
-  //         moduleId="FAKEMODULE1234"
-  //         hasAvailableUpdate={false}
-  //       />
-  //     </Provider>
-  //   )
-  //   expect(treeAvailableCanControl).toMatchSnapshot()
-  //   expect(treeNotAvailableCanControl).toMatchSnapshot()
-  //   expect(treeAvailableNoControl).toMatchSnapshot()
-  //   expect(treeNotAvailableNoControl).toMatchSnapshot()
-  // })
-
   it('displays a Warning for invalid files', () => {
     const SPECS = [
       {
@@ -110,4 +65,9 @@ describe('ModuleUpdate', () => {
       }
     )
   })
+
+  it.todo('is available and can control')
+  it.todo('is available and cannot control')
+  it.todo('is not available and can control')
+  it.todo('is not available and cannot control')
 })