From c22ae09f380aadcc38ca6c79083e3df63a6f885f Mon Sep 17 00:00:00 2001
From: Polo Li <65737086+pololi-stripe@users.noreply.github.com>
Date: Wed, 9 Oct 2024 16:14:23 -0700
Subject: [PATCH] Add onLineItemsChange option to EmbeddedCheckoutProvider
 (#536)

---
 package.json                                  |  2 +-
 .../EmbeddedCheckoutProvider.test.tsx         | 31 +++++++++++++++++++
 src/components/EmbeddedCheckoutProvider.tsx   | 12 +++++++
 yarn.lock                                     |  8 ++---
 4 files changed, 48 insertions(+), 5 deletions(-)

diff --git a/package.json b/package.json
index 2adc9fc..2ef44aa 100644
--- a/package.json
+++ b/package.json
@@ -69,7 +69,7 @@
     "@rollup/plugin-replace": "^5.0.5",
     "@rollup/plugin-terser": "^0.4.4",
     "@storybook/react": "^6.5.0-beta.8",
-    "@stripe/stripe-js": "^4.3.0",
+    "@stripe/stripe-js": "^4.8.0",
     "@testing-library/jest-dom": "^5.16.4",
     "@testing-library/react": "^13.1.1",
     "@testing-library/react-hooks": "^8.0.0",
diff --git a/src/components/EmbeddedCheckoutProvider.test.tsx b/src/components/EmbeddedCheckoutProvider.test.tsx
index 1001719..ac4de1f 100644
--- a/src/components/EmbeddedCheckoutProvider.test.tsx
+++ b/src/components/EmbeddedCheckoutProvider.test.tsx
@@ -414,6 +414,37 @@ describe('EmbeddedCheckoutProvider', () => {
     );
   });
 
+  it('does not allow changes to onLineItemsChange option', async () => {
+    const optionsProp1 = {
+      fetchClientSecret,
+      onLineItemsChange: () => Promise.resolve({type: 'accept' as const}),
+    };
+    const optionsProp2 = {
+      fetchClientSecret,
+      onLineItemsChange: () => Promise.resolve({type: 'reject' as const}),
+    };
+
+    // Silence console output so test output is less noisy
+    consoleWarn.mockImplementation(() => {});
+    const {rerender} = render(
+      <EmbeddedCheckoutProvider
+        stripe={mockStripe}
+        options={optionsProp1}
+      ></EmbeddedCheckoutProvider>
+    );
+    await act(() => mockEmbeddedCheckoutPromise);
+
+    rerender(
+      <EmbeddedCheckoutProvider
+        stripe={mockStripe}
+        options={optionsProp2}
+      ></EmbeddedCheckoutProvider>
+    );
+    expect(consoleWarn).toHaveBeenCalledWith(
+      'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onLineItemsChange option after setting it.'
+    );
+  });
+
   it('destroys Embedded Checkout when the component unmounts', async () => {
     const {rerender} = render(
       <div>
diff --git a/src/components/EmbeddedCheckoutProvider.tsx b/src/components/EmbeddedCheckoutProvider.tsx
index c33137b..2a25030 100644
--- a/src/components/EmbeddedCheckoutProvider.tsx
+++ b/src/components/EmbeddedCheckoutProvider.tsx
@@ -57,6 +57,9 @@ interface EmbeddedCheckoutProviderProps {
     onShippingDetailsChange?: (
       event: stripeJs.StripeEmbeddedCheckoutShippingDetailsChangeEvent
     ) => Promise<stripeJs.ResultAction>;
+    onLineItemsChange?: (
+      event: stripeJs.StripeEmbeddedCheckoutLineItemsChangeEvent
+    ) => Promise<stripeJs.ResultAction>;
   };
 }
 
@@ -220,6 +223,15 @@ export const EmbeddedCheckoutProvider: FunctionComponent<PropsWithChildren<
         'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onShippingDetailsChange option after setting it.'
       );
     }
+
+    if (
+      prevOptions.onLineItemsChange != null &&
+      options.onLineItemsChange !== prevOptions.onLineItemsChange
+    ) {
+      console.warn(
+        'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onLineItemsChange option after setting it.'
+      );
+    }
   }, [prevOptions, options]);
 
   return (
diff --git a/yarn.lock b/yarn.lock
index 17ecbeb..dceff8f 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -2350,10 +2350,10 @@
     regenerator-runtime "^0.13.7"
     resolve-from "^5.0.0"
 
-"@stripe/stripe-js@^4.3.0":
-  version "4.3.0"
-  resolved "https://registry.yarnpkg.com/@stripe/stripe-js/-/stripe-js-4.3.0.tgz#5e8fe1e654413698f33cbc7008fef1e86e16b030"
-  integrity sha512-bf8MxzzgD3dybtyIJUQSDMqxjEkJfsmj9IdRqDv609Zw08R41O7eoIy0f8KY41u8MbaFOYsn+XGJZtg1xwR2wQ==
+"@stripe/stripe-js@^4.8.0":
+  version "4.8.0"
+  resolved "https://registry.yarnpkg.com/@stripe/stripe-js/-/stripe-js-4.8.0.tgz#ba78f775347cb30e93e13aa408b345de15e3ff80"
+  integrity sha512-+4Cb0bVHlV4BJXxkJ3cCLSLuWxm3pXKtgcRacox146EuugjCzRRII5T5gUMgL4HpzrBLVwVxjKaZqntNWAXawQ==
 
 "@testing-library/dom@^8.5.0":
   version "8.13.0"