Skip to content

Commit 8fed29c

Browse files
committed
fix coverage
1 parent cfdacd3 commit 8fed29c

2 files changed

Lines changed: 45 additions & 1 deletion

File tree

src/views/domain-schedules/domain-schedules-create-advanced-form/__tests__/domain-schedules-create-advanced-form.test.tsx

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,18 +116,40 @@ describe(DomainSchedulesCreateAdvancedForm.name, () => {
116116
await user.click(screen.getByRole('radio', { name: 'Skip' }));
117117
expect(screen.queryByLabelText('Catch-up window')).not.toBeInTheDocument();
118118
});
119+
120+
it('clears schedule id and workflow id prefix when inputs are emptied', async () => {
121+
const { user, getValues } = setup();
122+
123+
await user.click(
124+
screen.getByRole('button', { name: /show advanced configurations/i })
125+
);
126+
127+
const scheduleId = screen.getByLabelText('Schedule Id');
128+
await user.type(scheduleId, 'my-schedule');
129+
await user.clear(scheduleId);
130+
expect(getValues().scheduleId).toBeUndefined();
131+
132+
const workflowIdPrefix = screen.getByLabelText('Workflow Id Prefix');
133+
await user.type(workflowIdPrefix, 'prefix');
134+
await user.clear(workflowIdPrefix);
135+
expect(getValues().workflowIdPrefix).toBeUndefined();
136+
});
119137
});
120138

121139
function setup() {
122140
const user = userEvent.setup();
141+
let getValues: () => DomainSchedulesCreateFormData;
123142

124143
function Wrapper() {
125144
const {
126145
control,
146+
getValues: readValues,
127147
formState: { errors: fieldErrors },
128148
} = useForm<DomainSchedulesCreateFormData>({
129149
defaultValues: {},
130150
});
151+
getValues = readValues;
152+
131153
return (
132154
<DomainSchedulesCreateAdvancedForm
133155
control={control}
@@ -137,5 +159,5 @@ function setup() {
137159
}
138160

139161
render(<Wrapper />);
140-
return { user };
162+
return { user, getValues: () => getValues() };
141163
}

src/views/domain-schedules/domain-schedules-create-modal/__tests__/transform-domain-schedules-create-form-to-body.test.ts

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,4 +134,26 @@ describe(transformDomainSchedulesCreateFormToBody.name, () => {
134134
);
135135
expect(result.catchUpWindowSeconds).toBeUndefined();
136136
});
137+
138+
it('omits catchUpPolicy when not set on form data', () => {
139+
const result = transformDomainSchedulesCreateFormToBody(baseForm);
140+
141+
expect(result.catchUpPolicy).toBeUndefined();
142+
});
143+
144+
it('omits buffer and concurrency limits when limit strings are empty', () => {
145+
const bufferResult = transformDomainSchedulesCreateFormToBody({
146+
...baseForm,
147+
overlapPolicy: ScheduleOverlapPolicy.SCHEDULE_OVERLAP_POLICY_BUFFER,
148+
bufferLimit: '',
149+
});
150+
expect(bufferResult.bufferLimit).toBeUndefined();
151+
152+
const concurrentResult = transformDomainSchedulesCreateFormToBody({
153+
...baseForm,
154+
overlapPolicy: ScheduleOverlapPolicy.SCHEDULE_OVERLAP_POLICY_CONCURRENT,
155+
concurrencyLimit: '',
156+
});
157+
expect(concurrentResult.concurrencyLimit).toBeUndefined();
158+
});
137159
});

0 commit comments

Comments
 (0)