diff options
author | Duarte Meneses <duarte.meneses@sonarsource.com> | 2018-04-13 15:59:23 +0200 |
---|---|---|
committer | SonarTech <sonartech@sonarsource.com> | 2018-04-26 20:20:51 +0200 |
commit | 6c5aad9a2d48cd783993a3a31911e307bf6827f5 (patch) | |
tree | e12910dde39fb187663a7f9bf6c93d73bb45efef /sonar-plugin-api/src/test/java | |
parent | 4cbf46ae604e6619c5b1fdaab81f5dd28f464a0e (diff) | |
download | sonarqube-6c5aad9a2d48cd783993a3a31911e307bf6827f5.tar.gz sonarqube-6c5aad9a2d48cd783993a3a31911e307bf6827f5.zip |
SONAR-10543 Update test to v7.2 and remove descriptionUrl and ruleTitle from scanner
Diffstat (limited to 'sonar-plugin-api/src/test/java')
2 files changed, 0 insertions, 31 deletions
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/internal/SensorContextTesterTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/internal/SensorContextTesterTest.java index b47ef208fed..0cee9bc2a96 100644 --- a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/internal/SensorContextTesterTest.java +++ b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/internal/SensorContextTesterTest.java @@ -117,7 +117,6 @@ public class SensorContextTesterTest { .forRule(RuleKey.of("repo", "rule")) .type(RuleType.BUG) .severity(Severity.BLOCKER) - .ruleTitle("title") .save(); newExternalIssue = tester.newExternalIssue(); newExternalIssue @@ -125,7 +124,6 @@ public class SensorContextTesterTest { .type(RuleType.BUG) .severity(Severity.BLOCKER) .forRule(RuleKey.of("repo", "rule")) - .ruleTitle("title") .save(); assertThat(tester.allExternalIssues()).hasSize(2); } diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssueTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssueTest.java index 57998dc4ba8..1176527f04c 100644 --- a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssueTest.java +++ b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssueTest.java @@ -52,19 +52,15 @@ public class DefaultExternalIssueTest { .message("Wrong way!")) .forRule(RuleKey.of("repo", "rule")) .remediationEffort(10l) - .descriptionUrl("url") .type(RuleType.BUG) - .ruleTitle("rule") .severity(Severity.BLOCKER); assertThat(issue.primaryLocation().inputComponent()).isEqualTo(inputFile); assertThat(issue.ruleKey()).isEqualTo(RuleKey.of("repo", "rule")); assertThat(issue.primaryLocation().textRange().start().line()).isEqualTo(1); assertThat(issue.remediationEffort()).isEqualTo(10l); - assertThat(issue.descriptionUrl()).isEqualTo("url"); assertThat(issue.type()).isEqualTo(RuleType.BUG); assertThat(issue.severity()).isEqualTo(Severity.BLOCKER); - assertThat(issue.ruleTitle()).isEqualTo("rule"); assertThat(issue.primaryLocation().message()).isEqualTo("Wrong way!"); issue.save(); @@ -82,8 +78,6 @@ public class DefaultExternalIssueTest { .message("Wrong way!")) .forRule(RuleKey.of("repo", "rule")) .remediationEffort(10l) - .descriptionUrl("url") - .ruleTitle("rule") .severity(Severity.BLOCKER); exception.expect(IllegalStateException.class); @@ -100,8 +94,6 @@ public class DefaultExternalIssueTest { .message("Wrong way!")) .forRule(RuleKey.of("repo", "rule")) .remediationEffort(10l) - .descriptionUrl("url") - .ruleTitle("rule") .severity(Severity.BLOCKER); exception.expect(IllegalStateException.class); @@ -119,8 +111,6 @@ public class DefaultExternalIssueTest { .message("Wrong way!")) .forRule(RuleKey.of("repo", "rule")) .remediationEffort(10l) - .descriptionUrl("url") - .ruleTitle("rule") .type(RuleType.BUG); exception.expect(IllegalStateException.class); @@ -128,23 +118,4 @@ public class DefaultExternalIssueTest { issue.save(); } - @Test - public void fail_to_store_if_no_rule_title() { - SensorStorage storage = mock(SensorStorage.class); - DefaultExternalIssue issue = new DefaultExternalIssue(storage) - .at(new DefaultIssueLocation() - .on(inputFile) - .at(inputFile.selectLine(1)) - .message("Wrong way!")) - .forRule(RuleKey.of("repo", "rule")) - .remediationEffort(10l) - .descriptionUrl("url") - .severity(Severity.BLOCKER) - .type(RuleType.BUG); - - exception.expect(IllegalStateException.class); - exception.expectMessage("Rule title is mandatory"); - issue.save(); - } - } |