SONARJAVA-1385 Remove activation by default

Author: benzonico

java-checks/src/main/java/org/sonar/java/checks/LoggersDeclarationCheck.java

@@ -34,7 +34,6 @@
 import org.sonar.plugins.java.api.tree.ModifiersTree;
 import org.sonar.plugins.java.api.tree.Tree;
 import org.sonar.plugins.java.api.tree.VariableTree;
-import org.sonar.squidbridge.annotations.ActivatedByDefault;
 import org.sonar.squidbridge.annotations.SqaleConstantRemediation;
 import org.sonar.squidbridge.annotations.SqaleSubCharacteristic;
 
@@ -45,7 +44,6 @@
   name = "Loggers should be \"private static final\" and should share a naming convention",
   priority = Priority.MINOR,
   tags = {Tag.CONVENTION})
-@ActivatedByDefault
 @SqaleSubCharacteristic(RulesDefinition.SubCharacteristics.UNDERSTANDABILITY)
 @SqaleConstantRemediation("5min")
 public class LoggersDeclarationCheck extends BaseTreeVisitor implements JavaFileScanner {

sonar-java-plugin/src/test/java/org/sonar/plugins/java/JavaSonarWayProfileTest.java

@@ -47,7 +47,7 @@ public void should_create_sonar_way_profile() {
 
     assertThat(profile.getLanguage()).isEqualTo(Java.KEY);
     List<ActiveRule> activeRules = profile.getActiveRulesByRepository(CheckList.REPOSITORY_KEY);
-    assertThat(activeRules.size()).as("Expected number of rules in profile").isGreaterThan(220);
+    assertThat(activeRules.size()).as("Expected number of rules in profile").isGreaterThanOrEqualTo(232);
     assertThat(profile.getName()).isEqualTo("Sonar way");
     assertThat(validation.hasErrors()).isFalse();
   }