Respect updates to default options

src/ConfirmProvider.js

@@ -51,13 +51,13 @@ const buildOptions = (defaultOptions, options) => {
 };
 
 const ConfirmProvider = ({ children, defaultOptions = {} }) => {
-  const [options, setOptions] = useState({ ...DEFAULT_OPTIONS, ...defaultOptions });
+  const [options, setOptions] = useState({});
   const [resolveReject, setResolveReject] = useState([]);
   const [resolve, reject] = resolveReject;
 
   const confirm = useCallback((options = {}) => {
     return new Promise((resolve, reject) => {
-      setOptions(buildOptions(defaultOptions, options));
+      setOptions(options);
       setResolveReject([resolve, reject]);
     });
   }, []);
@@ -87,7 +87,7 @@ const ConfirmProvider = ({ children, defaultOptions = {} }) => {
       </ConfirmContext.Provider>
       <ConfirmationDialog
         open={resolveReject.length === 2}
-        options={options}
+        options={buildOptions(defaultOptions, options)}
         onClose={handleClose}
         onCancel={handleCancel}
         onConfirm={handleConfirm}

test/useConfirm.test.js

@@ -1,4 +1,4 @@
-import React from 'react';
+import React, { useState } from 'react';
 import { render, fireEvent, waitForElementToBeRemoved } from '@testing-library/react'
 
 import { ConfirmProvider, useConfirm } from '../src/index';
@@ -99,4 +99,29 @@ describe('useConfirm', () => {
     expect(button.disabled).toBe(true);
     expect(button.getAttribute('aria-label')).toEqual('Confirm');
   });
+
+  test('respects updates to default options', () => {
+    function App() {
+      const [confirmationText, setConfirmationText] = useState("Yes");
+
+      return (
+        <ConfirmProvider defaultOptions={{ confirmationText }}>
+          <DeleteButton />
+          <button onClick={() => setConfirmationText("Ok")}>Change text</button>
+        </ConfirmProvider>
+      )
+    }
+
+    const { getByText, queryByText } = render(<App />);
+
+    fireEvent.click(getByText('Delete'));
+
+    expect(getByText('Yes')).toBeTruthy();
+    expect(queryByText('Ok')).toBeFalsy();
+
+    fireEvent.click(getByText('Change text'));
+
+    expect(queryByText('Yes')).toBeFalsy();
+    expect(getByText('Ok')).toBeTruthy();
+  });
 });