fix(test): fix test condition so that trip plans on load URL with search

Author: landonreed

lib/actions/api.js

@@ -6,10 +6,11 @@ import moment from 'moment'
 import haversine from 'haversine'
 
 import { rememberPlace } from './map'
-import { queryIsValid } from '../util/state'
-import queryParams from '../util/query-params'
-import { getTripOptionsFromQuery, getUrlParams } from '../util/query'
 import { hasCar } from '../util/itinerary'
+import { getTripOptionsFromQuery, getUrlParams } from '../util/query'
+import queryParams from '../util/query-params'
+import { queryIsValid } from '../util/state'
+import { randId } from '../util/storage'
 if (typeof (fetch) === 'undefined') require('isomorphic-fetch')
 
 // Generic API actions
@@ -22,14 +23,6 @@ export const toggleTracking = createAction('TOGGLE_TRACKING')
 export const rememberSearch = createAction('REMEMBER_SEARCH')
 export const forgetSearch = createAction('FORGET_SEARCH')
 
-/**
- * Generate a random ID. This might not quite be a UUID, but it serves our
- * purposes for now.
- */
-function randId () {
-  return Math.random().toString(36).substr(2, 9)
-}
-
 function formatRecentPlace (place) {
   return {
     ...place,

lib/actions/form.js

@@ -9,7 +9,7 @@ import {
   getUrlParams,
   planParamsToQuery
 } from '../util/query'
-import { getItem } from '../util/storage'
+import { getItem, randId } from '../util/storage'
 import { queryIsValid } from '../util/state'
 import { isMobile } from '../util/ui'
 import {
@@ -65,8 +65,7 @@ export function parseUrlQueryString (params = getUrlParams()) {
     Object.keys(params).forEach(key => {
       if (!key.startsWith('ui_')) planParams[key] = params[key]
     })
-    const searchId = params.ui_activeSearch
-    console.log(params, searchId)
+    const searchId = params.ui_activeSearch || randId()
     // Convert strings to numbers/objects and dispatch
     dispatch(setQueryParam(planParamsToQuery(planParams), searchId))
   }

lib/util/storage.js

@@ -32,3 +32,11 @@ export function getItem (key, notFoundValue = null) {
 export function removeItem (key) {
   window.localStorage.removeItem(`${STORAGE_PREFIX}.${key}`)
 }
+
+/**
+ * Generate a random ID. This might not quite be a UUID, but it serves our
+ * purposes for now.
+ */
+export function randId () {
+  return Math.random().toString(36).substr(2, 9)
+}