Move tests to appropriate testing class.

Author: JakeQZ

tests/CSSList/AtRuleBlockListTest.php

@@ -7,12 +7,46 @@
 use Sabberworm\CSS\CSSList\AtRuleBlockList;
 use Sabberworm\CSS\Parser;
 use Sabberworm\CSS\Renderable;
+use Sabberworm\CSS\Settings;
 
 /**
  * @covers \Sabberworm\CSS\CSSList\AtRuleBlockList
  */
 final class AtRuleBlockListTest extends TestCase
 {
+    /**
+     * @return array<string, array<int, string>>
+     */
+    public static function provideMinWidthMediaRule()
+    {
+        return [
+            'without spaces around arguments' => ['@media(min-width: 768px){.class{color:red}}'],
+            'with spaces around arguments' => ['@media (min-width: 768px) {.class{color:red}}'],
+        ];
+    }
+
+    /**
+     * @return array<string, array<int, string>>
+     */
+    public static function provideSyntacticlyCorrectAtRule()
+    {
+        return [
+            'media print' => ['@media print { html { background: white; color: black; } }'],
+            'keyframes' => ['@keyframes mymove { from { top: 0px; } }'],
+            'supports' => ['
+                @supports (display: flex) {
+                    .flex-container > * {
+                        text-shadow: 0 0 2px blue;
+                        float: none;
+                    }
+                    .flex-container {
+                        display: flex;
+                    }
+                }
+            '],
+        ];
+    }
+
     /**
      * @test
      */
@@ -43,23 +77,12 @@ public function implementsCommentable()
         self::assertInstanceOf(Commentable::class, $subject);
     }
 
-    /**
-     * @return array<string, array<int, string>>
-     */
-    public static function mediaRuleDataProvider()
-    {
-        return [
-            'without spaces around arguments' => ['@media(min-width: 768px){.class{color:red}}'],
-            'with spaces around arguments' => ['@media (min-width: 768px) {.class{color:red}}'],
-        ];
-    }
-
     /**
      * @test
      *
      * @param string $css
      *
-     * @dataProvider mediaRuleDataProvider
+     * @dataProvider provideMinWidthMediaRule
      */
     public function parsesRuleNameOfMediaQueries($css)
     {
@@ -74,7 +97,7 @@ public function parsesRuleNameOfMediaQueries($css)
      *
      * @param string $css
      *
-     * @dataProvider mediaRuleDataProvider
+     * @dataProvider provideMinWidthMediaRule
      */
     public function parsesArgumentsOfMediaQueries($css)
     {
@@ -83,4 +106,19 @@ public function parsesArgumentsOfMediaQueries($css)
 
         self::assertSame('(min-width: 768px)', $atRuleBlockList->atRuleArgs());
     }
+
+    /**
+     * @test
+     *
+     * @param string $css
+     *
+     * @dataProvider provideMinWidthMediaRule
+     * @dataProvider provideSyntacticlyCorrectAtRule
+     */
+    public function parsesSyntacticlyCorrectAtRuleInStrictMode($css)
+    {
+        $contents = (new Parser($css, Settings::create()->beStrict()))->parse()->getContents();
+
+        self::assertNotEmpty($contents, 'Failing CSS: `' . $css . '`');
+    }
 }

tests/ParserTest.php

@@ -74,10 +74,7 @@ public function files()
                     // or a future test of a as-of-now missing feature
                     continue;
                 }
-                $oParser = new Parser(
-                    \file_get_contents($sDirectory . '/' . $sFileName),
-                    Settings::create()->withLenientParsing($sFileName[0] !== '=')
-                );
+                $oParser = new Parser(file_get_contents($sDirectory . '/' . $sFileName));
                 try {
                     self::assertNotEquals('', $oParser->parse()->render());
                 } catch (\Exception $e) {