From ece62c0dceabcd3abc4ac852d6fe216eb2d7e2d0 Mon Sep 17 00:00:00 2001 From: Sam Burns Date: Wed, 24 Jan 2024 16:28:13 +0000 Subject: [PATCH] To improve readability, restore coding standards using ecs --- php/src/GildedRoseItem/BackstagePassItem.php | 7 +++++-- php/src/GildedRoseItem/BrieItem.php | 7 +++++-- php/src/GildedRoseItem/ConjuredItem.php | 7 +++++-- php/src/GildedRoseItem/DefaultItem.php | 7 +++++-- php/src/GildedRoseItem/SulfurasItem.php | 5 ++++- php/src/GildedRoseItemFactory.php | 4 ++-- .../behat/contexts/ServiceLevelContext.php | 18 ++++++++++-------- .../GildedRoseItem/BackstagePassItemTest.php | 3 --- .../phpunit/GildedRoseItem/BrieItemTest.php | 3 --- .../GildedRoseItem/ConjuredItemTest.php | 3 --- .../phpunit/GildedRoseItem/DefaultItemTest.php | 3 --- .../GildedRoseItem/SulfurasItemTest.php | 3 --- .../phpunit/GildedRoseItemFactoryTest.php | 3 +-- 13 files changed, 37 insertions(+), 36 deletions(-) diff --git a/php/src/GildedRoseItem/BackstagePassItem.php b/php/src/GildedRoseItem/BackstagePassItem.php index c68d2c40..c2263e9e 100644 --- a/php/src/GildedRoseItem/BackstagePassItem.php +++ b/php/src/GildedRoseItem/BackstagePassItem.php @@ -11,11 +11,14 @@ class BackstagePassItem implements GildedRoseItem { public const NAME = 'Backstage passes to a TAFKAL80ETC concert'; - public function __construct(private Item $item) {} + public function __construct( + private Item $item + ) { + } public function ageByOneDay(): void { - $this->item->sellIn -= 1; + --$this->item->sellIn; $this->item->quality = min(50, $this->getNewQuality($this->item->quality, $this->item->sellIn)); } diff --git a/php/src/GildedRoseItem/BrieItem.php b/php/src/GildedRoseItem/BrieItem.php index 8ef86989..92bdc3a4 100644 --- a/php/src/GildedRoseItem/BrieItem.php +++ b/php/src/GildedRoseItem/BrieItem.php @@ -11,11 +11,14 @@ class BrieItem implements GildedRoseItem { public const NAME = 'Aged Brie'; - public function __construct(private Item $item) {} + public function __construct( + private Item $item + ) { + } public function ageByOneDay(): void { - $this->item->sellIn -= 1; + --$this->item->sellIn; $this->item->quality = min(50, $this->getNewQuality($this->item->quality, $this->item->sellIn)); } diff --git a/php/src/GildedRoseItem/ConjuredItem.php b/php/src/GildedRoseItem/ConjuredItem.php index 732cee4d..757063cd 100644 --- a/php/src/GildedRoseItem/ConjuredItem.php +++ b/php/src/GildedRoseItem/ConjuredItem.php @@ -11,11 +11,14 @@ class ConjuredItem implements GildedRoseItem { public const NAME = 'Conjured Mana Cake'; - public function __construct(private Item $item) {} + public function __construct( + private Item $item + ) { + } public function ageByOneDay(): void { - $this->item->sellIn -= 1; + --$this->item->sellIn; $this->item->quality = max(0, $this->getNewQuality($this->item->quality, $this->item->sellIn)); } diff --git a/php/src/GildedRoseItem/DefaultItem.php b/php/src/GildedRoseItem/DefaultItem.php index c8f4c6f5..1c339d49 100644 --- a/php/src/GildedRoseItem/DefaultItem.php +++ b/php/src/GildedRoseItem/DefaultItem.php @@ -9,11 +9,14 @@ use GildedRose\Item; class DefaultItem implements GildedRoseItem { - public function __construct(private Item $item) {} + public function __construct( + private Item $item + ) { + } public function ageByOneDay(): void { - $this->item->sellIn -= 1; + --$this->item->sellIn; $this->item->quality = max(0, $this->getNewQuality($this->item->quality, $this->item->sellIn)); } diff --git a/php/src/GildedRoseItem/SulfurasItem.php b/php/src/GildedRoseItem/SulfurasItem.php index 1f7c004d..954f3761 100644 --- a/php/src/GildedRoseItem/SulfurasItem.php +++ b/php/src/GildedRoseItem/SulfurasItem.php @@ -11,7 +11,10 @@ class SulfurasItem implements GildedRoseItem { public const NAME = 'Sulfuras, Hand of Ragnaros'; - public function __construct(private Item $item) {} + public function __construct( + private Item $item + ) { + } public function ageByOneDay(): void { diff --git a/php/src/GildedRoseItemFactory.php b/php/src/GildedRoseItemFactory.php index 6885164a..69210ba1 100644 --- a/php/src/GildedRoseItemFactory.php +++ b/php/src/GildedRoseItemFactory.php @@ -4,11 +4,11 @@ declare(strict_types=1); namespace GildedRose; -use GildedRose\GildedRoseItem\BrieItem; use GildedRose\GildedRoseItem\BackstagePassItem; +use GildedRose\GildedRoseItem\BrieItem; use GildedRose\GildedRoseItem\ConjuredItem; -use GildedRose\GildedRoseItem\SulfurasItem; use GildedRose\GildedRoseItem\DefaultItem; +use GildedRose\GildedRoseItem\SulfurasItem; class GildedRoseItemFactory { diff --git a/php/tests/behat/contexts/ServiceLevelContext.php b/php/tests/behat/contexts/ServiceLevelContext.php index ad9b3e1e..b1b068d7 100644 --- a/php/tests/behat/contexts/ServiceLevelContext.php +++ b/php/tests/behat/contexts/ServiceLevelContext.php @@ -1,26 +1,22 @@ item = new Item($name, $initialSellIn, $initialQuality); - $this->gildedRose = new GildedRose([$this->item]); - } + private GildedRose $gildedRose; #[Then('I should see :expectedOutput')] public function iShouldSee(string $expectedOutput): void @@ -90,4 +86,10 @@ class ServiceLevelContext implements Context { $this->createItem('Conjured Mana Cake', $initialSellIn, $initialQuality); } + + private function createItem(string $name, int $initialSellIn, int $initialQuality): void + { + $this->item = new Item($name, $initialSellIn, $initialQuality); + $this->gildedRose = new GildedRose([$this->item]); + } } diff --git a/php/tests/phpunit/GildedRoseItem/BackstagePassItemTest.php b/php/tests/phpunit/GildedRoseItem/BackstagePassItemTest.php index d3c3f8ea..fff6464b 100644 --- a/php/tests/phpunit/GildedRoseItem/BackstagePassItemTest.php +++ b/php/tests/phpunit/GildedRoseItem/BackstagePassItemTest.php @@ -4,10 +4,7 @@ declare(strict_types=1); namespace PhpUnitTests\GildedRoseItem; -use ApprovalTests\Approvals; -use GildedRose\GildedRoseItem; use GildedRose\GildedRoseItem\BackstagePassItem; -use GildedRose\GildedRoseItemFactory; use GildedRose\Item; use PHPUnit\Framework\TestCase; diff --git a/php/tests/phpunit/GildedRoseItem/BrieItemTest.php b/php/tests/phpunit/GildedRoseItem/BrieItemTest.php index ee19bdca..3d751444 100644 --- a/php/tests/phpunit/GildedRoseItem/BrieItemTest.php +++ b/php/tests/phpunit/GildedRoseItem/BrieItemTest.php @@ -4,10 +4,7 @@ declare(strict_types=1); namespace PhpUnitTests\GildedRoseItem; -use ApprovalTests\Approvals; -use GildedRose\GildedRoseItem; use GildedRose\GildedRoseItem\BrieItem; -use GildedRose\GildedRoseItemFactory; use GildedRose\Item; use PHPUnit\Framework\TestCase; diff --git a/php/tests/phpunit/GildedRoseItem/ConjuredItemTest.php b/php/tests/phpunit/GildedRoseItem/ConjuredItemTest.php index 21d4122d..053a5645 100644 --- a/php/tests/phpunit/GildedRoseItem/ConjuredItemTest.php +++ b/php/tests/phpunit/GildedRoseItem/ConjuredItemTest.php @@ -4,10 +4,7 @@ declare(strict_types=1); namespace PhpUnitTests\GildedRoseItem; -use ApprovalTests\Approvals; -use GildedRose\GildedRoseItem; use GildedRose\GildedRoseItem\ConjuredItem; -use GildedRose\GildedRoseItemFactory; use GildedRose\Item; use PHPUnit\Framework\TestCase; diff --git a/php/tests/phpunit/GildedRoseItem/DefaultItemTest.php b/php/tests/phpunit/GildedRoseItem/DefaultItemTest.php index bb798542..bbde0725 100644 --- a/php/tests/phpunit/GildedRoseItem/DefaultItemTest.php +++ b/php/tests/phpunit/GildedRoseItem/DefaultItemTest.php @@ -4,10 +4,7 @@ declare(strict_types=1); namespace PhpUnitTests\GildedRoseItem; -use ApprovalTests\Approvals; -use GildedRose\GildedRoseItem; use GildedRose\GildedRoseItem\DefaultItem; -use GildedRose\GildedRoseItemFactory; use GildedRose\Item; use PHPUnit\Framework\TestCase; diff --git a/php/tests/phpunit/GildedRoseItem/SulfurasItemTest.php b/php/tests/phpunit/GildedRoseItem/SulfurasItemTest.php index 835a9f34..d682f51d 100644 --- a/php/tests/phpunit/GildedRoseItem/SulfurasItemTest.php +++ b/php/tests/phpunit/GildedRoseItem/SulfurasItemTest.php @@ -4,10 +4,7 @@ declare(strict_types=1); namespace PhpUnitTests\GildedRoseItem; -use ApprovalTests\Approvals; -use GildedRose\GildedRoseItem; use GildedRose\GildedRoseItem\SulfurasItem; -use GildedRose\GildedRoseItemFactory; use GildedRose\Item; use PHPUnit\Framework\TestCase; diff --git a/php/tests/phpunit/GildedRoseItemFactoryTest.php b/php/tests/phpunit/GildedRoseItemFactoryTest.php index f854ca11..d557dfa4 100644 --- a/php/tests/phpunit/GildedRoseItemFactoryTest.php +++ b/php/tests/phpunit/GildedRoseItemFactoryTest.php @@ -4,10 +4,9 @@ declare(strict_types=1); namespace PhpUnitTests; -use ApprovalTests\Approvals; use GildedRose\GildedRoseItem; -use GildedRose\GildedRoseItem\BrieItem; use GildedRose\GildedRoseItem\BackstagePassItem; +use GildedRose\GildedRoseItem\BrieItem; use GildedRose\GildedRoseItem\ConjuredItem; use GildedRose\GildedRoseItem\DefaultItem; use GildedRose\GildedRoseItem\SulfurasItem;