[7.x] [Fleet] Fix circular ref by moving code & tests into Fleet (#94171)

x-pack/plugins/fleet/server/saved_objects/index.ts

@@ -22,20 +22,22 @@ import {
 } from '../constants';
 
 import {
-  migratePackagePolicyToV7110,
-  migratePackagePolicyToV7120,
-  // @ts-expect-error
-} from './security_solution';
-import {
-  migrateAgentToV7100,
+  migrateAgentActionToV7100,
   migrateAgentEventToV7100,
   migrateAgentPolicyToV7100,
+  migrateAgentToV7100,
   migrateEnrollmentApiKeysToV7100,
   migratePackagePolicyToV7100,
   migrateSettingsToV7100,
-  migrateAgentActionToV7100,
 } from './migrations/to_v7_10_0';
-import { migrateAgentToV7120, migrateAgentPolicyToV7120 } from './migrations/to_v7_12_0';
+
+import { migratePackagePolicyToV7110 } from './migrations/to_v7_11_0';
+
+import {
+  migrateAgentPolicyToV7120,
+  migrateAgentToV7120,
+  migratePackagePolicyToV7120,
+} from './migrations/to_v7_12_0';
 
 /*
  * Saved object types and mappings

x-pack/plugins/fleet/server/saved_objects/migrations/security_solution/index.ts

@@ -0,0 +1,9 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+export { migratePackagePolicyToV7110 } from './to_v7_11_0';
+export { migratePackagePolicyToV7120 } from './to_v7_12_0';

x-pack/plugins/security_solution/common/endpoint/policy/migrations/to_v7_11_0.test.ts → x-pack/plugins/fleet/server/saved_objects/migrations/security_solution/to_v7_11_0.test.ts

@@ -6,7 +6,9 @@
  */
 
 import { SavedObjectMigrationContext, SavedObjectUnsanitizedDoc } from 'kibana/server';
-import { PackagePolicy } from '../../../../../fleet/common';
+
+import type { PackagePolicy } from '../../../../common';
+
 import { migratePackagePolicyToV7110 } from './to_v7_11_0';
 
 describe('7.11.0 Endpoint Package Policy migration', () => {

x-pack/plugins/security_solution/common/endpoint/policy/migrations/to_v7_11_0.ts → x-pack/plugins/fleet/server/saved_objects/migrations/security_solution/to_v7_11_0.ts

@@ -7,7 +7,8 @@
 
 import { SavedObjectMigrationFn, SavedObjectUnsanitizedDoc } from 'kibana/server';
 import { cloneDeep } from 'lodash';
-import { PackagePolicy } from '../../../../../fleet/common';
+
+import type { PackagePolicy } from '../../../../common';
 
 export const migratePackagePolicyToV7110: SavedObjectMigrationFn<PackagePolicy, PackagePolicy> = (
   packagePolicyDoc

x-pack/plugins/security_solution/common/endpoint/policy/migrations/to_v7_12_0.test.ts → x-pack/plugins/fleet/server/saved_objects/migrations/security_solution/to_v7_12_0.test.ts

@@ -6,14 +6,15 @@
  */
 
 import { SavedObjectMigrationContext, SavedObjectUnsanitizedDoc } from 'kibana/server';
-import { PackagePolicy } from '../../../../../fleet/common';
-import { PolicyData, ProtectionModes } from '../../types';
+
+import type { PackagePolicy } from '../../../../common';
+
 import { migratePackagePolicyToV7120 } from './to_v7_12_0';
 
 describe('7.12.0 Endpoint Package Policy migration', () => {
   const migration = migratePackagePolicyToV7120;
   it('adds ransomware option and notification customization', () => {
-    const doc: SavedObjectUnsanitizedDoc<PolicyData> = {
+    const doc = {
       id: 'mock-saved-object-id',
       attributes: {
         name: 'Some Policy Name',
@@ -41,7 +42,6 @@ describe('7.12.0 Endpoint Package Policy migration', () => {
               policy: {
                 value: {
                   windows: {
-                    // @ts-expect-error
                     popup: {
                       malware: {
                         message: '',
@@ -58,9 +58,7 @@ describe('7.12.0 Endpoint Package Policy migration', () => {
       type: ' nested',
     };
 
-    expect(
-      migration(doc, {} as SavedObjectMigrationContext) as SavedObjectUnsanitizedDoc<PolicyData>
-    ).toEqual({
+    expect(migration(doc, {} as SavedObjectMigrationContext)).toEqual({
       attributes: {
         name: 'Some Policy Name',
         package: {
@@ -88,7 +86,7 @@ describe('7.12.0 Endpoint Package Policy migration', () => {
                 value: {
                   windows: {
                     ransomware: {
-                      mode: ProtectionModes.off,
+                      mode: 'off',
                     },
                     popup: {
                       malware: {

x-pack/plugins/security_solution/common/endpoint/policy/migrations/to_v7_12_0.ts → x-pack/plugins/fleet/server/saved_objects/migrations/security_solution/to_v7_12_0.ts

@@ -7,8 +7,8 @@
 
 import { SavedObjectMigrationFn, SavedObjectUnsanitizedDoc } from 'kibana/server';
 import { cloneDeep } from 'lodash';
-import { PackagePolicy } from '../../../../../fleet/common';
-import { ProtectionModes } from '../../types';
+
+import type { PackagePolicy } from '../../../../common';
 
 export const migratePackagePolicyToV7120: SavedObjectMigrationFn<PackagePolicy, PackagePolicy> = (
   packagePolicyDoc
@@ -19,7 +19,7 @@ export const migratePackagePolicyToV7120: SavedObjectMigrationFn<PackagePolicy,
   if (packagePolicyDoc.attributes.package?.name === 'endpoint') {
     const input = updatedPackagePolicyDoc.attributes.inputs[0];
     const ransomware = {
-      mode: ProtectionModes.off,
+      mode: 'off',
     };
     const ransomwarePopup = {
       message: '',

x-pack/plugins/fleet/server/saved_objects/security_solution.js → x-pack/plugins/fleet/server/saved_objects/migrations/to_v7_11_0.ts

@@ -5,7 +5,4 @@
  * 2.0.
  */
 
-export {
-  migratePackagePolicyToV7110,
-  migratePackagePolicyToV7120,
-} from '../../../security_solution/common';
+export { migratePackagePolicyToV7110 } from './security_solution/to_v7_11_0';

x-pack/plugins/fleet/server/saved_objects/migrations/to_v7_12_0.ts

@@ -9,6 +9,8 @@ import type { SavedObjectMigrationFn } from 'kibana/server';
 
 import type { Agent, AgentPolicy } from '../../types';
 
+export { migratePackagePolicyToV7120 } from './security_solution/to_v7_12_0';
+
 export const migrateAgentToV7120: SavedObjectMigrationFn<Agent & { shared_id?: string }, Agent> = (
   agentDoc
 ) => {

x-pack/plugins/security_solution/common/shared_exports.ts

@@ -17,6 +17,4 @@ export { exactCheck } from './exact_check';
 export { getPaths, foldLeftRight, removeExternalLinkText } from './test_utils';
 export { validate, validateEither } from './validate';
 export { formatErrors } from './format_errors';
-export { migratePackagePolicyToV7110 } from './endpoint/policy/migrations/to_v7_11_0';
-export { migratePackagePolicyToV7120 } from './endpoint/policy/migrations/to_v7_12_0';
 export { addIdToItem, removeIdFromItem } from './add_remove_id_to_item';