aboutsummaryrefslogtreecommitdiffstats
path: root/server
diff options
context:
space:
mode:
authorJulien Lancelot <julien.lancelot@sonarsource.com>2015-09-29 13:42:37 +0200
committerJulien Lancelot <julien.lancelot@sonarsource.com>2015-09-29 13:42:53 +0200
commitae8cfc366beb622ea95e2b6654bbc6d404764f02 (patch)
treef9927b4e46b54a960cd4ccafab1342511ef4c323 /server
parent8610286a81782605b0ec803c4981d28ddae7fca0 (diff)
downloadsonarqube-ae8cfc366beb622ea95e2b6654bbc6d404764f02.tar.gz
sonarqube-ae8cfc366beb622ea95e2b6654bbc6d404764f02.zip
SONAR-6547 Improve log message
Diffstat (limited to 'server')
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/startup/ClearRulesOverloadedDebt.java5
-rw-r--r--server/sonar-server/src/test/java/org/sonar/server/startup/ClearRulesOverloadedDebtTest.java11
2 files changed, 7 insertions, 9 deletions
diff --git a/server/sonar-server/src/main/java/org/sonar/server/startup/ClearRulesOverloadedDebt.java b/server/sonar-server/src/main/java/org/sonar/server/startup/ClearRulesOverloadedDebt.java
index 5ef82d238a8..32172921815 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/startup/ClearRulesOverloadedDebt.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/startup/ClearRulesOverloadedDebt.java
@@ -82,9 +82,8 @@ public class ClearRulesOverloadedDebt implements Startable {
}
}
if (countClearedRules > 0) {
- LOG.warn("The SQALE model has been cleaned to remove useless data left over by previous migrations. The technical debt of {} rules was reset to their default values.",
- countClearedRules);
- LOG.warn("=> As a consequence, the overall technical debt of your projects might slightly evolve during the next analysis.");
+ LOG.warn("The SQALE model has been cleaned to remove any redundant data left over from previous migrations.");
+ LOG.warn("=> As a result, the technical debt of existing issues in your projects may change slightly when those projects are reanalyzed.");
}
}
diff --git a/server/sonar-server/src/test/java/org/sonar/server/startup/ClearRulesOverloadedDebtTest.java b/server/sonar-server/src/test/java/org/sonar/server/startup/ClearRulesOverloadedDebtTest.java
index e31f79e6bba..ebb0c59721a 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/startup/ClearRulesOverloadedDebtTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/startup/ClearRulesOverloadedDebtTest.java
@@ -88,7 +88,7 @@ public class ClearRulesOverloadedDebtTest {
verifyRuleHasNotOverriddenDebt(RULE_KEY_2);
verifyRuleHasNotOverriddenDebt(RULE_KEY_3);
verifyTaskRegistered();
- verifyLog(3);
+ verifyLog();
}
@Test
@@ -128,7 +128,7 @@ public class ClearRulesOverloadedDebtTest {
public void not_execute_task_when_already_executed() throws Exception {
insertRuleDto(RULE_KEY_1, SUB_CHARACTERISTIC_ID, "LINEAR", null, "1d");
underTest.start();
- verifyLog(1);
+ verifyLog();
verifyTaskRegistered();
logTester.clear();
@@ -169,11 +169,10 @@ public class ClearRulesOverloadedDebtTest {
assertThat(dbClient.loadedTemplateDao().countByTypeAndKey(ONE_SHOT_TASK_TYPE, "ClearRulesOverloadedDebt")).isEqualTo(1);
}
- private void verifyLog(int nbOfUpdatedRules) {
+ private void verifyLog() {
assertThat(logTester.logs(LoggerLevel.WARN)).containsOnly(
- "The SQALE model has been cleaned to remove useless data left over by previous migrations. The technical debt of " + nbOfUpdatedRules
- + " rules was reset to their default values.",
- "=> As a consequence, the overall technical debt of your projects might slightly evolve during the next analysis.");
+ "The SQALE model has been cleaned to remove any redundant data left over from previous migrations.",
+ "=> As a result, the technical debt of existing issues in your projects may change slightly when those projects are reanalyzed.");
}
private void verifyEmptyLog() {