Fix failing tests and add key

elastic · Sep 28, 2021 · 07a68fa · 07a68fa
1 parent 51f844f
commit 07a68fa
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 5 deletions.
diff --git a/.../enterprise_search/public/applications/workplace_search/__mocks__/content_sources.mock.ts b/.../enterprise_search/public/applications/workplace_search/__mocks__/content_sources.mock.ts
@@ -110,6 +110,7 @@ export const fullContentSources = [
     groups,
     custom: false,
     isIndexedSource: true,
+    isSyncConfigEnabled: true,
     areThumbnailsConfigEnabled: true,
     accessToken: '123token',
     urlField: 'myLink',
@@ -131,6 +132,7 @@ export const fullContentSources = [
     indexing: defaultIndexing,
     custom: true,
     isIndexedSource: true,
+    isSyncConfigEnabled: true,
     areThumbnailsConfigEnabled: true,
     accessToken: '123token',
     urlField: 'url',

diff --git a/...ic/applications/workplace_search/views/content_sources/components/source_sub_nav.test.tsx b/...ic/applications/workplace_search/views/content_sources/components/source_sub_nav.test.tsx
@@ -90,7 +90,7 @@ describe('useSourceSubNav', () => {
   it('returns extra nav items for synchronization', () => {
     setMockValues({
       isOrganization: true,
-      contentSource: { id: '2', isIndexedSource: true, name: 'foo' },
+      contentSource: { id: '2', isIndexedSource: true, name: 'foo', isSyncConfigEnabled: true },
     });
 
     expect(useSourceSubNav()).toEqual([

diff --git a/.../workplace_search/views/content_sources/components/synchronization/blocked_window_tab.tsx b/.../workplace_search/views/content_sources/components/synchronization/blocked_window_tab.tsx
@@ -41,8 +41,8 @@ export const BlockedWindows: React.FC = () => {
 
   const blockedWindowItems = (
     <>
-      {blockedWindows.map((blockedWindow) => (
-        <BlockedWindowItem blockedWindow={blockedWindow} />
+      {blockedWindows.map((blockedWindow, i) => (
+        <BlockedWindowItem key={i} blockedWindow={blockedWindow} />
       ))}
       <EuiSpacer />
       <EuiButton onClick={addBlockedWindow}>{ADD_LABEL}</EuiButton>

diff --git a/...orkplace_search/views/content_sources/components/synchronization/synchronization.test.tsx b/...orkplace_search/views/content_sources/components/synchronization/synchronization.test.tsx
@@ -5,19 +5,29 @@
  * 2.0.
  */
 
+import { setMockValues } from '../../../../../__mocks__/kea_logic';
+
 import React from 'react';
 
 import { shallow } from 'enzyme';
 
-import { EuiLink, EuiSwitch } from '@elastic/eui';
+import { EuiLink, EuiCallOut, EuiSwitch } from '@elastic/eui';
 
 import { Synchronization } from './synchronization';
 
 describe('Synchronization', () => {
-  it('renders', () => {
+  it('renders when config enabled', () => {
+    setMockValues({ contentSource: { isSyncConfigEnabled: true } });
     const wrapper = shallow(<Synchronization />);
 
     expect(wrapper.find(EuiLink)).toHaveLength(1);
     expect(wrapper.find(EuiSwitch)).toHaveLength(1);
   });
+
+  it('renders when config disabled', () => {
+    setMockValues({ contentSource: { isSyncConfigEnabled: false } });
+    const wrapper = shallow(<Synchronization />);
+
+    expect(wrapper.find(EuiCallOut)).toHaveLength(1);
+  });
 });