',
+ ],
+ ]);
+
+ // Assert
+ $settings = $entity->getExpressCheckoutSettings();
+ self::assertNull($settings->getButtonStyle());
+ self::assertTrue($settings->isPageEnabled('product'));
+ }
+
+ /**
+ * @return void
+ *
+ * @throws InvalidExpressCheckoutPageException
+ * @throws DuplicatedExpressCheckoutPageException
+ * @throws InvalidExpressCheckoutPageConfigException
+ */
+ public function testInflateKeepsAValidStoredButtonStyle(): void
+ {
+ // Arrange
+ $buttonStyle = '{"backgroundColor":"#123456"}';
+
+ // Act
+ $entity = ExpressCheckoutSettings::fromArray([
+ 'id' => 1,
+ 'storeId' => '1',
+ 'expressCheckoutSettings' => [
+ 'expressCheckoutConfigs' => [
+ ['page' => 'product', 'enabled' => true],
+ ],
+ 'buttonStyle' => $buttonStyle,
+ ],
+ ]);
+
+ // Assert
+ self::assertSame($buttonStyle, $entity->getExpressCheckoutSettings()->getButtonStyle());
+ }
+}
diff --git a/tests/BusinessLogic/DataAccess/ExpressCheckout/Repositories/ExpressCheckoutSettingsRepositoryTest.php b/tests/BusinessLogic/DataAccess/ExpressCheckout/Repositories/ExpressCheckoutSettingsRepositoryTest.php
index 79b8a287..2e038548 100644
--- a/tests/BusinessLogic/DataAccess/ExpressCheckout/Repositories/ExpressCheckoutSettingsRepositoryTest.php
+++ b/tests/BusinessLogic/DataAccess/ExpressCheckout/Repositories/ExpressCheckoutSettingsRepositoryTest.php
@@ -172,6 +172,132 @@ public function testDeleteExpressCheckoutSettingsIsNoopWhenNoEntityStored(): voi
self::assertNull($loaded);
}
+ /**
+ * @return void
+ *
+ * @throws Exception
+ */
+ public function testGetExpressCheckoutSettingsToleratesLegacyRowWithoutButtonStyle(): void
+ {
+ // Arrange
+ $entity = new ExpressCheckoutSettingsEntity();
+ $entity->inflate([
+ 'storeId' => '1',
+ 'expressCheckoutSettings' => [
+ 'expressCheckoutConfigs' => [],
+ ],
+ ]);
+ TestRepositoryRegistry::getRepository(ExpressCheckoutSettingsEntity::getClassName())->save($entity);
+
+ // Act
+ $loaded = StoreContext::doWithStore('1', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertNotNull($loaded);
+ self::assertNull($loaded->getButtonStyle());
+ }
+
+ /**
+ * @return void
+ *
+ * @throws Exception
+ */
+ public function testSetExpressCheckoutSettingsRoundTripsOpaqueButtonStyle(): void
+ {
+ // Arrange
+ $buttonStyle = '{"backgroundColor":"#123456","somethingNewerThanThisRelease":true}';
+ $settings = new ExpressCheckoutSettings(
+ [new ExpressCheckoutPageConfig(ExpressCheckoutPage::product(), true)],
+ $buttonStyle
+ );
+
+ // Act
+ StoreContext::doWithStore('1', [$this->repository, 'setExpressCheckoutSettings'], [$settings]);
+ $loaded = StoreContext::doWithStore('1', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertNotNull($loaded);
+ self::assertSame($buttonStyle, $loaded->getButtonStyle());
+ }
+
+ /**
+ * @return void
+ *
+ * @throws Exception
+ */
+ public function testSetExpressCheckoutSettingsScopesButtonStyleByStore(): void
+ {
+ // Arrange
+ $buttonStyleOne = '{"backgroundColor":"#123456"}';
+ $buttonStyleTwo = '{"backgroundColor":"#abcdef"}';
+ $storeOne = new ExpressCheckoutSettings(
+ [new ExpressCheckoutPageConfig(ExpressCheckoutPage::product(), true)],
+ $buttonStyleOne
+ );
+ $storeTwo = new ExpressCheckoutSettings(
+ [new ExpressCheckoutPageConfig(ExpressCheckoutPage::cart(), true)],
+ $buttonStyleTwo
+ );
+
+ // Act
+ StoreContext::doWithStore('1', [$this->repository, 'setExpressCheckoutSettings'], [$storeOne]);
+ StoreContext::doWithStore('2', [$this->repository, 'setExpressCheckoutSettings'], [$storeTwo]);
+ $loadedOne = StoreContext::doWithStore('1', [$this->repository, 'getExpressCheckoutSettings']);
+ $loadedTwo = StoreContext::doWithStore('2', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertSame($buttonStyleOne, $loadedOne->getButtonStyle());
+ self::assertSame($buttonStyleTwo, $loadedTwo->getButtonStyle());
+
+ // Act - updating store '1' must leave store '2' untouched
+ $buttonStyleOneUpdated = '{"backgroundColor":"#000000"}';
+ $storeOneUpdated = new ExpressCheckoutSettings(
+ [new ExpressCheckoutPageConfig(ExpressCheckoutPage::product(), true)],
+ $buttonStyleOneUpdated
+ );
+ StoreContext::doWithStore('1', [$this->repository, 'setExpressCheckoutSettings'], [$storeOneUpdated]);
+ $loadedOneAfterUpdate = StoreContext::doWithStore('1', [$this->repository, 'getExpressCheckoutSettings']);
+ $loadedTwoAfterUpdate = StoreContext::doWithStore('2', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertSame($buttonStyleOneUpdated, $loadedOneAfterUpdate->getButtonStyle());
+ self::assertSame($buttonStyleTwo, $loadedTwoAfterUpdate->getButtonStyle());
+
+ // Act - a store with no settings at all
+ $loadedThree = StoreContext::doWithStore('3', [$this->repository, 'getExpressCheckoutSettings']);
+ $reloadedTwo = StoreContext::doWithStore('2', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertNull($loadedThree);
+ self::assertSame($buttonStyleTwo, $reloadedTwo->getButtonStyle());
+ }
+
+ /**
+ * @return void
+ *
+ * @throws Exception
+ */
+ public function testGetExpressCheckoutSettingsDegradesNonStringButtonStyleToNull(): void
+ {
+ // Arrange
+ $entity = new ExpressCheckoutSettingsEntity();
+ $entity->inflate([
+ 'storeId' => '1',
+ 'expressCheckoutSettings' => [
+ 'expressCheckoutConfigs' => [],
+ 'buttonStyle' => ['backgroundColor' => '#123456'],
+ ],
+ ]);
+ TestRepositoryRegistry::getRepository(ExpressCheckoutSettingsEntity::getClassName())->save($entity);
+
+ // Act
+ $loaded = StoreContext::doWithStore('1', [$this->repository, 'getExpressCheckoutSettings']);
+
+ // Assert
+ self::assertNotNull($loaded);
+ self::assertNull($loaded->getButtonStyle());
+ }
+
/**
* @param string $storeId
*
diff --git a/tests/BusinessLogic/Domain/ExpressCheckout/Models/ExpressCheckoutSettingsTest.php b/tests/BusinessLogic/Domain/ExpressCheckout/Models/ExpressCheckoutSettingsTest.php
index 1e1ca9e1..823a87ef 100644
--- a/tests/BusinessLogic/Domain/ExpressCheckout/Models/ExpressCheckoutSettingsTest.php
+++ b/tests/BusinessLogic/Domain/ExpressCheckout/Models/ExpressCheckoutSettingsTest.php
@@ -4,6 +4,7 @@
use PHPUnit\Framework\TestCase;
use SeQura\Core\BusinessLogic\Domain\ExpressCheckout\Exceptions\DuplicatedExpressCheckoutPageException;
+use SeQura\Core\BusinessLogic\Domain\ExpressCheckout\Exceptions\InvalidExpressCheckoutButtonStyleException;
use SeQura\Core\BusinessLogic\Domain\ExpressCheckout\Exceptions\InvalidExpressCheckoutPageConfigException;
use SeQura\Core\BusinessLogic\Domain\ExpressCheckout\Models\ExpressCheckoutPage;
use SeQura\Core\BusinessLogic\Domain\ExpressCheckout\Models\ExpressCheckoutPageConfig;
@@ -168,6 +169,7 @@ public function testToArraySerializesConfigs(): void
['page' => 'cart', 'enabled' => false],
['page' => 'mini-cart', 'enabled' => true],
],
+ 'buttonStyle' => null,
], $settings->toArray());
}
@@ -178,6 +180,112 @@ public function testToArrayReturnsEmptyConfigsForDefaultConstruction(): void
{
$settings = new ExpressCheckoutSettings();
- self::assertSame(['expressCheckoutConfigs' => []], $settings->toArray());
+ self::assertSame(['expressCheckoutConfigs' => [], 'buttonStyle' => null], $settings->toArray());
+ }
+
+ /**
+ * @return void
+ */
+ public function testButtonStyleDefaultsToNull(): void
+ {
+ $settings = new ExpressCheckoutSettings();
+
+ self::assertNull($settings->getButtonStyle());
+ }
+
+ /**
+ * @return void
+ */
+ public function testConstructorTreatsAnEmptyButtonStyleAsUnset(): void
+ {
+ $settings = new ExpressCheckoutSettings([], '');
+
+ self::assertNull($settings->getButtonStyle());
+ }
+
+ /**
+ * @return void
+ *
+ * @throws DuplicatedExpressCheckoutPageException
+ * @throws InvalidExpressCheckoutPageConfigException
+ */
+ public function testButtonStyleIsCarriedVerbatim(): void
+ {
+ $buttonStyle = '{"backgroundColor":"#123456","somethingNewerThanThisRelease":true}';
+
+ $settings = new ExpressCheckoutSettings(
+ [new ExpressCheckoutPageConfig(ExpressCheckoutPage::product(), true)],
+ $buttonStyle
+ );
+
+ self::assertSame($buttonStyle, $settings->getButtonStyle());
+ self::assertSame($buttonStyle, $settings->toArray()['buttonStyle']);
+ }
+
+ /**
+ * @return array
+ */
+ public function invalidButtonStyleProvider(): array
+ {
+ return [
+ 'json null literal' => ['null'],
+ 'json number' => ['123'],
+ 'json string' => ['"#ffffff"'],
+ 'json list' => ['[{"backgroundColor":"#123456"}]'],
+ 'truncated object' => ['{"backgroundColor":'],
+ 'not json at all' => [''],
+ ];
+ }
+
+ /**
+ * @dataProvider invalidButtonStyleProvider
+ *
+ * @param string $buttonStyle
+ *
+ * @return void
+ *
+ * @throws DuplicatedExpressCheckoutPageException
+ * @throws InvalidExpressCheckoutPageConfigException
+ * @throws InvalidExpressCheckoutButtonStyleException
+ */
+ public function testConstructorRejectsAButtonStyleThatIsNotAJsonObject(string $buttonStyle): void
+ {
+ $this->expectException(InvalidExpressCheckoutButtonStyleException::class);
+
+ new ExpressCheckoutSettings([], $buttonStyle);
+ }
+
+ /**
+ * @return void
+ *
+ * @throws DuplicatedExpressCheckoutPageException
+ * @throws InvalidExpressCheckoutPageConfigException
+ * @throws InvalidExpressCheckoutButtonStyleException
+ */
+ public function testConstructorRejectsAnOversizedButtonStyle(): void
+ {
+ $buttonStyle = '{"backgroundColor":"' . str_repeat('a', 1515) . '"}';
+ self::assertSame(1537, strlen($buttonStyle));
+
+ $this->expectException(InvalidExpressCheckoutButtonStyleException::class);
+
+ new ExpressCheckoutSettings([], $buttonStyle);
+ }
+
+ /**
+ * @return void
+ *
+ * @throws DuplicatedExpressCheckoutPageException
+ * @throws InvalidExpressCheckoutPageConfigException
+ * @throws InvalidExpressCheckoutButtonStyleException
+ */
+ public function testConstructorAcceptsAButtonStyleAtTheSizeLimit(): void
+ {
+ $buttonStyle = '{"backgroundColor":"' . str_repeat('a', 1514) . '"}';
+ self::assertSame(1536, strlen($buttonStyle));
+
+ $settings = new ExpressCheckoutSettings([], $buttonStyle);
+
+ self::assertSame($buttonStyle, $settings->getButtonStyle());
}
}