Skip to content

Commit cefafea

Browse files
rusackasclaude
andcommitted
test: retry-wrap all array-button assertions consistently (bito suggestion)
The move-button assertions ran outside the waitFor guarding the remove buttons; same render pass so unlikely to flake, but consistent retry wrapping removes the risk entirely. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 1fe374f commit cefafea

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

superset-frontend/src/SqlLab/components/ScheduleQueryButton/ScheduleQueryButton.test.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -127,11 +127,11 @@ test('array fields render add/reorder/remove buttons with the rjsf 6 classes', a
127127
expect(
128128
document.querySelectorAll('button.rjsf-array-item-remove'),
129129
).toHaveLength(2);
130+
expect(
131+
document.querySelector('button.rjsf-array-item-move-up'),
132+
).toBeInTheDocument();
133+
expect(
134+
document.querySelector('button.rjsf-array-item-move-down'),
135+
).toBeInTheDocument();
130136
});
131-
expect(
132-
document.querySelector('button.rjsf-array-item-move-up'),
133-
).toBeInTheDocument();
134-
expect(
135-
document.querySelector('button.rjsf-array-item-move-down'),
136-
).toBeInTheDocument();
137137
});

0 commit comments

Comments
 (0)