Skip to content

Commit 584b203

Browse files
authored
Revert "refactor(checkout): CHECKOUT-9476 Convert Payment Components 1 (#2603)" (#2608)
This reverts commit 32a619c.
1 parent 76de7e1 commit 584b203

21 files changed

+1598
-1377
lines changed

packages/core/src/app/payment/Payment/Payment.test.tsx renamed to packages/core/src/app/payment/Payment.test.tsx

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import { rest } from 'msw';
1010
import React, { act, type FunctionComponent } from 'react';
1111

1212
import {
13+
type AnalyticsContextProps,
1314
type AnalyticsEvents,
1415
AnalyticsProviderMock,
1516
} from '@bigcommerce/checkout/analytics';
@@ -29,18 +30,18 @@ import {
2930
import { renderWithoutWrapper as render, screen } from '@bigcommerce/checkout/test-utils';
3031
import { ThemeProvider } from '@bigcommerce/checkout/ui';
3132

32-
import Checkout, { type CheckoutProps } from '../../checkout/Checkout';
33-
import { createErrorLogger } from '../../common/error';
33+
import Checkout, { type CheckoutProps } from '../checkout/Checkout';
34+
import { createErrorLogger } from '../common/error';
3435
import {
3536
createEmbeddedCheckoutStylesheet,
3637
createEmbeddedCheckoutSupport,
37-
} from '../../embeddedCheckout';
38+
} from '../embeddedCheckout';
3839

3940
describe('Payment step', () => {
4041
let checkout: CheckoutPageNodeObject;
4142
let CheckoutTest: FunctionComponent<CheckoutProps>;
4243
let checkoutService: CheckoutService;
43-
let defaultProps: CheckoutProps;
44+
let defaultProps: CheckoutProps & AnalyticsContextProps;
4445
let embeddedMessengerMock: EmbeddedCheckoutMessenger;
4546
let analyticsTracker: Partial<AnalyticsEvents>;
4647

@@ -77,14 +78,15 @@ describe('Payment step', () => {
7778
embeddedStylesheet: createEmbeddedCheckoutStylesheet(),
7879
embeddedSupport: createEmbeddedCheckoutSupport(getLanguageService()),
7980
errorLogger: createErrorLogger(),
81+
analyticsTracker,
8082
};
8183

8284
jest.spyOn(defaultProps.errorLogger, 'log').mockImplementation(noop);
8385

8486
CheckoutTest = (props) => (
8587
<CheckoutProvider checkoutService={checkoutService}>
8688
<LocaleProvider checkoutService={checkoutService}>
87-
<AnalyticsProviderMock analyticsTracker={analyticsTracker}>
89+
<AnalyticsProviderMock>
8890
<ExtensionProvider
8991
checkoutService={checkoutService}
9092
errorLogger={{
@@ -159,8 +161,8 @@ describe('Payment step', () => {
159161

160162
await checkout.waitForPaymentStep();
161163

162-
await act(async () => userEvent.click(screen.getByRole('radio', { name: 'Pay in Store' })));
163-
await act(async () => userEvent.click(screen.getAllByRole('button', { name: 'Edit' })[2]));
164+
await userEvent.click(screen.getByRole('radio', { name: 'Pay in Store' }));
165+
await userEvent.click(screen.getAllByRole('button', { name: 'Edit' })[2]);
164166

165167
expect(screen.queryByRole('radio')).not.toBeInTheDocument();
166168
expect(screen.queryByText('Pay in Store')).not.toBeInTheDocument();
@@ -305,11 +307,11 @@ describe('Payment step', () => {
305307
render(<CheckoutTest {...defaultProps} />);
306308

307309
await checkout.waitForPaymentStep();
308-
await act(async () => userEvent.click(screen.getByText('Place Order')));
310+
await userEvent.click(screen.getByText('Place Order'));
309311

310312
expect(screen.getByText('Something\'s gone wrong')).toBeInTheDocument();
311313

312-
await act(async () => userEvent.click(screen.getByText('Ok')));
314+
await userEvent.click(screen.getByText('Ok'));
313315

314316
expect(screen.queryByText('Something\'s gone wrong')).not.toBeInTheDocument();
315317
});

0 commit comments

Comments
 (0)