catch router.start

Author: chrisvfritz

rules/vue-router/router-start.js

@@ -0,0 +1,18 @@
+'use strict'
+
+var chalk = require('chalk')
+
+module.exports = {
+  pattern: /\b(router\s*?\.\s*?start)\s*?\(/,
+  warning: function (match, routerStart) {
+    return {
+      reason: 'Starting an app with routing no longer requires a special method - the router can simply be passed to the root Vue instance as option',
+      fix: (
+        'Replace ' + chalk.red(routerStart) + ' with ' +
+        chalk.green('router: router') +
+        ' on your root Vue instance (see link below for details)'
+      ),
+      docsHash: 'router-start-deprecated'
+    }
+  }
+}

rules/vue-router/router-start.spec.js

@@ -0,0 +1,28 @@
+'use strict'
+
+const check = createRuleChecker('vue-router/router-start')
+
+describe('Rule: router-start', () => {
+  it('does not match an empty line', () => {
+    const warning = check('')
+    expect(warning).toBe(null)
+  })
+
+  it('does not match "router"', () => {
+    const warning = check('router')
+    expect(warning).toBe(null)
+  })
+
+  it('does not match "start"', () => {
+    const warning = check('on')
+    expect(warning).toBe(null)
+  })
+
+  it('matches "router.start("', () => {
+    const warning = check(`
+      router.start(
+    `)
+    expect(warning).toBeTruthy()
+    expect(warning.fix).toBe('Replace router.start with router: router on your root Vue instance (see link below for details)')
+  })
+})