[rewrite] Add RemoveUnusedPrivateMethods

Signed-off-by: Vincent Potucek <[email protected]>

Author: Vincent Potucek

.github/ci-files/run-sanity-check.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+
+set +e
+
+./gradlew rewriteDryRun -Dorg.gradle.jvmargs=-Xmx8G \
+  --configure-on-demand \
+  --console plain \
+  --max-workers=1 \
+  -Pqa --profile
+
+gradle_return_code=$?
+
+set -e
+
+exit $gradle_return_code

build.gradle

@@ -3,16 +3,16 @@ import org.sonar.build.BlackBoxTest
 
 import static org.gradle.api.JavaVersion.VERSION_17
 
-plugins {
-  // Ordered alphabetically
+plugins { // natural order
   id 'com.github.hierynomus.license' version '0.16.1'
-  id "com.github.hierynomus.license-report" version "0.16.1" apply false
-  id 'com.gradleup.shadow' version '8.3.6' apply false
+  id 'com.github.hierynomus.license-report' version '0.16.1' apply false
   id 'com.google.protobuf' version '0.8.19' apply false
+  id 'com.gradleup.shadow' version '8.3.6' apply false
   id 'com.jfrog.artifactory' version '6.0.2'
-  id "de.undercouch.download" version "5.6.0" apply false
+  id 'de.undercouch.download' version '5.6.0' apply false
   id 'io.spring.dependency-management' version '1.1.7'
-  id "org.cyclonedx.bom" version "2.3.1" apply false
+  id 'org.cyclonedx.bom' version '2.3.1' apply false
+  id 'org.openrewrite.rewrite' version '7.19.0' apply false
   id 'org.sonarqube' version '7.0.0.6105'
 }
 
@@ -200,7 +200,10 @@ allprojects {
   }
 }
 
+apply from: 'gradle/rewrite.gradle'
+
 apply plugin: 'org.sonarqube'
+
 sonar {
   properties {
     property 'sonar.projectName', projectTitle

gradle/rewrite.gradle

@@ -0,0 +1,25 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+apply plugin: 'org.openrewrite.rewrite'
+
+rewrite {
+  activeRecipe('org.opensearch.openrewrite.SanityCheck')
+  exclusion('**package-info.java')
+  setExportDatatables(true)
+  setFailOnDryRunResults(true)
+}
+
+dependencies {
+  rewrite(platform('org.openrewrite.recipe:rewrite-recipe-bom:3.17.0'))
+  rewrite('org.openrewrite.recipe:rewrite-migrate-java:3.20.0')
+  rewrite('org.openrewrite.recipe:rewrite-java-security:3.19.2')
+  rewrite('org.openrewrite.recipe:rewrite-rewrite:0.14.1')
+  rewrite('org.openrewrite.recipe:rewrite-static-analysis:2.20.0')
+  rewrite('org.openrewrite.recipe:rewrite-third-party:0.30.0')
+}

rewrite.yml

@@ -0,0 +1,16 @@
+---
+type: specs.openrewrite.org/v1beta/recipe
+name: org.opensearch.openrewrite.SanityCheck
+displayName: Apply all Java & Gradle best practices
+description: Comprehensive code quality recipe combining modernization, security, and best practices.
+tags:
+  - java
+  - gradle
+  - static-analysis
+  - cleanup
+recipeList:
+  - org.openrewrite.gradle.EnableGradleBuildCache
+  - org.openrewrite.gradle.EnableGradleParallelExecution
+  - org.openrewrite.java.RemoveUnusedImports
+  - org.openrewrite.staticanalysis.RemoveUnusedPrivateMethods
+---

server/sonar-auth-github/src/main/java/org/sonar/auth/github/GitHubRestClient.java

@@ -20,18 +20,12 @@
 package org.sonar.auth.github;
 
 import com.github.scribejava.core.model.OAuth2AccessToken;
-import com.github.scribejava.core.model.OAuthRequest;
-import com.github.scribejava.core.model.Response;
-import com.github.scribejava.core.model.Verb;
 import com.github.scribejava.core.oauth.OAuth20Service;
 import java.io.IOException;
-import java.net.HttpURLConnection;
 import java.util.List;
-import java.util.concurrent.ExecutionException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import static java.lang.String.format;
 import static org.sonar.auth.OAuthRestClient.executePaginatedRequest;
 import static org.sonar.auth.OAuthRestClient.executeRequest;
 

server/sonar-ce-common/src/it/java/org/sonar/ce/queue/CeQueueImplIT.java

@@ -634,10 +634,6 @@ private CeTaskSubmit createTaskSubmit(String type, @Nullable Component component
       .build();
   }
 
-  private ComponentDto insertComponent(ComponentDto componentDto) {
-    return db.components().insertComponent(componentDto);
-  }
-
   private UserDto insertUser(UserDto userDto) {
     db.getDbClient().userDao().insert(session, userDto);
     session.commit();

server/sonar-ce-task-projectanalysis/src/it/java/org/sonar/ce/task/projectanalysis/step/LoadPeriodsStepIT.java

@@ -24,9 +24,7 @@
 import com.tngtech.java.junit.dataprovider.UseDataProvider;
 import java.time.ZoneId;
 import java.time.ZonedDateTime;
-import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.List;
 import java.util.Optional;
 import java.util.Random;
 import java.util.stream.Stream;
@@ -489,16 +487,6 @@ public static Object[][] anyValidLeakPeriodSettingValue() {
     };
   }
 
-  private List<SnapshotDto> createSnapshots(ComponentDto project) {
-    ArrayList<SnapshotDto> list = new ArrayList<>();
-    list.add(dbTester.components().insertSnapshot(project, snapshot -> snapshot.setCreatedAt(1226379600000L).setLast(false))); // 2008-11-11
-    list.add(dbTester.components().insertSnapshot(project, snapshot -> snapshot.setCreatedAt(1226494680000L).setLast(false))); // 2008-11-12
-    list.add(dbTester.components().insertSnapshot(project, snapshot -> snapshot.setCreatedAt(1227157200000L).setLast(false))); // 2008-11-20
-    list.add(dbTester.components().insertSnapshot(project, snapshot -> snapshot.setCreatedAt(1227358680000L).setLast(false))); // 2008-11-22
-    list.add(dbTester.components().insertSnapshot(project, snapshot -> snapshot.setCreatedAt(1227934800000L).setLast(true))); // 2008-11-29
-    return list;
-  }
-
   private long milisSinceEpoch(int year, int month, int day, int hour) {
     return ZonedDateTime.of(year, month, day, hour, 0, 0, 0, ZoneId.systemDefault())
       .toInstant().toEpochMilli();

server/sonar-ce-task-projectanalysis/src/it/java/org/sonar/ce/task/projectanalysis/step/ValidateProjectStepIT.java

@@ -20,7 +20,6 @@
 package org.sonar.ce.task.projectanalysis.step;
 
 import java.util.Date;
-import javax.annotation.Nullable;
 import org.junit.Rule;
 import org.junit.Test;
 import org.sonar.api.utils.DateUtils;
@@ -35,13 +34,10 @@
 import org.sonar.ce.task.step.TestComputationStepContext;
 import org.sonar.db.DbClient;
 import org.sonar.db.DbTester;
-import org.sonar.db.component.BranchType;
 import org.sonar.db.component.ComponentDto;
 import org.sonar.db.component.SnapshotTesting;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.when;
 import static org.sonar.db.component.BranchDto.DEFAULT_MAIN_BRANCH_NAME;
 
 public class ValidateProjectStepIT {
@@ -110,11 +106,4 @@ public void fail_when_project_key_is_invalid() {
         "Allowed characters are alphanumeric, '-', '_', '.' and ':', with at least one non-digit.",
         "You should update the project key with the expected format.");
   }
-
-  private void setBranch(BranchType type, @Nullable String mergeBranchUuid) {
-    Branch branch = mock(Branch.class);
-    when(branch.getType()).thenReturn(type);
-    when(branch.getReferenceBranchUuid()).thenReturn(mergeBranchUuid);
-    analysisMetadataHolder.setBranch(branch);
-  }
 }

server/sonar-ce-task-projectanalysis/src/it/java/org/sonar/ce/task/projectanalysis/step/ViewsPersistComponentsStepIT.java

@@ -516,22 +516,6 @@ private void assertDtoIsView(ComponentDto dto) {
     assertThat(dto.getCreatedAt()).isEqualTo(now);
   }
 
-  /**
-   * Assertions to verify the DTO created from {@link #createViewBuilder(ViewAttributes.Type)} ()}
-   */
-  private void assertDtoIsApplication(ComponentDto dto) {
-    assertThat(dto.name()).isEqualTo(VIEW_NAME);
-    assertThat(dto.longName()).isEqualTo(VIEW_NAME);
-    assertThat(dto.description()).isEqualTo(VIEW_DESCRIPTION);
-    assertThat(dto.path()).isNull();
-    assertThat(dto.uuid()).isEqualTo(VIEW_UUID);
-    assertThat(dto.branchUuid()).isEqualTo(VIEW_UUID);
-    assertThat(dto.qualifier()).isEqualTo(ComponentQualifiers.APP);
-    assertThat(dto.scope()).isEqualTo(ComponentScopes.PROJECT);
-    assertThat(dto.getCopyComponentUuid()).isNull();
-    assertThat(dto.getCreatedAt()).isEqualTo(now);
-  }
-
   /**
    * Assertions to verify the DTO created from {@link #createProjectView1Builder(ComponentDto, Long)}
    */

server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/measure/PostMeasuresComputationCheck.java

@@ -22,7 +22,6 @@
 import org.sonar.api.ExtensionPoint;
 import org.sonar.api.ce.ComputeEngineSide;
 import org.sonar.ce.common.scanner.ScannerReportReader;
-import org.sonar.ce.task.projectanalysis.analysis.AnalysisMetadataHolder;
 import org.sonar.ce.task.projectanalysis.analysis.Branch;
 import org.sonar.ce.task.projectanalysis.component.Component;