[TASK] Add Tests for Document::getAllDeclarationBlocks()

Also remove a misleading and incorrect comment.

Part of #757

Author: oliverklee

src/CSSList/Document.php

@@ -32,7 +32,6 @@ public static function parse(ParserState $parserState): Document
 
     /**
      * Gets all `DeclarationBlock` objects recursively, no matter how deeply nested the selectors are.
-     * Aliased as `getAllSelectors()`.
      *
      * @return array<int, DeclarationBlock>
      */

tests/Unit/CSSList/DocumentTest.php

@@ -6,11 +6,18 @@
 
 use PHPUnit\Framework\TestCase;
 use Sabberworm\CSS\Comment\Commentable;
+use Sabberworm\CSS\CSSList\AtRuleBlockList;
 use Sabberworm\CSS\CSSList\Document;
+use Sabberworm\CSS\Property\Charset;
+use Sabberworm\CSS\Property\Import;
 use Sabberworm\CSS\Renderable;
 use Sabberworm\CSS\RuleSet\DeclarationBlock;
+use Sabberworm\CSS\Value\CSSString;
+use Sabberworm\CSS\Value\URL;
 
 /**
+ * @covers \Sabberworm\CSS\CSSList\CSSBlockList
+ * @covers \Sabberworm\CSS\CSSList\CSSList
  * @covers \Sabberworm\CSS\CSSList\Document
  */
 final class DocumentTest extends TestCase
@@ -142,4 +149,80 @@ public function insertContentBeforeAppendsIfSiblingNotFound(): void
         self::assertCount(4, $this->subject->getContents());
         self::assertSame([$bogusOne, $sibling, $bogusTwo, $item], $this->subject->getContents());
     }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksForNoContentsReturnsEmptyArray(): void
+    {
+        self::assertSame([], $this->subject->getAllDeclarationBlocks());
+    }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksCanReturnOneDirectDeclarationBlockContent(): void
+    {
+        $declarationBlock = new DeclarationBlock();
+        $this->subject->setContents([$declarationBlock]);
+
+        $result = $this->subject->getAllDeclarationBlocks();
+
+        self::assertSame([$declarationBlock], $result);
+    }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksCanReturnMultipleDirectDeclarationBlockContents(): void
+    {
+        $declarationBlock1 = new DeclarationBlock();
+        $declarationBlock2 = new DeclarationBlock();
+        $this->subject->setContents([$declarationBlock1, $declarationBlock2]);
+
+        $result = $this->subject->getAllDeclarationBlocks();
+
+        self::assertSame([$declarationBlock1, $declarationBlock2], $result);
+    }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksAlsoReturnsDeclarationBlocksWithinAtRuleBlockList(): void
+    {
+        $declarationBlock = new DeclarationBlock();
+        $atRuleBlockList = new AtRuleBlockList('media');
+        $atRuleBlockList->setContents([$declarationBlock]);
+        $this->subject->setContents([$atRuleBlockList]);
+
+        $result = $this->subject->getAllDeclarationBlocks();
+
+        self::assertSame([$declarationBlock], $result);
+    }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksIgnoresImport(): void
+    {
+        $import = new Import(new URL(new CSSString('')), '');
+        $this->subject->setContents([$import]);
+
+        $result = $this->subject->getAllDeclarationBlocks();
+
+        self::assertSame([], $result);
+    }
+
+    /**
+     * @test
+     */
+    public function getAllDeclarationBlocksIgnoresCharset(): void
+    {
+        $Charset = new Charset(new CSSString(''));
+        $this->subject->setContents([$Charset]);
+
+        $result = $this->subject->getAllDeclarationBlocks();
+
+        self::assertSame([], $result);
+    }
 }