fixed eslint errors

Author: rydang

src/app/__tests__/ButtonsContainer.test.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-filename-extension */
 import { shallow, configure } from 'enzyme';
 import React from 'react';
 import Adapter from 'enzyme-adapter-react-16';

src/app/__tests__/StateContainter.test.js renamed to src/app/__tests__/StateContainer.test.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-filename-extension */
 import { mount, configure } from 'enzyme';
 import React from 'react';
 import { MemoryRouter, NavLink } from 'react-router-dom';

src/app/__tests__/TravelContainer.test.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-filename-extension */
 import { shallow, configure } from 'enzyme';
 import React from 'react';
 import Adapter from 'enzyme-adapter-react-16';

src/app/__tests__/actionContainer.test.js

@@ -1,3 +1,5 @@
+/* eslint-disable react/jsx-filename-extension */
+
 import { shallow, configure } from 'enzyme';
 import React from 'react';
 import Adapter from 'enzyme-adapter-react-16';

src/app/__tests__/dropdown.test.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-filename-extension */
 import React from 'react';
 import { configure, shallow } from 'enzyme';
 import Adapter from 'enzyme-adapter-react-16';

src/app/components/Chart.jsx

@@ -1,3 +1,7 @@
+/* eslint-disable no-underscore-dangle */
+/* eslint-disable no-param-reassign */
+/* eslint-disable no-use-before-define */
+/* eslint-disable react/no-string-refs */
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import '../styles/components/_d3Tree.scss';
@@ -60,7 +64,7 @@ class Chart extends Component {
       .attr('class', 'tooltip')
       .style('opacity', 1e-6)
       .on('mouseover', tipMouseover)
-      .on('mouseout', tipMouseout)
+      .on('mouseout', tipMouseout);
 
     root.x0 = height / 2;
     root.y0 = 0;
@@ -76,7 +80,10 @@ class Chart extends Component {
       // Update the nodes…
       const node = svg.selectAll('g.node')
         .data(nodes, (d) => {
-          if (!d.id) d.id = ++i;
+          if (!d.id) {
+            i += 1;
+            d.id = i;
+          }
           return d.id;
         });
 
@@ -175,7 +182,7 @@ class Chart extends Component {
       div.transition()
         .duration(300)
         .style('display', 'block')
-        .style('opacity', 1)
+        .style('opacity', 1);
     }
 
     function mouseout() {
@@ -205,14 +212,6 @@ class Chart extends Component {
         .style('top', `${d3.event.pageY}px`);
     }
 
-    function collapse(d) {
-      if (d.children) {
-        d._children = d.children;
-        d._children.forEach(collapse);
-        d.children = null;
-      }
-    }
-
     update(root);
     duration = 750;
 

src/app/components/Tree.jsx

@@ -12,6 +12,7 @@ const getItemString = (type, data) => {
   ) {
     return <span>{data.name}</span>;
   }
+  return null;
 };
 
 const Tree = (props) => {

src/app/containers/MainContainer.jsx

@@ -41,9 +41,8 @@ function MainContainer() {
       }
     });
 
-    // console log if the port with background script disconnects
-    port.onDisconnect.addListener((obj) => {
-      console.log('disconnected port', obj);
+    port.onDisconnect.addListener(() => {
+      // disconnecting
     });
 
     // assign port to state so it could be used by other components

src/app/index.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-filename-extension */
 import React from 'react';
 import ReactDOM from 'react-dom';
 import App from './components/App';

src/extension/background.js

@@ -98,7 +98,6 @@ chrome.runtime.onMessage.addListener((request, sender) => {
         // don't add anything to snapshot storage if mode is persisting for the initial snapshot
         if (!persist) tabsObj[tabId].snapshots.push(request.payload);
         if (bg) {
-          console.log('connecting');
           bg.postMessage({
             action: 'initialConnectSnapshots',
             payload: tabsObj,