Breaking: Remove useFragment option

Author: thomasthiebaud

src/config.ts

@@ -65,7 +65,6 @@ export interface DOMConfig {
 export interface Config {
   dom?: DOMConfig
   overrides?: SelectorsToElement,
-  useFragment?: boolean,
   useAsKey?: string[],
 }
 
@@ -89,7 +88,6 @@ const defaultConfig: Config = {
     ADD_ATTR: ['key'],
   },
   useAsKey: ['key'],
-  useFragment: false,
 }
 
 export function getConfig(config?: Partial<Config>): Config {

src/react.ts

@@ -104,7 +104,7 @@ function transform(element: EnrichedElement, options: Config) {
   }
 }
 
-export function render(nodes: NodeListOf<Node & ChildNode>, options: Config): React.ReactNode[] | React.ReactFragment {
+export function render(nodes: NodeListOf<Node & ChildNode>, options: Config): React.ReactNode[] {
   const elements: React.ReactNode[] = []
 
   for (let i = 0; i < nodes.length; i++) {
@@ -117,8 +117,5 @@ export function render(nodes: NodeListOf<Node & ChildNode>, options: Config): Re
     }
   }
 
-  if (options.useFragment) {
-    return React.createElement(React.Fragment, null, ...elements)
-  }
   return elements
 }

tests/config.spec.ts

@@ -12,7 +12,6 @@ describe('Options', () => {
           RETURN_DOM_IMPORT: false,
         },
         useAsKey: ['key'],
-        useFragment: false,
       })
     })
 
@@ -31,7 +30,6 @@ describe('Options', () => {
           RETURN_DOM_IMPORT: false,
         },
         useAsKey: ['key'],
-        useFragment: false,
       })
     })
 
@@ -49,7 +47,6 @@ describe('Options', () => {
           RETURN_DOM_IMPORT: false,
         },
         useAsKey: ['key'],
-        useFragment: false,
       })
     })
   })

tests/react.spec.ts

@@ -65,18 +65,6 @@ describe('React', () => {
   })
 
   describe('Options', () => {
-    describe('#useFragment', () => {
-      it('should return a Fragment', () => {
-        const div = document.createElement('div')
-        const link = document.createElement('a')
-        link.textContent = 'Link'
-        div.appendChild(link)
-
-        const wrapper = shallowWrapper(render(div.childNodes, { useFragment: true, useAsKey: ['key'] }))
-        expect(wrapper.find('a').text()).toEqual('Link')
-      })
-    })
-
     describe('#useAsKey', () => {
       it('should use key as a default key', () => {
         const div = document.createElement('div')
@@ -85,7 +73,7 @@ describe('React', () => {
         link.setAttribute('key', 'link')
         div.appendChild(link)
 
-        const wrapper = shallowWrapper(render(div.childNodes, { useFragment: true, useAsKey: ['key'] }))
+        const wrapper = shallowWrapper(render(div.childNodes, { useAsKey: ['key'] }))
         expect(wrapper.find('a').key()).toEqual('link')
       })
 
@@ -96,7 +84,7 @@ describe('React', () => {
         link.setAttribute('class', 'fallback key')
         div.appendChild(link)
 
-        const wrapper = shallowWrapper(render(div.childNodes, { useFragment: true, useAsKey: ['key', 'id', 'class'] }))
+        const wrapper = shallowWrapper(render(div.childNodes, { useAsKey: ['key', 'id', 'class'] }))
         expect(wrapper.find('a').key()).toEqual('fallback key')
       })
 
@@ -106,7 +94,7 @@ describe('React', () => {
         link.textContent = 'Link'
         div.appendChild(link)
 
-        const wrapper = shallowWrapper(render(div.childNodes, { useFragment: true, useAsKey: ['key', 'id'] }))
+        const wrapper = shallowWrapper(render(div.childNodes, { useAsKey: ['key', 'id'] }))
         expect(wrapper.find('a').key()).toEqual(null)
       })
     })