Skip to content

Commit 6784424

Browse files
authored
Merge pull request #23 from pagarme/release/1.1.2
Release 1.1.2
2 parents 7b5a390 + 097c329 commit 6784424

File tree

4 files changed

+53
-12
lines changed

4 files changed

+53
-12
lines changed

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"name": "pagarme/ecommerce-module-core",
33
"description": "Core component for Pagar.me e-commerce platform modules.",
44
"license": "MIT",
5-
"version": "1.1.1",
5+
"version": "1.1.2",
66
"authors": [
77
{
88
"name":"Open Source Team"

src/Maintenance/Assets/integrityData

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

src/Recurrence/Aggregates/Subscription.php

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -564,21 +564,12 @@ private function setCardData(CreateSubscriptionRequest $subscriptionRequest)
564564
return;
565565
}
566566

567-
if(!empty($subscriptionRequest->cardId)){
568-
return;
569-
}
570-
571-
if(!empty($subscriptionRequest->cardToken)){
572-
return;
573-
}
574-
575567
$card = new CreateCardRequest();
576568
if ($this->getCustomer()->getAddress() != null) {
577569
$card->billingAddress = $this->getCustomer()->getAddress()->convertToSDKRequest();
578570
}
579571

580572
$subscriptionRequest->card = $card;
581-
582573
}
583574

584575
public function getStatusValue()

tests/Recurrence/Aggregates/SubscriptionTest.php

Lines changed: 51 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
namespace Pagarme\Core\Test\Recurrence\Aggregates;
44

55
use Mockery;
6+
use MundiAPILib\Models\CreateAddressRequest;
7+
use MundiAPILib\Models\CreateCardRequest;
68
use MundiAPILib\Models\CreateSubscriptionRequest;
79
use Pagarme\Core\Kernel\Interfaces\PlatformOrderInterface;
810
use Pagarme\Core\Kernel\ValueObjects\Id\ChargeId;
@@ -187,4 +189,52 @@ public function testShouldSetAnArrayOfChargesOnSubscription()
187189
$this->assertContainsOnlyInstancesOf(Charge::class, $this->subscription->getCharges());
188190
$this->assertCount(2, $this->subscription->getCharges());
189191
}
190-
}
192+
193+
public function testShouldReturnACreateSubscriptionRequestObjectWithCardBillingAddress()
194+
{
195+
$this->subscription->setCustomer(new Customer());
196+
$this->subscription->setItems([new SubProduct]);
197+
$this->subscription->getCustomer()->setAddress(new Address());
198+
$this->subscription->setCardToken("cardToken");
199+
200+
$shipping = new Shipping;
201+
$shipping->setRecipientPhone(new Phone("021999999999"));
202+
$shipping->setAddress(new Address());
203+
204+
$this->subscription->setShipping($shipping);
205+
206+
$card = new CreateCardRequest();
207+
$card->billingAddress = $this->subscription->getCustomer()
208+
->getAddress()->convertToSDKRequest();
209+
$this->subscription->card = $card;
210+
211+
$sdkObject = $this->subscription->convertToSdkRequest();
212+
213+
$this->assertInstanceOf(CreateSubscriptionRequest::class, $sdkObject);
214+
$this->assertNotNull($sdkObject->card->billingAddress);
215+
}
216+
217+
public function testShouldReturnACreateSubscriptionRequestObjectWithSavedCardBillingAddress()
218+
{
219+
$this->subscription->setCustomer(new Customer());
220+
$this->subscription->setItems([new SubProduct]);
221+
$this->subscription->getCustomer()->setAddress(new Address());
222+
$this->subscription->setCardId("cardId");
223+
224+
$shipping = new Shipping;
225+
$shipping->setRecipientPhone(new Phone("021999999999"));
226+
$shipping->setAddress(new Address());
227+
228+
$this->subscription->setShipping($shipping);
229+
230+
$card = new CreateCardRequest();
231+
$card->billingAddress = $this->subscription->getCustomer()
232+
->getAddress()->convertToSDKRequest();
233+
$this->subscription->card = $card;
234+
235+
$sdkObject = $this->subscription->convertToSdkRequest();
236+
237+
$this->assertInstanceOf(CreateSubscriptionRequest::class, $sdkObject);
238+
$this->assertNotNull($sdkObject->card->billingAddress);
239+
}
240+
}

0 commit comments

Comments
 (0)