[Fleet] use agentless API from package policy editor UI (#244424)

Author: nchaulet

x-pack/platform/plugins/shared/fleet/common/experimental_features.ts

@@ -20,7 +20,7 @@ const _allowedExperimentalValues = {
   installIntegrationsKnowledge: false,
   enableFleetPolicyRevisionsCleanupTask: true,
   agentlessPoliciesAPI: true, // When enabled, agentless policies API will be enabled.
-  useAgentlessAPIInUI: false, // When enabled, Fleet UI will use agentless policies API to create agentless policies.
+  useAgentlessAPIInUI: true, // When enabled, Fleet UI will use agentless policies API to create agentless policies.
   disableAgentlessLegacyAPI: false, // When enabled, it will disable creating agentless policies via agent or package policies API.
 };
 

x-pack/platform/plugins/shared/fleet/public/applications/fleet/sections/agent_policy/create_package_policy_page/single_page_layout/index.test.tsx

@@ -9,9 +9,16 @@ import { Route } from '@kbn/shared-ux-router';
 import React from 'react';
 import { fireEvent, act, waitFor } from '@testing-library/react';
 
+import { sendCreateAgentlessPolicy } from '../../../../../../hooks/use_request/agentless_policy';
+
 import type { MockedFleetStartServices, TestRenderer } from '../../../../../../mock';
 import { createFleetTestRendererMock } from '../../../../../../mock';
-import { FLEET_ROUTING_PATHS, pagePathGetters, PLUGIN_ID } from '../../../../constants';
+import {
+  FLEET_ROUTING_PATHS,
+  pagePathGetters,
+  PLUGIN_ID,
+  INTEGRATIONS_PLUGIN_ID,
+} from '../../../../constants';
 import type { CreatePackagePolicyRouteState } from '../../../../types';
 import {
   sendCreatePackagePolicyForRq,
@@ -37,6 +44,8 @@ jest.mock('../components/steps/components/use_policies', () => {
   };
 });
 
+jest.mock('../../../../../../hooks/use_request/agentless_policy');
+
 jest.mock('../../../../hooks', () => {
   return {
     ...jest.requireActual('../../../../hooks'),
@@ -718,9 +727,18 @@ describe('When on the package policy create page', () => {
             isCloudEnabled: true,
           },
         });
-        (sendCreateAgentPolicy as jest.MockedFunction<any>).mockResolvedValue({
-          data: {
-            item: { id: 'agentless-policy-1', name: 'Agentless policy 1', namespace: 'default' },
+        (sendCreateAgentlessPolicy as jest.MockedFunction<any>).mockResolvedValue({
+          item: {
+            name: 'Nginx',
+            id: 'policy-1',
+            inputs: [],
+            policy_ids: ['agent-policy-1'],
+            supports_agentless: true,
+            package: {
+              name: 'nginx',
+              title: 'Nginx',
+              version: '1.3.0',
+            },
           },
         });
 
@@ -741,17 +759,19 @@ describe('When on the package policy create page', () => {
           fireEvent.click(renderResult.getByText(/Save and continue/).closest('button')!);
         });
 
-        expect(sendCreateAgentPolicy).toHaveBeenCalledWith(
+        expect(sendCreateAgentlessPolicy).toHaveBeenCalledWith(
           expect.objectContaining({
-            monitoring_enabled: ['logs', 'metrics'],
-            name: 'Agentless policy for nginx-1',
+            name: 'nginx-1',
           }),
-          { withSysMonitoring: true }
+          { format: 'legacy' }
         );
-        expect(sendCreatePackagePolicyForRq).toHaveBeenCalled();
+        expect(sendCreatePackagePolicyForRq).not.toHaveBeenCalled();
 
         await waitFor(() => {
-          expect(renderResult.getByText('Nginx integration added')).toBeInTheDocument();
+          expect(useStartServices().application.navigateToApp).toHaveBeenCalledWith(
+            INTEGRATIONS_PLUGIN_ID,
+            { path: '/detail/nginx-1.3.0/policies?openEnrollmentFlyout=agent-policy-1' }
+          );
         });
       });
 
@@ -766,19 +786,18 @@ describe('When on the package policy create page', () => {
           fireEvent.click(renderResult.getByText(/Save and continue/).closest('button')!);
         });
 
-        expect(sendCreateAgentPolicy).toHaveBeenCalledWith(
+        expect(sendCreateAgentlessPolicy).toHaveBeenCalledWith(
           expect.objectContaining({
-            monitoring_enabled: ['logs', 'metrics'],
-            name: 'Agentless policy for nginx-1',
-            supports_agentless: true,
+            name: 'nginx-1',
           }),
-          { withSysMonitoring: true }
+          { format: 'legacy' }
         );
-        expect(sendCreatePackagePolicyForRq).toHaveBeenCalled();
+        expect(sendCreatePackagePolicyForRq).not.toHaveBeenCalled();
 
-        await waitFor(() => {
-          expect(renderResult.getByText('Nginx integration added')).toBeInTheDocument();
-        });
+        expect(useStartServices().application.navigateToApp).toHaveBeenCalledWith(
+          INTEGRATIONS_PLUGIN_ID,
+          { path: '/detail/nginx-1.3.0/policies?openEnrollmentFlyout=agent-policy-1' }
+        );
       });
     });
   });