refactor(migrations): automergeMajor (#14634)

lib/config/migration.ts

@@ -202,10 +202,6 @@ export function migrateConfig(
         migratedConfig.minor = migratedConfig.minor || {};
         migratedConfig.minor.automerge = !!val;
         delete migratedConfig[key];
-      } else if (key === 'automergeMajor') {
-        migratedConfig.major = migratedConfig.major || {};
-        migratedConfig.major.automerge = !!val;
-        delete migratedConfig[key];
       } else if (key === 'separateMajorReleases') {
         delete migratedConfig.separateMultipleMajor;
         migratedConfig.separateMajorMinor = val;

lib/config/migrations/custom/automerge-major-migration.spec.ts

@@ -0,0 +1,47 @@
+import { AutomergeMajorMigration } from './automerge-major-migration';
+
+describe('config/migrations/custom/automerge-major-migration', () => {
+  it('should migrate value to object', () => {
+    expect(AutomergeMajorMigration).toMigrate(
+      {
+        automergeMajor: 'some-value',
+      },
+      {
+        major: {
+          automerge: true,
+        },
+      }
+    );
+  });
+
+  it('should migrate value to object and concat with existing minor object', () => {
+    expect(AutomergeMajorMigration).toMigrate(
+      {
+        automergeMajor: 'some-value',
+        major: {
+          matchFiles: ['test'],
+        },
+      },
+      {
+        major: {
+          automerge: true,
+          matchFiles: ['test'],
+        },
+      }
+    );
+  });
+
+  it('should ignore non object minor value', () => {
+    expect(AutomergeMajorMigration).toMigrate(
+      {
+        automergeMajor: 'some-value',
+        major: null,
+      },
+      {
+        major: {
+          automerge: true,
+        },
+      }
+    );
+  });
+});

lib/config/migrations/custom/automerge-major-migration.ts

@@ -0,0 +1,15 @@
+import is from '@sindresorhus/is';
+import { AbstractMigration } from '../base/abstract-migration';
+
+export class AutomergeMajorMigration extends AbstractMigration {
+  override readonly deprecated = true;
+  override readonly propertyName = 'automergeMajor';
+
+  override run(value: unknown): void {
+    const major = this.get('major');
+
+    const newMajor = is.object(major) ? major : {};
+    newMajor.automerge = Boolean(value);
+    this.setHard('major', newMajor);
+  }
+}

lib/config/migrations/migrations-service.ts

@@ -2,6 +2,7 @@ import { dequal } from 'dequal';
 import type { RenovateConfig } from '../types';
 import { RemovePropertyMigration } from './base/remove-property-migration';
 import { RenamePropertyMigration } from './base/rename-property-migration';
+import { AutomergeMajorMigration } from './custom/automerge-major-migration';
 import { AutomergeTypeMigration } from './custom/automerge-type-migration';
 import { BinarySourceMigration } from './custom/binary-source-migration';
 import { CompatibilityMigration } from './custom/compatibility-migration';
@@ -55,6 +56,7 @@ export class MigrationsService {
   ]);
 
   static readonly customMigrations: ReadonlyArray<MigrationConstructor> = [
+    AutomergeMajorMigration,
     AutomergeTypeMigration,
     BinarySourceMigration,
     CompatibilityMigration,