test for hook merging

Author: yyx990803

test/unit/lib/pipeline-test-util.js

@@ -24,6 +24,9 @@ exports.test = function (configs, cb) {
     var config = configs[route]
     Object.keys(config).forEach(function (hook) {
       var fn = config[hook]
+      if (Array.isArray(fn) || hook === 'mixins') {
+        return
+      }
       if (fn.length) {
         config[hook] = function (transition) {
           var event = route + '.' + hook
@@ -84,12 +87,15 @@ exports.test = function (configs, cb) {
     '/data/:msg': {
       component: {
         route: configs.data,
+        mixins: configs.data && configs.data.mixins,
         template:
           '<span v-if="$loadingRouteData">loading...</span>' +
-          '<span v-if="!$loadingRouteData">{{msg}}</span>',
+          '<span v-if="!$loadingRouteData">{{msg}}{{otherMsg}}{{thirdMsg}}</span>',
         data: function () {
           return {
-            msg: 'default'
+            msg: 'default',
+            otherMsg: '',
+            thirdMsg: ''
           }
         }
       }

test/unit/specs/pipeline/activate.js

@@ -168,4 +168,34 @@ describe('activate', function () {
       done()
     })
   })
+
+  it('multiple', function (done) {
+    var calls = []
+    test({
+      a: {
+        activate: [
+          function (transition) {
+            calls.push(1)
+            setTimeout(function () {
+              transition.next()
+            }, wait)
+          },
+          function () {
+            calls.push(2)
+            return new Promise(function (resolve) {
+              setTimeout(resolve, wait)
+            })
+          }
+        ]
+      }
+    }, function (router, _, emitter) {
+      router.go('/a')
+      expect(router.app.$el.textContent).toBe('')
+      setTimeout(function () {
+        assertCalls(calls, [1, 2])
+        expect(router.app.$el.textContent).toBe('A ')
+        done()
+      }, wait * 3)
+    })
+  })
 })

test/unit/specs/pipeline/data.js

@@ -196,4 +196,49 @@ describe('data', function () {
       }, wait * 2)
     })
   })
+
+  it('multiple data hooks', function (done) {
+    test({
+      data: {
+        data: [
+          function (transition) {
+            return {
+              msg: new Promise(function (resolve) {
+                setTimeout(function () {
+                  resolve(transition.to.params.msg)
+                }, wait)
+              })
+            }
+          },
+          function (transition) {
+            return new Promise(function (resolve) {
+              setTimeout(function () {
+                resolve({
+                  otherMsg: ' ' + transition.to.params.msg + '2'
+                })
+              }, wait)
+            })
+          }
+        ],
+        mixins: [
+          {
+            route: {
+              data: function (transition) {
+                transition.next({
+                  thirdMsg: ' ' + transition.to.params.msg + '3'
+                })
+              }
+            }
+          }
+        ]
+      }
+    }, function (router, calls, emitter) {
+      router.go('/data/hello')
+      expect(router.app.$el.textContent).toBe('loading...')
+      setTimeout(function () {
+        expect(router.app.$el.textContent).toBe('hello hello2 hello3')
+        done()
+      }, wait * 3)
+    })
+  })
 })

test/unit/specs/pipeline/deactivate.js

@@ -154,4 +154,36 @@ describe('deactivate', function () {
       }, wait * 2)
     })
   })
+
+  it('multiple', function (done) {
+    var calls = []
+    test({
+      a: {
+        deactivate: [
+          function (transition) {
+            calls.push(1)
+            setTimeout(function () {
+              transition.next()
+            }, wait)
+          },
+          function () {
+            calls.push(2)
+            return new Promise(function (resolve) {
+              setTimeout(resolve, wait)
+            })
+          }
+        ]
+      }
+    }, function (router, _, emitter) {
+      router.go('/a')
+      expect(router.app.$el.textContent).toBe('A ')
+      router.go('/b')
+      expect(router.app.$el.textContent).toBe('A ')
+      setTimeout(function () {
+        assertCalls(calls, [1, 2])
+        expect(router.app.$el.textContent).toBe('')
+        done()
+      }, wait * 3)
+    })
+  })
 })