SONAR-24021 Invoke method(s) only conditionally

3 files changed, 68 insertions, 0 deletions

diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/DeleteAllFilterTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/DeleteAllFilterTest.java
index bade600449e..0744d249be7 100644
--- a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/DeleteAllFilterTest.java
+++ b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/DeleteAllFilterTest.java
@@ -22,6 +22,9 @@ package org.sonar.db.purge.period;
 import java.util.Arrays;
 import java.util.List;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.RegisterExtension;
+import org.slf4j.event.Level;
+import org.sonar.api.testfixtures.log.LogTesterJUnit5;
 import org.sonar.api.utils.DateUtils;
 import org.sonar.db.purge.DbCleanerTestUtils;
 import org.sonar.db.purge.PurgeableAnalysisDto;
@@ -30,6 +33,9 @@ import static org.assertj.core.api.Assertions.assertThat;
 
 class DeleteAllFilterTest {
 
+  @RegisterExtension
+  private final LogTesterJUnit5 logs = new LogTesterJUnit5();
+
   @Test
   void shouldDeleteAllSnapshotsPriorToDate() {
     Filter filter = new DeleteAllFilter(DateUtils.parseDate("2011-12-25"));
@@ -41,4 +47,21 @@ class DeleteAllFilterTest {
 
     assertThat(toDelete).extracting("analysisUuid").containsOnly("u1", "u2");
   }
+
+  @Test
+  void log_should_log_debug_message_when_debug_enabled() {
+    DeleteAllFilter filter = new DeleteAllFilter(DateUtils.parseDate("2011-12-25"));
+    logs.setLevel(Level.DEBUG);
+    filter.log();
+    assertThat(logs.logs(Level.DEBUG)).contains("-> Delete data prior to: 2011-12-25");
+  }
+
+  @Test
+  void log_should_not_log_debug_message_when_debug_disabled() {
+    DeleteAllFilter filter = new DeleteAllFilter(DateUtils.parseDate("2011-12-25"));
+    logs.setLevel(Level.INFO);
+    filter.log();
+    assertThat(logs.logs()).isEmpty();
+  }
+
 }
diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepOneFilterTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepOneFilterTest.java
index 00dfa99f709..3f3ae1f6857 100644
--- a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepOneFilterTest.java
+++ b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepOneFilterTest.java
@@ -23,6 +23,9 @@ import java.util.Arrays;
 import java.util.Calendar;
 import java.util.List;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.RegisterExtension;
+import org.slf4j.event.Level;
+import org.sonar.api.testfixtures.log.LogTesterJUnit5;
 import org.sonar.api.utils.DateUtils;
 import org.sonar.db.purge.DbCleanerTestUtils;
 import org.sonar.db.purge.PurgeableAnalysisDto;
@@ -35,6 +38,9 @@ class KeepOneFilterTest {
     return snapshotDtos.stream().map(PurgeableAnalysisDto::getAnalysisUuid).toList();
   }
 
+  @RegisterExtension
+  private final LogTesterJUnit5 logs = new LogTesterJUnit5();
+
   @Test
   void shouldOnlyOneSnapshotPerInterval() {
     Filter filter = new KeepOneFilter(DateUtils.parseDate("2011-03-25"), DateUtils.parseDate("2011-08-25"), Calendar.MONTH, "month");
@@ -76,4 +82,20 @@ class KeepOneFilterTest {
     assertThat(KeepOneFilter.isDeletable(DbCleanerTestUtils.createAnalysisWithDate("u1", "2011-05-01").setHasEvents(true))).isFalse();
   }
 
+  @Test
+  void log_should_log_debug_message_when_debug_enabled() {
+    KeepOneFilter filter = new KeepOneFilter(DateUtils.parseDate("2011-03-25"), DateUtils.parseDate("2011-08-25"), Calendar.MONTH, "month");
+    logs.setLevel(Level.DEBUG);
+    filter.log();
+    assertThat(logs.logs(Level.DEBUG)).contains("-> Keep one snapshot per month between 2011-03-25 and 2011-08-25");
+  }
+
+  @Test
+  void log_should_not_log_debug_message_when_debug_disabled() {
+    KeepOneFilter filter = new KeepOneFilter(DateUtils.parseDate("2011-03-25"), DateUtils.parseDate("2011-08-25"), Calendar.MONTH, "month");
+    logs.setLevel(Level.INFO);
+    filter.log();
+    assertThat(logs.logs()).isEmpty();
+  }
+
 }
diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepWithVersionFilterTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepWithVersionFilterTest.java
index e6e7a435293..50cb3655ae1 100644
--- a/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepWithVersionFilterTest.java
+++ b/server/sonar-db-dao/src/test/java/org/sonar/db/purge/period/KeepWithVersionFilterTest.java
@@ -22,6 +22,9 @@ package org.sonar.db.purge.period;
 import java.util.Arrays;
 import java.util.List;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.RegisterExtension;
+import org.slf4j.event.Level;
+import org.sonar.api.testfixtures.log.LogTesterJUnit5;
 import org.sonar.db.purge.DbCleanerTestUtils;
 import org.sonar.db.purge.PurgeableAnalysisDto;
 
@@ -30,6 +33,9 @@ import static org.sonar.api.utils.DateUtils.parseDate;
 
 class KeepWithVersionFilterTest {
 
+  @RegisterExtension
+  private final LogTesterJUnit5 logs = new LogTesterJUnit5();
+
   @Test
   void keep_only_analyses_with_a_version() {
     Filter underTest = new KeepWithVersionFilter(parseDate("2015-10-18"));
@@ -41,4 +47,21 @@ class KeepWithVersionFilterTest {
 
     assertThat(result).extracting(PurgeableAnalysisDto::getAnalysisUuid).containsExactlyInAnyOrder("u2");
   }
+
+  @Test
+  void log_should_log_debug_message_when_debug_enabled() {
+    KeepWithVersionFilter filter = new KeepWithVersionFilter(parseDate("2015-10-18"));
+    logs.setLevel(Level.DEBUG);
+    filter.log();
+    assertThat(logs.logs(Level.DEBUG)).contains("-> Keep analyses with a version prior to 2015-10-18");
+  }
+
+  @Test
+  void log_should_not_log_debug_message_when_debug_disabled() {
+    KeepWithVersionFilter filter = new KeepWithVersionFilter(parseDate("2015-10-18"));
+    logs.setLevel(Level.INFO);
+    filter.log();
+    assertThat(logs.logs()).isEmpty();
+  }
+
 }