fix: unmountComponentAtNode(...): Target container is not a DOM eleme…

…ntd fyf (#108)

* Fixed error where crash will happen if fired twice in a row.

* undid change to see if test fails in CI.

* re-fixed error:  unmountComponentAtNode(...): Target container is not a DOM element.

src/index.js

@@ -44,7 +44,9 @@ export default function withReactContent (ParentSwal) {
           const superOnDestroy = params.onDestroy
           params.onDestroy = (element) => {
             superOnDestroy(element)
-            ReactDOM.unmountComponentAtNode(domElement)
+            if (domElement) {
+              ReactDOM.unmountComponentAtNode(domElement)
+            }
           }
         }
       })

test/integration.test.js

@@ -53,6 +53,25 @@ describe('integration', () => {
       },
     })
   })
+  it('can fire twice without crashing', async () => {
+    await cleanSwalState()
+    const MySwal = withReactContent(SwalWithoutAnimation)
+    MySwal.fire({
+      title: <span>React element</span>,
+      footer: 'plain text'
+    })
+    await MySwal.fire({
+      title: <span>React element</span>,
+      footer: 'plain text',
+      onOpen: () => {
+        expect(MySwal.getTitle().innerHTML).toEqual(
+          '<span>React element</span>',
+        )
+        expect(MySwal.getFooter().innerHTML).toEqual('plain text')
+        MySwal.clickConfirm()
+      },
+    })
+  })
   it('returns a class with the same instance & static properties as Swal', async () => {
     const MySwal = withReactContent(Swal)
     Object.keys(Swal).forEach(key => {