From 2ff90ee6a8e2f3d741bca8973b96b5bac5b46f58 Mon Sep 17 00:00:00 2001
From: Serhii Teplov <steplov@users.noreply.github.com>
Date: Sun, 25 Nov 2018 17:43:31 +0200
Subject: [PATCH] fix: use correct source map names for script files imported
 over src (#131)

---
 lib/process.js                   | 20 ++++++-------
 test/resources/SourceMapsSrc.vue |  3 ++
 test/sourceMaps.spec.js          | 51 ++++++++++++++++++++++++++++++++
 3 files changed, 63 insertions(+), 11 deletions(-)
 create mode 100644 test/resources/SourceMapsSrc.vue
 create mode 100644 test/sourceMaps.spec.js

diff --git a/lib/process.js b/lib/process.js
index 0443b526..cd153f26 100644
--- a/lib/process.js
+++ b/lib/process.js
@@ -31,25 +31,23 @@ function processScript(scriptPart, filePath, config) {
 
 module.exports = function(src, filePath, config) {
   const vueJestConfig = getVueJestConfig(config)
-  var parts = vueCompiler.parseComponent(src, { pad: true })
+  let parts = vueCompiler.parseComponent(src, { pad: true })
+  let scriptSrc = src
+  let sourceMapPath = filePath
 
   if (parts.script && parts.script.src) {
-    parts.script.content = fs.readFileSync(
-      join(filePath, '..', parts.script.src),
-      'utf8'
-    )
+    const externalScrPath = join(filePath, '..', parts.script.src)
+
+    parts.script.content = fs.readFileSync(externalScrPath, 'utf8')
+    scriptSrc = parts.script.content
+    sourceMapPath = externalScrPath
   }
 
   const result = processScript(parts.script, filePath, config)
   const script = result.code
   const inputMap = result.map
 
-  let scriptSrc = src
-  if (parts.script && parts.script.src) {
-    scriptSrc = parts.script.content
-  }
-
-  const map = generateSourceMap(script, '', filePath, scriptSrc, inputMap)
+  const map = generateSourceMap(script, '', sourceMapPath, scriptSrc, inputMap)
 
   let output =
     ';(function(){\n' +
diff --git a/test/resources/SourceMapsSrc.vue b/test/resources/SourceMapsSrc.vue
new file mode 100644
index 00000000..2fbf9f1e
--- /dev/null
+++ b/test/resources/SourceMapsSrc.vue
@@ -0,0 +1,3 @@
+<template src="./BasicSrc.html"></template>
+
+<script src="./BasicSrc.js"></script>
diff --git a/test/sourceMaps.spec.js b/test/sourceMaps.spec.js
new file mode 100644
index 00000000..9a75850c
--- /dev/null
+++ b/test/sourceMaps.spec.js
@@ -0,0 +1,51 @@
+import { resolve } from 'path'
+import { readFileSync } from 'fs'
+import clearModule from 'clear-module'
+import cache from '../lib/cache'
+import jestVue from '../vue-jest'
+
+beforeEach(() => {
+  cache.flushAll()
+  clearModule.all()
+})
+
+const getSourceMaps = code => {
+  const sourceMapBase64 = /\/\/# sourceMappingURL=data:application\/json;charset=utf-8;base64,(.*)/gim
+
+  let matches
+  let values = []
+
+  while ((matches = sourceMapBase64.exec(code)) !== null) {
+    values.push(JSON.parse(Buffer.from(matches[1], 'base64').toString('ascii')))
+  }
+
+  return values
+}
+
+test('generates source maps for .vue files', () => {
+  const filePath = resolve(__dirname, './resources/Basic.vue')
+  const fileString = readFileSync(filePath, { encoding: 'utf8' })
+
+  const { code } = jestVue.process(fileString, filePath, {
+    moduleFileExtensions: ['js', 'vue']
+  })
+
+  const [template, js] = getSourceMaps(code)
+
+  expect(js.sources[0]).toBe('Basic.vue')
+  expect(template.sources[0]).toBe('Basic.vue')
+})
+
+test('generates source maps using src attributes', () => {
+  const filePath = resolve(__dirname, './resources/SourceMapsSrc.vue')
+  const fileString = readFileSync(filePath, { encoding: 'utf8' })
+
+  const { code } = jestVue.process(fileString, filePath, {
+    moduleFileExtensions: ['js', 'vue']
+  })
+
+  const [template, js] = getSourceMaps(code)
+
+  expect(js.sources[0]).toBe('BasicSrc.js')
+  expect(template.sources[0]).toBe('SourceMapsSrc.vue')
+})