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

Fix Subscribers intergration test with refactor

Browse Source 
[MAILPOET-3033]
This commit is contained in:
Jan Lysý
2020-10-05 09:44:53 +02:00
committed by Veljko V
parent 6ef68f7625
commit 00b0cf184d
1 changed files with 18 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
tests/integration/API/JSON/v1/SubscribersTest.php
View File

@ -3,6 +3,7 @@
namespace MailPoet\Test\API\JSON\v1; namespace MailPoet\Test\API\JSON\v1;
use Codeception\Util\Fixtures; use Codeception\Util\Fixtures;
use MailPoet\API\JSON\Error;
use MailPoet\API\JSON\Response as APIResponse; use MailPoet\API\JSON\Response as APIResponse;
use MailPoet\API\JSON\ResponseBuilders\SubscribersResponseBuilder; use MailPoet\API\JSON\ResponseBuilders\SubscribersResponseBuilder;
use MailPoet\API\JSON\v1\Subscribers; use MailPoet\API\JSON\v1\Subscribers;
@ -28,6 +29,7 @@ use MailPoet\Models\SendingQueue;
use MailPoet\Models\Subscriber; use MailPoet\Models\Subscriber;
use MailPoet\Models\SubscriberIP; use MailPoet\Models\SubscriberIP;
use MailPoet\Models\SubscriberSegment; use MailPoet\Models\SubscriberSegment;
use MailPoet\Segments\SegmentsRepository;
use MailPoet\Settings\SettingsController; use MailPoet\Settings\SettingsController;
use MailPoet\Settings\SettingsRepository; use MailPoet\Settings\SettingsRepository;
use MailPoet\Statistics\Track\Unsubscribes; use MailPoet\Statistics\Track\Unsubscribes;
@ -42,6 +44,7 @@ use MailPoet\Subscription\Captcha;
use MailPoet\Subscription\CaptchaSession; use MailPoet\Subscription\CaptchaSession;
use MailPoet\Subscription\SubscriptionUrlFactory; use MailPoet\Subscription\SubscriptionUrlFactory;
use MailPoet\Test\DataFactories\DynamicSegment; use MailPoet\Test\DataFactories\DynamicSegment;
use MailPoet\UnexpectedValueException;
use MailPoet\WP\Functions; use MailPoet\WP\Functions;
use MailPoetVendor\Carbon\Carbon; use MailPoetVendor\Carbon\Carbon;
use MailPoetVendor\Idiorm\ORM; use MailPoetVendor\Idiorm\ORM;
@ -97,7 +100,7 @@ class SubscribersTest extends \MailPoetTest {
$container->get(SubscribersRepository::class), $container->get(SubscribersRepository::class),
$container->get(SubscribersResponseBuilder::class), $container->get(SubscribersResponseBuilder::class),
$container->get(SubscriberListingRepository::class), $container->get(SubscriberListingRepository::class),
$container->get(SubscribersRepository::class), $container->get(SegmentsRepository::class),
$obfuscator $obfuscator
); );
$this->obfuscatedEmail = $obfuscator->obfuscate('email'); $this->obfuscatedEmail = $obfuscator->obfuscate('email');
@ -269,9 +272,11 @@ class SubscribersTest extends \MailPoetTest {
public function testItCanTrashASubscriber() { public function testItCanTrashASubscriber() {
$response = $this->endpoint->trash(['id' => $this->subscriber2->getId()]); $response = $this->endpoint->trash(['id' => $this->subscriber2->getId()]);
expect($response->status)->equals(APIResponse::STATUS_OK); expect($response->status)->equals(APIResponse::STATUS_OK);
expect($response->data)->equals( $subscriberRepository = $this->diContainer->get(SubscribersRepository::class);
Subscriber::findOne($this->subscriber2->getId())->asArray() $subscriber = $subscriberRepository->findOneById($this->subscriber2->getId());
); expect($response->data['id'])->equals($subscriber->getId());
expect($response->data['email'])->equals($subscriber->getEmail());
expect($response->data['status'])->equals($subscriber->getStatus());
expect($response->data['deleted_at'])->notNull(); expect($response->data['deleted_at'])->notNull();
expect($response->meta['count'])->equals(1); expect($response->meta['count'])->equals(1);
} }
@ -520,12 +525,15 @@ class SubscribersTest extends \MailPoetTest {
} }
public function testItCannotRunAnInvalidBulkAction() { public function testItCannotRunAnInvalidBulkAction() {
$response = $this->endpoint->bulkAction([ try {
'action' => 'invalidAction', $this->endpoint->bulkAction([
'listing' => [], 'action' => 'invalidAction',
]); 'listing' => [],
expect($response->status)->equals(APIResponse::STATUS_NOT_FOUND); ]);
expect($response->errors[0]['message'])->contains('has no method'); } catch (UnexpectedValueException $exception) {
expect($exception->getHttpStatusCode())->equals(APIResponse::STATUS_BAD_REQUEST);
expect($exception->getErrors()[Error::BAD_REQUEST])->contains('Invalid bulk action');
}
} }
public function testItFailsWithEmailFilled() { public function testItFailsWithEmailFilled() {