Skip to content

Commit

Permalink
Merge branch 'main' into MAIN-B-22206
Browse files Browse the repository at this point in the history
  • Loading branch information
@danieljordan-caci
danieljordan-caci authored Jan 24, 2025
2 parents 278b1fc + cb302f4 commit 1548227
Show file tree
Hide file tree
Showing 6 changed files with 185 additions and 15 deletions.
24 changes: 23 additions & 1 deletion src/components/Office/ReviewServiceItems/ReviewAccountingCodes.jsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,20 @@ const ReviewAccountingCodesItem = ({ shipmentId, shipmentType, tac, sac, amount
);
};

// Renders Move level services such as Move management (MS) and Counseling(CS) that not associated to a shipment.
const ReviewMoveLevelServiceItems = ({ services }) => {
return (
<div className={`${styles.MoveLevelServiceItems}`}>
{services.map((item) => (
<div key={item.id}>
<div className={styles.MoveLevelServiceItemsAmount}>{toDollarString(item.amount)}</div>
<div className={styles.MoveLevelServiceItemsName}>{item.mtoServiceItemName} fee</div>
</div>
))}
</div>
);
};

ReviewAccountingCodesItem.propTypes = {
shipmentId: PropTypes.string,
shipmentType: PropTypes.string,
Expand Down Expand Up @@ -58,7 +72,13 @@ const ReviewAccountingCodes = ({ TACs, SACs, cards }) => {
}, {}),
);

if (shipments.length === 0) {
// Create move level service items. Move level service will not have an associated MTO Shipment ID
const moveLevelServices = [];
cards
.filter((card) => !card.mtoShipmentID && card.status === PAYMENT_SERVICE_ITEM_STATUS.APPROVED)
.forEach((item) => moveLevelServices.push(item));

if (shipments.length === 0 && moveLevelServices.length === 0) {
return null;
}

Expand All @@ -77,6 +97,8 @@ const ReviewAccountingCodes = ({ TACs, SACs, cards }) => {
/>
);
})}

{moveLevelServices.length > 0 && <ReviewMoveLevelServiceItems services={moveLevelServices} />}
</div>
);
};
Expand Down
33 changes: 33 additions & 0 deletions src/components/Office/ReviewServiceItems/ReviewAccountingCodes.module.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -43,3 +43,36 @@
.SAC{
word-wrap: break-word;
}

.MoveLevelServiceItems {
@include u-margin-bottom(1);
@include u-padding-y(1.5);
@include u-padding-x(1);
border: solid 1px $base-lighter;
border-left: solid 5px $base-lighter;
border-radius: 4px;
line-height: 18px;
display: flex;
flex-direction: column;
gap: 10px;

.MoveLevelServiceItemsAmount {
float: right;
}

.MoveLevelServiceItemsName {
@include u-text('bold');
@include u-text('base-darker');
}

> div:not(:last-child) {
@include u-border-bottom('1px');
@include u-border-color('base-lighter');
@include u-padding-top(.5);
@include u-padding-bottom(1.5);
}

> div:last-child {
@include u-padding-top(.5);
}
}
31 changes: 26 additions & 5 deletions src/components/Office/ReviewServiceItems/ReviewAccountingCodes.stories.jsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react';

import ReviewAccountingCodes from './ReviewAccountingCodes';

import { LOA_TYPE, SHIPMENT_OPTIONS } from 'shared/constants';
import { LOA_TYPE, SHIPMENT_OPTIONS, PAYMENT_SERVICE_ITEM_STATUS } from 'shared/constants';

export default {
title: 'Office Components/ReviewServiceItems/ReviewAccountingCodes',
Expand All @@ -14,14 +14,14 @@ const SACs = { HHG: 'AB12', NTS: 'CD34' };

const serviceItemsHHG = [
{
status: 'APPROVED',
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.HHG,
mtoShipmentID: '10',
mtoShipmentType: SHIPMENT_OPTIONS.HHG,
amount: 23.45,
},
{
status: 'APPROVED',
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.HHG,
mtoShipmentID: '10',
mtoShipmentType: SHIPMENT_OPTIONS.HHG,
Expand All @@ -31,15 +31,15 @@ const serviceItemsHHG = [

const serviceItemsNTSR = [
{
status: 'APPROVED',
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.NTS,
mtoShipmentSacType: LOA_TYPE.NTS,
mtoShipmentID: '20',
mtoShipmentType: SHIPMENT_OPTIONS.NTSR,
amount: 559,
},
{
status: 'APPROVED',
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.NTS,
mtoShipmentSacType: LOA_TYPE.NTS,
mtoShipmentID: '20',
Expand All @@ -48,8 +48,29 @@ const serviceItemsNTSR = [
},
];

const moveLevelServices = [
{
amount: 44.33,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Move management',
},
{
amount: 20.65,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Counseling',
},
];

export const withOneShipment = () => <ReviewAccountingCodes TACs={TACs} SACs={SACs} cards={[...serviceItemsHHG]} />;

export const withMultipleShipments = () => (
<ReviewAccountingCodes TACs={TACs} SACs={SACs} cards={[...serviceItemsHHG, ...serviceItemsNTSR]} />
);

export const withMultipleShipmentsAndMoveLevelServices = () => (
<ReviewAccountingCodes
TACs={TACs}
SACs={SACs}
cards={[...serviceItemsHHG, ...serviceItemsNTSR, ...moveLevelServices]}
/>
);
98 changes: 91 additions & 7 deletions src/components/Office/ReviewServiceItems/ReviewAccountingCodes.test.jsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,25 @@ import { LOA_TYPE, PAYMENT_SERVICE_ITEM_STATUS, SHIPMENT_OPTIONS } from 'shared/

describe('components/Office/ReviewServiceItems/ReviewAccountingCodes', () => {
describe('can display nothing if there are no valid service items', () => {
it('should not display service items not attached to a shipment', () => {
it('should not display service items when cards is empty', () => {
render(
<ReviewAccountingCodes TACs={{ HHG: '1234', NTS: '5678' }} SACs={{ HHG: 'AB12', NTS: 'CD34' }} cards={[]} />,
);

expect(screen.queryByRole('heading', { level: 4, name: 'Accounting codes' })).not.toBeInTheDocument();
});

it('should not display service items that are rejected', () => {
render(
<ReviewAccountingCodes
TACs={{ HHG: '1234', NTS: '5678' }}
SACs={{ HHG: 'AB12', NTS: 'CD34' }}
cards={[
{
amount: 0.01,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentID: 'X',
mtoShipmentType: SHIPMENT_OPTIONS.HHG,
status: PAYMENT_SERVICE_ITEM_STATUS.DENIED,
},
]}
/>,
Expand All @@ -24,23 +34,21 @@ describe('components/Office/ReviewServiceItems/ReviewAccountingCodes', () => {
expect(screen.queryByRole('heading', { level: 4, name: 'Accounting codes' })).not.toBeInTheDocument();
});

it('should not display service items that are rejected', () => {
it('should not display shipment card for service items not attached to a shipment', () => {
render(
<ReviewAccountingCodes
TACs={{ HHG: '1234', NTS: '5678' }}
SACs={{ HHG: 'AB12', NTS: 'CD34' }}
cards={[
{
amount: 0.01,
mtoShipmentID: 'X',
mtoShipmentType: SHIPMENT_OPTIONS.HHG,
status: PAYMENT_SERVICE_ITEM_STATUS.DENIED,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
},
]}
/>,
);

expect(screen.queryByRole('heading', { level: 4, name: 'Accounting codes' })).not.toBeInTheDocument();
expect(screen.queryByText('HHG')).not.toBeInTheDocument();
});
});

Expand Down Expand Up @@ -124,5 +132,81 @@ describe('components/Office/ReviewServiceItems/ReviewAccountingCodes', () => {
expect(within(thirdShipment).getByText('SAC: CD34 (NTS)')).toBeInTheDocument();
expect(within(thirdShipment).getByText('$30.00')).toBeInTheDocument();
});

it('can display a move level service item card', () => {
render(
<ReviewAccountingCodes
cards={[
{
amount: 44.33,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Move management',
},
{
amount: 20,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Counseling',
},
]}
/>,
);

expect(screen.getByRole('heading', { level: 4, name: 'Accounting codes' })).toBeInTheDocument();
expect(screen.getByText('Move management fee')).toBeInTheDocument();
expect(screen.getByText('$44.33')).toBeInTheDocument();
expect(screen.getByText('Counseling fee')).toBeInTheDocument();
expect(screen.getByText('$20.00')).toBeInTheDocument();
});

it('can display a move level service item card and multiple shipment cards', () => {
render(
<ReviewAccountingCodes
TACs={{ HHG: '1234', NTS: '5678' }}
SACs={{ HHG: 'AB12', NTS: 'CD34' }}
cards={[
{
amount: 44.33,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Move management',
},
{
amount: 20.65,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: 'Counseling',
},
{
amount: 10,
mtoShipmentID: '1',
mtoShipmentType: SHIPMENT_OPTIONS.HHG,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.HHG,
},
{
amount: 20,
mtoShipmentID: '2',
mtoShipmentType: SHIPMENT_OPTIONS.NTS,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoShipmentTacType: LOA_TYPE.NTS,
mtoShipmentSacType: LOA_TYPE.HHG,
},
]}
/>,
);

expect(screen.getByRole('heading', { level: 4, name: 'Accounting codes' })).toBeInTheDocument();
const firstShipment = screen.getByTestId('shipment-1');
expect(within(firstShipment).getByText('HHG')).toBeInTheDocument();
expect(within(firstShipment).queryByText('SAC: AB12 (HHG)')).not.toBeInTheDocument();
expect(within(firstShipment).getByText('$10.00')).toBeInTheDocument();
const secondShipment = screen.getByTestId('shipment-2');
expect(within(secondShipment).getByText('NTS')).toBeInTheDocument();
expect(within(secondShipment).getByText('TAC: 5678 (NTS)')).toBeInTheDocument();
expect(within(secondShipment).getByText('SAC: AB12 (HHG)')).toBeInTheDocument();
expect(within(secondShipment).getByText('$20.00')).toBeInTheDocument();
expect(screen.getByText('Move management fee')).toBeInTheDocument();
expect(screen.getByText('$44.33')).toBeInTheDocument();
expect(screen.getByText('Counseling fee')).toBeInTheDocument();
expect(screen.getByText('$20.65')).toBeInTheDocument();
});
});
});
12 changes: 11 additions & 1 deletion src/components/Office/ReviewServiceItems/ReviewServiceItems.stories.jsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -248,8 +248,18 @@ export const WithPaymentReviewedApproved = () => (
amount: 5678.05,
createdAt: '2020-01-01T00:08:00.999Z',
},
{
amount: 54.89,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: serviceItemCodes.MS,
},
{
amount: 61.32,
status: PAYMENT_SERVICE_ITEM_STATUS.APPROVED,
mtoServiceItemName: serviceItemCodes.CS,
},
]}
curCardIndex={1}
curCardIndex={3}
requestReviewed
handleClose={action('clicked')}
onCompleteReview={action('clicked')}
Expand Down
2 changes: 1 addition & 1 deletion src/shared/styles/colors.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
//Group 1: Brand Colors
$mm-blue: #252f3e;
$mm-gold: #ffbe2e;
$mm-periwinkle: #8cafea;
$mm-periwinkle: #8fb2ed;

//Group 2: Background Colors
$bg-white: #fff;
Expand Down

0 comments on commit 1548227

Please sign in to comment.