Skip to content

Commit 93e3382

Browse files
committed
test(kbn-elastic-assistant): remove redundant comment
1 parent ec83b03 commit 93e3382

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

  • x-pack/platform/packages/shared/kbn-elastic-assistant/impl/assistant/common/components/assistant_settings_management/flyout

x-pack/platform/packages/shared/kbn-elastic-assistant/impl/assistant/common/components/assistant_settings_management/flyout/index.test.tsx

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,8 @@ describe('Assistant settings flyout', () => {
4040
</Flyout>
4141
);
4242

43-
// Here we check that EuiFlyout was called with the correct aria-label and trust that EuiFlyout sets it correctly
44-
// manual testing with a screen reader confirms this works as expected
45-
// see https://eui.elastic.co/docs/components/containers/flyout/#props for aria-label usage
46-
expect(mockedEuiFlyout).toHaveBeenCalledWith(
47-
expect.objectContaining({ 'aria-label': title }),
48-
expect.anything()
49-
);
43+
const firstCallProps = mockedEuiFlyout.mock.calls[0]?.[0];
44+
expect(firstCallProps?.['aria-label']).toBe(title);
5045
});
5146

5247
it('does not pass aria-label when the title is missing', () => {

0 commit comments

Comments
 (0)