[TASK] Make Selector a Renderable

This is required to be able to drop `__toString` for this class.

Author: oliverklee

CHANGELOG.md

@@ -10,6 +10,7 @@ Please also have a look at our
 
 ### Added
 
+- Make `Selector` a `Renderable` (#1017)
 - `OutputFormat` properties for space around specific list separators (#880)
 - Partial support for CSS Color Module Level 4:
     - `rgb` and `rgba`, and `hsl` and `hsla` are now aliases (#797}

src/Property/Selector.php

@@ -4,11 +4,14 @@
 
 namespace Sabberworm\CSS\Property;
 
+use Sabberworm\CSS\OutputFormat;
+use Sabberworm\CSS\Renderable;
+
 /**
  * Class representing a single CSS selector. Selectors have to be split by the comma prior to being passed into this
  * class.
  */
-class Selector
+class Selector implements Renderable
 {
     /**
      * regexp for specificity calculations
@@ -137,4 +140,19 @@ public function getSpecificity()
         }
         return $this->specificity;
     }
+
+    public function render(OutputFormat $outputFormat): string
+    {
+        return $this->getSelector();
+    }
+
+    /**
+     * @return never
+     *
+     * @throws \BadMethodCallException
+     */
+    public function getLineNo(): int
+    {
+        throw new \BadMethodCallException('Not implemented', 1740654131);
+    }
 }

tests/Functional/Property/SelectorTest.php

@@ -0,0 +1,59 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sabberworm\CSS\Tests\Functional\Selector;
+
+use PHPUnit\Framework\TestCase;
+use Sabberworm\CSS\OutputFormat;
+use Sabberworm\CSS\Property\Selector;
+
+/**
+ * @covers \Sabberworm\CSS\Property\Selector
+ */
+final class SelectorTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function renderWithVirginOutputFormatRendersPatternPassedToConstructor(): void
+    {
+        $pattern = 'a';
+        $subject = new Selector($pattern);
+
+        self::assertSame($pattern, $subject->render(new OutputFormat()));
+    }
+
+    /**
+     * @test
+     */
+    public function renderWithDefaultOutputFormatRendersPatternPassedToConstructor(): void
+    {
+        $pattern = 'a';
+        $subject = new Selector($pattern);
+
+        self::assertSame($pattern, $subject->render(OutputFormat::create()));
+    }
+
+    /**
+     * @test
+     */
+    public function renderWithCompactOutputFormatRendersPatternPassedToConstructor(): void
+    {
+        $pattern = 'a';
+        $subject = new Selector($pattern);
+
+        self::assertSame($pattern, $subject->render(OutputFormat::createCompact()));
+    }
+
+    /**
+     * @test
+     */
+    public function renderWithPrettyOutputFormatRendersPatternPassedToConstructor(): void
+    {
+        $pattern = 'a';
+        $subject = new Selector($pattern);
+
+        self::assertSame($pattern, $subject->render(OutputFormat::createPretty()));
+    }
+}

tests/FunctionalDeprecated/Property/SelectorTest.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sabberworm\CSS\Tests\FunctionalDeperecated\Selector;
+
+use PHPUnit\Framework\TestCase;
+use Sabberworm\CSS\Property\Selector;
+
+/**
+ * @covers \Sabberworm\CSS\Property\Selector
+ */
+final class SelectorTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function toStringReturnsPatternPassedToConstructor(): void
+    {
+        $pattern = 'a';
+        $subject = new Selector($pattern);
+
+        self::assertSame($pattern, (string) $subject);
+    }
+}

tests/Unit/Property/SelectorTest.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sabberworm\CSS\Tests\Unit\Property;
+
+use PHPUnit\Framework\TestCase;
+use Sabberworm\CSS\Property\Selector;
+use Sabberworm\CSS\Renderable;
+
+/**
+ * @covers \Sabberworm\CSS\Property\Selector
+ */
+final class SelectorTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function implementsRenderable(): void
+    {
+        $subject = new Selector('a');
+
+        self::assertInstanceOf(Renderable::class, $subject);
+    }
+}