fixes on mark as seen and delete reseult features

Author: Anisha Swain

mock/overview.js

@@ -4,67 +4,67 @@ export const expirationLimit = 20;
 export const resultData = [
   {
     key: 1,
-    result: ['fio_2020.09.10T12.42.11', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -1', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-09-10T 11:52:34.552478',
     seen: false,
     description: 'More content can be added here',
   },
   {
     key: 2,
-    result: ['fio_2020.09.10T12.42.12', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -2', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-09-10T11:52:34.552478',
     seen: false,
     description: 'More content can be added here',
   },
   {
     key: 3,
-    result: ['fio_2020.09.10T12.42.13', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -3', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-09-10T11:52:34.552478',
     seen: false,
     description: 'More content can be added here',
   },
   {
     key: 4,
-    result: ['fio_2020.09.10T12.42.14', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -4', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-09-10T11:52:34.552478',
     seen: false,
     description: 'More content can be added here',
   },
   {
     key: 5,
-    result: ['fio_2020.09.10T12.42.11', 'virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -5', 'virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-11-13T11:52:34.552478',
     deletion: '2020-11-13T11:52:34.552478',
     seen: true,
     description: 'More content can be added here',
   },
   {
     key: 6,
-    result: ['fio_2020.09.10T12.42.12', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -6', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-11-20T11:52:34.552478',
     deletion: '2020-11-20T11:52:34.552478',
     seen: true,
     description: 'More content can be added here',
   },
   {
     key: 7,
-    result: ['fio_2020.09.10T12.42.13', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -7', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-11-15T11:52:34.552478',
     deletion: '2020-11-15T11:52:34.552478',
     seen: true,
     description: 'More content can be added here',
   },
   {
     key: 8,
-    result: ['fio_2020.09.10T12.42.14', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -8', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-11-25T11:52:34.552478',
     deletion: '2020-11-25T11:52:34.552478',
     seen: true,
     description: 'More content can be added here',
   },
   {
     key: 9,
-    result: ['fio_2020.09.10T12.42.14', ' virtual506.virt.lab.eng.bos.redhat.com'],
+    result: ['mock result -9', ' virtual506.virt.lab.eng.bos.redhat.com'],
     end: '2020-11-25T11:52:34.552478',
     deletion: '2020-11-25T11:52:34.552478',
     seen: true,

src/pages/Overview/index.js

@@ -57,15 +57,15 @@ class Overview extends React.Component {
     this.getSeperatedResults();
   }
 
-  getSeperatedResults = () => {
+  getSeperatedResults() {
     const { totalResultData } = this.state;
     const unlabledData = totalResultData.filter(x => x.seen === true);
     const newData = totalResultData.filter(x => x.seen !== true);
     this.setState({
       newData,
       unlabledData,
     });
-  };
+  }
 
   onCollapse = (event, rowKey, isOpen) => {
     const { rows } = this.state;
@@ -79,12 +79,6 @@ class Overview extends React.Component {
     console.log(e);
   };
 
-  handleTabClick = (event, tabIndex) => {
-    this.setState({
-      activeTabKey: tabIndex,
-    });
-  };
-
   showDrowpdown = id => {
     const dropdownElem = document.getElementById(id);
     if (dropdownElem.style.display === 'none') {
@@ -95,16 +89,36 @@ class Overview extends React.Component {
   };
 
   markSeen = row => {
-    console.log(row);
+    const { totalResultData } = this.state;
+    const objIndex = totalResultData.findIndex(x => x.key === row.key);
+    totalResultData[objIndex].seen = true;
+    this.setState(
+      {
+        totalResultData,
+      },
+      () => {
+        this.getSeperatedResults();
+      }
+    );
   };
 
   deleteResult = row => {
     const { totalResultData } = this.state;
     const updatedResult = totalResultData.filter(x => x.key !== row.key);
+    this.setState(
+      {
+        totalResultData: updatedResult,
+      },
+      () => {
+        this.getSeperatedResults();
+      }
+    );
+  };
+
+  handleTabClick = (event, tabIndex) => {
     this.setState({
-      totalResultData: updatedResult,
+      activeTabKey: tabIndex,
     });
-    this.getSeperatedResults();
   };
 
   render() {
@@ -154,7 +168,7 @@ class Overview extends React.Component {
                   <div className={styles.dropdownLink} onClick={() => this.markSeen(row)}>
                     Mark as seen
                   </div>
-                  <div className={styles.dropdownLink} href="#delete">
+                  <div className={styles.dropdownLink} onClick={() => this.deleteResult(row)}>
                     Delete
                   </div>
                 </div>

src/pages/PrivateRoute/index.test.js

@@ -1,7 +1,7 @@
 import React from 'react';
 import { shallow, configure } from 'enzyme';
 import Adapter from 'enzyme-adapter-react-16';
-import AuthLayout from '@/components/AuthLayout';
+import LoginModal from '@/components/LoginModal';
 import PrivateRoute from './index';
 
 const mockProps = {
@@ -20,7 +20,7 @@ describe('test Login page component', () => {
     expect(wrapper).toMatchSnapshot();
   });
 
-  it('render AuthLayout component', () => {
-    expect(wrapper.find(AuthLayout).length).toBe(1);
+  it('render LoginModal component', () => {
+    expect(wrapper.find(LoginModal).length).toBe(1);
   });
 });