Add sanity check on adding a new rule property

sonar-plugin/css/src/test/java/org/sonar/css/rules/CssRuleTest.java

@@ -22,14 +22,18 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import com.google.gson.Gson;
+import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
 import java.util.Set;
 import org.junit.jupiter.api.Test;
 import org.mockito.internal.util.collections.Sets;
+import org.sonar.check.RuleProperty;
 import org.sonar.css.CssRules;
 
 class CssRuleTest {
 
+  private static final int RULES_PROPERTIES_COUNT = 9;
+
   @Test
   void class_name_should_match_stylelint_key()
     throws NoSuchMethodException, IllegalAccessException, InvocationTargetException, InstantiationException {
@@ -64,6 +68,23 @@ void rules_default_is_empty()
     }
   }
 
+  /*
+   * This test raises awareness of the consequence of a rule adding or removing a rule property.
+   * If a new rule property is added, we should inform the SonarCloud team about it on release.
+   */
+  @Test
+  void rules_properties_count() {
+    var count = 0;
+    for (Class clazz : CssRules.getRuleClasses()) {
+      for (Field field : clazz.getDeclaredFields()) {
+        if (field.isAnnotationPresent(RuleProperty.class)) {
+          count++;
+        }
+      }
+    }
+    assertThat(count).isEqualTo(RULES_PROPERTIES_COUNT);
+  }
+
   @Test
   void selector_pseudo_class_options() {
     SelectorPseudoClassNoUnknown selectorPseudoClassNoUnknown = new SelectorPseudoClassNoUnknown();

sonar-plugin/javascript-checks/src/test/java/org/sonar/javascript/checks/CheckListTest.java

@@ -21,6 +21,7 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
+import java.lang.reflect.Field;
 import java.nio.file.Files;
 import java.nio.file.Paths;
 import java.util.ArrayList;
@@ -29,11 +30,14 @@
 import java.util.Set;
 import java.util.stream.Collectors;
 import org.junit.jupiter.api.Test;
+import org.sonar.check.RuleProperty;
 import org.sonar.plugins.javascript.api.EslintBasedCheck;
 import org.sonar.plugins.javascript.api.JavaScriptCheck;
 
 class CheckListTest {
 
+  private static final int CHECKS_PROPERTIES_COUNT = 35;
+
   /**
    * Enforces that each check declared in list.
    */
@@ -133,6 +137,23 @@ void testEveryCheckBelongsToLanguage() {
     assertThat(allChecks).isEqualTo(tsAndJsChecks);
   }
 
+  /*
+   * This test raises awareness of the consequence of a rule adding or removing a rule property.
+   * If a new rule property is added, we should inform the SonarCloud team about it on release.
+   */
+  @Test
+  void testChecksPropertiesCount() {
+    var count = 0;
+    for (Class clazz : CheckList.getAllChecks()) {
+      for (Field field : clazz.getDeclaredFields()) {
+        if (field.isAnnotationPresent(RuleProperty.class)) {
+          count++;
+        }
+      }
+    }
+    assertThat(count).isEqualTo(CHECKS_PROPERTIES_COUNT);
+  }
+
   private boolean isEslintBasedCheck(Class<? extends JavaScriptCheck> cls) {
     try {
       cls.getMethod("eslintKey");