Merge pull request #141 from open-source-labs/cleanup

codebase cleaned up

Author: nusanam

package-lock.json

@@ -62,7 +62,6 @@
   "dependencies": {
     "bower": "^1.8.8",
     "d3": "^4.13.0",
-    "d3-tip": "^0.9.1",
     "d3-zoom": "^1.8.3",
     "immer": "^3.2.0",
     "jest-runner": "^24.9.0",

package.json

@@ -1,6 +1,6 @@
 const acorn = require('acorn');
+// eslint-disable-next-line import/newline-after-import
 const jsx = require('acorn-jsx');
-
 const JSXParser = acorn.Parser.extend(jsx());
 
 // Helper function to grab the getters/setters from `elementType`

package/astParser.js

@@ -138,13 +138,5 @@ module.exports = (snap, mode) => {
     window.addEventListener('message', ({ data: { action } }) => {
       if (action === 'contentScriptStarted') sendSnapshot();
     });
-    // Testing sending back a function def to client
-    // function getNextImport(filePath) {
-    // return loadable(() => import(`${filePath}`))
-    // Got relative file path to return back to client code
-    // return (`myTestString${filePath}`);
-    // return getNextImport('./UseStateHook');
-    // return 'Testing outside';
-    // const OtherComponent = loadable(() => import('./OtherComponent'))
   };
 };

package/linkFiber.js

@@ -1,7 +1,7 @@
 /* eslint-disable no-console */
 /* eslint-disable no-param-reassign */
 
-// this is the current snapshot that is being sent to the snapshots array. it is an object
+// this is the current snapshot that is being sent to the snapshots array.
 class Tree {
   constructor(component, useStateInstead = false, name) {
     // special case when component is root

package/tree.js

@@ -1,3 +1,4 @@
+/* eslint-disable react/jsx-props-no-spreading */
 /* eslint-disable import/no-extraneous-dependencies */
 /* eslint-disable import/no-named-as-default-member */
 /* eslint-disable import/no-named-as-default */

src/app/__tests__/Chart.test.jsx

@@ -1,5 +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__/MainContainer.test.js

@@ -81,25 +81,4 @@ describe('testing the play-button', () => {
     wrapper = shallow(<TravelContainer snapshotsLength={2} />);
     expect(wrapper.find('.play-button').text()).toBe('Play');
   });
-});
-
-// describe('testing the playback speed', () => {
-//   test('if the playback dropdown states 0.5x the speed should be 0.5x', () => {
-//     const wrapper = shallow(<TravelContainer {...props} />);
-//     wrapper.find('Dropdown').simulate('change', { value: ['val'] });
-//     wrapper.find('select').simulate('change', { value: 'hello' });
-//     console.log('val', wrapper.find('Dropdown').simulate('select', { value: ['val'] }));
-//     expect(wrapper.find('Dropdown').text()).toBe('0.5x');
-//     expect(wrapper.find('select [selected]').val()).toEqual('key');
-//   });
-//   test('if the playback dropdown states 1x the speed should be 1x', () => {
-//     const wrapper = shallow(<TravelContainer {...dropdownProps} />);
-
-//     expect(wrapper.find('Dropdown').label).toBe('1.0x');
-//   });
-//   test('if the playback dropdown states 2x the speed should be 2x', () => {
-//     const wrapper = shallow(<TravelContainer {...dropdownProps} />);
-//     wrapper.find('Dropdown').simulate('click');
-//     expect(wrapper.find('Dropdown').label).toBe('2.0x');
-//   });
-// });
+});

src/app/__tests__/TravelContainer.test.js

@@ -1,12 +1,7 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-
 import { changeView, changeSlider } from '../actions/actions';
 
-// Launch Feature: Figure out changeView vs changeSlider
-// Should we make the btn bigger instead and keep the functionality?
-// div onclick event triggers the changeView method
-
 const Action = props => {
   const {
     selected, index, sliderIndex, dispatch,