Merge branch 'master' into fixes-search-bar

Author: elasticmachine

.ci/Jenkinsfile_security_cypress

@@ -16,6 +16,6 @@ kibanaPipeline(timeoutMinutes: 180) {
   }
 
   if (params.NOTIFY_ON_FAILURE) {
-    kibanaPipeline.sendMail(to: 'gloria.delatorre@elastic.co')
+    kibanaPipeline.sendMail(to: 'siem_dev_team@elastic.co')
   }
 }

src/plugins/data/public/ui/query_string_input/_query_bar.scss

@@ -59,6 +59,7 @@
     &.kbnQueryBar__datePickerWrapper-isHidden {
       width: 0;
       overflow: hidden;
+      max-width: 0;
     }
   }
 }

src/plugins/discover/public/application/angular/doc.html

@@ -1,6 +1,5 @@
 <div class="app-container">
     <discover-doc
-      es-client="esClient"
       id="id"
       index="index"
       index-pattern-id="indexPatternId"

src/plugins/discover/public/application/angular/doc.ts

@@ -36,7 +36,6 @@ app.directive('discoverDoc', function (reactDirective: any) {
       ['index', { watchDepth: 'value' }],
       ['indexPatternId', { watchDepth: 'reference' }],
       ['indexPatternService', { watchDepth: 'reference' }],
-      ['esClient', { watchDepth: 'reference' }],
     ],
     { restrict: 'E' }
   );
@@ -51,10 +50,9 @@ app.config(($routeProvider: any) => {
     .when('/doc/:indexPattern/:index', {
       // have to be written as function expression, because it's not compiled in dev mode
       // eslint-disable-next-line object-shorthand
-      controller: function ($scope: LazyScope, $route: any, es: any) {
+      controller: function ($scope: LazyScope, $route: any) {
         timefilter.disableAutoRefreshSelector();
         timefilter.disableTimeRangeSelector();
-        $scope.esClient = es;
         $scope.id = $route.current.params.id;
         $scope.index = $route.current.params.index;
         $scope.indexPatternId = $route.current.params.indexPattern;

src/plugins/discover/public/application/components/doc/doc.test.tsx

@@ -16,13 +16,16 @@
  * specific language governing permissions and limitations
  * under the License.
  */
+import { throwError, of } from 'rxjs';
 import React from 'react';
 import { act } from 'react-dom/test-utils';
 import { mountWithIntl } from 'test_utils/enzyme_helpers';
 import { ReactWrapper } from 'enzyme';
 import { findTestSubject } from '@elastic/eui/lib/test';
 import { Doc, DocProps } from './doc';
 
+const mockSearchApi = jest.fn();
+
 jest.mock('../../../kibana_services', () => {
   let registry: any[] = [];
 
@@ -31,6 +34,11 @@ jest.mock('../../../kibana_services', () => {
       metadata: {
         branch: 'test',
       },
+      data: {
+        search: {
+          search: mockSearchApi,
+        },
+      },
     }),
     getDocViewsRegistry: () => ({
       addDocView(view: any) {
@@ -59,7 +67,7 @@ const waitForPromises = async () =>
  * this works but logs ugly error messages until we're using React 16.9
  * should be adapted when we upgrade
  */
-async function mountDoc(search: () => void, update = false, indexPatternGetter: any = null) {
+async function mountDoc(update = false, indexPatternGetter: any = null) {
   const indexPattern = {
     getComputedFields: () => [],
   };
@@ -70,7 +78,6 @@ async function mountDoc(search: () => void, update = false, indexPatternGetter:
   const props = {
     id: '1',
     index: 'index1',
-    esClient: { search } as any,
     indexPatternId: 'xyz',
     indexPatternService,
   } as DocProps;
@@ -88,32 +95,33 @@ async function mountDoc(search: () => void, update = false, indexPatternGetter:
 
 describe('Test of <Doc /> of Discover', () => {
   test('renders loading msg', async () => {
-    const comp = await mountDoc(jest.fn());
+    const comp = await mountDoc();
     expect(findTestSubject(comp, 'doc-msg-loading').length).toBe(1);
   });
 
   test('renders IndexPattern notFound msg', async () => {
     const indexPatternGetter = jest.fn(() => Promise.reject({ savedObjectId: '007' }));
-    const comp = await mountDoc(jest.fn(), true, indexPatternGetter);
+    const comp = await mountDoc(true, indexPatternGetter);
     expect(findTestSubject(comp, 'doc-msg-notFoundIndexPattern').length).toBe(1);
   });
 
   test('renders notFound msg', async () => {
-    const search = jest.fn(() => Promise.reject({ status: 404 }));
-    const comp = await mountDoc(search, true);
+    mockSearchApi.mockImplementation(() => throwError({ status: 404 }));
+    const comp = await mountDoc(true);
     expect(findTestSubject(comp, 'doc-msg-notFound').length).toBe(1);
   });
 
   test('renders error msg', async () => {
-    const search = jest.fn(() => Promise.reject('whatever'));
-    const comp = await mountDoc(search, true);
+    mockSearchApi.mockImplementation(() => throwError({ error: 'something else' }));
+    const comp = await mountDoc(true);
     expect(findTestSubject(comp, 'doc-msg-error').length).toBe(1);
   });
 
   test('renders elasticsearch hit ', async () => {
-    const hit = { hits: { total: 1, hits: [{ _id: 1, _source: { test: 1 } }] } };
-    const search = jest.fn(() => Promise.resolve(hit));
-    const comp = await mountDoc(search, true);
+    mockSearchApi.mockImplementation(() =>
+      of({ rawResponse: { hits: { total: 1, hits: [{ _id: 1, _source: { test: 1 } }] } } })
+    );
+    const comp = await mountDoc(true);
     expect(findTestSubject(comp, 'doc-hit').length).toBe(1);
   });
 });

src/plugins/discover/public/application/components/doc/doc.tsx

@@ -23,17 +23,6 @@ import { IndexPatternsContract } from 'src/plugins/data/public';
 import { ElasticRequestState, useEsDocSearch } from './use_es_doc_search';
 import { getServices } from '../../../kibana_services';
 import { DocViewer } from '../doc_viewer/doc_viewer';
-import { ElasticSearchHit } from '../../doc_views/doc_views_types';
-
-export interface ElasticSearchResult {
-  hits: {
-    hits: [ElasticSearchHit];
-    max_score: number;
-  };
-  timed_out: boolean;
-  took: number;
-  shards: Record<string, unknown>;
-}
 
 export interface DocProps {
   /**
@@ -53,12 +42,6 @@ export interface DocProps {
    * IndexPatternService to get a given index pattern by ID
    */
   indexPatternService: IndexPatternsContract;
-  /**
-   * Client of ElasticSearch to use for the query
-   */
-  esClient: {
-    search: (payload: { index: string; body: Record<string, any> }) => Promise<ElasticSearchResult>;
-  };
 }
 
 export function Doc(props: DocProps) {

src/plugins/discover/public/application/components/doc/use_es_doc_search.test.tsx

@@ -19,6 +19,21 @@
 import { renderHook, act } from '@testing-library/react-hooks';
 import { buildSearchBody, useEsDocSearch, ElasticRequestState } from './use_es_doc_search';
 import { DocProps } from './doc';
+import { Observable } from 'rxjs';
+
+const mockSearchResult = new Observable();
+
+jest.mock('../../../kibana_services', () => ({
+  getServices: () => ({
+    data: {
+      search: {
+        search: jest.fn(() => {
+          return mockSearchResult;
+        }),
+      },
+    },
+  }),
+}));
 
 describe('Test of <Doc /> helper / hook', () => {
   test('buildSearchBody', () => {
@@ -53,7 +68,6 @@ describe('Test of <Doc /> helper / hook', () => {
     const props = {
       id: '1',
       index: 'index1',
-      esClient: { search: jest.fn(() => new Promise(() => {})) },
       indexPatternId: 'xyz',
       indexPatternService,
     } as DocProps;

src/plugins/discover/public/application/components/doc/use_es_doc_search.ts

@@ -17,7 +17,7 @@
  * under the License.
  */
 import { useEffect, useState } from 'react';
-import { IndexPattern } from '../../../kibana_services';
+import { IndexPattern, getServices } from '../../../kibana_services';
 import { DocProps } from './doc';
 import { ElasticSearchHit } from '../../doc_views/doc_views_types';
 
@@ -53,7 +53,6 @@ export function buildSearchBody(id: string, indexPattern: IndexPattern): Record<
  * Custom react hook for querying a single doc in ElasticSearch
  */
 export function useEsDocSearch({
-  esClient,
   id,
   index,
   indexPatternId,
@@ -69,12 +68,18 @@ export function useEsDocSearch({
         const indexPatternEntity = await indexPatternService.get(indexPatternId);
         setIndexPattern(indexPatternEntity);
 
-        const { hits } = await esClient.search({
-          index,
-          body: buildSearchBody(id, indexPatternEntity),
-        });
+        const { rawResponse } = await getServices()
+          .data.search.search({
+            params: {
+              index,
+              body: buildSearchBody(id, indexPatternEntity),
+            },
+          })
+          .toPromise();
 
-        if (hits && hits.hits && hits.hits[0]) {
+        const hits = rawResponse.hits;
+
+        if (hits?.hits?.[0]) {
           setStatus(ElasticRequestState.Found);
           setHit(hits.hits[0]);
         } else {
@@ -91,6 +96,6 @@ export function useEsDocSearch({
       }
     }
     requestData();
-  }, [esClient, id, index, indexPatternId, indexPatternService]);
+  }, [id, index, indexPatternId, indexPatternService]);
   return [status, hit, indexPattern];
 }

src/plugins/discover/public/application/components/json_code_block/__snapshots__/json_code_block.test.tsx.snap

@@ -23,7 +23,7 @@ import { IndexPattern } from '../../../../../data/public';
 
 it('returns the `JsonCodeEditor` component', () => {
   const props = {
-    hit: { _index: 'test', _source: { test: 123 } },
+    hit: { _index: 'test', _type: 'doc', _id: 'foo', _score: 1, _source: { test: 123 } },
     columns: [],
     indexPattern: {} as IndexPattern,
     filter: jest.fn(),