fix test isolation in stack_trace.spec.js

Author: simon-id

test/stack_trace.spec.js

@@ -7,7 +7,7 @@
 
 const fs = require('fs')
 const path = require('path')
-const proxyquire = require('proxyquire').noPreserveCache()
+const proxyquire = require('proxyquire')
 
 const { getPrepareStackTrace } = require('../js/stack-trace')
 const { getSourcePathAndLineFromSourceMaps } = require('../js/source-map')
@@ -147,36 +147,45 @@ describe('getFilenameFromSourceMap', () => {
 })
 
 describe('getOriginalPathAndLineFromSourceMapFromSourceMap', () => {
-  const sourceMapsMap = new Map()
-  const setLRU = sinon.spy()
-  const getLRU = sinon.spy()
-  class LRU {
-    set (key, value) {
-      setLRU(key, value)
-      return sourceMapsMap.set(key, value)
-    }
+  let sourceMapsMap
+  let setLRU
+  let getLRU
+  let getOriginalPathAndLineFromSourceMap
+
+  beforeEach(() => {
+    sourceMapsMap = new Map()
+    setLRU = sinon.spy()
+    getLRU = sinon.spy()
+    class LRU {
+      set (key, value) {
+        setLRU(key, value)
+        return sourceMapsMap.set(key, value)
+      }
 
-    get (key) {
-      getLRU(key)
-      return sourceMapsMap.get(key)
+      get (key) {
+        getLRU(key)
+        return sourceMapsMap.get(key)
+      }
     }
-  }
 
-  const { getOriginalPathAndLineFromSourceMap } = proxyquire('../js/source-map', {
-    'lru-cache': LRU,
-    './node_source_map': nodeSourceMap,
-    fs: {
-      readFileSync: (filename) => {
-        switch (filename) {
-          case 'no-sourcemap':
-            return ''
-          case 'error':
-            throw new Error('errr')
-          default:
-            return readFileSync(filename)
+    const sourceMap = proxyquire('../js/source-map', {
+      'lru-cache': LRU,
+      './node_source_map': nodeSourceMap,
+      fs: {
+        readFileSync: (filename) => {
+          switch (filename) {
+            case 'no-sourcemap':
+              return ''
+            case 'error':
+              throw new Error('errr')
+            default:
+              return readFileSync(filename)
+          }
         }
       }
-    }
+    })
+
+    getOriginalPathAndLineFromSourceMap = sourceMap.getOriginalPathAndLineFromSourceMap
   })
 
   const fileName = 'test-inline.js'
@@ -185,13 +194,6 @@ describe('getOriginalPathAndLineFromSourceMapFromSourceMap', () => {
     line: 5
   }
 
-  afterEach(() => {
-    sinon.reset()
-    setLRU.resetHistory()
-    getLRU.resetHistory()
-    sourceMapsMap.clear()
-  })
-
   it('should add SourceMap to cache', () => {
     const pathAndLine = getOriginalPathAndLineFromSourceMap(originalPathAndLine.path, originalPathAndLine.line, 12)