Skip to content

Commit 1572326

Browse files
committed
Make customUnitRateID optional and apply Prettier to tests
The duplicate-policy clone helper already treated newDefaultRateID as optional at runtime; mirror that in the parameter type so callers that don't have a rate ID (existing unit tests for per-diem-only paths) can pass through without an explicit pass-through. No production-call-site behavior change — buildDuplicatePolicyData still always provides it.
1 parent 3ce4e8b commit 1572326

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

src/libs/PolicyUtils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,7 @@ function getCustomUnitsForDuplication(
335335
customUnitIDs: {
336336
distanceCustomUnitID: string;
337337
perDiemCustomUnitID: string;
338-
customUnitRateID: string;
338+
customUnitRateID?: string;
339339
},
340340
): Record<string, CustomUnit> | undefined {
341341
const customUnits = policy?.customUnits;

tests/unit/PolicyUtilsTest.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -370,7 +370,11 @@ describe('PolicyUtils', () => {
370370
...createRandomPolicy(0),
371371
customUnits: {[distanceUnitWithMultipleRates.customUnitID]: distanceUnitWithMultipleRates},
372372
};
373-
const result = getCustomUnitsForDuplication(policyWithMultipleRates, true, false, {distanceCustomUnitID: 'newDist', perDiemCustomUnitID: 'newPerDiem', customUnitRateID: 'newRate'});
373+
const result = getCustomUnitsForDuplication(policyWithMultipleRates, true, false, {
374+
distanceCustomUnitID: 'newDist',
375+
perDiemCustomUnitID: 'newPerDiem',
376+
customUnitRateID: 'newRate',
377+
});
374378
expect(result).toEqual({
375379
newDist: {
376380
...distanceUnitWithMultipleRates,

0 commit comments

Comments
 (0)