From c1ab2ffdccceaba7bbd28ea894664edff4af0639 Mon Sep 17 00:00:00 2001
From: Oliver Klee <github@oliverklee.de>
Date: Sat, 1 Feb 2025 09:55:39 +0100
Subject: [PATCH 1/2] [TASK] Add accessor tests for `OutputFormat` (part 2)

Currently, this class uses magic getters and setters. These
should be proper methods instead.

But first, we should have tests for the accessors. This is
part 2.
---
 tests/Unit/OutputFormatTest.php | 326 ++++++++++++++++++++++++++++++++
 1 file changed, 326 insertions(+)

diff --git a/tests/Unit/OutputFormatTest.php b/tests/Unit/OutputFormatTest.php
index 482be82f..53bf9e19 100644
--- a/tests/Unit/OutputFormatTest.php
+++ b/tests/Unit/OutputFormatTest.php
@@ -304,4 +304,330 @@ public function setSpaceBetweenBlocksProvidesFluentInterface(): void
     {
         self::assertSame($this->subject, $this->subject->setSpaceBetweenBlocks(' '));
     }
+
+    /**
+     * @test
+     */
+    public function getBeforeAtRuleBlockInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getBeforeAtRuleBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setBeforeAtRuleBlockSetsBeforeAtRuleBlock(): void
+    {
+        $value = ' ';
+        $this->subject->setBeforeAtRuleBlock($value);
+
+        self::assertSame($value, $this->subject->getBeforeAtRuleBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setBeforeAtRuleBlockProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setBeforeAtRuleBlock(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getAfterAtRuleBlockInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getAfterAtRuleBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterAtRuleBlockSetsAfterAtRuleBlock(): void
+    {
+        $value = ' ';
+        $this->subject->setAfterAtRuleBlock($value);
+
+        self::assertSame($value, $this->subject->getAfterAtRuleBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterAtRuleBlockProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setAfterAtRuleBlock(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getSpaceBeforeSelectorSeparatorInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getSpaceBeforeSelectorSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceBeforeSelectorSeparatorSetsSpaceBeforeSelectorSeparator(): void
+    {
+        $value = ' ';
+        $this->subject->setSpaceBeforeSelectorSeparator($value);
+
+        self::assertSame($value, $this->subject->getSpaceBeforeSelectorSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceBeforeSelectorSeparatorProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setSpaceBeforeSelectorSeparator(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getSpaceAfterSelectorSeparatorInitiallyReturnsSpace(): void
+    {
+        self::assertSame(' ', $this->subject->getSpaceAfterSelectorSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceAfterSelectorSeparatorSetsSpaceAfterSelectorSeparator(): void
+    {
+        $value = '    ';
+        $this->subject->setSpaceAfterSelectorSeparator($value);
+
+        self::assertSame($value, $this->subject->getSpaceAfterSelectorSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceAfterSelectorSeparatorProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setSpaceAfterSelectorSeparator(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getSpaceBeforeListArgumentSeparatorInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getSpaceBeforeListArgumentSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceBeforeListArgumentSeparatorSetsSpaceBeforeListArgumentSeparator(): void
+    {
+        $value = ' ';
+        $this->subject->setSpaceBeforeListArgumentSeparator($value);
+
+        self::assertSame($value, $this->subject->getSpaceBeforeListArgumentSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceBeforeListArgumentSeparatorProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setSpaceBeforeListArgumentSeparator(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getSpaceAfterListArgumentSeparatorInitiallyEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getSpaceAfterListArgumentSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceAfterListArgumentSeparatorSetsSpaceAfterListArgumentSeparator(): void
+    {
+        $value = '    ';
+        $this->subject->setSpaceAfterListArgumentSeparator($value);
+
+        self::assertSame($value, $this->subject->getSpaceAfterListArgumentSeparator());
+    }
+
+    /**
+     * @test
+     */
+    public function setSpaceAfterListArgumentSeparatorProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setSpaceAfterListArgumentSeparator(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getBeforeDeclarationBlockInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getBeforeDeclarationBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setBeforeDeclarationBlockSetsBeforeDeclarationBlock(): void
+    {
+        $value = ' ';
+        $this->subject->setBeforeDeclarationBlock($value);
+
+        self::assertSame($value, $this->subject->getBeforeDeclarationBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setBeforeDeclarationBlockProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setBeforeDeclarationBlock(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getAfterDeclarationBlockSelectorsInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getAfterDeclarationBlockSelectors());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterDeclarationBlockSelectorsSetsAfterDeclarationBlockSelectors(): void
+    {
+        $value = ' ';
+        $this->subject->setAfterDeclarationBlockSelectors($value);
+
+        self::assertSame($value, $this->subject->getAfterDeclarationBlockSelectors());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterDeclarationBlockSelectorsProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setAfterDeclarationBlockSelectors(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getAfterDeclarationBlockInitiallyReturnsEmptyString(): void
+    {
+        self::assertSame('', $this->subject->getAfterDeclarationBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterDeclarationBlockSetsAfterDeclarationBlock(): void
+    {
+        $value = ' ';
+        $this->subject->setAfterDeclarationBlock($value);
+
+        self::assertSame($value, $this->subject->getAfterDeclarationBlock());
+    }
+
+    /**
+     * @test
+     */
+    public function setAfterDeclarationBlockProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setAfterDeclarationBlock(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getIndentationInitiallyReturnsTab(): void
+    {
+        self::assertSame("\t", $this->subject->getIndentation());
+    }
+
+    /**
+     * @test
+     */
+    public function setIndentationSetsIndentation(): void
+    {
+        $value = ' ';
+        $this->subject->setIndentation($value);
+
+        self::assertSame($value, $this->subject->getIndentation());
+    }
+
+    /**
+     * @test
+     */
+    public function setIndentationProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setIndentation(' '));
+    }
+
+    /**
+     * @test
+     */
+    public function getIgnoreExceptionsInitiallyReturnsFalse(): void
+    {
+        self::assertFalse($this->subject->getIgnoreExceptions());
+    }
+
+    /**
+     * @test
+     *
+     * @dataProvider provideBooleans
+     */
+    public function setIgnoreExceptionsSetsIgnoreExceptions(bool $value): void
+    {
+        $this->subject->setIgnoreExceptions($value);
+
+        self::assertSame($value, $this->subject->getIgnoreExceptions());
+    }
+
+    /**
+     * @test
+     */
+    public function setIgnoreExceptionsProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setIgnoreExceptions(true));
+    }
+
+    /**
+     * @test
+     */
+    public function getRenderCommentsInitiallyReturnsFalse(): void
+    {
+        self::assertFalse($this->subject->getRenderComments());
+    }
+
+    /**
+     * @test
+     *
+     * @dataProvider provideBooleans
+     */
+    public function setRenderCommentsSetsRenderComments(bool $value): void
+    {
+        $this->subject->setRenderComments($value);
+
+        self::assertSame($value, $this->subject->getRenderComments());
+    }
+
+    /**
+     * @test
+     */
+    public function setRenderCommentsProvidesFluentInterface(): void
+    {
+        self::assertSame($this->subject, $this->subject->setRenderComments(true));
+    }
 }

From 34495bb5b26d3649af9fd2e129c2f1a62e5788ed Mon Sep 17 00:00:00 2001
From: Oliver Klee <github@oliverklee.de>
Date: Sat, 1 Feb 2025 20:16:02 +0100
Subject: [PATCH 2/2] Update tests/Unit/OutputFormatTest.php

Co-authored-by: JakeQZ <jake.github@qzdesign.co.uk>
---
 tests/Unit/OutputFormatTest.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/Unit/OutputFormatTest.php b/tests/Unit/OutputFormatTest.php
index 53bf9e19..ddb87579 100644
--- a/tests/Unit/OutputFormatTest.php
+++ b/tests/Unit/OutputFormatTest.php
@@ -443,7 +443,7 @@ public function setSpaceBeforeListArgumentSeparatorProvidesFluentInterface(): vo
     /**
      * @test
      */
-    public function getSpaceAfterListArgumentSeparatorInitiallyEmptyString(): void
+    public function getSpaceAfterListArgumentSeparatorInitiallyReturnsEmptyString(): void
     {
         self::assertSame('', $this->subject->getSpaceAfterListArgumentSeparator());
     }