refactor: typos and refactor in tests

diff --git a/__tests__/errors.spec.ts b/__tests__/errors.spec.ts
index 1bfea05f325d4f3481fe24b61045af05c94e5b86..92e115b2428ee839e47fd26c43307f9563b15bab 100644 (file)
--- a/__tests__/errors.spec.ts
+++ b/__tests__/errors.spec.ts
@@ -54,7 +54,7 @@ describe('Errors', () => {
     )
   })
 
-  it('triggers erros caused by new navigations of a next(redirect) trigered by history', async () => {
+  it('triggers errors caused by new navigations of a next(redirect) triggered by history', async () => {
     const { router, history } = createRouter()
     await router.push('/p/0')
     await router.push('/p/other')

diff --git a/__tests__/guards/component-beforeRouteEnter.spec.ts b/__tests__/guards/component-beforeRouteEnter.spec.ts
index f9002fe60f6c2a56861ccef35c5102db2e812275..9513c3ac53ed2856d4a488e41f2645b44868ef62 100644 (file)
--- a/__tests__/guards/component-beforeRouteEnter.spec.ts
+++ b/__tests__/guards/component-beforeRouteEnter.spec.ts
@@ -1,17 +1,6 @@
-import { RouterOptions, createRouter as newRouter } from '../../src/router'
 import fakePromise from 'faked-promise'
-import { createDom, noGuard } from '../utils'
+import { createDom, noGuard, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw, NavigationGuard } from '../../src/types'
-import { createWebHistory } from '../../src'
-
-function createRouter(
-  options: Partial<RouterOptions> & { routes: RouteRecordRaw[] }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
 
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }

diff --git a/__tests__/guards/component-beforeRouteLeave.spec.ts b/__tests__/guards/component-beforeRouteLeave.spec.ts
index be1e3ea24dd3b14af3c1e7334a696db6aede59cd..7e22e7273f785710611d95494df63706a87455dd 100644 (file)
--- a/__tests__/guards/component-beforeRouteLeave.spec.ts
+++ b/__tests__/guards/component-beforeRouteLeave.spec.ts
@@ -1,17 +1,5 @@
-import { RouterOptions, createRouter as newRouter } from '../../src/router'
-import { createDom, noGuard } from '../utils'
+import { createDom, noGuard, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw } from '../../src/types'
-import { createWebHistory } from '../../src'
-
-// TODO: refactor in utils
-function createRouter(
-  options: Partial<RouterOptions> & { routes: RouteRecordRaw[] }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
 
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }

diff --git a/__tests__/guards/component-beforeRouteUpdate.spec.ts b/__tests__/guards/component-beforeRouteUpdate.spec.ts
index 517d70fe3e233580263ccae4030cb9a8684f80e8..4c6a724a5750ad9ace38eb9234d89544cfb4fd11 100644 (file)
--- a/__tests__/guards/component-beforeRouteUpdate.spec.ts
+++ b/__tests__/guards/component-beforeRouteUpdate.spec.ts
@@ -1,19 +1,7 @@
 import fakePromise from 'faked-promise'
-import { createDom, noGuard } from '../utils'
-import { createRouter as newRouter, createWebHistory } from '../../src'
+import { createDom, noGuard, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw } from '../../src/types'
 
-function createRouter(
-  options: Partial<import('../../src/router').RouterOptions> & {
-    routes: import('../../src/types').RouteRecordRaw[]
-  }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
-
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }
 

diff --git a/__tests__/guards/global-after.spec.ts b/__tests__/guards/global-after.spec.ts
index f2b96db1fc1dae32db5e8e49fd6f068e13651758..ae458c0774c31575ef9f406b869c866402c096c3 100644 (file)
--- a/__tests__/guards/global-after.spec.ts
+++ b/__tests__/guards/global-after.spec.ts
@@ -1,18 +1,6 @@
-import { createDom } from '../utils'
-import { createWebHistory, createRouter as newRouter } from '../../src'
+import { createDom, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw } from 'src/types'
 
-function createRouter(
-  options: Partial<import('../../src/router').RouterOptions> & {
-    routes: import('../../src/types').RouteRecordRaw[]
-  }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
-
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }
 const Nested = { template: `<div>Nested<router-view/></div>` }

diff --git a/__tests__/guards/global-beforeEach.spec.ts b/__tests__/guards/global-beforeEach.spec.ts
index 797333cbbc4f22886ec6c529c16d8e45f3b7a15b..4cc45e0894712b649503095535c40d10470b5cc8 100644 (file)
--- a/__tests__/guards/global-beforeEach.spec.ts
+++ b/__tests__/guards/global-beforeEach.spec.ts
@@ -1,17 +1,6 @@
-import { RouterOptions } from '../../src/router'
 import fakePromise from 'faked-promise'
-import { createDom, tick, noGuard } from '../utils'
+import { createDom, tick, noGuard, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw, RouteLocationRaw } from '../../src/types'
-import { createWebHistory, createRouter as newRouter } from '../../src'
-
-function createRouter(
-  options: Partial<RouterOptions> & { routes: RouteRecordRaw[] }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
 
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }

diff --git a/__tests__/guards/route-beforeEnter.spec.ts b/__tests__/guards/route-beforeEnter.spec.ts
index 825549d012bdb18a35018cb6faf8b15a6ccfb4aa..913b0d041f5e1055b3a7fb6cff41c2d0daeecc19 100644 (file)
--- a/__tests__/guards/route-beforeEnter.spec.ts
+++ b/__tests__/guards/route-beforeEnter.spec.ts
@@ -1,17 +1,6 @@
-import { RouterOptions, createRouter as newRouter } from '../../src/router'
 import fakePromise from 'faked-promise'
-import { createDom, noGuard, tick } from '../utils'
+import { createDom, noGuard, tick, newRouter as createRouter } from '../utils'
 import { RouteRecordRaw } from '../../src/types'
-import { createWebHistory } from '../../src'
-
-function createRouter(
-  options: Partial<RouterOptions> & { routes: RouteRecordRaw[] }
-) {
-  return newRouter({
-    history: createWebHistory(),
-    ...options,
-  })
-}
 
 const Home = { template: `<div>Home</div>` }
 const Foo = { template: `<div>Foo</div>` }

diff --git a/__tests__/history/memory.spec.ts b/__tests__/history/memory.spec.ts
index 72aaf23ac4938b733374ae8eee65cd1d70d9dcb0..aee86f581c7cd456ea9991c46970f222a26c304d 100644 (file)
--- a/__tests__/history/memory.spec.ts
+++ b/__tests__/history/memory.spec.ts
@@ -115,7 +115,6 @@ describe('Memory history', () => {
     expect(spy).toHaveBeenCalledWith(START, normaliezedLoc, {
       direction: 'back',
       distance: -1,
-      // TODO: should be something else
       type: 'pop',
     })
     history.forward()
@@ -123,7 +122,6 @@ describe('Memory history', () => {
     expect(spy).toHaveBeenLastCalledWith(normaliezedLoc, START, {
       direction: 'forward',
       distance: 1,
-      // TODO: should be something else
       type: 'pop',
     })
   })

diff --git a/__tests__/matcher/path-parser.spec.ts b/__tests__/matcher/path-parser.spec.ts
index 1e314e3e63c9b62f672032344db9da384a27fce9..446d274d6b7e87a84c772f0844577d44b4ed1f84 100644 (file)
--- a/__tests__/matcher/path-parser.spec.ts
+++ b/__tests__/matcher/path-parser.spec.ts
@@ -621,6 +621,12 @@ describe('Path parser', () => {
       matchParams('/a/:a(?:b-([^/]+\\)?)', '/a/b-one', {
         a: 'one',
       })
+      matchParams('/a/:a(?:b-([^/]+\\)?)', '/a/b-', {
+        a: '',
+      })
+      matchParams('/a/:a(?:b-([^/]+\\))', '/a/b-one', {
+        a: 'one',
+      })
     })
 
     it('catch all', () => {

diff --git a/__tests__/matcher/records.spec.ts b/__tests__/matcher/records.spec.ts
index 1446565c1b0fe29a21821595a067d273abf548dc..afb168c311d2fe1511a9e0408b02190b157857ee 100644 (file)
--- a/__tests__/matcher/records.spec.ts
+++ b/__tests__/matcher/records.spec.ts
@@ -83,11 +83,4 @@ describe('normalizeRouteRecord', () => {
       props: false,
     })
   })
-
-  // TODO: move to router
-  it.todo('beforeEnter is called with the string redirect')
-
-  it.todo('beforeEnter is called with object redirect')
-
-  it.todo('function redirect is invoked by beforeEnter')
 })

diff --git a/__tests__/utils.ts b/__tests__/utils.ts
index 04167bcfc70deea20f819f405ea149a9cde2d9c8..09ab815141e616ccaab3464effd025c7ac7465e3 100644 (file)
--- a/__tests__/utils.ts
+++ b/__tests__/utils.ts
@@ -6,8 +6,10 @@ import {
   RouteLocationNormalized,
   _RouteRecordBase,
   RouteComponent,
+  RouteRecordRaw,
 } from '../src/types'
 import { h, resolveComponent, ComponentOptions } from 'vue'
+import { RouterOptions, createWebHistory, createRouter } from '../src'
 
 export const tick = (time?: number) =>
   new Promise(resolve => {
@@ -132,3 +134,12 @@ export const components = {
     },
   } as RouteComponent,
 }
+
+export function newRouter(
+  options: Partial<RouterOptions> & { routes: RouteRecordRaw[] }
+) {
+  return createRouter({
+    history: options.history || createWebHistory(),
+    ...options,
+  })
+}