Make the tests for AtRuleBlockList more fine-grained

Author: oliverklee

tests/CSSList/AtRuleBlockListTest.php

@@ -5,27 +5,49 @@
 use PHPUnit\Framework\TestCase;
 use Sabberworm\CSS\Parser;
 
+/**
+ * @covers \Sabberworm\CSS\CSSList\AtRuleBlockList
+ */
 class AtRuleBlockListTest extends TestCase
 {
+    /**
+     * @return array<string, array<int, string>>
+     */
+    public 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
+     */
+    public function parsesRuleNameOfMediaQueries($css)
+    {
+        $contents = (new Parser($css))->parse()->getContents();
+        $atRuleBlockList = $contents[0];
+
+        self::assertSame('media', $atRuleBlockList->atRuleName());
+    }
+
     /**
      * @test
+     *
+     * @param string $css
+     *
+     * @dataProvider mediaRuleDataProvider
      */
-    public function mediaQueries()
+    public function parsesArgumentsOfMediaQueries($css)
     {
-        $sCss = '@media(min-width: 768px){.class{color:red}}';
-        $oParser = new Parser($sCss);
-        $oDoc = $oParser->parse();
-        $aContents = $oDoc->getContents();
-        $oMediaQuery = $aContents[0];
-        self::assertSame('media', $oMediaQuery->atRuleName(), 'Does not interpret the type as a function');
-        self::assertSame('(min-width: 768px)', $oMediaQuery->atRuleArgs(), 'The media query is the value');
+        $contents = (new Parser($css))->parse()->getContents();
+        $atRuleBlockList = $contents[0];
 
-        $sCss = '@media (min-width: 768px) {.class{color:red}}';
-        $oParser = new Parser($sCss);
-        $oDoc = $oParser->parse();
-        $aContents = $oDoc->getContents();
-        $oMediaQuery = $aContents[0];
-        self::assertSame('media', $oMediaQuery->atRuleName(), 'Does not interpret the type as a function');
-        self::assertSame('(min-width: 768px)', $oMediaQuery->atRuleArgs(), 'The media query is the value');
+        self::assertSame('(min-width: 768px)', $atRuleBlockList->atRuleArgs());
     }
 }