SONAR-26205 Clean up issue_stats_by_rule_key after project is deleted

Author: dbmeneses

server/sonar-db-dao/src/it/java/org/sonar/db/purge/PurgeDaoIT.java

@@ -2071,6 +2071,35 @@ void deleteBranch_purgesScaActivity() {
     assertThat(db.countRowsOfTable(dbSession, "sca_analyses")).isEqualTo(1);
   }
 
+  @Test
+  void deleteBranch_purgeIssueStatsByRuleKey() {
+    ProjectDto project = db.components().insertPublicProject().getProjectDto();
+    BranchDto branch1 = db.components().insertProjectBranch(project);
+    BranchDto branch2 = db.components().insertProjectBranch(project);
+
+    db.executeInsert("issue_stats_by_rule_key",
+      "aggregation_type", "PROJECT",
+      "aggregation_id", branch1.getUuid(),
+      "rule_key", "rule1",
+      "issue_count", 1,
+      "rating", 2,
+      "hotspot_count", 3,
+      "hotspots_reviewed", 4);
+
+    db.executeInsert("issue_stats_by_rule_key",
+      "aggregation_type", "PROJECT",
+      "aggregation_id", branch2.getUuid(),
+      "rule_key", "rule1",
+      "issue_count", 1,
+      "rating", 2,
+      "hotspot_count", 3,
+      "hotspots_reviewed", 4);
+
+    assertThat(db.countRowsOfTable(dbSession, "issue_stats_by_rule_key")).isEqualTo(2);
+    underTest.deleteBranch(dbSession, branch1.getUuid());
+    assertThat(db.countRowsOfTable(dbSession, "issue_stats_by_rule_key")).isEqualTo(1);
+  }
+
   @Test
   void deleteProject_purgesScaLicenseProfiles() {
     ProjectDto project = db.components().insertPublicProject().getProjectDto();

server/sonar-db-dao/src/main/java/org/sonar/db/purge/PurgeCommands.java

@@ -524,6 +524,13 @@ public void deleteArchitectureGraphs(String branchUuid) {
     profiler.stop();
   }
 
+  public void deleteIssueStatsByRuleKey(String branchUuid) {
+    profiler.start("deleteIssueStatsByRuleKey (issue_stats_by_rule_key)");
+    purgeMapper.deleteIssueStatsByRuleKey(branchUuid);
+    session.commit();
+    profiler.stop();
+  }
+
   public void deleteAnticipatedTransitions(String projectUuid, long createdAt) {
     profiler.start("deleteAnticipatedTransitions (anticipated_transitions)");
     purgeMapper.deleteAnticipatedTransitionsByProjectUuidAndCreationDate(projectUuid, createdAt);

server/sonar-db-dao/src/main/java/org/sonar/db/purge/PurgeDao.java

@@ -283,6 +283,7 @@ private static void deleteBranch(String branchUuid, PurgeCommands commands) {
     commands.deleteIssuesFixed(branchUuid);
     commands.deleteScaActivity(branchUuid);
     commands.deleteArchitectureGraphs(branchUuid);
+    commands.deleteIssueStatsByRuleKey(branchUuid);
   }
 
   private static void deleteProject(String projectUuid, PurgeMapper mapper, PurgeCommands commands) {

server/sonar-db-dao/src/main/java/org/sonar/db/purge/PurgeMapper.java

@@ -208,4 +208,7 @@ void deleteCeActivityByRootUuidOrBefore(@Nullable @Param("rootUuid") String root
   void deleteScaLicenseProfileProjectsByProjectUuid(@Param("projectUuid") String projectUuid);
 
   void deleteArchitectureGraphsByBranchUuid(@Param("branchUuid") String branchUuid);
+
+  void deleteIssueStatsByRuleKey(@Param("branchUuid") String branchUuid);
+
 }

server/sonar-db-dao/src/main/resources/org/sonar/db/purge/PurgeMapper.xml

@@ -694,4 +694,7 @@
   <delete id="deleteScaLicenseProfileProjectsByProjectUuid">
     delete from sca_lic_prof_projects where project_uuid = #{projectUuid,jdbcType=VARCHAR}
   </delete>
+  <delete id="deleteIssueStatsByRuleKey">
+    delete from issue_stats_by_rule_key where aggregation_type = 'PROJECT' and aggregation_id = #{branchUuid,jdbcType=VARCHAR}
+  </delete>
 </mapper>