Skip to content

Commit

Permalink
fix(order): CHECKOUT-4575 Display different message if order is await…
Browse files Browse the repository at this point in the history
…ing payment
  • Loading branch information
davidchin committed Feb 24, 2020
1 parent 309772c commit 930472c
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 2 deletions.
18 changes: 18 additions & 0 deletions src/app/order/OrderStatus.spec.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ import { Order } from '@bigcommerce/checkout-sdk';
import { shallow, ShallowWrapper } from 'enzyme';
import React from 'react';

import { TranslatedString } from '../locale';

import { getOrder } from './orders.mock';
import OrderStatus from './OrderStatus';

Expand Down Expand Up @@ -57,6 +59,22 @@ describe('OrderStatus', () => {
});
});

describe('when order is awaiting payment', () => {
beforeEach(() => {
orderStatus = shallow(<OrderStatus
order={ {
...getOrder(),
status: 'AWAITING_PAYMENT',
} }
/>);
});

it('displays order is pending text', () => {
expect(orderStatus.find(TranslatedString).prop('id'))
.toEqual('order_confirmation.order_pending_review_text');
});
});

describe('when order has digital items', () => {
beforeEach(() => {
order = {
Expand Down
5 changes: 3 additions & 2 deletions src/app/order/OrderStatus.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ const OrderStatus: FunctionComponent<OrderStatusProps> = ({
supportPhoneNumber,
}) => {
const isPendingReview = order.status === 'MANUAL_VERIFICATION_REQUIRED';
const isAwaitingPayment = order.status === 'AWAITING_PAYMENT';
const orderNumber = order.orderId;

return <OrderConfirmationSection>
Expand All @@ -28,14 +29,14 @@ const OrderStatus: FunctionComponent<OrderStatusProps> = ({
/>
</p> }

{ isPendingReview &&
{ (isPendingReview || isAwaitingPayment) &&
<p>
<TranslatedString
id="order_confirmation.order_pending_review_text"
/>
</p> }

{ !isPendingReview &&
{ !(isPendingReview || isAwaitingPayment) &&
<p>
<TranslatedHtml
data={ { orderNumber, supportEmail, supportPhoneNumber } }
Expand Down

0 comments on commit 930472c

Please sign in to comment.