diff options
author | Léo Geoffroy <leo.geoffroy@sonarsource.com> | 2022-06-23 12:21:38 +0200 |
---|---|---|
committer | sonartech <sonartech@sonarsource.com> | 2022-06-30 20:03:09 +0000 |
commit | b2079411492f9c056c92c619d2ccf08e86e88e5c (patch) | |
tree | d41cad00a40352d3dc93d9ec46eb4092319c9254 /server/sonar-server-common | |
parent | a410c8e18142d77c4e19ef10ce6d420526d5c236 (diff) | |
download | sonarqube-b2079411492f9c056c92c619d2ccf08e86e88e5c.tar.gz sonarqube-b2079411492f9c056c92c619d2ccf08e86e88e5c.zip |
SONAR-16518 Add context to RuleDescriptionSectionDto
Diffstat (limited to 'server/sonar-server-common')
2 files changed, 9 insertions, 8 deletions
diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexTest.java index 41b786e2751..94f38180b32 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexTest.java @@ -216,20 +216,20 @@ public class RuleIndexTest { // otherwise the generated random values may raise false-positives RuleDto rule1 = createJavaRule(rule -> rule.setRuleKey("123") .setName("rule 123") - .addOrReplaceRuleDescriptionSectionDto(createDefaultRuleDescriptionSection(uuidFactory.create(), "My great rule CWE-123 which makes your code 1000 times better!"))); + .replaceRuleDescriptionSectionDtos(createDefaultRuleDescriptionSection(uuidFactory.create(), "My great rule CWE-123 which makes your code 1000 times better!"))); RuleDto rule2 = createJavaRule(rule -> rule.setRuleKey("124") .setName("rule 124") - .addOrReplaceRuleDescriptionSectionDto(createDefaultRuleDescriptionSection(uuidFactory.create(), "Another great and shiny rule CWE-124"))); + .replaceRuleDescriptionSectionDtos(createDefaultRuleDescriptionSection(uuidFactory.create(), "Another great and shiny rule CWE-124"))); RuleDto rule3 = createJavaRule(rule -> rule.setRuleKey("1000") .setName("rule 1000") - .addOrReplaceRuleDescriptionSectionDto(createDefaultRuleDescriptionSection(uuidFactory.create(), "Another great rule CWE-1000"))); + .replaceRuleDescriptionSectionDtos(createDefaultRuleDescriptionSection(uuidFactory.create(), "Another great rule CWE-1000"))); RuleDto rule4 = createJavaRule(rule -> rule.setRuleKey("404") .setName("rule 404") - .addOrReplaceRuleDescriptionSectionDto(createDefaultRuleDescriptionSection(uuidFactory.create(), + .replaceRuleDescriptionSectionDtos(createDefaultRuleDescriptionSection(uuidFactory.create(), "<h1>HTML-Geeks</h1><p style=\"color:blue\">special formatting!</p><table><tr><td>inside</td><td>tables</td></tr></table>"))); RuleDto rule5 = createJavaRule(rule -> rule.setRuleKey("405") .setName("rule 405") - .addOrReplaceRuleDescriptionSectionDto(createDefaultRuleDescriptionSection(uuidFactory.create(), "internationalization missunderstandings alsdkjfnadklsjfnadkdfnsksdjfn"))); + .replaceRuleDescriptionSectionDtos(createDefaultRuleDescriptionSection(uuidFactory.create(), "internationalization missunderstandings alsdkjfnadklsjfnadkdfnsksdjfn"))); index(); // partial match at word boundary diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexerTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexerTest.java index c47c8ab560c..348a1f4c3d3 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexerTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/rule/index/RuleIndexerTest.java @@ -139,7 +139,8 @@ public class RuleIndexerTest { public void index_long_rule_description() { String description = IntStream.range(0, 100000).map(i -> i % 100).mapToObj(Integer::toString).collect(joining(" ")); RuleDescriptionSectionDto ruleDescriptionSectionDto = createDefaultRuleDescriptionSection(uuidFactory.create(), description); - RuleDto rule = dbTester.rules().insert(r -> r.addOrReplaceRuleDescriptionSectionDto(ruleDescriptionSectionDto)); + + RuleDto rule = dbTester.rules().insert(r -> r.replaceRuleDescriptionSectionDtos(ruleDescriptionSectionDto)); underTest.commitAndIndex(dbTester.getSession(), rule.getUuid()); assertThat(es.countDocuments(TYPE_RULE)).isOne(); @@ -148,7 +149,7 @@ public class RuleIndexerTest { @Test public void index_long_rule_with_several_sections() { RuleDto rule = dbTester.rules().insert(r -> { - r.addOrReplaceRuleDescriptionSectionDto(RULE_DESCRIPTION_SECTION_DTO); + r.replaceRuleDescriptionSectionDtos(RULE_DESCRIPTION_SECTION_DTO); r.addRuleDescriptionSectionDto(RULE_DESCRIPTION_SECTION_DTO2); }); @@ -166,7 +167,7 @@ public class RuleIndexerTest { public void index_long_rule_with_section_in_markdown() { RuleDto rule = dbTester.rules().insert(r -> { r.setDescriptionFormat(RuleDto.Format.MARKDOWN); - r.addOrReplaceRuleDescriptionSectionDto(RULE_DESCRIPTION_SECTION_DTO); + r.replaceRuleDescriptionSectionDtos(RULE_DESCRIPTION_SECTION_DTO); }); underTest.commitAndIndex(dbTester.getSession(), rule.getUuid()); |