diff options
author | Julien HENRY <julien.henry@sonarsource.com> | 2018-09-06 18:02:56 +0200 |
---|---|---|
committer | SonarTech <sonartech@sonarsource.com> | 2018-09-24 20:20:58 +0200 |
commit | 1cacbb1d2ed72d720058580746d678ba8da1b453 (patch) | |
tree | 3d08e44365ae38fb975239385d301fc64973f541 /sonar-plugin-api/src/test/java/org/sonar | |
parent | cfba7fcb6500d8217bd81ecfcb8f47ec48ad55f2 (diff) | |
download | sonarqube-1cacbb1d2ed72d720058580746d678ba8da1b453.tar.gz sonarqube-1cacbb1d2ed72d720058580746d678ba8da1b453.zip |
SONAR-11209 Store ad hoc rules coming from scanner in rules_metadata
Diffstat (limited to 'sonar-plugin-api/src/test/java/org/sonar')
-rw-r--r-- | sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/rule/internal/DefaultAdHocRuleTest.java | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/rule/internal/DefaultAdHocRuleTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/rule/internal/DefaultAdHocRuleTest.java index 7a2b38084bf..241ae701379 100644 --- a/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/rule/internal/DefaultAdHocRuleTest.java +++ b/sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/rule/internal/DefaultAdHocRuleTest.java @@ -51,6 +51,21 @@ public class DefaultAdHocRuleTest { verify(storage).store(any(DefaultAdHocRule.class)); } + + @Test + public void description_is_optional() { + SensorStorage storage = mock(SensorStorage.class); + new DefaultAdHocRule(storage) + .engineId("engine") + .ruleId("ruleId") + .name("name") + .severity(Severity.BLOCKER) + .type(RuleType.CODE_SMELL) + .save(); + + verify(storage).store(any(DefaultAdHocRule.class)); + } + @Test public void fail_to_store_if_no_engine_id() { SensorStorage storage = mock(SensorStorage.class); @@ -99,21 +114,6 @@ public class DefaultAdHocRuleTest { rule.save(); } - @Test - public void fail_to_store_if_no_description() { - SensorStorage storage = mock(SensorStorage.class); - NewAdHocRule rule = new DefaultAdHocRule(storage) - .engineId("engine") - .ruleId("ruleId") - .name("name") - .description(" ") - .severity(Severity.BLOCKER) - .type(RuleType.CODE_SMELL); - - exception.expect(IllegalStateException.class); - exception.expectMessage("Description is mandatory"); - rule.save(); - } @Test public void fail_to_store_if_no_severity() { |