From 21d1bc113c068d0b9a204d24ce410429d2fb0325 Mon Sep 17 00:00:00 2001 From: Oliver Davies Date: Mon, 20 May 2019 22:40:38 +0100 Subject: [PATCH] Only confirmed sponsors are returned --- src/Sponsors/src/TwigExtension/SponsorsExtension.php | 1 + src/Sponsors/tests/SponsorsTest.php | 12 +++++++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/src/Sponsors/src/TwigExtension/SponsorsExtension.php b/src/Sponsors/src/TwigExtension/SponsorsExtension.php index 0769a07..966e1c4 100644 --- a/src/Sponsors/src/TwigExtension/SponsorsExtension.php +++ b/src/Sponsors/src/TwigExtension/SponsorsExtension.php @@ -28,6 +28,7 @@ class SponsorsExtension extends AbstractExtension }) ->filter->isConfirmed() ->map->getData() + ->values() ->toArray(); } } diff --git a/src/Sponsors/tests/SponsorsTest.php b/src/Sponsors/tests/SponsorsTest.php index bed0c7a..cd108bd 100644 --- a/src/Sponsors/tests/SponsorsTest.php +++ b/src/Sponsors/tests/SponsorsTest.php @@ -39,6 +39,16 @@ class SponsorsTest extends TestCase /** @test */ public function only_confirmed_sponsors_are_returned() { - $this->markTestIncomplete(); + $data = [ + Sponsor::LEVEL_GOLD => [ + ['name' => 'Acquia', 'confirmed' => false], + ['name' => 'Microserve', 'confirmed' => true], + ], + ]; + + $sponsors = $this->extension->getSponsors($data, Sponsor::LEVEL_GOLD); + + $this->assertCount(1, $sponsors); + $this->assertSame('Microserve', $sponsors[0]['name']); } }