[7.x] [Enterprise Search] Misc test branch coverage (#86847) (#87202)

* [Enterprise Search] Misc test branch coverage (#86847) * Remove unnecessary || {} branch schema is always required / should never be undefined * Cover if (addFieldFormErrors) branch * Increase coverage of determineTooltipContent by simplifying branching - most of these checks aren't necessary due to early returns * Cover branch in WS source_icon Co-authored-by: Kibana Machine <[email protected]> * Update x-pack/plugins/enterprise_search/public/applications/workplace_search/components/shared/source_icon/source_icon.test.tsx Co-authored-by: Kibana Machine <[email protected]>
elastic · Jan 4, 2021 · ae2f5fa · ae2f5fa
1 parent 8b201aa
commit ae2f5fa
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 27 deletions.
diff --git a/.../applications/app_search/components/documents/search_experience/build_search_ui_config.ts b/.../applications/app_search/components/documents/search_experience/build_search_ui_config.ts
@@ -16,19 +16,16 @@ export const buildSearchUIConfig = (apiConnector: object, schema: Schema) => {
       sortField: 'id',
     },
     searchQuery: {
-      result_fields: Object.keys(schema || {}).reduce(
-        (acc: { [key: string]: object }, key: string) => {
-          acc[key] = {
-            snippet: {
-              size: 300,
-              fallback: true,
-            },
-            raw: {},
-          };
-          return acc;
-        },
-        {}
-      ),
+      result_fields: Object.keys(schema).reduce((acc: { [key: string]: object }, key: string) => {
+        acc[key] = {
+          snippet: {
+            size: 300,
+            fallback: true,
+          },
+          raw: {},
+        };
+        return acc;
+      }, {}),
     },
   };
 };
diff --git a/...tions/app_search/components/settings/log_retention/messaging/determine_tooltip_content.ts b/...tions/app_search/components/settings/log_retention/messaging/determine_tooltip_content.ts
@@ -26,21 +26,12 @@ export const determineTooltipContent = (
   if (!logRetentionSettings.enabled) {
     return renderOrReturnMessage(messages.noLogging);
   }
-  if (logRetentionSettings.enabled && !ilmEnabled) {
+  if (!ilmEnabled) {
     return renderOrReturnMessage(messages.ilmDisabled);
   }
-  if (
-    logRetentionSettings.enabled &&
-    ilmEnabled &&
-    !logRetentionSettings.retentionPolicy?.isDefault
-  ) {
+  if (!logRetentionSettings.retentionPolicy?.isDefault) {
     return renderOrReturnMessage(messages.customPolicy);
-  }
-  if (
-    logRetentionSettings.enabled &&
-    ilmEnabled &&
-    logRetentionSettings.retentionPolicy?.isDefault
-  ) {
+  } else {
     return renderOrReturnMessage(messages.defaultPolicy);
   }
 };
diff --git a/...ugins/enterprise_search/public/applications/shared/schema/schema_add_field_modal.test.tsx b/...ugins/enterprise_search/public/applications/shared/schema/schema_add_field_modal.test.tsx
@@ -41,10 +41,10 @@ describe('SchemaAddFieldModal', () => {
     expect(wrapper.find(EuiModal)).toHaveLength(1);
   });
 
-  // No matter what I try I can't get this to actually achieve coverage.
   it('sets loading state in useEffect', () => {
     setState(true);
-    const wrapper = mount(<SchemaAddFieldModal {...props} {...errors} />);
+    const wrapper = mount(<SchemaAddFieldModal {...props} />);
+    wrapper.setProps({ ...errors });
     const input = wrapper.find(EuiFieldText);
 
     expect(input.prop('isLoading')).toEqual(false);