From c96c4b8fd2d355f11e9a59a3152b65e5f7e840d8 Mon Sep 17 00:00:00 2001
From: Sharyn <sharyn@nonacreative.com>
Date: Fri, 6 Jul 2018 14:36:22 +0200
Subject: [PATCH] lint fixes

---
 components/browseArchives/index.js            | 32 ++++++++++++-------
 components/interviews/interviewsList/index.js | 25 +++++++--------
 .../interviews/relatedInterviewsList/index.js | 25 +++++++--------
 .../interviews/singleInterview/index.js       |  6 ++--
 components/searchResults/index.js             |  2 +-
 5 files changed, 50 insertions(+), 40 deletions(-)

diff --git a/components/browseArchives/index.js b/components/browseArchives/index.js
index 8c6e433..3b4f7ad 100644
--- a/components/browseArchives/index.js
+++ b/components/browseArchives/index.js
@@ -1,5 +1,5 @@
 import React from 'react';
-import RelatedInterviewsList from '../interviews/relatedInterviewsList'
+import RelatedInterviewsList from '../interviews/relatedInterviewsList';
 import InterviewsList from '../interviews/interviewsList';
 import SingleInterview from '../interviews/singleInterview';
 import TopicsList from '../topicsList';
@@ -31,10 +31,16 @@ class Search extends React.Component {
     });
 
     if (event.target.value.length === 0) {
-      this.setState({isSearchActive: false});
+      this.setState({ isSearchActive: false });
     }
   }
 
+  getSelectedInterview() {
+    const { activeSingleInterviewId } = this.state;
+    const selectedInterview = Data.find(item => item.id === activeSingleInterviewId);
+    return selectedInterview;
+  }
+
   toggleInterviewsListModal() {
     const { isInterviewsListModalOpen } = this.state;
     this.setState({
@@ -52,14 +58,13 @@ class Search extends React.Component {
     });
   }
 
-  getSelectedInterview() {
-    const { activeSingleInterviewId } = this.state;
-    const selectedInterview = Data.find( item => item.id === activeSingleInterviewId);
-    return selectedInterview;
-  }
-
   render() {
-    const { isSingleInterviewModalOpen, isInterviewsListModalOpen, isSearchActive, activeSingleInterviewId } = this.state;
+    const {
+      isSingleInterviewModalOpen,
+      isInterviewsListModalOpen,
+      isSearchActive,
+      activeSingleInterviewId,
+    } = this.state;
 
     return (
       <div className="browse-wrap">
@@ -77,11 +82,16 @@ class Search extends React.Component {
         </div>
         <div className="browse-content-wrap container">
           <div className="browse-content-left">
-            {isSearchActive && <RelatedInterviewsList data={Data} toggleSingleInterview={this.toggleSingleInterview} />}
+            {isSearchActive &&
+              (<RelatedInterviewsList
+                data={Data}
+                toggleSingleInterview={this.toggleSingleInterview}
+              />)
+            }
           </div>
           <div className="browse-content-right">
             {isSearchActive ? <SearchResults /> :
-             (
+              (
                 <React.Fragment>
                   <InterviewsList
                     data={Data}
diff --git a/components/interviews/interviewsList/index.js b/components/interviews/interviewsList/index.js
index 990e70b..7be36c7 100644
--- a/components/interviews/interviewsList/index.js
+++ b/components/interviews/interviewsList/index.js
@@ -17,17 +17,16 @@ const InterviewsList = props => (
       <div className="interviews-list">
         <h4>Interviews ({ props.data.length })</h4>
         <ul>
-          {
-            props.data.map(interview =>
-              (<li
-                id={interview.id}
-                key={interview.id}
-                role="button"
-                tabIndex="0"
-                onClick={props.toggleSingleInterview}
-              >
-                { interview.name }
-              </li>))
+          { props.data.map(interview => (
+            // eslint-disable-next-line
+            <li
+              id={interview.id}
+              key={interview.id}
+              onClick={props.toggleSingleInterview}
+            >
+              { interview.name }
+            </li>
+            ))
           }
         </ul>
       </div>
@@ -37,9 +36,9 @@ const InterviewsList = props => (
 
 InterviewsList.propTypes = {
   isInterviewsListModalOpen: PropTypes.bool.isRequired,
-  data: PropTypes.array.isRequired,
+  data: PropTypes.arrayOf(PropTypes.node).isRequired,
   toggleSingleInterview: PropTypes.func.isRequired,
   toggleInterviewsListModal: PropTypes.func.isRequired,
-}
+};
 
 export default InterviewsList;
diff --git a/components/interviews/relatedInterviewsList/index.js b/components/interviews/relatedInterviewsList/index.js
index c28c470..0696bf4 100644
--- a/components/interviews/relatedInterviewsList/index.js
+++ b/components/interviews/relatedInterviewsList/index.js
@@ -6,25 +6,24 @@ const RelatedInterviewsList = props => (
   <div className="related-interviews-list">
     <h4>Related <br />Interviews</h4>
     <ul>
-      {
-        props.data.map(interview =>
-          (<li
-            id={interview.id}
-            key={interview.id}
-            role="button"
-            tabIndex="0"
-            onClick={props.toggleSingleInterview}
-          >
-            { interview.name }
-          </li>))
+      { props.data.map(interview => (
+        // eslint-disable-next-line
+        <li
+          id={interview.id}
+          key={interview.id}
+          onClick={props.toggleSingleInterview}
+        >
+          { interview.name }
+        </li>
+        ))
       }
     </ul>
   </div>
 );
 
 RelatedInterviewsList.propTypes = {
-  data: PropTypes.array.isRequired,
+  data: PropTypes.arrayOf(PropTypes.node).isRequired,
   toggleSingleInterview: PropTypes.func.isRequired,
-}
+};
 
 export default RelatedInterviewsList;
diff --git a/components/interviews/singleInterview/index.js b/components/interviews/singleInterview/index.js
index cf1d023..07bfa49 100644
--- a/components/interviews/singleInterview/index.js
+++ b/components/interviews/singleInterview/index.js
@@ -6,7 +6,7 @@ import './style.scss';
 
 const SingleInterview = props => (
   <Modal
-    isModalOpen={true}
+    isModalOpen
     closeModal={props.toggleSingleInterview}
   >
     <div className="single-interview">
@@ -18,7 +18,9 @@ const SingleInterview = props => (
 
 SingleInterview.propTypes = {
   activeSingleInterviewId: PropTypes.number.isRequired,
-  selectedInterview: PropTypes.shape({}).isRequired,
+  selectedInterview: PropTypes.shape({
+    content: PropTypes.string.isRequired,
+  }).isRequired,
   toggleSingleInterview: PropTypes.func.isRequired,
 };
 
diff --git a/components/searchResults/index.js b/components/searchResults/index.js
index ea0f19c..4f4e8fa 100644
--- a/components/searchResults/index.js
+++ b/components/searchResults/index.js
@@ -1,7 +1,7 @@
 import React from 'react';
 import './style.scss';
 
-const SearchResults = props => (
+const SearchResults = () => (
   <div className="search-results">
     <p>Search results go here</p>
   </div>