Update tests to include check for changing state

URL stays the same, but state changes.
remix-run · Dec 8, 2015 · 379c0ad · 379c0ad
1 parent 0dda517
commit 379c0ad
Show file tree

Hide file tree

Showing 2 changed files with 68 additions and 2 deletions.
diff --git a/modules/__tests__/describePush.js b/modules/__tests__/describePush.js
@@ -94,6 +94,43 @@ function describePush(createHistory) {
       })
 
       it('becomes a REPLACE if path is unchanged', function (done) {
+        let steps = [
+          function (location) {
+            expect(location.pathname).toEqual('/')
+            expect(location.search).toEqual('')
+            expect(location.state).toEqual(null)
+            expect(location.action).toEqual(POP)
+
+            history.push({
+              pathname: '/home',
+              search: '?the=query',
+              state: { the: 'state' }
+            })
+          },
+          function (location) {
+            expect(location.pathname).toEqual('/home')
+            expect(location.search).toEqual('?the=query')
+            expect(location.state).toEqual({ the: 'state' })
+            expect(location.action).toEqual(PUSH)
+
+            history.push({
+              pathname: '/home',
+              search: '?the=query',
+              state: { the: 'state' }
+            })
+          },
+          function (location) {
+            expect(location.pathname).toEqual('/home')
+            expect(location.search).toEqual('?the=query')
+            expect(location.state).toEqual({ the: 'state' })
+            expect(location.action).toEqual(REPLACE)
+          }
+        ]
+
+        unlisten = history.listen(execSteps(steps, done))
+      })
+
+      it('stays PUSH if state is changed', function (done) {
         let steps = [
           function (location) {
             expect(location.pathname).toEqual('/')
@@ -123,7 +160,7 @@ function describePush(createHistory) {
             expect(location.pathname).toEqual('/home')
             expect(location.search).toEqual('?the=query')
             expect(location.state).toEqual({ different: 'state' })
-            expect(location.action).toEqual(REPLACE)
+            expect(location.action).toEqual(PUSH)
           }
         ]
 

diff --git a/modules/__tests__/describePushState.js b/modules/__tests__/describePushState.js
@@ -36,6 +36,35 @@ function describePushState(createHistory) {
     })
 
     it('becomes a REPLACE if path is unchanged', function (done) {
+      let steps = [
+        function (location) {
+          expect(location.pathname).toEqual('/')
+          expect(location.search).toEqual('')
+          expect(location.state).toEqual(null)
+          expect(location.action).toEqual(POP)
+
+          history.pushState({ the: 'state' }, '/home?the=query')
+        },
+        function (location) {
+          expect(location.pathname).toEqual('/home')
+          expect(location.search).toEqual('?the=query')
+          expect(location.state).toEqual({ the: 'state' })
+          expect(location.action).toEqual(PUSH)
+
+          history.pushState({ the: 'state' }, '/home?the=query')
+        },
+        function (location) {
+          expect(location.pathname).toEqual('/home')
+          expect(location.search).toEqual('?the=query')
+          expect(location.state).toEqual({ the: 'state' })
+          expect(location.action).toEqual(REPLACE)
+        }
+      ]
+
+      unlisten = history.listen(execSteps(steps, done))
+    })
+
+    it('stays PUSH if state is changed', function (done) {
       let steps = [
         function (location) {
           expect(location.pathname).toEqual('/')
@@ -57,7 +86,7 @@ function describePushState(createHistory) {
           expect(location.pathname).toEqual('/home')
           expect(location.search).toEqual('?the=query')
           expect(location.state).toEqual({ different: 'state' })
-          expect(location.action).toEqual(REPLACE)
+          expect(location.action).toEqual(PUSH)
         }
       ]