[CLEANUP] Reorder and group some tests for CSSList

This is a pre-patch to having dedicated tests for some of our
abstract classes in order to avoid duplicated tests.

This change merely moves tests around in the test classes
(and fixing two incorrectly named tests), but does not
change the tests themselves in any way.

Author: oliverklee

tests/Unit/CSSList/AtRuleBlockListTest.php

@@ -62,7 +62,7 @@ public function isCSSList(): void
     }
 
     /*
-     * not grouped yet
+     * Tests for the constructor and its parameters
      */
 
     /**
@@ -77,6 +77,10 @@ public function atRuleNameReturnsTypeProvidedToConstructor(): void
         self::assertSame($type, $subject->atRuleName());
     }
 
+    /*
+     * Tests for functionality implemented by the tested class itself
+     */
+
     /**
      * @test
      */
@@ -102,22 +106,26 @@ public function atRuleArgsReturnsArgumentsProvidedToConstructor(): void
     /**
      * @test
      */
-    public function getLineNoReturnsLineNumberProvidedToConstructor(): void
+    public function isRootListAlwaysReturnsFalse(): void
     {
-        $lineNumber = 42;
-
-        $subject = new AtRuleBlockList('', '', $lineNumber);
+        $subject = new AtRuleBlockList('supports');
 
-        self::assertSame($lineNumber, $subject->getLineNo());
+        self::assertFalse($subject->isRootList());
     }
 
+    /*
+     * Tests for functionality implemented in `CSSBlockList`
+     */
+
     /**
      * @test
      */
-    public function isRootListAlwaysReturnsFalse(): void
+    public function getLineNoReturnsLineNumberProvidedToConstructor(): void
     {
-        $subject = new AtRuleBlockList('supports');
+        $lineNumber = 42;
 
-        self::assertFalse($subject->isRootList());
+        $subject = new AtRuleBlockList('', '', $lineNumber);
+
+        self::assertSame($lineNumber, $subject->getLineNo());
     }
 }

tests/Unit/CSSList/DocumentTest.php

@@ -54,7 +54,7 @@ public function isCSSList(): void
     }
 
     /*
-     * not grouped yet
+     * Tests for functionality implemented by the tested class itself
      */
 
     /**

tests/Unit/CSSList/KeyFrameTest.php

@@ -60,20 +60,4 @@ public function isCSSList(): void
 
         self::assertInstanceOf(CSSList::class, $subject);
     }
-
-    /*
-     * not grouped yet
-     */
-
-    /**
-     * @test
-     */
-    public function getLineNoReturnsLineNumberProvidedToConstructor(): void
-    {
-        $lineNumber = 42;
-
-        $subject = new KeyFrame($lineNumber);
-
-        self::assertSame($lineNumber, $subject->getLineNo());
-    }
 }