From c77af6e5f06db954814dab92ef88baf0a02ce5cb Mon Sep 17 00:00:00 2001 From: Cee Chen Date: Thu, 5 Oct 2023 10:31:24 -0700 Subject: [PATCH] This time, for real, fix combobox CI failures in main - Apologies for the shenanigans y'all, should've made this a PR and not assumed I knew what I was doing by pushing a quick fix to main --- .../__snapshots__/combo_box.test.tsx.snap | 7 ----- src/components/combo_box/combo_box.test.tsx | 29 ++++++++++++------- 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/src/components/combo_box/__snapshots__/combo_box.test.tsx.snap b/src/components/combo_box/__snapshots__/combo_box.test.tsx.snap index 43288f4a70e..a1e7f5f4028 100644 --- a/src/components/combo_box/__snapshots__/combo_box.test.tsx.snap +++ b/src/components/combo_box/__snapshots__/combo_box.test.tsx.snap @@ -96,13 +96,6 @@ exports[`EuiComboBox renders the options list dropdown 1`] = ` style="inline-size: 2px;" value="" /> - - Combo box. Selected. Combo box input. Type some text or, to display a list of choices, press Down Arrow. To exit the list of choices, press Escape. -
{ render(); }); - it('renders the options list dropdown', async () => { - const { baseElement } = render( - - ); - await showEuiComboBoxOptions(); + // React 16 for some reason doesn't snapshot the screen reader text + testOnReactVersion(['17', '18'])( + 'renders the options list dropdown', + async () => { + const { baseElement } = render( + + ); + await showEuiComboBoxOptions(); - expect(baseElement).toMatchSnapshot(); - }); + expect(baseElement).toMatchSnapshot(); + } + ); it('renders selectedOptions as pills', () => { const { getAllByTestSubject } = render(