fix(create-local-vue): fix max range stack size (fix #1768) (#1786)

* test(create-local-vue): add a failing test

* fix(create-local-vue): avoid Vue instance pollution

Author: antoinerey

packages/shared/create-local-vue.js

@@ -36,7 +36,7 @@ function _createLocalVue(
   instance.config = cloneDeep(Vue.config)
 
   // if a user defined errorHandler is defined by a localVue instance via createLocalVue, register it
-  instance.config.errorHandler = config.errorHandler || Vue.config.errorHandler
+  instance.config.errorHandler = config.errorHandler
 
   // option merge strategies need to be exposed by reference
   // so that merge strats registered by plugins can work properly

test/specs/create-local-vue.spec.js

@@ -231,4 +231,27 @@ describeWithShallowAndMount('createLocalVue', mountingMethod => {
       }
     }
   )
+
+  // This test is related to the issue 1768.
+  // See: https://github.com/vuejs/vue-test-utils/issues/1768
+  itSkipIf(
+    process.env.TEST_ENV === 'browser' || vueVersion < 2.6,
+    'Does not exceed maximum call stack size when no custom error handler is defined',
+    async () => {
+      const { error } = global.console
+      const spy = jest.spyOn(global.console, 'error')
+      const localVue = createLocalVue()
+
+      try {
+        mountingMethod(ComponentWithSyncError, { localVue })
+      } catch {
+        const expected = expect.stringMatching(
+          /Maximum call stack size exceeded/
+        )
+
+        global.console.error = error
+        expect(spy).not.toHaveBeenCalledWith(expected)
+      }
+    }
+  )
 })