From dcebb6dd8bd426b263856a52dd23f6181baf8611 Mon Sep 17 00:00:00 2001 From: Miki Date: Thu, 18 Jul 2024 21:14:52 -0700 Subject: [PATCH] Fix unit tests Signed-off-by: Miki --- .../fatal_errors_screen.test.tsx.snap | 8 +- .../field/__snapshots__/field.test.tsx.snap | 412 ++++---- .../__snapshots__/import_flyout.test.tsx.snap | 156 +-- .../import_mode_control.test.tsx.snap | 2 +- .../components/import_mode_control.test.tsx | 2 +- .../__snapshots__/create_button.test.tsx.snap | 207 ++-- .../__snapshots__/clone_modal.test.js.snap | 10 +- .../__snapshots__/save_modal.test.js.snap | 16 +- .../query_editor/language_selector.test.tsx | 6 +- .../shard_failure_modal.test.tsx.snap | 4 +- .../__snapshots__/create_button.test.tsx.snap | 4 +- .../create_data_source_form.test.tsx.snap | 168 ++-- .../data_source_table.test.tsx.snap | 151 +-- ...ery_data_source_delete_modal.test.tsx.snap | 16 +- ...query_data_connections_table.test.tsx.snap | 20 +- ...rect_query_data_connections_table.test.tsx | 4 +- .../query_permissions.test.tsx.snap | 185 ++-- ...figure_amazon_s3_data_source.test.tsx.snap | 890 ++++++++++-------- ...review_amazon_s3_data_source.test.tsx.snap | 63 +- ...igure_prometheus_data_source.test.tsx.snap | 785 ++++++++------- ...eview_prometheus_data_source.test.tsx.snap | 63 +- .../query_permissions.test.tsx | 12 +- .../update_aws_credential_modal.test.tsx | 6 +- .../update_password_modal.test.tsx.snap | 576 ++++++------ .../open_search_panel.test.tsx.snap | 4 +- .../sample_data_view_data_button.test.js.snap | 8 +- .../__snapshots__/footer.test.js.snap | 4 +- .../instruction_set.test.js.snap | 20 +- .../__snapshots__/introduction.test.js.snap | 4 +- .../saved_objects_installer.test.js.snap | 144 +-- .../header/__snapshots__/header.test.tsx.snap | 36 +- .../advanced_options.test.tsx.snap | 16 +- .../__snapshots__/time_field.test.tsx.snap | 24 +- .../header/__snapshots__/header.test.tsx.snap | 2 +- .../__snapshots__/add_filter.test.tsx.snap | 12 +- .../components/add_filter/add_filter.test.tsx | 6 +- .../components/table/table.test.tsx | 4 +- .../__snapshots__/field_editor.test.tsx.snap | 246 ++--- .../bytes/__snapshots__/bytes.test.tsx.snap | 8 +- .../date/__snapshots__/date.test.tsx.snap | 8 +- .../__snapshots__/date_nanos.test.tsx.snap | 8 +- .../__snapshots__/duration.test.tsx.snap | 48 +- .../number/__snapshots__/number.test.tsx.snap | 8 +- .../__snapshots__/percent.test.tsx.snap | 8 +- .../__snapshots__/static_lookup.test.tsx.snap | 16 +- .../string/__snapshots__/string.test.tsx.snap | 8 +- .../__snapshots__/truncate.test.tsx.snap | 8 +- .../editors/truncate/truncate.test.tsx | 6 +- .../url/__snapshots__/url.test.tsx.snap | 112 +-- .../__snapshots__/samples.test.tsx.snap | 6 +- .../field_editor/field_editor.test.tsx | 7 + .../__snapshots__/controls_tab.test.tsx.snap | 18 +- .../list_control_editor.test.tsx.snap | 96 +- .../__snapshots__/options_tab.test.tsx.snap | 24 +- .../vis/__snapshots__/form_row.test.tsx.snap | 18 +- .../input_control_vis.test.tsx.snap | 48 +- .../__snapshots__/list_control.test.tsx.snap | 6 +- .../__snapshots__/range_control.test.tsx.snap | 1 + .../getting_started.test.tsx.snap | 8 +- .../overview_page_header.test.tsx.snap | 4 +- .../saved_object_save_modal.test.tsx.snap | 24 +- .../saved_objects_table.test.tsx.snap | 18 +- .../__snapshots__/flyout.test.tsx.snap | 32 +- .../__snapshots__/table.test.tsx.snap | 52 +- .../objects_table/components/flyout.test.tsx | 6 +- .../url_panel_content.test.tsx.snap | 62 +- .../components/url_panel_content.test.tsx | 31 +- .../__snapshots__/agg.test.tsx.snap | 2 +- .../extended_bounds.test.tsx.snap | 6 +- .../__snapshots__/number_row.test.tsx.snap | 2 +- .../number_list/number_row.test.tsx | 2 +- .../components/controls/extended_bounds.tsx | 23 +- .../components/add_delete_buttons.test.tsx | 6 +- .../components/aggs/agg_select.test.tsx | 8 +- .../components/aggs/filter_ratio.test.js | 8 +- .../__snapshots__/icon_select.test.js.snap | 4 +- .../splits/__snapshots__/terms.test.js.snap | 52 +- .../components/splits/terms.test.js | 4 + .../application/components/yes_no.test.js | 4 +- .../value_axes_panel.test.tsx.snap | 6 +- .../__snapshots__/new_vis_modal.test.tsx.snap | 54 +- .../delete_workspace_modal.test.tsx.snap | 8 +- .../workspace_fatal_error.test.tsx.snap | 4 +- .../__snapshots__/index.test.tsx.snap | 2 +- 84 files changed, 2815 insertions(+), 2375 deletions(-) diff --git a/src/core/public/fatal_errors/__snapshots__/fatal_errors_screen.test.tsx.snap b/src/core/public/fatal_errors/__snapshots__/fatal_errors_screen.test.tsx.snap index 2165a2112f24..5fd679baf7e3 100644 --- a/src/core/public/fatal_errors/__snapshots__/fatal_errors_screen.test.tsx.snap +++ b/src/core/public/fatal_errors/__snapshots__/fatal_errors_screen.test.tsx.snap @@ -18,7 +18,7 @@ exports[`FatalErrorsScreen rendering render matches snapshot 1`] = ` - , - , + @@ -39,7 +39,7 @@ exports[`FatalErrorsScreen rendering render matches snapshot 1`] = ` id="core.fatalErrors.goBackButtonLabel" values={Object {}} /> - , + , ] } body={ diff --git a/src/plugins/advanced_settings/public/management_app/components/field/__snapshots__/field.test.tsx.snap b/src/plugins/advanced_settings/public/management_app/components/field/__snapshots__/field.test.tsx.snap index 9ab66e07807d..0e08b53cc28d 100644 --- a/src/plugins/advanced_settings/public/management_app/components/field/__snapshots__/field.test.tsx.snap +++ b/src/plugins/advanced_settings/public/management_app/components/field/__snapshots__/field.test.tsx.snap @@ -28,10 +28,10 @@ exports[`Field for array setting should render as read only if saving is disable } > - - - + `; @@ -101,10 +101,10 @@ exports[`Field for array setting should render as read only with help text if ov } > - - - + `; @@ -173,10 +173,10 @@ exports[`Field for array setting should render custom setting icon if it is cust } > - - - + `; @@ -224,10 +224,10 @@ exports[`Field for array setting should render default value if there is no user } > - - - + `; @@ -292,10 +292,10 @@ exports[`Field for array setting should render unsaved value if there are unsave } > - - - + `; @@ -377,10 +377,10 @@ exports[`Field for array setting should render user value if there is user value } > - - - + `; @@ -445,10 +445,10 @@ exports[`Field for boolean setting should render as read only if saving is disab } > - - - + `; @@ -524,10 +524,10 @@ exports[`Field for boolean setting should render as read only with help text if } > - - - + `; @@ -602,10 +602,10 @@ exports[`Field for boolean setting should render custom setting icon if it is cu } > - - - + `; @@ -659,10 +659,10 @@ exports[`Field for boolean setting should render default value if there is no us } > - - - + `; @@ -733,10 +733,10 @@ exports[`Field for boolean setting should render unsaved value if there are unsa } > - - - + `; @@ -820,10 +820,10 @@ exports[`Field for boolean setting should render user value if there is user val } > - - - + `; @@ -894,10 +894,10 @@ exports[`Field for image setting should render as read only if saving is disable } > - - - + `; @@ -970,10 +970,10 @@ exports[`Field for image setting should render as read only with help text if ov } > - - + `; @@ -1040,10 +1040,10 @@ exports[`Field for image setting should render custom setting icon if it is cust } > - - - + `; @@ -1094,10 +1094,10 @@ exports[`Field for image setting should render default value if there is no user } > - - - + `; @@ -1165,10 +1165,10 @@ exports[`Field for image setting should render unsaved value if there are unsave } > - - - + `; @@ -1248,10 +1248,10 @@ exports[`Field for image setting should render user value if there is user value } > - - + `; @@ -1352,10 +1352,10 @@ exports[`Field for json setting should render as read only if saving is disabled } > - - + `; @@ -1448,10 +1448,10 @@ exports[`Field for json setting should render as read only with help text if ove } > - - + `; @@ -1540,10 +1540,10 @@ exports[`Field for json setting should render custom setting icon if it is custo } > - - + `; @@ -1636,10 +1636,10 @@ exports[`Field for json setting should render default value if there is no user } > - - + `; @@ -1741,10 +1741,10 @@ exports[`Field for json setting should render unsaved value if there are unsaved } > - - + `; @@ -1849,10 +1849,10 @@ exports[`Field for json setting should render user value if there is user value } > - - + `; @@ -1937,10 +1937,10 @@ exports[`Field for markdown setting should render as read only if saving is disa } > - - + `; @@ -2030,10 +2030,10 @@ exports[`Field for markdown setting should render as read only with help text if } > - - + `; @@ -2122,10 +2122,10 @@ exports[`Field for markdown setting should render custom setting icon if it is c } > - - + `; @@ -2193,10 +2193,10 @@ exports[`Field for markdown setting should render default value if there is no u } > - - + `; @@ -2281,10 +2281,10 @@ exports[`Field for markdown setting should render unsaved value if there are uns } > - - + `; @@ -2382,10 +2382,10 @@ exports[`Field for markdown setting should render user value if there is user va } > - - + `; @@ -2470,10 +2470,10 @@ exports[`Field for number setting should render as read only if saving is disabl } > - - - + `; @@ -2543,10 +2543,10 @@ exports[`Field for number setting should render as read only with help text if o } > - - - + `; @@ -2615,10 +2615,10 @@ exports[`Field for number setting should render custom setting icon if it is cus } > - - - + `; @@ -2666,10 +2666,10 @@ exports[`Field for number setting should render default value if there is no use } > - - - + `; @@ -2734,10 +2734,10 @@ exports[`Field for number setting should render unsaved value if there are unsav } > - - - + `; @@ -2815,10 +2815,10 @@ exports[`Field for number setting should render user value if there is user valu } > - - - + `; @@ -2883,10 +2883,10 @@ exports[`Field for select setting should render as read only if saving is disabl } > - - - + `; @@ -2972,10 +2972,10 @@ exports[`Field for select setting should render as read only with help text if o } > - - - + `; @@ -3060,10 +3060,10 @@ exports[`Field for select setting should render custom setting icon if it is cus } > - - - + `; @@ -3127,10 +3127,10 @@ exports[`Field for select setting should render default value if there is no use } > - - - + `; @@ -3211,10 +3211,10 @@ exports[`Field for select setting should render unsaved value if there are unsav } > - - - + `; @@ -3308,10 +3308,10 @@ exports[`Field for select setting should render user value if there is user valu } > - - - + `; @@ -3392,10 +3392,10 @@ exports[`Field for string setting should render as read only if saving is disabl } > - - - + `; @@ -3465,10 +3465,10 @@ exports[`Field for string setting should render as read only with help text if o } > - - - + `; @@ -3537,10 +3537,10 @@ exports[`Field for string setting should render custom setting icon if it is cus } > - - - + `; @@ -3588,10 +3588,10 @@ exports[`Field for string setting should render default value if there is no use } > - - - + `; @@ -3656,10 +3656,10 @@ exports[`Field for string setting should render unsaved value if there are unsav } > - - - + `; @@ -3737,10 +3737,10 @@ exports[`Field for string setting should render user value if there is user valu } > - - - + `; @@ -3805,10 +3805,10 @@ exports[`Field for stringWithValidation setting should render as read only if sa } > - - - + `; @@ -3878,10 +3878,10 @@ exports[`Field for stringWithValidation setting should render as read only with } > - - - + `; @@ -3950,10 +3950,10 @@ exports[`Field for stringWithValidation setting should render custom setting ico } > - - - + `; @@ -4001,10 +4001,10 @@ exports[`Field for stringWithValidation setting should render default value if t } > - - - + `; @@ -4069,10 +4069,10 @@ exports[`Field for stringWithValidation setting should render unsaved value if t } > - - - + `; @@ -4150,10 +4150,10 @@ exports[`Field for stringWithValidation setting should render user value if ther } > - - - + `; diff --git a/src/plugins/console/public/application/components/__snapshots__/import_flyout.test.tsx.snap b/src/plugins/console/public/application/components/__snapshots__/import_flyout.test.tsx.snap index bac6be2733fd..04bb8ddcff99 100644 --- a/src/plugins/console/public/application/components/__snapshots__/import_flyout.test.tsx.snap +++ b/src/plugins/console/public/application/components/__snapshots__/import_flyout.test.tsx.snap @@ -61,9 +61,9 @@ exports[`ImportFlyout Component renders correctly 1`] = `
-
-
- +
- - +
- -
- -
+
+ -
- -
- - -
+
+ +
+ + -
- -
- -
- + +
+ +
+ +
+ +
- +
diff --git a/src/plugins/console/public/application/components/__snapshots__/import_mode_control.test.tsx.snap b/src/plugins/console/public/application/components/__snapshots__/import_mode_control.test.tsx.snap index 343c6b351229..29e14c2c986d 100644 --- a/src/plugins/console/public/application/components/__snapshots__/import_mode_control.test.tsx.snap +++ b/src/plugins/console/public/application/components/__snapshots__/import_mode_control.test.tsx.snap @@ -14,7 +14,7 @@ exports[`ImportModeControl Component should render correclty 1`] = ` } } > - { let component: ShallowWrapper, React.Component<{}, {}, any>>; diff --git a/src/plugins/dashboard/public/application/components/dashboard_listing/__snapshots__/create_button.test.tsx.snap b/src/plugins/dashboard/public/application/components/dashboard_listing/__snapshots__/create_button.test.tsx.snap index 4512135a4927..1ca3fd80e890 100644 --- a/src/plugins/dashboard/public/application/components/dashboard_listing/__snapshots__/create_button.test.tsx.snap +++ b/src/plugins/dashboard/public/application/components/dashboard_listing/__snapshots__/create_button.test.tsx.snap @@ -245,7 +245,7 @@ exports[`create button with props renders button dropdown menu when two provider - + } closePopover={[Function]} display="inlineBlock" @@ -274,79 +274,89 @@ exports[`create button with props renders button dropdown menu when two provider
- - - - - + + + + +
@@ -480,78 +490,87 @@ exports[`create button with props renders single button when one provider given diff --git a/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/clone_modal.test.js.snap b/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/clone_modal.test.js.snap index 1ea30b7fe0a6..559f69752b6e 100644 --- a/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/clone_modal.test.js.snap +++ b/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/clone_modal.test.js.snap @@ -34,7 +34,7 @@ exports[`renders DashboardCloneModal 1`] = `

- - @@ -53,8 +53,8 @@ exports[`renders DashboardCloneModal 1`] = ` id="dashboard.topNav.cloneModal.cancelButtonLabel" values={Object {}} /> - - + - + `; diff --git a/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/save_modal.test.js.snap b/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/save_modal.test.js.snap index bc4ed477d9ee..727dae147a96 100644 --- a/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/save_modal.test.js.snap +++ b/src/plugins/dashboard/public/application/components/dashboard_top_nav/top_nav/__snapshots__/save_modal.test.js.snap @@ -7,9 +7,9 @@ exports[`renders DashboardSaveModal 1`] = ` onSave={[Function]} options={ - - - - + - - + } showCopyOnSave={true} diff --git a/src/plugins/data/public/ui/query_editor/language_selector.test.tsx b/src/plugins/data/public/ui/query_editor/language_selector.test.tsx index da3543deb74f..709678e29d66 100644 --- a/src/plugins/data/public/ui/query_editor/language_selector.test.tsx +++ b/src/plugins/data/public/ui/query_editor/language_selector.test.tsx @@ -8,7 +8,7 @@ import { QueryLanguageSelector } from './language_selector'; import { OpenSearchDashboardsContextProvider } from 'src/plugins/opensearch_dashboards_react/public'; import { coreMock } from '../../../../../core/public/mocks'; import { mountWithIntl } from 'test_utils/enzyme_helpers'; -import { EuiComboBox } from '@elastic/eui'; +import { EuiCompressedComboBox } from '@elastic/eui'; import { QueryEnhancement } from '../types'; const startMock = coreMock.createStart(); @@ -53,7 +53,7 @@ describe('LanguageSelector', () => { }, }) ); - const euiComboBox = component.find(EuiComboBox); + const euiComboBox = component.find(EuiCompressedComboBox); expect(euiComboBox.prop('selectedOptions')).toEqual( expect.arrayContaining([ { @@ -72,7 +72,7 @@ describe('LanguageSelector', () => { }, }) ); - const euiComboBox = component.find(EuiComboBox); + const euiComboBox = component.find(EuiCompressedComboBox); expect(euiComboBox.prop('selectedOptions')).toEqual( expect.arrayContaining([ { diff --git a/src/plugins/data/public/ui/shard_failure_modal/__snapshots__/shard_failure_modal.test.tsx.snap b/src/plugins/data/public/ui/shard_failure_modal/__snapshots__/shard_failure_modal.test.tsx.snap index f7f3d1c1fbd0..ef9d1f0fb923 100644 --- a/src/plugins/data/public/ui/shard_failure_modal/__snapshots__/shard_failure_modal.test.tsx.snap +++ b/src/plugins/data/public/ui/shard_failure_modal/__snapshots__/shard_failure_modal.test.tsx.snap @@ -173,7 +173,7 @@ exports[`ShardFailureModal renders matching snapshot given valid properties 1`] > - - + `; diff --git a/src/plugins/data_source_management/public/components/create_button/__snapshots__/create_button.test.tsx.snap b/src/plugins/data_source_management/public/components/create_button/__snapshots__/create_button.test.tsx.snap index f0e2965c6678..03c20a8093ee 100644 --- a/src/plugins/data_source_management/public/components/create_button/__snapshots__/create_button.test.tsx.snap +++ b/src/plugins/data_source_management/public/components/create_button/__snapshots__/create_button.test.tsx.snap @@ -1,7 +1,7 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[`CreateButton should render normally 1`] = ` - - + `; diff --git a/src/plugins/data_source_management/public/components/create_data_source_wizard/components/create_form/__snapshots__/create_data_source_form.test.tsx.snap b/src/plugins/data_source_management/public/components/create_data_source_wizard/components/create_form/__snapshots__/create_data_source_form.test.tsx.snap index 2bb1bd8053d7..be2a18192fe9 100644 --- a/src/plugins/data_source_management/public/components/create_data_source_wizard/components/create_form/__snapshots__/create_data_source_form.test.tsx.snap +++ b/src/plugins/data_source_management/public/components/create_data_source_wizard/components/create_form/__snapshots__/create_data_source_form.test.tsx.snap @@ -1,8 +1,8 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[`Datasource Management: Create Datasource form with different authType configurations should render normally with all authMethod combinations 1`] = ` -