diff --git a/docs/development/plugins/data/public/kibana-plugin-plugins-data-public.searchbar.md b/docs/development/plugins/data/public/kibana-plugin-plugins-data-public.searchbar.md index 0d5e0e42af27f..b015ebfcbaada 100644 --- a/docs/development/plugins/data/public/kibana-plugin-plugins-data-public.searchbar.md +++ b/docs/development/plugins/data/public/kibana-plugin-plugins-data-public.searchbar.md @@ -7,7 +7,7 @@ Signature: ```typescript -SearchBar: React.ComponentClass, "query" | "isLoading" | "filters" | "indexPatterns" | "refreshInterval" | "customSubmitButton" | "screenTitle" | "dataTestSubj" | "showQueryBar" | "showQueryInput" | "showFilterBar" | "showDatePicker" | "showAutoRefreshOnly" | "isRefreshPaused" | "dateRangeFrom" | "dateRangeTo" | "showSaveQuery" | "savedQuery" | "onQueryChange" | "onQuerySubmit" | "onSaved" | "onSavedQueryUpdated" | "onClearSavedQuery" | "onRefresh" | "timeHistory" | "onFiltersUpdated" | "onRefreshChange">, any> & { - WrappedComponent: React.ComponentType & ReactIntl.InjectedIntlProps>; +SearchBar: React.ComponentClass, "query" | "isLoading" | "filters" | "onRefresh" | "onRefreshChange" | "refreshInterval" | "indexPatterns" | "customSubmitButton" | "screenTitle" | "dataTestSubj" | "showQueryBar" | "showQueryInput" | "showFilterBar" | "showDatePicker" | "showAutoRefreshOnly" | "isRefreshPaused" | "dateRangeFrom" | "dateRangeTo" | "showSaveQuery" | "savedQuery" | "onQueryChange" | "onQuerySubmit" | "onSaved" | "onSavedQueryUpdated" | "onClearSavedQuery" | "timeHistory" | "onFiltersUpdated">, any> & { + WrappedComponent: React.ComponentType & ReactIntl.InjectedIntlProps>; } ``` diff --git a/package.json b/package.json index 634720d6d3a2f..901b30ab5efc5 100644 --- a/package.json +++ b/package.json @@ -126,7 +126,7 @@ "@elastic/charts": "19.2.0", "@elastic/datemath": "5.0.3", "@elastic/ems-client": "7.8.0", - "@elastic/eui": "22.3.1", + "@elastic/eui": "23.3.1", "@elastic/filesaver": "1.1.2", "@elastic/good": "8.1.1-kibana2", "@elastic/numeral": "2.4.0", diff --git a/packages/kbn-ui-shared-deps/package.json b/packages/kbn-ui-shared-deps/package.json index 8259f251a9be3..eddc215db5a62 100644 --- a/packages/kbn-ui-shared-deps/package.json +++ b/packages/kbn-ui-shared-deps/package.json @@ -10,7 +10,7 @@ }, "dependencies": { "@elastic/charts": "19.2.0", - "@elastic/eui": "22.3.1", + "@elastic/eui": "23.3.1", "@kbn/i18n": "1.0.0", "abortcontroller-polyfill": "^1.4.0", "angular": "^1.7.9", diff --git a/src/core/public/chrome/ui/header/__snapshots__/collapsible_nav.test.tsx.snap b/src/core/public/chrome/ui/header/__snapshots__/collapsible_nav.test.tsx.snap index 3e3d8c422b978..866ea5f45d986 100644 --- a/src/core/public/chrome/ui/header/__snapshots__/collapsible_nav.test.tsx.snap +++ b/src/core/public/chrome/ui/header/__snapshots__/collapsible_nav.test.tsx.snap @@ -275,6 +275,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = `
@@ -469,6 +471,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="discover" + rel="noreferrer" > @@ -588,6 +592,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="metrics" + rel="noreferrer" > @@ -691,6 +696,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="siem" + rel="noreferrer" >
@@ -1108,6 +1118,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="discover" + rel="noreferrer" > @@ -1227,6 +1239,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="metrics" + rel="noreferrer" > @@ -1330,6 +1343,7 @@ exports[`CollapsibleNav renders links grouped by category 1`] = ` class="euiListGroupItem__button" data-test-subj="collapsibleNavAppLink" href="siem" + rel="noreferrer" >
() => `generated-id`); +jest.mock('@elastic/eui/lib/services/accessibility/html_id_generator', () => { + return { + htmlIdGenerator: () => () => `generated-id`, + }; +}); describe('AddLicense component when license is active', () => { test('should display correct verbiage', () => { diff --git a/x-pack/plugins/license_management/__jest__/request_trial_extension.test.js b/x-pack/plugins/license_management/__jest__/request_trial_extension.test.js index 6d5a9fdd3fb38..192f496e41a27 100644 --- a/x-pack/plugins/license_management/__jest__/request_trial_extension.test.js +++ b/x-pack/plugins/license_management/__jest__/request_trial_extension.test.js @@ -6,7 +6,11 @@ import { RequestTrialExtension } from '../public/application/sections/license_dashboard/request_trial_extension'; import { createMockLicense, getComponent } from './util'; -jest.mock(`@elastic/eui/lib/components/form/form_row/make_id`, () => () => `generated-id`); +jest.mock('@elastic/eui/lib/services/accessibility/html_id_generator', () => { + return { + htmlIdGenerator: () => () => `generated-id`, + }; +}); describe('RequestTrialExtension component', () => { test('should not display when license is active and trial has not been used', () => { diff --git a/x-pack/plugins/license_management/__jest__/revert_to_basic.test.js b/x-pack/plugins/license_management/__jest__/revert_to_basic.test.js index c223c39a8f12c..7639d7ad1a1a4 100644 --- a/x-pack/plugins/license_management/__jest__/revert_to_basic.test.js +++ b/x-pack/plugins/license_management/__jest__/revert_to_basic.test.js @@ -6,7 +6,11 @@ import { RevertToBasic } from '../public/application/sections/license_dashboard/revert_to_basic'; import { createMockLicense, getComponent } from './util'; -jest.mock(`@elastic/eui/lib/components/form/form_row/make_id`, () => () => `generated-id`); +jest.mock('@elastic/eui/lib/services/accessibility/html_id_generator', () => { + return { + htmlIdGenerator: () => () => `generated-id`, + }; +}); describe('RevertToBasic component', () => { test('should display when trial is active', () => { diff --git a/x-pack/plugins/license_management/__jest__/start_trial.test.js b/x-pack/plugins/license_management/__jest__/start_trial.test.js index 5bd005bc1adbd..e4c5436981edf 100644 --- a/x-pack/plugins/license_management/__jest__/start_trial.test.js +++ b/x-pack/plugins/license_management/__jest__/start_trial.test.js @@ -7,7 +7,11 @@ import { StartTrial } from '../public/application/sections/license_dashboard/start_trial'; import { createMockLicense, getComponent } from './util'; -jest.mock(`@elastic/eui/lib/components/form/form_row/make_id`, () => () => `generated-id`); +jest.mock('@elastic/eui/lib/services/accessibility/html_id_generator', () => { + return { + htmlIdGenerator: () => () => `generated-id`, + }; +}); describe('StartTrial component when trial is allowed', () => { test('display for basic license', () => { diff --git a/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_enabled/__tests__/__snapshots__/collection_enabled.test.js.snap b/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_enabled/__tests__/__snapshots__/collection_enabled.test.js.snap index fa266b30fcb94..f891aab34c4a1 100644 --- a/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_enabled/__tests__/__snapshots__/collection_enabled.test.js.snap +++ b/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_enabled/__tests__/__snapshots__/collection_enabled.test.js.snap @@ -187,58 +187,24 @@ exports[`ExplainCollectionEnabled should explain about xpack.monitoring.collecti We checked the cluster settings and found that - - xpack.monitoring.collection.enabled -
- } - > - xpack.monitoring.collection.enabled - - - + + + xpack.monitoring.collection.enabled + is set to - - -1 -
- } - > - -1 - - - + + + -1 + diff --git a/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_interval/__tests__/__snapshots__/collection_interval.test.js.snap b/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_interval/__tests__/__snapshots__/collection_interval.test.js.snap index 8b3675bf32753..49cbe092e0e20 100644 --- a/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_interval/__tests__/__snapshots__/collection_interval.test.js.snap +++ b/x-pack/plugins/monitoring/public/components/no_data/explanations/collection_interval/__tests__/__snapshots__/collection_interval.test.js.snap @@ -360,58 +360,24 @@ exports[`ExplainCollectionInterval collection interval setting updates should sh We checked the cluster settings and found that - - xpack.monitoring.collection.interval - - } - > - xpack.monitoring.collection.interval - - - + + + xpack.monitoring.collection.interval + is set to - - -1 - - } - > - -1 - - - + + + -1 + @@ -692,58 +658,24 @@ exports[`ExplainCollectionInterval should explain about xpack.monitoring.collect We checked the cluster settings and found that - - xpack.monitoring.collection.interval - - } - > - xpack.monitoring.collection.interval - - - + + + xpack.monitoring.collection.interval + is set to - - -1 - - } - > - -1 - - - + + + -1 + diff --git a/x-pack/plugins/monitoring/public/components/no_data/explanations/exporters/__tests__/exporters.test.js b/x-pack/plugins/monitoring/public/components/no_data/explanations/exporters/__tests__/exporters.test.js index 88f1eedfc3bcc..44b97bc807ce9 100644 --- a/x-pack/plugins/monitoring/public/components/no_data/explanations/exporters/__tests__/exporters.test.js +++ b/x-pack/plugins/monitoring/public/components/no_data/explanations/exporters/__tests__/exporters.test.js @@ -8,19 +8,6 @@ import React from 'react'; import { renderWithIntl } from 'test_utils/enzyme_helpers'; import { ExplainExporters, ExplainExportersCloud } from '../exporters'; -// Mocking to prevent errors with React portal. -// Temporary until https://github.com/elastic/kibana/pull/55877 provides other alternatives. -jest.mock('@elastic/eui/lib/components/code/code', () => { - const React = require.requireActual('react'); - return { - EuiCode: ({ children }) => ( - - {children} - - ), - }; -}); - describe('ExplainExporters', () => { test('should explain about xpack.monitoring.exporters setting', () => { const reason = { diff --git a/x-pack/plugins/monitoring/public/components/no_data/explanations/plugin_enabled/__tests__/plugin_enabled.test.js b/x-pack/plugins/monitoring/public/components/no_data/explanations/plugin_enabled/__tests__/plugin_enabled.test.js index ece58ad6aeed2..031c4baa0a424 100644 --- a/x-pack/plugins/monitoring/public/components/no_data/explanations/plugin_enabled/__tests__/plugin_enabled.test.js +++ b/x-pack/plugins/monitoring/public/components/no_data/explanations/plugin_enabled/__tests__/plugin_enabled.test.js @@ -8,19 +8,6 @@ import React from 'react'; import { renderWithIntl } from 'test_utils/enzyme_helpers'; import { ExplainPluginEnabled } from '../plugin_enabled'; -// Mocking to prevent errors with React portal. -// Temporary until https://github.com/elastic/kibana/pull/55877 provides other alternatives. -jest.mock('@elastic/eui/lib/components/code/code', () => { - const React = require.requireActual('react'); - return { - EuiCode: ({ children }) => ( - - {children} - - ), - }; -}); - describe('ExplainPluginEnabled', () => { test('should explain about xpack.monitoring.enabled setting', () => { const reason = { diff --git a/x-pack/plugins/monitoring/public/components/no_data/reasons/__tests__/reason_found.test.js b/x-pack/plugins/monitoring/public/components/no_data/reasons/__tests__/reason_found.test.js index 7f15cacb1ebb9..a5f9691880b2c 100644 --- a/x-pack/plugins/monitoring/public/components/no_data/reasons/__tests__/reason_found.test.js +++ b/x-pack/plugins/monitoring/public/components/no_data/reasons/__tests__/reason_found.test.js @@ -8,19 +8,6 @@ import React from 'react'; import { renderWithIntl } from 'test_utils/enzyme_helpers'; import { ReasonFound } from '../'; -// Mocking to prevent errors with React portal. -// Temporary until https://github.com/elastic/kibana/pull/55877 provides other alternatives. -jest.mock('@elastic/eui/lib/components/code/code', () => { - const React = require.requireActual('react'); - return { - EuiCode: ({ children }) => ( - - {children} - - ), - }; -}); - const enabler = {}; describe('ReasonFound', () => { diff --git a/x-pack/plugins/remote_clusters/public/application/sections/components/remote_cluster_form/__snapshots__/remote_cluster_form.test.js.snap b/x-pack/plugins/remote_clusters/public/application/sections/components/remote_cluster_form/__snapshots__/remote_cluster_form.test.js.snap index 35c566548f158..bcc9a3f14d48b 100644 --- a/x-pack/plugins/remote_clusters/public/application/sections/components/remote_cluster_form/__snapshots__/remote_cluster_form.test.js.snap +++ b/x-pack/plugins/remote_clusters/public/application/sections/components/remote_cluster_form/__snapshots__/remote_cluster_form.test.js.snap @@ -251,7 +251,7 @@ exports[`RemoteClusterForm proxy mode renders correct connection settings when u
Use proxy mode @@ -1509,7 +1509,7 @@ Array [
Seed nodes @@ -1526,7 +1526,7 @@ Array [ class="euiFormRow__fieldWrapper" >