Removed old code and fixed tests

README.md

@@ -37,6 +37,24 @@ npx jest --watch
 
 
 
+## Code Style
+
+This project uses eslint and prettier for style consistency. 
+
+Run the linter:
+
+```
+npm run lint
+```
+
+Run the code formatter:
+
+```
+npm run format
+```
+
+
+
 ## Release
 
 ```

package.json

@@ -8,7 +8,8 @@
   "author": "Looky Labs, Inc.",
   "scripts": {
     "start": "npm-run-all -p electron watch:*",
-    "test": "jest src/js",
+    "test": "jest --roots src/js",
+    "pretest": "npm run lint",
     "electron": "electron .",
     "release": "electron-packager . Looky --out releases --overwrite --icon dist/static/looky.smaller.icns",
     "prerelease": "npm run build",
@@ -20,7 +21,9 @@
     "watch:js": "npm run build:js -- --watch",
     "watch:css": "npm run build:css -- --watch",
     "watch:static": "npm run build:static -- --watch",
-    "watch:reload": "livereload dist"
+    "watch:reload": "livereload dist",
+    "lint": "eslint src",
+    "format": "prettier"
   },
   "jest": {
     "notify": true,

src/js/.eslintrc.yml

@@ -1,11 +1,11 @@
 parserOptions:
-  ecmaVersion: 6
+  ecmaVersion: 2018
   ecmaFeatures:
     jsx: true
-    experimentalObjectRestSpread: true
   sourceType: "module"
 
 env:
+  node: true
   browser: true
   jest: true
   es6: true

src/js/actions/boomd.js

@@ -21,7 +21,7 @@ export const connectBoomd = () => (dispatch, getState, api) => {
         dispatch(setBoomdError("Incorrect user and pass combination."))
       else dispatch(connectedBoomd())
     })
-    .catch(res => {
+    .catch(_res => {
       dispatch(setBoomdError("No server running at that host and port."))
     })
 }

src/js/actions/fetchMainSearch.js

@@ -1,6 +1,6 @@
 import * as actions from "."
 import * as selectors from "../selectors"
-import * as outMessages from "../treehouse/outMessages"
+import * as outMessages from "../boom/outMessages"
 import eventsReceiver from "../receivers/eventsReceiver"
 import countByTimeReceiver from "../receivers/countByTimeReceiver"
 import analyticsReceiver from "../receivers/analyticsReceiver"

src/js/actions/fetchNextPage.js

@@ -1,6 +1,6 @@
 import * as actions from "."
 import * as selectors from "../selectors/index"
-import outMessages from "../treehouse/outMessages"
+import outMessages from "../boom/outMessages"
 import eventsReceiver from "../receivers/eventsReceiver"
 
 export function fetchMainSearchPage() {

src/js/actions/logDetails.js

@@ -1,5 +1,5 @@
 import * as actions from "."
-import * as outMessages from "../treehouse/outMessages"
+import * as outMessages from "../boom/outMessages"
 import {getCurrentSpace} from "../reducers/spaces"
 import * as selectors from "../selectors"
 

src/js/actions/mainSearch.js

@@ -1,5 +1,5 @@
 import {getCurrentSpace} from "../reducers/spaces"
-import * as outMessages from "../treehouse/outMessages"
+import * as outMessages from "../boom/outMessages"
 import uniq from "lodash/uniq"
 
 export function requestMainSearch({saveToHistory, query}) {

src/js/actions/spaces.js

@@ -1,4 +1,4 @@
-import * as outMessages from "../treehouse/outMessages"
+import * as outMessages from "../boom/outMessages"
 
 export function fetchSpaceInfo(name) {
   return (dispatch, getState, api) => {

src/js/reducers/index.test.js

@@ -3,11 +3,12 @@ import reducer from "."
 test("NEW_BRO_SCHEMA", () => {
   const newState = reducer(undefined, {
     type: "NEW_BRO_SCHEMA",
+    spaceName: "default",
     id: "1",
     descriptor: ["...schema stuff"]
   })
 
   expect(newState.broSchemas).toEqual({
-    1: ["...schema stuff"]
+    "default.1": ["...schema stuff"]
   })
 })

src/js/reducers/searchBar.js

@@ -3,7 +3,6 @@ import isNumber from "lodash/isNumber"
 import trim from "lodash/trim"
 import {changeProgramTimeWindow} from "../changeProgramTimeWindow"
 import {createSelector} from "reselect"
-import compact from "lodash/compact"
 
 export const initialState = {
   current: "",