From 9fdc9ab29f2236c7b1af8cecce6ffd27e4c9fbd8 Mon Sep 17 00:00:00 2001
From: Sam Burns <sam.burns@upmind.com>
Date: Wed, 24 Jan 2024 15:14:06 +0000
Subject: [PATCH] To prepare to move business logic out of god object
 antipattern, provide domain objects and factory pattern

---
 php/src/GildedRoseItem.php                    | 10 +++
 php/src/GildedRoseItem/BackstagePassItem.php  | 19 ++++++
 php/src/GildedRoseItem/BrieItem.php           | 19 ++++++
 php/src/GildedRoseItem/DefaultItem.php        | 17 ++++++
 php/src/GildedRoseItem/SulfurasItem.php       | 19 ++++++
 php/src/GildedRoseItemFactory.php             | 27 ++++++++
 ...re => default-quality-degradation.feature} |  0
 php/tests/phpunit/ExampleTest.php             | 18 ------
 .../phpunit/GildedRoseItemFactoryTest.php     | 61 +++++++++++++++++++
 9 files changed, 172 insertions(+), 18 deletions(-)
 create mode 100644 php/src/GildedRoseItem.php
 create mode 100644 php/src/GildedRoseItem/BackstagePassItem.php
 create mode 100644 php/src/GildedRoseItem/BrieItem.php
 create mode 100644 php/src/GildedRoseItem/DefaultItem.php
 create mode 100644 php/src/GildedRoseItem/SulfurasItem.php
 create mode 100644 php/src/GildedRoseItemFactory.php
 rename php/tests/behat/feature_files/{default-quality-degradation copy 2.feature => default-quality-degradation.feature} (100%)
 delete mode 100644 php/tests/phpunit/ExampleTest.php
 create mode 100644 php/tests/phpunit/GildedRoseItemFactoryTest.php

diff --git a/php/src/GildedRoseItem.php b/php/src/GildedRoseItem.php
new file mode 100644
index 00000000..64e5df2b
--- /dev/null
+++ b/php/src/GildedRoseItem.php
@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose;
+
+interface GildedRoseItem
+{
+    public function ageByOneDay(): void;
+}
diff --git a/php/src/GildedRoseItem/BackstagePassItem.php b/php/src/GildedRoseItem/BackstagePassItem.php
new file mode 100644
index 00000000..633860f3
--- /dev/null
+++ b/php/src/GildedRoseItem/BackstagePassItem.php
@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose\GildedRoseItem;
+
+use GildedRose\GildedRoseItem;
+use GildedRose\Item;
+
+class BackstagePassItem implements GildedRoseItem
+{
+    public const NAME = 'Backstage passes to a TAFKAL80ETC concert';
+
+    public function __construct(private Item $item) {}
+
+    public function ageByOneDay(): void
+    {
+    }
+}
diff --git a/php/src/GildedRoseItem/BrieItem.php b/php/src/GildedRoseItem/BrieItem.php
new file mode 100644
index 00000000..9a114a5f
--- /dev/null
+++ b/php/src/GildedRoseItem/BrieItem.php
@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose\GildedRoseItem;
+
+use GildedRose\GildedRoseItem;
+use GildedRose\Item;
+
+class BrieItem implements GildedRoseItem
+{
+    public const NAME = 'Aged Brie';
+
+    public function __construct(private Item $item) {}
+
+    public function ageByOneDay(): void
+    {
+    }
+}
diff --git a/php/src/GildedRoseItem/DefaultItem.php b/php/src/GildedRoseItem/DefaultItem.php
new file mode 100644
index 00000000..9647f4b1
--- /dev/null
+++ b/php/src/GildedRoseItem/DefaultItem.php
@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose\GildedRoseItem;
+
+use GildedRose\GildedRoseItem;
+use GildedRose\Item;
+
+class DefaultItem implements GildedRoseItem
+{
+    public function __construct(private Item $item) {}
+
+    public function ageByOneDay(): void
+    {
+    }
+}
diff --git a/php/src/GildedRoseItem/SulfurasItem.php b/php/src/GildedRoseItem/SulfurasItem.php
new file mode 100644
index 00000000..1f7c004d
--- /dev/null
+++ b/php/src/GildedRoseItem/SulfurasItem.php
@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose\GildedRoseItem;
+
+use GildedRose\GildedRoseItem;
+use GildedRose\Item;
+
+class SulfurasItem implements GildedRoseItem
+{
+    public const NAME = 'Sulfuras, Hand of Ragnaros';
+
+    public function __construct(private Item $item) {}
+
+    public function ageByOneDay(): void
+    {
+    }
+}
diff --git a/php/src/GildedRoseItemFactory.php b/php/src/GildedRoseItemFactory.php
new file mode 100644
index 00000000..8817401d
--- /dev/null
+++ b/php/src/GildedRoseItemFactory.php
@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GildedRose;
+
+use GildedRose\GildedRoseItem\BrieItem;
+use GildedRose\GildedRoseItem\BackstagePassItem;
+use GildedRose\GildedRoseItem\SulfurasItem;
+use GildedRose\GildedRoseItem\DefaultItem;
+
+class GildedRoseItemFactory
+{
+    public function createGildedRoseItem(Item $item): GildedRoseItem
+    {
+        switch ($item->name) {
+            case BrieItem::NAME:
+                return new BrieItem($item);
+            case BackstagePassItem::NAME:
+                return new BackstagePassItem($item);
+            case SulfurasItem::NAME:
+                return new SulfurasItem($item);
+            default:
+                return new DefaultItem($item);
+        }
+    }
+}
diff --git a/php/tests/behat/feature_files/default-quality-degradation copy 2.feature b/php/tests/behat/feature_files/default-quality-degradation.feature
similarity index 100%
rename from php/tests/behat/feature_files/default-quality-degradation copy 2.feature
rename to php/tests/behat/feature_files/default-quality-degradation.feature
diff --git a/php/tests/phpunit/ExampleTest.php b/php/tests/phpunit/ExampleTest.php
deleted file mode 100644
index ea7b98a0..00000000
--- a/php/tests/phpunit/ExampleTest.php
+++ /dev/null
@@ -1,18 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace PhpUnitTests;
-
-use ApprovalTests\Approvals;
-use GildedRose\GildedRose;
-use GildedRose\Item;
-use PHPUnit\Framework\TestCase;
-
-class ExamplelTest extends TestCase
-{
-    public function testFoo(): void
-    {
-        $this->assertTrue(true);
-    }
-}
diff --git a/php/tests/phpunit/GildedRoseItemFactoryTest.php b/php/tests/phpunit/GildedRoseItemFactoryTest.php
new file mode 100644
index 00000000..66237fdb
--- /dev/null
+++ b/php/tests/phpunit/GildedRoseItemFactoryTest.php
@@ -0,0 +1,61 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpUnitTests;
+
+use ApprovalTests\Approvals;
+use GildedRose\GildedRoseItem;
+use GildedRose\GildedRoseItem\BrieItem;
+use GildedRose\GildedRoseItem\BackstagePassItem;
+use GildedRose\GildedRoseItem\DefaultItem;
+use GildedRose\GildedRoseItem\SulfurasItem;
+use GildedRose\GildedRoseItemFactory;
+use GildedRose\Item;
+use PHPUnit\Framework\TestCase;
+
+class GildedRoseItemFactoryTest extends TestCase
+{
+    private GildedRoseItemFactory $factory;
+
+    protected function setUp(): void
+    {
+        $this->factory = new GildedRoseItemFactory();
+    }
+
+    public function testCreatingDefaultItem(): void
+    {
+        $item = new Item('foo', 0, 0);
+        $gildedRoseItem = $this->factory->createGildedRoseItem($item);
+
+        $this->assertInstanceOf(GildedRoseItem::class, $gildedRoseItem);
+        $this->assertInstanceOf(DefaultItem::class, $gildedRoseItem);
+    }
+
+    public function testCreatingBrieItem(): void
+    {
+        $item = new Item('Aged Brie', 0, 0);
+        $gildedRoseItem = $this->factory->createGildedRoseItem($item);
+
+        $this->assertInstanceOf(GildedRoseItem::class, $gildedRoseItem);
+        $this->assertInstanceOf(BrieItem::class, $gildedRoseItem);
+    }
+
+    public function testCreatingBackstagePassItem(): void
+    {
+        $item = new Item('Backstage passes to a TAFKAL80ETC concert', 0, 0);
+        $gildedRoseItem = $this->factory->createGildedRoseItem($item);
+
+        $this->assertInstanceOf(GildedRoseItem::class, $gildedRoseItem);
+        $this->assertInstanceOf(BackstagePassItem::class, $gildedRoseItem);
+    }
+
+    public function testCreatingSulfuraItem(): void
+    {
+        $item = new Item('Sulfuras, Hand of Ragnaros', 0, 0);
+        $gildedRoseItem = $this->factory->createGildedRoseItem($item);
+
+        $this->assertInstanceOf(GildedRoseItem::class, $gildedRoseItem);
+        $this->assertInstanceOf(SulfurasItem::class, $gildedRoseItem);
+    }
+}