refactor(types): improve types (#785)

Author: 38elements

packages/create-instance/add-mocks.js

@@ -2,7 +2,10 @@
 import $$Vue from 'vue'
 import { warn } from 'shared/util'
 
-export default function addMocks (mockedProperties: Object, Vue: Component) {
+export default function addMocks (
+  mockedProperties: Object,
+  Vue: Component
+): void {
   Object.keys(mockedProperties).forEach(key => {
     try {
       Vue.prototype[key] = mockedProperties[key]

packages/create-instance/add-slots.js

@@ -2,8 +2,8 @@
 
 import { compileToFunctions } from 'vue-template-compiler'
 
-function startsWithTag (str) {
-  return str && str.trim()[0] === '<'
+function startsWithTag (str: SlotValue): boolean {
+  return typeof str === 'string' && str.trim()[0] === '<'
 }
 
 function createVNodesForSlot (

packages/create-instance/create-functional-component.js

@@ -7,7 +7,7 @@ import { createSlotVNodes } from './add-slots'
 export default function createFunctionalComponent (
   component: Component,
   mountingOptions: Options
-) {
+): Component {
   if (mountingOptions.context && typeof mountingOptions.context !== 'object') {
     throwError('mount.context must be an object')
   }

packages/create-instance/extract-instance-options.js

@@ -1,3 +1,5 @@
+// @flow
+
 const MOUNTING_OPTIONS = [
   'attachToDocument',
   'mocks',
@@ -11,7 +13,9 @@ const MOUNTING_OPTIONS = [
   'propsData'
 ]
 
-export default function extractInstanceOptions (options) {
+export default function extractInstanceOptions (
+  options: Object
+): Object {
   const instanceOptions = { ...options }
   MOUNTING_OPTIONS.forEach(mountingOption => {
     delete instanceOptions[mountingOption]

packages/create-instance/log-events.js

@@ -4,7 +4,7 @@ export function logEvents (
   vm: Component,
   emitted: Object,
   emittedByOrder: Array<any>
-) {
+): void {
   const emit = vm.$emit
   vm.$emit = (name, ...args) => {
     (emitted[name] || (emitted[name] = [])).push(args)
@@ -13,7 +13,7 @@ export function logEvents (
   }
 }
 
-export function addEventLogger (vue: Component) {
+export function addEventLogger (vue: Component): void {
   vue.mixin({
     beforeCreate: function () {
       this.__emitted = Object.create(null)

packages/create-instance/validate-slots.js

@@ -11,7 +11,7 @@ function isValidSlot (slot: any): boolean {
   )
 }
 
-function requiresTemplateCompiler (slot) {
+function requiresTemplateCompiler (slot: any): void {
   if (typeof slot === 'string' && !compileToFunctions) {
     throwError(
       `vueTemplateCompiler is undefined, you must pass ` +

packages/shared/compile-template.js

@@ -2,7 +2,7 @@
 
 import { compileToFunctions } from 'vue-template-compiler'
 
-export function compileTemplate (component: Component) {
+export function compileTemplate (component: Component): void {
   if (component.template) {
     Object.assign(component, compileToFunctions(component.template))
   }

packages/shared/util.js

@@ -1,16 +1,16 @@
 // @flow
 import Vue from 'vue'
 
-export function throwError (msg: string) {
+export function throwError (msg: string): void {
   throw new Error(`[vue-test-utils]: ${msg}`)
 }
 
-export function warn (msg: string) {
+export function warn (msg: string): void {
   console.error(`[vue-test-utils]: ${msg}`)
 }
 
 const camelizeRE = /-(\w)/g
-export const camelize = (str: string) => {
+export const camelize = (str: string): string => {
   const camelizedStr = str.replace(
     camelizeRE,
     (_, c) => (c ? c.toUpperCase() : '')
@@ -21,14 +21,14 @@ export const camelize = (str: string) => {
 /**
  * Capitalize a string.
  */
-export const capitalize = (str: string) =>
+export const capitalize = (str: string): string =>
   str.charAt(0).toUpperCase() + str.slice(1)
 
 /**
  * Hyphenate a camelCase string.
  */
 const hyphenateRE = /\B([A-Z])/g
-export const hyphenate = (str: string) =>
+export const hyphenate = (str: string): string =>
   str.replace(hyphenateRE, '-$1').toLowerCase()
 
 export const vueVersion = Number(

packages/shared/validators.js

@@ -1,7 +1,7 @@
 // @flow
 import { throwError, capitalize, camelize, hyphenate } from './util'
 
-export function isDomSelector (selector: any) {
+export function isDomSelector (selector: any): boolean {
   if (typeof selector !== 'string') {
     return false
   }
@@ -28,7 +28,7 @@ export function isDomSelector (selector: any) {
   }
 }
 
-export function isVueComponent (component: any) {
+export function isVueComponent (component: any): boolean {
   if (typeof component === 'function' && component.options) {
     return true
   }
@@ -44,7 +44,7 @@ export function isVueComponent (component: any) {
   return typeof component.render === 'function'
 }
 
-export function componentNeedsCompiling (component: Component) {
+export function componentNeedsCompiling (component: Component): boolean {
   return (
     component &&
     !component.render &&
@@ -53,7 +53,7 @@ export function componentNeedsCompiling (component: Component) {
   )
 }
 
-export function isRefSelector (refOptionsObject: any) {
+export function isRefSelector (refOptionsObject: any): boolean {
   if (
     typeof refOptionsObject !== 'object' ||
     Object.keys(refOptionsObject || {}).length !== 1
@@ -64,15 +64,18 @@ export function isRefSelector (refOptionsObject: any) {
   return typeof refOptionsObject.ref === 'string'
 }
 
-export function isNameSelector (nameOptionsObject: any) {
+export function isNameSelector (nameOptionsObject: any): boolean {
   if (typeof nameOptionsObject !== 'object' || nameOptionsObject === null) {
     return false
   }
 
   return !!nameOptionsObject.name
 }
 
-export function templateContainsComponent (template: string, name: string) {
+export function templateContainsComponent (
+  template: string,
+  name: string
+): boolean {
   return [capitalize, camelize, hyphenate].some(format => {
     const re = new RegExp(`<${format(name)}\\s*(\\s|>|(\/>))`, 'g')
     return re.test(template)

packages/test-utils/src/add-scoped-slots.js

@@ -32,7 +32,10 @@ function getVueTemplateCompilerHelpers (proxy: Object): Object {
   return helpers
 }
 
-export function addScopedSlots (vm: Component, scopedSlots: any) {
+export function addScopedSlots (
+  vm: Component,
+  scopedSlots: { [name: string]: string }
+): void {
   if (window.navigator.userAgent.match(/PhantomJS/i)) {
     throwError(
       `the scopedSlots option does not support PhantomJS. ` +