feat(payment): PAYPAL-4995 added check for teardown call in deinitial…

…ize method
bigcommerce · Jan 8, 2025 · 1e16475 · 1e16475
1 parent e050766
commit 1e16475
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/packages/core/src/payment/strategies/braintree/braintree-hosted-form.spec.ts b/packages/core/src/payment/strategies/braintree/braintree-hosted-form.spec.ts
@@ -206,6 +206,16 @@ describe('BraintreeHostedForm', () => {
 
             expect(cardFields.teardown).toHaveBeenCalled();
         });
+
+        it('do not call teardown if fields are not initialized', async () => {
+            await subject.initialize({
+                ...formOptions,
+                fields: {},
+            });
+            await subject.deinitialize();
+
+            expect(cardFields.teardown).not.toHaveBeenCalled();
+        });
     });
 
     describe('#tokenize', () => {

diff --git a/packages/core/src/payment/strategies/braintree/braintree-hosted-form.ts b/packages/core/src/payment/strategies/braintree/braintree-hosted-form.ts
@@ -75,9 +75,11 @@ export default class BraintreeHostedForm {
     }
 
     async deinitialize(): Promise<void> {
-        this._isInitializedHostedForm = false;
+        if (this._isInitializedHostedForm) {
+            this._isInitializedHostedForm = false;
 
-        await this._cardFields?.teardown();
+            await this._cardFields?.teardown();
+        }
     }
 
     validate() {