SONAR-5195 Revert preventing rule template to declare technical debt

2 files changed, 0 insertions, 25 deletions

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
index ba5870588ca..831a05f1155 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
@@ -687,9 +687,6 @@ public interface RulesDefinition extends ServerExtension {
       if ((Strings.isNullOrEmpty(debtSubCharacteristic) && debtRemediationFunction != null) || (!Strings.isNullOrEmpty(debtSubCharacteristic) && debtRemediationFunction == null)) {
         throw new IllegalStateException(String.format("Both debt sub-characteristic and debt remediation function should be defined on rule '%s'", this));
       }
-      if (!Strings.isNullOrEmpty(debtSubCharacteristic) && template) {
-        throw new IllegalStateException(String.format("'%s' is a rule template, it should not define technical debt.", this));
-      }
     }
 
     @Override
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/server/rule/RulesDefinitionTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/server/rule/RulesDefinitionTest.java
index 43a88eaffe0..1027e8ba3fc 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/server/rule/RulesDefinitionTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/server/rule/RulesDefinitionTest.java
@@ -340,26 +340,4 @@ public class RulesDefinitionTest {
     }
   }
 
-  /**
-   * SONAR-5195
-   */
-  @Test
-  public void fail_if_rule_template_define_technical_debt() {
-    RulesDefinition.NewRepository newRepo = context.createRepository("squid", "java");
-    RulesDefinition.NewRule newRule = newRepo.createRule("XPath rule")
-      .setTemplate(true)
-      .setName("Insufficient branch coverage")
-      .setHtmlDescription("This rule allows to define some homemade Java rules with help of an XPath expression.")
-      .setSeverity(Severity.MAJOR)
-      .setDebtSubCharacteristic(RulesDefinition.SubCharacteristics.UNIT_TESTS);
-    newRule.setDebtRemediationFunction(newRule.debtRemediationFunctions().linearWithOffset("1h", "10min"));
-
-    try {
-      newRepo.done();
-      fail();
-    } catch (IllegalStateException e) {
-      assertThat(e).hasMessage("'[repository=squid, key=XPath rule]' is a rule template, it should not define technical debt.");
-    }
-  }
-
 }