updated jest tests to avoid generating unhandled promise rejections

elastic · Sep 22, 2021 · 78c031b · 78c031b
1 parent 1fb0982
commit 78c031b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 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
@@ -12,7 +12,7 @@ import { SearchSource, SearchSourceDependencies, SortDirection } from './';
 import { AggConfigs, AggTypesRegistryStart } from '../../';
 import { mockAggTypesRegistry } from '../aggs/test_helpers';
 import { RequestResponder } from 'src/plugins/inspector/common';
-import { switchMap } from 'rxjs/operators';
+import { switchMap, catchError } from 'rxjs/operators';
 import { Filter } from '@kbn/es-query';
 
 const getComputedFields = () => ({
@@ -1117,8 +1117,10 @@ describe('SearchSource', () => {
 
         searchSource = new SearchSource({}, searchSourceDependencies);
         searchSource.setField('index', indexPattern);
+
         await searchSource
           .fetch$(options)
+          .pipe(catchError((e) => e))
           .toPromise()
           .catch(() => {});
 

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: {