test(drawer): Remove nesting breaking tests

Author: m7kvqbe1

packages/react-component-library/src/components/Drawer/Drawer.test.tsx

@@ -1,12 +1,7 @@
 import React from 'react'
 import { renderToStaticMarkup } from 'react-dom/server'
 import '@testing-library/jest-dom/extend-expect'
-import {
-  render,
-  RenderResult,
-  fireEvent,
-  cleanup,
-} from '@testing-library/react'
+import { render, RenderResult, fireEvent } from '@testing-library/react'
 
 import { Drawer } from '.'
 
@@ -33,45 +28,45 @@ describe('Drawer', () => {
         renderToStaticMarkup(children)
       )
     })
+  })
 
-    describe('and the `isOpen` attribute and `onClose` callback are provided', () => {
-      beforeEach(() => {
-        cleanup()
-        onCloseSpy = jest.fn()
+  describe('and the `isOpen` attribute and `onClose` callback are provided', () => {
+    beforeEach(() => {
+      children = <h1>Arbitrary JSX</h1>
+      onCloseSpy = jest.fn()
 
-        wrapper = render(
-          <Drawer isOpen onClose={onCloseSpy}>
-            {children}
-          </Drawer>
-        )
-      })
+      wrapper = render(
+        <Drawer isOpen onClose={onCloseSpy}>
+          {children}
+        </Drawer>
+      )
+    })
 
-      it('applies the `is-open` class', () => {
-        expect(wrapper.getByTestId('drawer-wrapper').classList).toContain(
-          'is-open'
+    it('applies the `is-open` class', () => {
+      expect(wrapper.getByTestId('drawer-wrapper').classList).toContain(
+        'is-open'
+      )
+    })
+
+    describe('and a user clicks on the close button', () => {
+      beforeEach(() => {
+        fireEvent(
+          wrapper.getByTestId('drawer-close'),
+          new MouseEvent('click', {
+            bubbles: true,
+            cancelable: true,
+          })
         )
       })
 
-      describe('and a user clicks on the close button', () => {
-        beforeEach(() => {
-          fireEvent(
-            wrapper.getByTestId('drawer-close'),
-            new MouseEvent('click', {
-              bubbles: true,
-              cancelable: true,
-            })
-          )
-        })
-
-        it('invokes the `onClose` callback', () => {
-          expect(onCloseSpy).toHaveBeenCalled()
-        })
+      it('invokes the `onClose` callback', () => {
+        expect(onCloseSpy).toHaveBeenCalled()
+      })
 
-        it('removes the `is-open` class', () => {
-          expect(wrapper.getByTestId('drawer-wrapper').classList).not.toContain(
-            'is-open'
-          )
-        })
+      it('removes the `is-open` class', () => {
+        expect(wrapper.getByTestId('drawer-wrapper').classList).not.toContain(
+          'is-open'
+        )
       })
     })
   })