Fix remaining code style issues
[MAILPOET-4850]
This commit is contained in:
@@ -44,7 +44,6 @@ class UnsubscribeAnalyticsTest extends \MailPoetTest {
|
|||||||
expect($properties[sprintf(UnsubscribeReporter::COUNT_PER_METHOD_PATTERN, 'Link')])->equals(1);
|
expect($properties[sprintf(UnsubscribeReporter::COUNT_PER_METHOD_PATTERN, 'Link')])->equals(1);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
private function createStatisticsUnsubscribe(\DateTimeInterface $createdAt, $method): StatisticsUnsubscribeEntity {
|
private function createStatisticsUnsubscribe(\DateTimeInterface $createdAt, $method): StatisticsUnsubscribeEntity {
|
||||||
$subscriber = (new Subscriber())->create();
|
$subscriber = (new Subscriber())->create();
|
||||||
$newsletter = (new Newsletter())->create();
|
$newsletter = (new Newsletter())->create();
|
||||||
@@ -63,5 +62,4 @@ class UnsubscribeAnalyticsTest extends \MailPoetTest {
|
|||||||
parent::_after();
|
parent::_after();
|
||||||
$this->truncateEntity(StatisticsUnsubscribeEntity::class);
|
$this->truncateEntity(StatisticsUnsubscribeEntity::class);
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@@ -16,8 +16,8 @@ use MailPoet\Automation\Integrations\MailPoet\Triggers\UserRegistrationTrigger;
|
|||||||
use MailPoet\Entities\SegmentEntity;
|
use MailPoet\Entities\SegmentEntity;
|
||||||
use MailPoet\Segments\SegmentsRepository;
|
use MailPoet\Segments\SegmentsRepository;
|
||||||
|
|
||||||
class TriggerHandlerTest extends \MailPoetTest
|
class TriggerHandlerTest extends \MailPoetTest {
|
||||||
{
|
|
||||||
|
|
||||||
/** @var TriggerHandler */
|
/** @var TriggerHandler */
|
||||||
private $testee;
|
private $testee;
|
||||||
@@ -61,7 +61,7 @@ class TriggerHandlerTest extends \MailPoetTest
|
|||||||
Step::TYPE_TRIGGER,
|
Step::TYPE_TRIGGER,
|
||||||
$trigger->getKey(),
|
$trigger->getKey(),
|
||||||
[
|
[
|
||||||
'segment_ids' => [$this->segments['segment_1']->getId()]
|
'segment_ids' => [$this->segments['segment_1']->getId()],
|
||||||
],
|
],
|
||||||
[]
|
[]
|
||||||
)
|
)
|
||||||
@@ -73,7 +73,7 @@ class TriggerHandlerTest extends \MailPoetTest
|
|||||||
Step::TYPE_TRIGGER,
|
Step::TYPE_TRIGGER,
|
||||||
$trigger->getKey(),
|
$trigger->getKey(),
|
||||||
[
|
[
|
||||||
'segment_ids' => [$this->segments['segment_2']->getId()]
|
'segment_ids' => [$this->segments['segment_2']->getId()],
|
||||||
],
|
],
|
||||||
[]
|
[]
|
||||||
)
|
)
|
||||||
@@ -108,7 +108,7 @@ class TriggerHandlerTest extends \MailPoetTest
|
|||||||
Step::TYPE_TRIGGER,
|
Step::TYPE_TRIGGER,
|
||||||
$trigger->getKey(),
|
$trigger->getKey(),
|
||||||
[
|
[
|
||||||
'segment_ids' => [$this->segments['segment_1']->getId()]
|
'segment_ids' => [$this->segments['segment_1']->getId()],
|
||||||
],
|
],
|
||||||
[]
|
[]
|
||||||
)
|
)
|
||||||
@@ -120,7 +120,7 @@ class TriggerHandlerTest extends \MailPoetTest
|
|||||||
Step::TYPE_TRIGGER,
|
Step::TYPE_TRIGGER,
|
||||||
$trigger->getKey(),
|
$trigger->getKey(),
|
||||||
[
|
[
|
||||||
'segment_ids' => []
|
'segment_ids' => [],
|
||||||
],
|
],
|
||||||
[]
|
[]
|
||||||
)
|
)
|
||||||
@@ -190,5 +190,4 @@ class TriggerHandlerTest extends \MailPoetTest
|
|||||||
$this->automationStorage->truncate();
|
$this->automationStorage->truncate();
|
||||||
$this->segmentRepository->truncate();
|
$this->segmentRepository->truncate();
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@@ -304,7 +304,7 @@ class WooCommercePurchasesTest extends \MailPoetTest {
|
|||||||
$this->createClick($this->link, $this->subscriber, 0); // wrong click, should not be tracked
|
$this->createClick($this->link, $this->subscriber, 0); // wrong click, should not be tracked
|
||||||
$this->entityManager->flush();
|
$this->entityManager->flush();
|
||||||
|
|
||||||
$orderMock = $this->createOrderMock($this->subscriber->getEmail(),15.0, 123, new DateTime("-1 minute"));
|
$orderMock = $this->createOrderMock($this->subscriber->getEmail(), 15.0, 123, new DateTime("-1 minute"));
|
||||||
$woocommercePurchases = new WooCommercePurchases(
|
$woocommercePurchases = new WooCommercePurchases(
|
||||||
$this->createWooCommerceHelperMock($orderMock),
|
$this->createWooCommerceHelperMock($orderMock),
|
||||||
$this->diContainer->get(StatisticsWooCommercePurchasesRepository::class),
|
$this->diContainer->get(StatisticsWooCommercePurchasesRepository::class),
|
||||||
|
Reference in New Issue
Block a user