Fix backwards incompatible change in SonarLint metadata file (#3712)

SonarSource · Feb 16, 2023 · f58ba51 · f58ba51
1 parent ebc205f
commit f58ba51
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/...pt-plugin/src/main/java/org/sonar/plugins/javascript/utils/RulesMetadataForSonarLint.java b/...pt-plugin/src/main/java/org/sonar/plugins/javascript/utils/RulesMetadataForSonarLint.java
@@ -49,7 +49,6 @@
 import org.sonar.plugins.javascript.JavaScriptLanguage;
 import org.sonar.plugins.javascript.JavaScriptProfilesDefinition;
 import org.sonar.plugins.javascript.api.EslintBasedCheck;
-import org.sonar.plugins.javascript.api.JavaScriptCheck;
 import org.sonar.plugins.javascript.css.CssProfileDefinition;
 import org.sonar.plugins.javascript.css.CssRules;
 import org.sonar.plugins.javascript.css.CssRulesDefinition;
@@ -175,7 +174,7 @@ static class Rule {
     private RuleStatus status = RuleStatus.defaultStatus();
     private Set<String> tags;
     private List<RulesDefinition.Param> params;
-    private List<Object> defaultParams;
+    private List<Object> defaultParams = Collections.emptyList();
     private RuleScope scope;
     private String eslintKey;
     private boolean activatedByDefault;

diff --git a/...lugin/src/test/java/org/sonar/plugins/javascript/utils/RulesMetadataForSonarLintTest.java b/...lugin/src/test/java/org/sonar/plugins/javascript/utils/RulesMetadataForSonarLintTest.java
@@ -32,12 +32,10 @@
 import org.sonar.javascript.checks.StringLiteralsQuotesCheck;
 import org.sonar.plugins.javascript.JavaScriptProfilesDefinition;
 import org.sonar.plugins.javascript.css.CssProfileDefinition;
-import org.sonar.plugins.javascript.css.CssRules;
 import org.sonar.plugins.javascript.css.CssRulesDefinition;
 import org.sonar.plugins.javascript.css.rules.AtRuleNoUnknown;
 import org.sonar.plugins.javascript.rules.JavaScriptRulesDefinition;
 
-import static java.util.Arrays.asList;
 import static org.assertj.core.api.Assertions.assertThat;
 
 class RulesMetadataForSonarLintTest {
@@ -156,6 +154,8 @@ void test_all() throws Exception {
       .map(element -> element.getAsJsonObject().get("scope").getAsString())
       .collect(Collectors.toSet());
     assertThat(scopes).isEqualTo(Set.of("ALL", "MAIN", "TEST"));
+
+    assertThat(jsonArray).extracting(j -> j.getAsJsonObject().get("defaultParams")).doesNotContainNull();
   }
 
 }