[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 <42973632+kibanamachine@users.noreply.github.com>

x-pack/plugins/enterprise_search/public/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;
+      }, {}),
     },
   };
 };

x-pack/plugins/enterprise_search/public/applications/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);
   }
 };

x-pack/plugins/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);

x-pack/plugins/enterprise_search/public/applications/workplace_search/components/shared/source_icon/source_icon.test.tsx

@@ -24,4 +24,10 @@ describe('SourceIcon', () => {
 
     expect(wrapper.find('.wrapped-icon')).toHaveLength(1);
   });
+
+  it('renders a full bleed icon', () => {
+    const wrapper = shallow(<SourceIcon name="foo" fullBleed serviceType="custom" />);
+
+    expect(wrapper.find(EuiIcon).prop('type')).toEqual('test-file-stub');
+  });
 });