Fix #57 FP on rule class-selector-naming-convention within mixin calls

Author: racodond

css-checks/src/main/java/org/sonar/css/checks/common/ClassSelectorNamingConventionCheck.java

@@ -28,6 +28,7 @@
 import org.sonar.css.checks.Tags;
 import org.sonar.plugins.css.api.tree.css.ClassSelectorTree;
 import org.sonar.plugins.css.api.tree.css.IdentifierTree;
+import org.sonar.plugins.css.api.tree.less.LessMixinCallTree;
 import org.sonar.plugins.css.api.visitors.DoubleDispatchVisitorCheck;
 import org.sonar.squidbridge.annotations.ActivatedByDefault;
 import org.sonar.squidbridge.annotations.SqaleConstantRemediation;
@@ -56,7 +57,9 @@ public class ClassSelectorNamingConventionCheck extends DoubleDispatchVisitorChe
 
   @Override
   public void visitClassSelector(ClassSelectorTree tree) {
-    if (!tree.className().isInterpolated() && !tree.className().text().matches(format)) {
+    if (!tree.className().isInterpolated()
+      && !tree.className().text().matches(format)
+      && !tree.parent().hasAncestor(LessMixinCallTree.class)) {
       addIssue(tree.className());
     }
     super.visitClassSelector(tree);

css-checks/src/test/resources/checks/common/class-selector-naming-convention/selectorNamingConvention.less

@@ -15,3 +15,9 @@
 
 .abc@{DEF}ghi { /* Compliant */
 }
+
+.abc {
+  .Mymixin;
+  .Mymixin();
+  .Mymxin
+}

css-frontend/src/test/java/org/sonar/css/parser/less/StyleSheetTreeTest.java

@@ -39,6 +39,9 @@ public StyleSheetTreeTest() {
   public void stylesheet() throws Exception {
     StyleSheetTree tree;
 
+    tree = checkParsed(".abc { .space; }");
+    assertThat(tree.atRules()).isEmpty();
+
     checkParsed("");
     checkParsed(" ");
     checkParsed("  ");

its/ruling/projects/custom/common/class-selector-naming-convention/selectorNamingConvention.less

@@ -15,3 +15,9 @@
 
 .abc@{DEF}ghi { /* Compliant */
 }
+
+.abc {
+  .Mymixin;
+  .Mymixin();
+  .Mymxin
+}

its/ruling/tests/src/test/expected/css-alphabetize-declarations.json

@@ -4,7 +4,7 @@
 59,
 65,
 71,
-87,
+87,g
 101,
 106,
 111,

its/ruling/tests/src/test/expected/less-selector-naming-convention.json

@@ -4,6 +4,9 @@
 7,
 13,
 13,
+20,
+21,
+22,
 ],
 'project:custom/common/id-selector-naming-convention/selectorNamingConvention.less':[
 7,