Make the existing tests for Document for fine-grained

Author: oliverklee

tests/CSSList/DocumentTest.php

@@ -3,28 +3,78 @@
 namespace Sabberworm\CSS\Tests\CSSList;
 
 use PHPUnit\Framework\TestCase;
-use Sabberworm\CSS\Parser;
+use Sabberworm\CSS\CSSList\Document;
+use Sabberworm\CSS\RuleSet\DeclarationBlock;
 
+/**
+ * @covers \Sabberworm\CSS\CSSList\Document
+ */
 class DocumentTest extends TestCase
 {
+    /**
+     * @var Document
+     */
+    private $subject;
+
+    protected function setUp()
+    {
+        $this->subject = new Document();
+    }
+
+    /**
+     * @test
+     */
+    public function getContentsInitiallyReturnsEmptyArray()
+    {
+        self::assertSame([], $this->subject->getContents());
+    }
+
+    /**
+     * @test
+     */
+    public function setContentsWithEmptyArrayDoesNotAddAnyContents()
+    {
+        $this->subject->setContents([]);
+
+        self::assertSame([], $this->subject->getContents());
+    }
+
     /**
      * @test
      */
-    public function overrideContents()
+    public function setContentsCanAddOneItem()
     {
-        $sCss = '.thing { left: 10px; }';
-        $oParser = new Parser($sCss);
-        $oDoc = $oParser->parse();
-        $aContents = $oDoc->getContents();
-        self::assertCount(1, $aContents);
-
-        $sCss2 = '.otherthing { right: 10px; }';
-        $oParser2 = new Parser($sCss);
-        $oDoc2 = $oParser2->parse();
-        $aContents2 = $oDoc2->getContents();
-
-        $oDoc->setContents([$aContents[0], $aContents2[0]]);
-        $aFinalContents = $oDoc->getContents();
-        self::assertCount(2, $aFinalContents);
+        $item = new DeclarationBlock();
+
+        $this->subject->setContents([$item]);
+
+        self::assertSame([$item], $this->subject->getContents());
+    }
+
+    /**
+     * @test
+     */
+    public function setContentsCanAddTwoItemsInOneCall()
+    {
+        $item1 = new DeclarationBlock();
+        $item2 = new DeclarationBlock();
+
+        $this->subject->setContents([$item1, $item2]);
+
+        self::assertSame([$item1, $item2], $this->subject->getContents());
+    }
+
+    /**
+     * @test
+     */
+    public function setContentsReplacesItemsSetInPreviousCall()
+    {
+        $item1 = new DeclarationBlock();
+        $item2 = new DeclarationBlock();
+
+        $this->subject->setContents([$item1]);
+        $this->subject->setContents([$item2]);
+
+        self::assertSame([$item2], $this->subject->getContents());
     }
 }

tests/ParserTest.php

@@ -14,6 +14,7 @@
 use Sabberworm\CSS\Property\Selector;
 use Sabberworm\CSS\RuleSet\AtRuleSet;
 use Sabberworm\CSS\RuleSet\DeclarationBlock;
+use Sabberworm\CSS\RuleSet\RuleSet;
 use Sabberworm\CSS\Settings;
 use Sabberworm\CSS\Value\Color;
 use Sabberworm\CSS\Value\Size;
@@ -33,6 +34,23 @@
  */
 class ParserTest extends TestCase
 {
+    /**
+     * @test
+     */
+    public function parseForOneRuleSetReturnsDocumentWithOneRuleSet()
+    {
+        $css = '.thing { left: 10px; }';
+        $parser = new Parser($css);
+
+        $document = $parser->parse();
+
+        self::assertInstanceOf(Document::class, $document);
+
+        $cssList = $document->getContents();
+        self::assertCount(1, $cssList);
+        self::assertInstanceOf(RuleSet::class, $cssList[0]);
+    }
+
     /**
      * @test
      */