diff options
| author | Elena Vilchik | 2019-06-12 11:04:18 +0200 |
|---|---|---|
| committer | Tibor Blenessy | 2019-06-12 11:04:18 +0200 |
| commit | 5e1b4ef423a1ca5943232ad332d3230d9d09ec82 (patch) | |
| tree | 8b6f9bb15b3c4d141809ae5d1aef35026e08a1c1 /sonar-css-plugin | |
| parent | 1410b857a28a37feb6268c27fd3b075b691a234f (diff) | |
| download | sonar-css-5e1b4ef423a1ca5943232ad332d3230d9d09ec82.tar.bz2 | |
Ignore Angular Material selector types in S4670 (#177)
Diffstat (limited to 'sonar-css-plugin')
| -rw-r--r-- | sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/SelectorTypeNoUnknown.java | 12 | ||||
| -rw-r--r-- | sonar-css-plugin/src/test/java/org/sonar/css/plugin/rules/CssRuleTest.java | 9 |
2 files changed, 20 insertions, 1 deletions
diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/SelectorTypeNoUnknown.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/SelectorTypeNoUnknown.java index 9511d1b..a42fa9a 100644 --- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/SelectorTypeNoUnknown.java +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/SelectorTypeNoUnknown.java @@ -19,6 +19,8 @@ */ package org.sonar.css.plugin.rules; +import java.util.Arrays; +import java.util.List; import org.sonar.check.Rule; @Rule(key = "S4670") @@ -28,4 +30,14 @@ public class SelectorTypeNoUnknown implements CssRule { public String stylelintKey() { return "selector-type-no-unknown"; } + + @Override + public List<Object> stylelintOptions() { + return Arrays.asList(true, new StylelintIgnoreOption()); + } + + private static class StylelintIgnoreOption { + // Used by GSON serialization + private final String[] ignoreTypes = {"/^mat-/"}; + } } diff --git a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/rules/CssRuleTest.java b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/rules/CssRuleTest.java index feca51e..9016115 100644 --- a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/rules/CssRuleTest.java +++ b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/rules/CssRuleTest.java @@ -45,7 +45,8 @@ public class CssRuleTest { AtRuleNoUnknown.class, DeclarationBlockNoDuplicateProperties.class, PropertyNoUnknown.class, - SelectorPseudoClassNoUnknown.class); + SelectorPseudoClassNoUnknown.class, + SelectorTypeNoUnknown.class); for (Class ruleClass : CssRules.getRuleClasses()) { CssRule rule = (CssRule)ruleClass.getConstructor().newInstance(); @@ -70,6 +71,12 @@ public class CssRuleTest { } @Test + public void selector_type_no_unknown_options() { + String optionsAsJson = new Gson().toJson(new SelectorTypeNoUnknown().stylelintOptions()); + assertThat(optionsAsJson).isEqualTo("[true,{\"ignoreTypes\":[\"/^mat-/\"]}]"); + } + + @Test public void at_rule_unknown_default() { String optionsAsJson = new Gson().toJson(new AtRuleNoUnknown().stylelintOptions()); assertThat(optionsAsJson).isEqualTo("[true,{\"ignoreAtRules\":[\"value\",\"at-root\",\"content\",\"debug\",\"each\",\"else\",\"error\",\"for\",\"function\",\"if\",\"include\",\"mixin\",\"return\",\"warn\",\"while\",\"extend\"]}]"); |
