Fix #105 Support CSS Fonts Module Level 4 - font-variant-emoji

css-checks/src/test/java/org/sonar/css/checks/common/ValidatePropertyValueCheckTest.java

@@ -75,9 +75,9 @@ public void test_ignore_scss_nested_properties() {
 
   @Test
   public void test_number_of_validated_properties() {
-    assertEquals(316, CheckTestUtils.getCommonTestFile("properties/css").listFiles().length);
-    assertEquals(316, CheckTestUtils.getCommonTestFile("properties/less").listFiles().length);
-    assertEquals(316, CheckTestUtils.getCommonTestFile("properties/scss").listFiles().length);
+    assertEquals(317, CheckTestUtils.getCommonTestFile("properties/css").listFiles().length);
+    assertEquals(317, CheckTestUtils.getCommonTestFile("properties/less").listFiles().length);
+    assertEquals(317, CheckTestUtils.getCommonTestFile("properties/scss").listFiles().length);
   }
 
 }

css-checks/src/test/resources/checks/common/properties/css/font-variant-emoji.css

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

css-checks/src/test/resources/checks/common/properties/less/font-variant-emoji.less

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

css-checks/src/test/resources/checks/common/properties/scss/font-variant-emoji.scss

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

css-frontend/src/main/java/org/sonar/css/model/property/StandardPropertyFactory.java

@@ -248,6 +248,7 @@ public class StandardPropertyFactory {
     FontVariantAlternates.class,
     FontVariantCaps.class,
     FontVariantEastAsian.class,
+    FontVariantEmoji.class,
     FontVariantLigatures.class,
     FontVariantNumeric.class,
     FontVariantPosition.class,

css-frontend/src/main/java/org/sonar/css/model/property/standard/FontVariantEmoji.java

@@ -0,0 +1,33 @@
+/*
+ * SonarQube CSS / SCSS / Less Analyzer
+ * Copyright (C) 2013-2017 David RACODON
+ * mailto: [email protected]
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+ */
+package org.sonar.css.model.property.standard;
+
+import org.sonar.css.model.property.StandardProperty;
+import org.sonar.css.model.property.validator.valueelement.IdentifierValidator;
+
+public class FontVariantEmoji extends StandardProperty {
+
+  public FontVariantEmoji() {
+    setExperimental(true);
+    addLinks("https://drafts.csswg.org/css-fonts-4/#propdef-font-variant-emoji");
+    addValidators(new IdentifierValidator("auto", "text", "emoji"));
+  }
+
+}

css-frontend/src/test/java/org/sonar/css/model/property/StandardPropertyFactoryTest.java

@@ -92,13 +92,13 @@ public void should_return_an_unknown_property_object() {
 
   @Test
   public void number_of_standard_properties() {
-    assertEquals(624, StandardPropertyFactory.getAll().size());
+    assertEquals(625, StandardPropertyFactory.getAll().size());
   }
 
   @Test
   public void number_of_experimental_properties() {
     assertEquals(
-      338,
+      339,
       StandardPropertyFactory
         .getAll()
         .stream()

its/ruling/projects/custom/common/properties/css/font-variant-emoji.css

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

its/ruling/projects/custom/common/properties/less/font-variant-emoji.less

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

its/ruling/projects/custom/common/properties/scss/font-variant-emoji.scss

@@ -0,0 +1,10 @@
+.mybox {
+  font-variant-emoji: inherit;
+  font-variant-emoji: auto;
+  font-variant-emoji: text;
+  font-variant-emoji: emoji;
+  font-variant-emoji: 0; /* Noncompliant {Update the invalid value of property "font-variant-emoji". Expected format: auto | none}! */
+  font-variant-emoji: 10px; /* Noncompliant */
+  font-variant-emoji: normal; /* Noncompliant */
+  font-variant-emoji: auto none; /* Noncompliant */
+}

its/ruling/tests/src/test/expected/css-experimental-property-usage.json

@@ -701,6 +701,16 @@
 11,
 12,
 ],
+'project:custom/common/properties/css/font-variant-emoji.css':[
+2,
+3,
+4,
+5,
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/css/font-variant-position.css':[
 2,
 3,

its/ruling/tests/src/test/expected/css-line-length.json

@@ -589,6 +589,9 @@
 'project:custom/common/properties/css/font-variant-caps.css':[
 10,
 ],
+'project:custom/common/properties/css/font-variant-emoji.css':[
+6,
+],
 'project:custom/common/properties/css/font-variant-position.css':[
 6,
 ],

its/ruling/tests/src/test/expected/css-validate-property-value.json

@@ -754,6 +754,12 @@
 11,
 12,
 ],
+'project:custom/common/properties/css/font-variant-emoji.css':[
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/css/font-variant-position.css':[
 6,
 7,

its/ruling/tests/src/test/expected/less-experimental-property-usage.json

@@ -686,6 +686,16 @@
 11,
 12,
 ],
+'project:custom/common/properties/less/font-variant-emoji.less':[
+2,
+3,
+4,
+5,
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/less/font-variant-position.less':[
 2,
 3,

its/ruling/tests/src/test/expected/less-line-length.json

@@ -483,6 +483,9 @@
 'project:custom/common/properties/less/font-variant-caps.less':[
 10,
 ],
+'project:custom/common/properties/less/font-variant-emoji.less':[
+6,
+],
 'project:custom/common/properties/less/font-variant-position.less':[
 6,
 ],

its/ruling/tests/src/test/expected/less-prefer-single-line-comments.json

@@ -1002,6 +1002,12 @@
 11,
 12,
 ],
+'project:custom/common/properties/less/font-variant-emoji.less':[
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/less/font-variant-position.less':[
 6,
 7,

its/ruling/tests/src/test/expected/less-validate-property-value.json

@@ -737,6 +737,12 @@
 11,
 12,
 ],
+'project:custom/common/properties/less/font-variant-emoji.less':[
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/less/font-variant-position.less':[
 6,
 7,

its/ruling/tests/src/test/expected/scss-experimental-property-usage.json

@@ -683,6 +683,16 @@
 11,
 12,
 ],
+'project:custom/common/properties/scss/font-variant-emoji.scss':[
+2,
+3,
+4,
+5,
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/scss/font-variant-position.scss':[
 2,
 3,

its/ruling/tests/src/test/expected/scss-line-length.json

@@ -507,6 +507,9 @@
 'project:custom/common/properties/scss/font-variant-caps.scss':[
 10,
 ],
+'project:custom/common/properties/scss/font-variant-emoji.scss':[
+6,
+],
 'project:custom/common/properties/scss/font-variant-position.scss':[
 6,
 ],

its/ruling/tests/src/test/expected/scss-prefer-single-line-comments.json

@@ -1031,6 +1031,12 @@
 11,
 12,
 ],
+'project:custom/common/properties/scss/font-variant-emoji.scss':[
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/scss/font-variant-position.scss':[
 6,
 7,

its/ruling/tests/src/test/expected/scss-validate-property-value.json

@@ -708,6 +708,12 @@
 11,
 12,
 ],
+'project:custom/common/properties/scss/font-variant-emoji.scss':[
+6,
+7,
+8,
+9,
+],
 'project:custom/common/properties/scss/font-variant-position.scss':[
 6,
 7,