fix(abstract): call afterHooks with go

Fix #3250

Author: posva

src/history/abstract.js

@@ -46,8 +46,12 @@ export class AbstractHistory extends History {
     this.confirmTransition(
       route,
       () => {
+        const prev = this.current
         this.index = targetIndex
         this.updateRoute(route)
+        this.router.afterHooks.forEach(hook => {
+          hook && hook(route, prev)
+        })
       },
       err => {
         if (isNavigationFailure(err, NavigationFailureType.duplicated)) {

test/unit/specs/abstract-history.spec.js

@@ -0,0 +1,42 @@
+import Vue from 'vue'
+import VueRouter from '../../../src/index'
+
+Vue.use(VueRouter)
+
+const delay = t => new Promise(resolve => setTimeout(resolve, t))
+
+describe('abstract history', () => {
+  it('run afterEach after initial navigation', done => {
+    const router = new VueRouter({ mode: 'abstract' })
+    const afterEach = jasmine.createSpy('afterEach')
+    const onReady = jasmine.createSpy('ready')
+    const onError = jasmine.createSpy('error')
+    router.afterEach(afterEach)
+    router.onReady(onReady, onError)
+
+    router.push('/').finally(() => {
+      expect(onReady).toHaveBeenCalled()
+      expect(onError).not.toHaveBeenCalled()
+      expect(afterEach).toHaveBeenCalled()
+      done()
+    })
+  })
+
+  it('run afterEach after router.go', done => {
+    const router = new VueRouter({ mode: 'abstract' })
+    const afterEach = jasmine.createSpy('afterEach')
+
+    router
+      .push('/')
+      .then(() => router.push('/foo'))
+      .then(() => {
+        router.afterEach(afterEach)
+        router.go(-1)
+        return delay(30)
+      })
+      .finally(() => {
+        expect(afterEach).toHaveBeenCalled()
+        done()
+      })
+  })
+})