fix(gitlab): remove approval rules only if automerging

lib/config/migrate-validate.spec.ts

@@ -44,5 +44,6 @@ describe('config/migrate-validate', () => {
       expect(res.warnings).toBeUndefined();
       expect(res).toMatchSnapshot();
     });
+
   });
 });

lib/config/presets/gitlab/index.spec.ts

@@ -124,6 +124,44 @@ describe('config/presets/gitlab/index', () => {
       });
       expect(content).toEqual({ foo: 'bar' });
     });
+
+    it('should remove approval rules for merge requests with automerge enabled', async () => {
+      httpMock
+        .scope(gitlabApiHost)
+        .get(projectPath)
+        .reply(200, {
+          default_branch: 'main',
+        })
+        .get(`${basePath}/files/default.json/raw?ref=main`)
+        .reply(200, { foo: 'bar' }, {});
+
+      const content = await gitlab.getPreset({ repo: 'some/repo' });
+      expect(content).toEqual({ foo: 'bar' });
+
+      // Simulate automerge enabled
+      const automergeEnabled = true;
+      const approvalRulesRemoved = await gitlab.tryPrAutomerge(1, { gitLabIgnoreApprovals: true }, automergeEnabled);
+      expect(approvalRulesRemoved).toBe(true);
+    });
+
+    it('should not remove approval rules for merge requests without automerge enabled', async () => {
+      httpMock
+        .scope(gitlabApiHost)
+        .get(projectPath)
+        .reply(200, {
+          default_branch: 'main',
+        })
+        .get(`${basePath}/files/default.json/raw?ref=main`)
+        .reply(200, { foo: 'bar' }, {});
+
+      const content = await gitlab.getPreset({ repo: 'some/repo' });
+      expect(content).toEqual({ foo: 'bar' });
+
+      // Simulate automerge disabled
+      const automergeDisabled = false;
+      const approvalRulesRemoved = await gitlab.tryPrAutomerge(1, { gitLabIgnoreApprovals: true }, automergeDisabled);
+      expect(approvalRulesRemoved).toBe(false);
+    });
   });
 
   describe('getPresetFromEndpoint()', () => {

lib/modules/platform/gitlab/index.ts

@@ -652,10 +652,6 @@ async function tryPrAutomerge(
   platformPrOptions: PlatformPrOptions | undefined,
 ): Promise<void> {
   try {
-    if (platformPrOptions?.gitLabIgnoreApprovals) {
-      await ignoreApprovals(pr);
-    }
-
     if (platformPrOptions?.usePlatformAutomerge) {
       // https://docs.gitlab.com/ee/api/merge_requests.html#merge-status
       const desiredDetailedMergeStatus = [
@@ -757,6 +753,7 @@ export async function createPr({
   draftPR,
   labels,
   platformPrOptions,
+  automerge,
 }: CreatePRConfig): Promise<Pr> {
   let title = prTitle;
   if (draftPR) {
@@ -790,7 +787,13 @@ export async function createPr({
     await approvePr(pr.iid);
   }
 
-  await tryPrAutomerge(pr.iid, platformPrOptions);
+  if (platformPrOptions?.gitLabIgnoreApprovals && automerge) {
+    await ignoreApprovals(pr.iid);
+  }
+
+  if (automerge) {
+    await tryPrAutomerge(pr.iid, platformPrOptions);
+  }
 
   return massagePr(pr);
 }