Merge pull request #9468 from brave/state-paths

don’t convert paths to immutable
brave · Jun 15, 2017 · 02a7878 · 02a7878
1 parent c9869a2
commit 02a7878
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/app/common/state/tabState.js b/app/common/state/tabState.js
@@ -280,7 +280,7 @@ const tabState = {
     if (index === tabState.TAB_ID_NONE) {
       return null
     }
-    return makeImmutable(['tabs', index])
+    return ['tabs', index]
   },
 
   getByTabId: (state, tabId) => {
@@ -534,7 +534,7 @@ const tabState = {
       if (path == null) {
         return null
       }
-      return path.push('frame')
+      return path.concat(['frame'])
     } else {
       return frameState.getPathByTabId(state, tabId)
     }

diff --git a/test/unit/app/common/state/tabStateTest.js b/test/unit/app/common/state/tabStateTest.js
@@ -195,6 +195,15 @@ const shouldValidateAction = function (cb) {
 }
 
 describe('tabState unit tests', function () {
+  describe('getPathByTabId', function () {
+    it('returns null if tab is not found', function () {
+      assert.equal(tabState.getPathByTabId(twoTabsAppState, 333), null)
+    })
+    it('returns path if index found (as mutable array)', function () {
+      assert.deepEqual(tabState.getPathByTabId(twoTabsAppState, 1), ['tabs', 0])
+    })
+  })
+
   describe('getByTabId', function () {
     before(function () {
       this.appState = twoTabsAppState