test(warn): test warnings

__tests__/warnings.spec.ts

@@ -0,0 +1,41 @@
+import { mockWarn } from 'jest-mock-warn'
+import { createMemoryHistory, createRouter } from '../src'
+import { defineComponent } from 'vue'
+
+let component = defineComponent({})
+
+describe('warnings', () => {
+  mockWarn()
+  it('warns on missing name and path for redirect', async () => {
+    const history = createMemoryHistory()
+    const router = createRouter({
+      history,
+      routes: [
+        { path: '/', component },
+        { path: '/redirect', redirect: { params: { foo: 'f' } } },
+      ],
+    })
+    await router.push('/redirect').catch(() => {})
+    expect('Invalid redirect found').toHaveBeenWarned()
+  })
+
+  it('warns when resolving a route with path and params', async () => {
+    const history = createMemoryHistory()
+    const router = createRouter({
+      history,
+      routes: [{ path: '/:p', name: 'p', component }],
+    })
+    router.resolve({ path: '/', params: { p: 'p' } })
+    expect('Path "/" was passed with params').toHaveBeenWarned()
+  })
+
+  it('does not warn when resolving a route with path, params and name', async () => {
+    const history = createMemoryHistory()
+    const router = createRouter({
+      history,
+      routes: [{ path: '/:p', name: 'p', component }],
+    })
+    router.resolve({ path: '/p', name: 'p', params: { p: 'p' } })
+    expect('Path "/" was passed with params').not.toHaveBeenWarned()
+  })
+})

src/router.ts

@@ -353,6 +353,7 @@ export function createRouter(options: RouterOptions): Router {
             targetLocation.fullPath
           }". A redirect must contain a name or path.`
         )
+        return Promise.reject(new Error('Invalid redirect'))
       }
       return pushWithRedirect(
         {