From e766007679a913e498584004acf96d55eb58329d Mon Sep 17 00:00:00 2001
From: Wojtek Wajerowicz
 <115081248+wojciech-wajerowicz-sonarsource@users.noreply.github.com>
Date: Thu, 15 Jun 2023 13:37:23 +0200
Subject: SONAR-19383 Remove unnecessary methods from CeTaskMessagesDao.
 (#8539)

---
 .../server/startup/UpgradeSuggestionsCleanerIT.java  | 20 +++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

(limited to 'server/sonar-webserver-core')

diff --git a/server/sonar-webserver-core/src/it/java/org/sonar/server/startup/UpgradeSuggestionsCleanerIT.java b/server/sonar-webserver-core/src/it/java/org/sonar/server/startup/UpgradeSuggestionsCleanerIT.java
index fdf008ac82a..b621a7e8238 100644
--- a/server/sonar-webserver-core/src/it/java/org/sonar/server/startup/UpgradeSuggestionsCleanerIT.java
+++ b/server/sonar-webserver-core/src/it/java/org/sonar/server/startup/UpgradeSuggestionsCleanerIT.java
@@ -22,6 +22,7 @@ package org.sonar.server.startup;
 import com.tngtech.java.junit.dataprovider.DataProvider;
 import com.tngtech.java.junit.dataprovider.DataProviderRunner;
 import com.tngtech.java.junit.dataprovider.UseDataProvider;
+import java.util.List;
 import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
@@ -30,6 +31,8 @@ import org.sonar.api.SonarEdition;
 import org.sonar.api.SonarRuntime;
 import org.sonar.api.utils.System2;
 import org.sonar.db.DbTester;
+import org.sonar.db.ce.CeActivityDto;
+import org.sonar.db.ce.CeQueueDto;
 import org.sonar.db.ce.CeTaskMessageDto;
 import org.sonar.db.ce.CeTaskMessageType;
 import org.sonar.db.user.UserDismissedMessageDto;
@@ -79,6 +82,7 @@ public class UpgradeSuggestionsCleanerIT {
   @UseDataProvider("editionsWithCleanup")
   public void start_cleans_up_obsolete_upgrade_suggestions(SonarEdition edition) {
     when(sonarRuntime.getEdition()).thenReturn(edition);
+    insertTask(TASK_UUID);
     insertCeTaskMessage("ctm1", CeTaskMessageType.GENERIC, "msg1");
     insertCeTaskMessage("ctm2", CeTaskMessageType.GENERIC, "msg2");
     insertCeTaskMessage("ctm3", CeTaskMessageType.SUGGEST_DEVELOPER_EDITION_UPGRADE, "upgrade-msg-1");
@@ -88,7 +92,7 @@ public class UpgradeSuggestionsCleanerIT {
     underTest.start();
     underTest.stop();
 
-    assertThat(dbTester.getDbClient().ceTaskMessageDao().selectByTask(dbTester.getSession(), TASK_UUID))
+    assertThat(getTaskMessages(TASK_UUID))
       .extracting(CeTaskMessageDto::getUuid)
       .containsExactly("ctm1", "ctm2");
     assertThat(dbTester.getDbClient().userDismissedMessagesDao().selectByUser(dbTester.getSession(), user))
@@ -99,6 +103,7 @@ public class UpgradeSuggestionsCleanerIT {
   @Test
   public void start_does_nothing_in_community_edition() {
     when(sonarRuntime.getEdition()).thenReturn(SonarEdition.COMMUNITY);
+    insertTask(TASK_UUID);
     insertCeTaskMessage("ctm1", CeTaskMessageType.GENERIC, "msg1");
     insertCeTaskMessage("ctm2", CeTaskMessageType.GENERIC, "msg2");
     insertCeTaskMessage("ctm3", CeTaskMessageType.SUGGEST_DEVELOPER_EDITION_UPGRADE, "upgrade-msg-1");
@@ -107,7 +112,7 @@ public class UpgradeSuggestionsCleanerIT {
 
     underTest.start();
 
-    assertThat(dbTester.getDbClient().ceTaskMessageDao().selectByTask(dbTester.getSession(), TASK_UUID))
+    assertThat(getTaskMessages(TASK_UUID))
       .extracting(CeTaskMessageDto::getUuid)
       .containsExactly("ctm1", "ctm2", "ctm3");
     assertThat(dbTester.getDbClient().userDismissedMessagesDao().selectByUser(dbTester.getSession(), user))
@@ -122,10 +127,19 @@ public class UpgradeSuggestionsCleanerIT {
 
     underTest.start();
 
-    assertThat(dbTester.getDbClient().ceTaskMessageDao().selectByTask(dbTester.getSession(), TASK_UUID)).isEmpty();
+    assertThat(getTaskMessages(TASK_UUID)).isEmpty();
     assertThat(dbTester.getDbClient().userDismissedMessagesDao().selectByUser(dbTester.getSession(), user)).isEmpty();
   }
 
+  private List<CeTaskMessageDto> getTaskMessages(String taskUuid) {
+    return dbTester.getDbClient().ceActivityDao().selectByUuid(dbTester.getSession(), taskUuid).map(CeActivityDto::getCeTaskMessageDtos).orElse(List.of());
+  }
+
+  private void insertTask(String taskUuid) {
+    dbTester.getDbClient().ceActivityDao().insert(dbTester.getSession(),
+      new CeActivityDto(new CeQueueDto().setUuid(taskUuid).setTaskType("ISSUE_SYNC")).setStatus(CeActivityDto.Status.FAILED));
+  }
+
   private void insertCeTaskMessage(String uuid, CeTaskMessageType messageType, String msg) {
     CeTaskMessageDto dto = new CeTaskMessageDto()
       .setUuid(uuid)
-- 
cgit v1.2.3