Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix code style

Browse Source 
[MAILPOET-2537]
This commit is contained in:
Amine Ben hammou
2020-01-15 17:40:53 +01:00
committed by Jack Kitterhing
parent d9c86b8af2
commit 856589da3d
1 changed files with 13 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
tests/integration/Subscription/ManageTest.php
View File

@ -16,31 +16,30 @@ use MailPoetVendor\Idiorm\ORM;
class ManageTest extends \MailPoetTest { class ManageTest extends \MailPoetTest {
private $settings; private $settings;
private $segment_a; private $segmentA;
private $segment_b; private $segmentB;
private $hidden_segment; private $hiddenSegment;
private $subscriber; private $subscriber;
public function _before() { public function _before() {
parent::_before(); parent::_before();
$this->_after(); $this->_after();
$di = $this->di_container; $di = $this->diContainer;
$this->settings = $di->get(SettingsController::class); $this->settings = $di->get(SettingsController::class);
$this->segment_a = Segment::createOrUpdate(['name' => 'List A']); $this->segmentA = Segment::createOrUpdate(['name' => 'List A']);
$this->segment_b = Segment::createOrUpdate(['name' => 'List B']); $this->segmentB = Segment::createOrUpdate(['name' => 'List B']);
$this->hidden_segment = Segment::createOrUpdate(['name' => 'Hidden List']); $this->hiddenSegment = Segment::createOrUpdate(['name' => 'Hidden List']);
$this->settings->set('subscription.segments', [$this->segment_a->id, $this->segment_b->id]); $this->settings->set('subscription.segments', [$this->segmentA->id, $this->segmentB->id]);
$this->subscriber = Subscriber::createOrUpdate([ $this->subscriber = Subscriber::createOrUpdate([
'first_name' => 'John', 'first_name' => 'John',
'last_name' => 'John', 'last_name' => 'John',
'email' => 'john.doe@example.com', 'email' => 'john.doe@example.com',
'status' => Subscriber::STATUS_SUBSCRIBED, 'status' => Subscriber::STATUS_SUBSCRIBED,
'segments' => [$this->segment_a->id, $this->hidden_segment->id], 'segments' => [$this->segmentA->id, $this->hiddenSegment->id],
]); ]);
} }
public function testItDoesntRemoveHiddenSegments() { public function testItDoesntRemoveHiddenSegments() {
$di = $this->di_container;
$manage = new Manage( $manage = new Manage(
Stub::make(UrlHelper::class, [ Stub::make(UrlHelper::class, [
'redirectBack' => null, 'redirectBack' => null,
@ -64,7 +63,7 @@ class ManageTest extends \MailPoetTest {
'last_name' => 'John', 'last_name' => 'John',
'email' => 'john.doe@example.com', 'email' => 'john.doe@example.com',
'status' => Subscriber::STATUS_SUBSCRIBED, 'status' => Subscriber::STATUS_SUBSCRIBED,
'segments' => [$this->segment_b->id], 'segments' => [$this->segmentB->id],
]; ];
$manage->onSave(); $manage->onSave();
@ -79,9 +78,9 @@ class ManageTest extends \MailPoetTest {
}); });
expect($subscriber->status)->equals(Subscriber::STATUS_SUBSCRIBED); expect($subscriber->status)->equals(Subscriber::STATUS_SUBSCRIBED);
expect($subscriptions)->equals([ expect($subscriptions)->equals([
['segment_id' => $this->segment_a->id, 'status' => Subscriber::STATUS_UNSUBSCRIBED], ['segment_id' => $this->segmentA->id, 'status' => Subscriber::STATUS_UNSUBSCRIBED],
['segment_id' => $this->segment_b->id, 'status' => Subscriber::STATUS_SUBSCRIBED], ['segment_id' => $this->segmentB->id, 'status' => Subscriber::STATUS_SUBSCRIBED],
['segment_id' => $this->hidden_segment->id, 'status' => Subscriber::STATUS_SUBSCRIBED], ['segment_id' => $this->hiddenSegment->id, 'status' => Subscriber::STATUS_SUBSCRIBED],
]); ]);
} }