[7.x] [AppServices] Fix unhandled promise rejections in search tests (#…

…112849) (#113264) * [AppServices] Fix unhandled promise rejections in search tests (#112849) * updated jest tests to avoid generating unhandled promise rejections * rather use throwError * remove unused imports Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> # Conflicts: # src/plugins/data/common/search/search_source/search_source.test.ts * Fix lint issues
elastic · Sep 29, 2021 · 09778a3 · 09778a3
1 parent df7ed2f
commit 09778a3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/plugins/data/common/search/search_source/search_source.test.ts b/src/plugins/data/common/search/search_source/search_source.test.ts
@@ -6,7 +6,7 @@
  * Side Public License, v 1.
  */
 
-import { of } from 'rxjs';
+import { of, throwError } from 'rxjs';
 import { GetConfigFn } from '../../types';
 import { IndexPattern } from '../..';
 import { SearchSource, SearchSourceDependencies, SortDirection } from './';
@@ -1153,7 +1153,7 @@ describe('SearchSource', () => {
           },
         };
 
-        searchSourceDependencies.search = jest.fn().mockReturnValue(of(Promise.reject('aaaaa')));
+        searchSourceDependencies.search = jest.fn().mockReturnValue(throwError('aaaaa'));
 
         searchSource = new SearchSource({}, searchSourceDependencies);
         searchSource.setField('index', indexPattern);

diff --git a/src/plugins/data/server/search/search_service.test.ts b/src/plugins/data/server/search/search_service.test.ts
@@ -38,10 +38,8 @@ describe('Search service', () => {
   let mockCoreStart: MockedKeys<CoreStart>;
 
   beforeEach(() => {
-    const mockLogger: any = {
-      debug: () => {},
-    };
     const context = coreMock.createPluginInitializerContext({});
+    const mockLogger = context.logger.get();
     context.config.create = jest.fn().mockImplementation(() => {
       return of({
         search: {