[8.19] fix Failing test: Jest Tests.src/platform/plugins/shared/dashboard/public/dashboard_api - initializeUnifiedSearchManager startComparing$ timeRange Should not return timeRanage change when timeRestore resets to false (elastic#240715) (elastic#240895)

# Backport

This will backport the following commits from `main` to `8.19`:
- [fix Failing test: Jest
Tests.src/platform/plugins/shared/dashboard/public/dashboard_api -
initializeUnifiedSearchManager startComparing$ timeRange Should not
return timeRanage change when timeRestore resets to false
(elastic#240715)](elastic#240715)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-10-27T20:28:11Z","message":"fix
Failing test: Jest
Tests.src/platform/plugins/shared/dashboard/public/dashboard_api -
initializeUnifiedSearchManager startComparing$ timeRange Should not
return timeRanage change when timeRestore resets to false
(elastic#240715)\n\nFixes https://github.com/elastic/kibana/issues/240273\n\nPR
removes timing issues by removing awaits and moves all expect
logic\ninto subscription.\n\n---------\n\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"2076cc6c32282a2ee72b1e05f09badedfc2fddb5","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Presentation","release_note:skip","backport:version","v9.1.0","v8.19.0","v9.2.0","v9.3.0"],"title":"fix
Failing test: Jest
Tests.src/platform/plugins/shared/dashboard/public/dashboard_api -
initializeUnifiedSearchManager startComparing$ timeRange Should not
return timeRanage change when timeRestore resets to
false","number":240715,"url":"https://github.com/elastic/kibana/pull/240715","mergeCommit":{"message":"fix
Failing test: Jest
Tests.src/platform/plugins/shared/dashboard/public/dashboard_api -
initializeUnifiedSearchManager startComparing$ timeRange Should not
return timeRanage change when timeRestore resets to false
(elastic#240715)\n\nFixes https://github.com/elastic/kibana/issues/240273\n\nPR
removes timing issues by removing awaits and moves all expect
logic\ninto subscription.\n\n---------\n\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"2076cc6c32282a2ee72b1e05f09badedfc2fddb5"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","9.2"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/240715","number":240715,"mergeCommit":{"message":"fix
Failing test: Jest
Tests.src/platform/plugins/shared/dashboard/public/dashboard_api -
initializeUnifiedSearchManager startComparing$ timeRange Should not
return timeRanage change when timeRestore resets to false
(elastic#240715)\n\nFixes https://github.com/elastic/kibana/issues/240273\n\nPR
removes timing issues by removing awaits and moves all expect
logic\ninto subscription.\n\n---------\n\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"2076cc6c32282a2ee72b1e05f09badedfc2fddb5"}}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>

Author: 3 people

src/platform/plugins/shared/dashboard/public/dashboard_api/unified_search_manager.test.ts

@@ -7,10 +7,10 @@
  * License v3.0 only", or the "Server Side Public License, v 1".
  */
 
-import { BehaviorSubject, Subject } from 'rxjs';
+import { BehaviorSubject, Subject, take } from 'rxjs';
 import { getSampleDashboardState } from '../mocks';
 import type { DashboardState } from '../../common';
-import { COMPARE_DEBOUNCE, initializeUnifiedSearchManager } from './unified_search_manager';
+import { initializeUnifiedSearchManager } from './unified_search_manager';
 import type { ControlGroupApi } from '@kbn/controls-plugin/public';
 
 describe('initializeUnifiedSearchManager', () => {
@@ -90,7 +90,7 @@ describe('initializeUnifiedSearchManager', () => {
         });
       });
 
-      test('Should not return timeRanage change when timeRestore resets to false', async () => {
+      test('Should not return timeRanage change when timeRestore resets to false', (done) => {
         const lastSavedState$ = new BehaviorSubject<DashboardState>(getSampleDashboardState());
         const timeRestore$ = new BehaviorSubject<boolean | undefined>(false);
         const unifiedSearchManager = initializeUnifiedSearchManager(
@@ -103,34 +103,37 @@ describe('initializeUnifiedSearchManager', () => {
             useUnifiedSearchIntegration: false,
           }
         );
-        let unsavedChanges: object | undefined;
-        unifiedSearchManager.internalApi.startComparing$(lastSavedState$).subscribe((changes) => {
-          unsavedChanges = changes;
-        });
+        let emitCount = 0;
+        unifiedSearchManager.internalApi
+          .startComparing$(lastSavedState$)
+          .pipe(take(2))
+          .subscribe((changes) => {
+            emitCount++;
+
+            if (emitCount === 1) {
+              expect(changes).toMatchInlineSnapshot(`
+              Object {
+                "timeRange": Object {
+                  "from": "now-30m",
+                  "to": "now",
+                },
+              }
+            `);
+              // reset timeRestore to false
+              timeRestore$.next(false);
+            }
+
+            if (emitCount === 2) {
+              expect(changes).toMatchInlineSnapshot(`Object {}`);
+              done();
+            }
+          });
 
         timeRestore$.next(true);
         unifiedSearchManager.api.setTimeRange({
           to: 'now',
           from: 'now-30m',
         });
-
-        await new Promise((resolve) => setTimeout(resolve, COMPARE_DEBOUNCE + 1));
-
-        expect(unsavedChanges).toMatchInlineSnapshot(`
-            Object {
-              "timeRange": Object {
-                "from": "now-30m",
-                "to": "now",
-              },
-            }
-          `);
-
-        // reset timeRestore to false
-        timeRestore$.next(false);
-
-        await new Promise((resolve) => setTimeout(resolve, COMPARE_DEBOUNCE + 1));
-
-        expect(unsavedChanges).toMatchInlineSnapshot(`Object {}`);
       });
     });
   });