Fix repeated test runs

src/linodes/reducers/index.js

@@ -8,16 +8,14 @@ import {
   TOGGLE_SELECTED
 } from '../actions/index';
 
-const view = getStorage("linodes/view") || "grid";
-
-const default_state = {
-  view,
-  selected: { } // set
-};
-
 const array_to_set = arr => arr.reduce((s, v) => ({ ...s, [v]: true }), { });
 
-export function index(state=default_state, action) {
+export function index(state=null, action) {
+  if (state === null) {
+    const view = getStorage("linodes/view") || "grid";
+    state = { view, selected: { } };
+  }
+
   switch (action.type) {
   case CHANGE_VIEW:
     const { view } = action;

src/reducers/authentication.js

@@ -1,23 +1,23 @@
 import { CALLBACK, SET_TOKEN } from '../actions/authentication';
 import { getStorage, setStorage } from '~/storage';
 
-const token = getStorage("authentication/oauth-token");
-
-const default_state = { token };
-
-export default function authentication(state = default_state, action) {
-    switch (action.type) {
-    case SET_TOKEN:
-        let newState = {
-            ...state,
-            scopes: action.scopes,
-            username: action.username,
-            email: action.email,
-            token: action.token
-        };
-        setStorage("authentication/oauth-token", action.token);
-        return newState;
-    default:
-        return state;
-    }
+export default function authentication(state = null, action) {
+  if (state === null) {
+    const token = getStorage("authentication/oauth-token");
+    state = { token };
+  }
+  switch (action.type) {
+  case SET_TOKEN:
+      let newState = {
+          ...state,
+          scopes: action.scopes,
+          username: action.username,
+          email: action.email,
+          token: action.token
+      };
+      setStorage("authentication/oauth-token", action.token);
+      return newState;
+  default:
+      return state;
+  }
 }

test/reducers/authentication.spec.js

@@ -5,12 +5,14 @@ import * as actions from '../../src/actions/authentication';
 
 describe("authentication reducer", () => {
   it('should handle initial state', () => {
+    window.localStorage.clear();
     expect(
       authentication(undefined, {})
     ).to.be.eql({ token: null });
   });
 
   it('should handle SET_TOKEN', () => {
+    window.localStorage.clear();
     const state = authentication(undefined, {});
     const auth = { token: null };
 
@@ -33,6 +35,7 @@ describe("authentication reducer", () => {
   });
 
   it('should handle anything else', () => {
+    window.localStorage.clear();
     const state = authentication(undefined, {});
     const auth = { token: null };
 

test/setup.js

@@ -1 +1,3 @@
+window.localStorage.clear();
+
 import '../src/index';