fix unit tests in FF/IE

Author: yyx990803

package.json

@@ -8,7 +8,7 @@
     "lint": "eslint src build test/e2e test/unit/specs",
     "test": "npm run lint && ./node_modules/karma/bin/karma start build/karma.config.js",
     "build": "webpack --config build/webpack.build.dev.config.js && webpack --config build/webpack.build.min.config.js",
-    "serve": "webpack-dev-server --quiet --hot --config example/webpack.config.js --content-base example --history-api-fallback",
+    "serve": "webpack-dev-server --quiet --hot --config example/webpack.config.js --content-base example --history-api-fallback --host 0.0.0.0",
     "watch-test": "webpack --watch test/unit/specs/index.js test/unit/test.build.js",
     "e2e-local": "nightwatch -c build/nightwatch.local.json -e chrome,firefox",
     "e2e-sauce": "nightwatch -c build/nightwatch.sauce.json -e chrome,firefox,ie10,ie11",

test/unit/specs/core.js

@@ -473,10 +473,12 @@ describe('Core', function () {
       router.go('/a')
       nextTick(function () {
         window.addEventListener('popstate', function onPop () {
-          expect(window.scrollTo).toHaveBeenCalledWith(x, y)
-          window.removeEventListener('popstate', onPop)
-          router.stop()
-          done()
+          nextTick(function () {
+            expect(window.scrollTo).toHaveBeenCalledWith(x, y)
+            window.removeEventListener('popstate', onPop)
+            router.stop()
+            done()
+          })
         })
         history.back()
       })

test/unit/specs/index.js

@@ -1,5 +1,9 @@
-window.isIE9 = navigator.userAgent.toLowerCase().indexOf('msie 9.0') > 0
-window.wait = 16
+var UA = navigator.userAgent.toLowerCase()
+window.isIE9 = UA.indexOf('msie 9.0') > 0
+window.isIE = UA.indexOf('trident') > 0
+
+// IE has some shaky timer precision issues when using the Promise polyfill...
+window.wait = isIE ? 100 : 30
 
 require('es6-promise').polyfill()
 

test/unit/specs/pipeline/activate.js

@@ -97,7 +97,7 @@ describe('activate', function () {
         assertCalls(calls, ['a.activate'])
         expect(router.app.$el.textContent).toBe('A ')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -121,7 +121,7 @@ describe('activate', function () {
         expect(router.app.$el.textContent).toBe('')
         expect(router.history.currentPath).toBe('/')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 })

test/unit/specs/pipeline/can-activate.js

@@ -97,7 +97,7 @@ describe('canActivate', function () {
         assertCalls(calls, ['a.canActivate'])
         expect(router.app.$el.textContent).toBe('A ')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -123,7 +123,7 @@ describe('canActivate', function () {
         expect(router.app.$el.textContent).toBe('')
         expect(router.history.currentPath).toBe('/')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -147,7 +147,7 @@ describe('canActivate', function () {
         expect(router.app.$el.textContent).toBe('')
         expect(router.history.currentPath).toBe('/')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 })

test/unit/specs/pipeline/can-deactivate.js

@@ -108,7 +108,7 @@ describe('canDeactivate', function () {
         expect(router.app.$el.textContent).toBe('C ')
         expect(router.history.currentPath).toBe('/c')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -136,7 +136,7 @@ describe('canDeactivate', function () {
         expect(router.app.$el.textContent).toBe('A ')
         expect(router.history.currentPath).toBe('/a')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -162,7 +162,7 @@ describe('canDeactivate', function () {
         expect(router.app.$el.textContent).toBe('A ')
         expect(router.history.currentPath).toBe('/a')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 })

test/unit/specs/pipeline/data.js

@@ -22,7 +22,7 @@ describe('data', function () {
       setTimeout(function () {
         expect(router.app.$el.textContent).toBe('hello')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -50,9 +50,9 @@ describe('data', function () {
           setTimeout(function () {
             expect(router.app.$el.textContent).toBe('reload')
             done()
-          }, wait)
+          }, wait * 2)
         })
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -81,9 +81,9 @@ describe('data', function () {
           setTimeout(function () {
             expect(router.app.$el.textContent).toBe('reload')
             done()
-          }, wait)
+          }, wait * 2)
         })
-      }, wait)
+      }, wait * 2)
     })
   })
 

test/unit/specs/pipeline/deactivate.js

@@ -107,7 +107,7 @@ describe('deactivate', function () {
         assertCalls(calls, ['a.deactivate'])
         expect(router.app.$el.textContent).toBe('')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 
@@ -134,7 +134,7 @@ describe('deactivate', function () {
         expect(router.app.$el.textContent).toBe('A ')
         expect(router.history.currentPath).toBe('/a')
         done()
-      }, wait)
+      }, wait * 2)
     })
   })
 })