[Reporting] Fix job completion notification to disappear after 24 hou…

…rs (#133381) (#133408)

(cherry picked from commit 1e39a2c)

Co-authored-by: Michael Dokolin <mikhail.dokolin@elastic.co>

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

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

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

@@ -22,6 +22,12 @@ import {
 import { Job } from './job';
 import { ReportingAPIClient } from './reporting_api_client';
 
+/**
+ * @todo Replace with `Infinity` once elastic/eui#5945 is resolved.
+ * @see https://github.com/elastic/eui/issues/5945
+ */
+const COMPLETED_JOB_TOAST_TIMEOUT = 24 * 60 * 60 * 1000; // 24 hours
+
 function updateStored(jobIds: JobId[]): void {
   sessionStorage.setItem(JOB_COMPLETION_NOTIFICATIONS_SESSION_KEY, JSON.stringify(jobIds));
 }
@@ -54,6 +60,8 @@ export class ReportingNotifierStreamHandler {
     failed: failedJobs,
   }: JobSummarySet): Rx.Observable<JobSummarySet> {
     const showNotificationsAsync = async () => {
+      const completedOptions = { toastLifeTimeMs: COMPLETED_JOB_TOAST_TIMEOUT };
+
       // notifications with download link
       for (const job of completedJobs) {
         if (job.csvContainsFormulas) {
@@ -63,7 +71,8 @@ export class ReportingNotifierStreamHandler {
               this.apiClient.getManagementLink,
               this.apiClient.getDownloadLink,
               this.theme
-            )
+            ),
+            completedOptions
           );
         } else if (job.maxSizeReached) {
           this.notifications.toasts.addWarning(
@@ -72,7 +81,8 @@ export class ReportingNotifierStreamHandler {
               this.apiClient.getManagementLink,
               this.apiClient.getDownloadLink,
               this.theme
-            )
+            ),
+            completedOptions
           );
         } else if (job.status === JOB_STATUSES.WARNINGS) {
           this.notifications.toasts.addWarning(
@@ -81,7 +91,8 @@ export class ReportingNotifierStreamHandler {
               this.apiClient.getManagementLink,
               this.apiClient.getDownloadLink,
               this.theme
-            )
+            ),
+            completedOptions
           );
         } else {
           this.notifications.toasts.addSuccess(
@@ -90,7 +101,8 @@ export class ReportingNotifierStreamHandler {
               this.apiClient.getManagementLink,
               this.apiClient.getDownloadLink,
               this.theme
-            )
+            ),
+            completedOptions
           );
         }
       }

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

@@ -37,12 +37,5 @@ export const getSuccessToast = (
     </>,
     { theme$: theme.theme$ }
   ),
-  /**
-   * 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',
 });