Merge branch '7.17' into backport/7.17/pr-132122

docs/api/alerting/find_rules.asciidoc

@@ -43,7 +43,7 @@ NOTE: Rule `params` are stored as a {ref}/flattened.html[flattened field type] a
   (Optional, array|string) The fields to perform the `simple_query_string` parsed query against.
 
 `fields`::
-  (Optional, array|string) The fields to return in the `attributes` key of the response.
+  (Optional, array of strings) The fields to return in the `attributes` key of the response.
 
 `sort_field`::
   (Optional, string) Sorts the response. Could be a rule field returned in the `attributes` key of the response.

test/plugin_functional/test_suites/saved_objects_management/export_transform.ts

@@ -19,7 +19,8 @@ export default function ({ getService }: PluginFunctionalProviderContext) {
   const supertest = getService('supertest');
   const esArchiver = getService('esArchiver');
 
-  describe('export transforms', () => {
+  // Failing: See https://github.com/elastic/kibana/issues/131834
+  describe.skip('export transforms', () => {
     describe('root objects export transforms', () => {
       before(async () => {
         await esArchiver.load(

x-pack/plugins/reporting/public/lib/stream_handler.test.ts

@@ -5,6 +5,7 @@
  * 2.0.
  */
 
+import { omit } from 'lodash';
 import sinon, { stub } from 'sinon';
 import { NotificationsStart } from 'src/core/public';
 import { coreMock } from '../../../../../src/core/public/mocks';
@@ -98,7 +99,7 @@ describe('stream handler', () => {
         expect(mockShowDanger.callCount).toBe(0);
         expect(mockShowSuccess.callCount).toBe(1);
         expect(mockShowWarning.callCount).toBe(0);
-        expect(mockShowSuccess.args[0]).toMatchSnapshot();
+        expect(omit(mockShowSuccess.args[0][0], 'toastLifeTimeMs')).toMatchSnapshot();
         done();
       });
     });

x-pack/plugins/reporting/public/notifier/job_success.tsx

@@ -34,5 +34,12 @@ export const getSuccessToast = (
       <DownloadButton getUrl={getDownloadLink} job={job} />
     </Fragment>
   ),
+  /**
+   * If timeout is an Infinity value, a Not-a-Number (NaN) value, or negative, then timeout will be zero.
+   * And we cannot use `Number.MAX_SAFE_INTEGER` because EUI's Timer implementation
+   * subtracts it from the current time to evaluate the remainder.
+   * @see https://www.w3.org/TR/2011/WD-html5-20110525/timers.html
+   */
+  toastLifeTimeMs: Number.MAX_SAFE_INTEGER - Date.now(),
   'data-test-subj': 'completeReportSuccess',
 });

x-pack/test/functional/apps/discover/reporting.ts

@@ -151,6 +151,10 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
 
       beforeEach(() => PageObjects.common.navigateToApp('discover'));
 
+      afterEach(async () => {
+        await PageObjects.reporting.checkForReportingToasts();
+      });
+
       it('generates a report with data', async () => {
         await setupPage();
         await PageObjects.discover.loadSavedSearch('Ecommerce Data');

x-pack/test/functional/apps/lens/drag_and_drop.ts

@@ -12,8 +12,7 @@ export default function ({ getPageObjects }: FtrProviderContext) {
   const PageObjects = getPageObjects(['visualize', 'lens', 'common', 'header']);
 
   describe('lens drag and drop tests', () => {
-    // FLAKY: https://github.com/elastic/kibana/issues/108352
-    describe.skip('basic drag and drop', () => {
+    describe('basic drag and drop', () => {
       it('should construct the basic split xy chart', async () => {
         await PageObjects.visualize.navigateToNewVisualization();
         await PageObjects.visualize.clickVisType('lens');