fix(config): Do not use config.renderStubDefaultSlot

* docs mention config.global.renderStubDefaultSlot * get rid of double definition
vuejs · Oct 6, 2022 · d16ceb7 · d16ceb7
1 parent 77bfbe2
commit d16ceb7
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 13 deletions.
diff --git a/docs/guide/advanced/stubs-shallow-mount.md b/docs/guide/advanced/stubs-shallow-mount.md
@@ -275,11 +275,11 @@ For this use case, you can use `config.renderStubDefaultSlot`, which will render
 import { config, mount } from '@vue/test-utils'
 
 beforeAll(() => {
-  config.renderStubDefaultSlot = true
+  config.global.renderStubDefaultSlot = true
 })
 
 afterAll(() => {
-  config.renderStubDefaultSlot = false
+  config.global.renderStubDefaultSlot = false
 })
 
 test('shallow with stubs', () => {
@@ -316,4 +316,4 @@ So regardless of which mounting method you choose, we suggest keeping these guid
 
 - use `global.stubs` to replace a component with a dummy one to simplify your tests
 - use `shallow: true` (or `shallowMount`) to stub out all child components
-- use `config.renderStubDefaultSlot` to render the default `<slot>` for a stubbed component
+- use `global.renderStubDefaultSlot` to render the default `<slot>` for a stubbed component
diff --git a/docs/migration/index.md b/docs/migration/index.md
@@ -206,7 +206,7 @@ You can enable the old behavior like this:
 ```js
 import { config } from '@vue/test-utils'
 
-config.renderStubDefaultSlot = true
+config.global.renderStubDefaultSlot = true
 ```
 
 ### `destroy` is now `unmount` to match Vue 3

diff --git a/src/config.ts b/src/config.ts
@@ -12,7 +12,6 @@ export interface GlobalConfigOptions {
     DOMWrapper: Pluggable<DOMWrapper<Node>>
     createStubs?: CustomCreateStub
   }
-  renderStubDefaultSlot: boolean
 }
 
 interface Plugin<Instance, O> {
@@ -80,6 +79,5 @@ export const config: GlobalConfigOptions = {
   plugins: {
     VueWrapper: new Pluggable(),
     DOMWrapper: new Pluggable()
-  },
-  renderStubDefaultSlot: false
+  }
 }
diff --git a/src/utils.ts b/src/utils.ts
@@ -40,8 +40,8 @@ export function mergeGlobalProperties(
 
   const renderStubDefaultSlot =
     mountGlobal.renderStubDefaultSlot ??
-    config?.renderStubDefaultSlot ??
-    configGlobal?.renderStubDefaultSlot
+    configGlobal?.renderStubDefaultSlot ??
+    false
 
   return {
     mixins: [...(configGlobal.mixins || []), ...(mountGlobal.mixins || [])],

diff --git a/tests/mountingOptions/global.stubs.spec.ts b/tests/mountingOptions/global.stubs.spec.ts
@@ -731,11 +731,11 @@ describe('mounting options: stubs', () => {
     }
 
     afterEach(() => {
-      config.renderStubDefaultSlot = false
+      config.global.renderStubDefaultSlot = false
     })
 
     it('renders only the default stub slot only behind flag', () => {
-      config.renderStubDefaultSlot = true
+      config.global.renderStubDefaultSlot = true
 
       const wrapper = mount(Component, {
         global: {
@@ -750,7 +750,7 @@ describe('mounting options: stubs', () => {
     })
 
     it('renders none of the slots on a stub', () => {
-      config.renderStubDefaultSlot = false
+      config.global.renderStubDefaultSlot = false
       const wrapper = mount(Component, {
         global: {
           stubs: ['ComponentWithSlots']
@@ -764,7 +764,7 @@ describe('mounting options: stubs', () => {
     })
 
     it('renders the default slot of deeply nested stubs when renderStubDefaultSlot=true', () => {
-      config.renderStubDefaultSlot = true
+      config.global.renderStubDefaultSlot = true
 
       const SimpleSlot = {
         name: 'SimpleSlot',