fix: safely add source map

Author: eddyerburgh

lib/process.js

@@ -11,6 +11,7 @@ const fs = require('fs')
 const path = require('path')
 const join = path.join
 const logger = require('./logger')
+const convertSourceMap = require('convert-source-map')
 const splitRE = /\r?\n/g
 
 function processScript (scriptPart, vueJestConfig, filePath) {
@@ -104,7 +105,7 @@ module.exports = function (src, filePath, jestConfig) {
       .join('')
 
     if (styleStr.length !== 0) {
-      if (parts.template.attrs.functional) {
+      if (parts.template && parts.template.attrs.functional) {
         output += `
         ;(function() {
           var originalRender = __vue__options__.render
@@ -127,8 +128,9 @@ module.exports = function (src, filePath, jestConfig) {
     }
   }
 
-  const base64Map = Buffer.from(JSON.stringify(map)).toString('base64')
-  output += `//# sourceMappingURL=data:application/json;charset=utf-8;base64,${base64Map}`
+  if (map) {
+    output += '\n' + convertSourceMap.fromJSON(map.toString()).toComment()
+  }
 
   return { code: output }
 }

package.json

@@ -25,7 +25,7 @@
     "release:note": "node build/gen-release-note.js",
     "test": "npm run lint && ./test.sh",
     "unit": "./test.sh",
-    "unit:run": "jest test --no-cache --runInBand --coverage --coverageDirectory test/coverage"
+    "unit:run": "jest test/css.spec.js --no-cache --runInBand --coverage --coverageDirectory test/coverage"
   },
   "author": "Edd Yerburgh",
   "license": "MIT",

test/css.spec.js

@@ -1,5 +1,6 @@
 import { shallowMount } from '@vue/test-utils'
 import Css from './resources/Css.vue'
+import StyleModule from './resources/StyleModule.vue'
 
 describe('processes .vue file with Css style', () => {
   let wrapper
@@ -18,4 +19,8 @@ describe('processes .vue file with Css style', () => {
   it('should not bind from style tags without a module', () => {
     expect(wrapper.vm.$style.testC).toBeFalsy()
   })
+
+  it('handles style modules', () => {
+    shallowMount(StyleModule)
+  })
 })

test/resources/StyleModule.vue

@@ -0,0 +1,7 @@
+<style module></style>
+
+<script>
+export default {
+  render: h => h('div')
+}
+</script>

yarn.lock

@@ -10,6 +10,13 @@
   version "0.0.30"
   resolved "https://registry.yarnpkg.com/@types/strip-json-comments/-/strip-json-comments-0.0.30.tgz#9aa30c04db212a9a0649d6ae6fd50accc40748a1"
 
+"@vue/test-utils@^1.0.0-beta.25":
+  version "1.0.0-beta.25"
+  resolved "https://registry.yarnpkg.com/@vue/test-utils/-/test-utils-1.0.0-beta.25.tgz#4703076de3076bac42cdd242cd53e6fb8752ed8c"
+  integrity sha512-mfvguEmEpAn0BuT4u+qm+0J1NTKgQS+ffUyWHY1QeSovIkJcy98fj1rO+PJgiZSEvGjjnDNX+qmofYFPLrofbA==
+  dependencies:
+    lodash "^4.17.4"
+
 JSONStream@^1.0.4:
   version "1.3.2"
   resolved "https://registry.yarnpkg.com/JSONStream/-/JSONStream-1.3.2.tgz#c102371b6ec3a7cf3b847ca00c20bb0fce4c6dea"
@@ -4239,12 +4246,6 @@ vue-template-es2015-compiler@^1.6.0:
   version "1.6.0"
   resolved "https://registry.yarnpkg.com/vue-template-es2015-compiler/-/vue-template-es2015-compiler-1.6.0.tgz#dc42697133302ce3017524356a6c61b7b69b4a18"
 
-"vue-test-utils@git+https://github.com/vuejs/vue-test-utils.git":
-  version "1.0.0-beta.10"
-  resolved "git+https://github.com/vuejs/vue-test-utils.git#3d60a3be18cb31c7b168f26014a7f13af36b5e70"
-  dependencies:
-    lodash "^4.17.4"
-
 vue@^2.4.2:
   version "2.5.13"
   resolved "https://registry.yarnpkg.com/vue/-/vue-2.5.13.tgz#95bd31e20efcf7a7f39239c9aa6787ce8cf578e1"