fixed failing test issue

Author: nusanam

src/app/__tests__/mainReducer.test.jsx

@@ -94,7 +94,7 @@ describe('mainReducer testing', () => {
   });
 
   describe('empty', () => {
-    xit('should empty snapshots except the first one', () => {
+    it('should empty snapshots except the first one', () => {
       expect(mainReducer(state, emptySnapshots()).tabs[currentTab].sliderIndex).toEqual(0);
       expect(mainReducer(state, emptySnapshots()).tabs[currentTab].viewIndex).toEqual(-1);
       expect(mainReducer(state, emptySnapshots()).tabs[currentTab].playing).toEqual(false);

src/app/components/Chart.jsx

@@ -3,9 +3,9 @@
 /* eslint-disable no-use-before-define */
 /* eslint-disable react/no-string-refs */
 import React, { Component } from 'react';
-import PropTypes from 'prop-types';
+// import PropTypes from 'prop-types';
 import * as d3 from 'd3';
-import d3Tip from "d3-tip";
+import d3Tip from 'd3-tip';
 
 let root = {};
 class Chart extends Component {
@@ -36,15 +36,13 @@ class Chart extends Component {
 
   maked3Tree() {
     this.removed3Tree();
-    let width = 600;
-    // let height = 1060;
     const margin = {
       top: 0,
       right: 60,
-      bottom: 80,
+      bottom: 200,
       left: 120,
     };
-    // const width = 600 - margin.right - margin.left;
+    const width = 600 - margin.right - margin.left;
     const height = 700 - margin.top - margin.bottom;
 
     let chartContainer = d3.select(this.chartRef.current)
@@ -93,7 +91,7 @@ class Chart extends Component {
       });
 
     node.append("circle")
-      .attr("r", 12)
+      .attr("r", 10)
 
     //creating a d3.tip method where the html has a function that returns the data we passed into tip.show from line 120
     let tip = d3Tip()

src/app/reducers/mainReducer.js

@@ -70,7 +70,7 @@ export default (state, action) => produce(state, draft => {
       tabs[currentTab].playing = false;
       tabs[currentTab].snapshots.splice(1);
       // reset children in root node to reset graph
-      tabs[currentTab].hierarchy.children = [];
+      if (tabs[currentTab].hierarchy) tabs[currentTab].hierarchy.children = [];
       // reassigning pointer to the appropriate node to branch off of
       tabs[currentTab].currLocation = tabs[currentTab].hierarchy;
       // reset index

src/app/styles/components/d3graph.css

@@ -55,18 +55,17 @@ body {
   font-family: "Overpass Mono", monospace;
 }
 
-/* Creates a small triangle extender for the tooltip */
+/* Creates a small triangle extender for the tooltip
 .d3-tip:after {
   box-sizing: border-box;
   display: inline;
   font-size: 15px;
-  width: 100%;
   line-height: 1;
   color: #9cf4df;
   content: "\25BC";
   position: absolute;
   text-align: center;
-}
+} */
 
 /* Style northward tooltips specifically */
 .d3-tip.n:after {

src/extension/background.js

@@ -155,7 +155,7 @@ chrome.runtime.onMessage.addListener((request, sender) => {
       if (!persist) {
         tabsObj[tabId].snapshots.splice(1);
         // reset children in root node to reset graph
-        tabsObj[tabId].hierarchy.children = [];
+        if (tabsObj[tabId].hierarchy) tabsObj[tabId].hierarchy.children = [];
         // reassigning pointer to the appropriate node to branch off of
         tabsObj[tabId].currLocation = tabsObj[tabId].hierarchy;
         // reset index