test: add linting to lib and fix errors

Author: eddyerburgh

lib/add-template-mapping.js

@@ -1,4 +1,3 @@
-const sourceMap = require('source-map')
 const splitRE = /\r?\n/g
 
 module.exports = function addTemplateMapping (content, parts, output, map, beforeLines) {

lib/compilers/babel-compiler.js

@@ -1,8 +1,5 @@
 const babel = require('babel-core')
-const path = require('path')
-const fs = require('fs')
 const findBabelConfig = require('find-babel-config')
-const throwError = require('../throw-error')
 const logger = require('../logger')
 
 var defaultBabelOptions = {
@@ -11,16 +8,16 @@ var defaultBabelOptions = {
 }
 
 module.exports = function compileBabel (scriptContent) {
-  const { file, config } = findBabelConfig.sync(process.cwd(), 0);
+  const { file, config } = findBabelConfig.sync(process.cwd(), 0)
 
   if (!file) {
     logger.info('no .babelrc found, defaulting to default babel options')
   }
 
   const baseBabelOptions = file ? config : defaultBabelOptions
-  const babelOptions = Object.assign({sourceMaps: true}, baseBabelOptions)
+  const babelOptions = Object.assign({ sourceMaps: true }, baseBabelOptions)
 
-  const res =  babel.transform(scriptContent, babelOptions)
+  const res = babel.transform(scriptContent, babelOptions)
 
   return {
     code: res.code,

lib/compilers/coffee-compiler.js

@@ -14,7 +14,7 @@ module.exports = function (raw, cb, compiler) {
     throwError(err)
   }
   return {
-      code: compiled.js,
-      map: compiled.v3SourceMap
-    }
+    code: compiled.js,
+    map: compiled.v3SourceMap
   }
+}

lib/compilers/jade-compiler.js

@@ -2,11 +2,11 @@ var ensureRequire = require('../ensure-require.js')
 const throwError = require('../throw-error')
 
 module.exports = function (raw) {
-  var html;
+  var html
   ensureRequire('jade', 'jade')
   var jade = require('jade')
   try {
-    var html = jade.compile(raw)()
+    html = jade.compile(raw)()
   } catch (err) {
     throwError(err)
   }

lib/compilers/pug-compiler.js

@@ -2,11 +2,11 @@ var ensureRequire = require('../ensure-require.js')
 const throwError = require('../throw-error')
 
 module.exports = function (raw) {
-  var html;
+  var html
   ensureRequire('pug', 'pug')
   var jade = require('pug')
   try {
-    var html = jade.compile(raw)()
+    html = jade.compile(raw)()
   } catch (err) {
     throwError(err)
   }

lib/compilers/typescript-compiler.js

@@ -1,9 +1,9 @@
 const typescript = require('typescript')
 
 module.exports = function compileTypescript (scriptContent) {
-  const res = typescript.transpileModule(scriptContent,{"compilerOptions": {
-    "sourceMap": true
-    }})
+  const res = typescript.transpileModule(scriptContent, { 'compilerOptions': {
+    'sourceMap': true
+  }})
 
   return {
     code: res.outputText,

lib/get_cache_key.js

@@ -1,7 +1,7 @@
 var crypto = require('crypto')
 
-function getCacheKey (fileData, filename, configString) {
+module.exports = function getCacheKey (fileData, filename, configString) {
   return crypto.createHash('md5')
-    .update(fileData + filePath + configString, 'utf8')
+    .update(fileData + filename + configString, 'utf8')
     .digest('hex')
 }

lib/logger.js

@@ -1,3 +1,3 @@
-module.exports.info = function info(msg) {
+module.exports.info = function info (msg) {
   console.info('\n[jest-vue] Info: ' + (msg) + '\n')
 }

lib/process.js

@@ -21,7 +21,7 @@ function processScript (scriptPart) {
   return compileBabel(scriptPart.content)
 }
 
-module.exports = function(src, path){
+module.exports = function (src, path) {
   var parts = vueCompiler.parseComponent(src, { pad: true })
   const renderFunctions = compileTemplate(parts.template)
 
@@ -36,10 +36,10 @@ module.exports = function(src, path){
     'var __vue__options__ = (typeof module.exports === "function"' +
     '? module.exports.options' +
     ': module.exports)\n'
-  
+
   output += '__vue__options__.render = ' + renderFunctions.render + '\n' +
     '__vue__options__.staticRenderFns = ' + renderFunctions.staticRenderFns + '\n'
-  
+
   if (map) {
     const beforeLines = output.split(splitRE).length
     addTemplateMapping(script, parts, output, map, beforeLines)
@@ -48,6 +48,6 @@ module.exports = function(src, path){
   if (map) {
     output += '\n' + convertSourceMap.fromJSON(map.toString()).toComment()
   }
-  
+
   return output
 }

lib/template-compiler.js

@@ -5,11 +5,11 @@ var compilePug = require('./compilers/pug-compiler')
 var compileJade = require('./compilers/jade-compiler')
 const throwError = require('./throw-error')
 
-function getTemplateContent(templatePart) {
-  if(templatePart.lang === 'pug') {
+function getTemplateContent (templatePart) {
+  if (templatePart.lang === 'pug') {
     return compilePug(templatePart.content)
   }
-  if(templatePart.lang === 'jade') {
+  if (templatePart.lang === 'jade') {
     return compileJade(templatePart.content)
   }
   return templatePart.content

lib/throw-error.js

@@ -1,3 +1,3 @@
-module.exports = function error (msg){
+module.exports = function error (msg) {
   throw new Error('\n[jest-vue] Error: ' + (msg) + '\n')
 }

package.json

@@ -15,7 +15,7 @@
     "vue jest preprocessor"
   ],
   "scripts": {
-    "lint": "eslint jest-vue.js test",
+    "lint": "eslint lib jest-vue.js test",
     "lint:fix": "npm run lint -- --fix",
     "release": "build/release.sh",
     "release:note": "node build/gen-release-note.js",