Fix potentially flaky acceptance test

[MAILPOET-1987]
This commit is contained in:
Rostislav Wolny
2019-05-13 09:09:04 +02:00
committed by M. Shull
parent 240c11eed0
commit d574ac09a6

View File

@@ -2,6 +2,7 @@
namespace MailPoet\Test\Acceptance; namespace MailPoet\Test\Acceptance;
use MailPoet\Test\DataFactories\ScheduledTask;
use MailPoet\Test\DataFactories\Segment; use MailPoet\Test\DataFactories\Segment;
use MailPoet\Test\DataFactories\Settings; use MailPoet\Test\DataFactories\Settings;
use MailPoet\Test\DataFactories\Subscriber; use MailPoet\Test\DataFactories\Subscriber;
@@ -9,6 +10,7 @@ use MailPoet\Test\DataFactories\Subscriber;
require_once __DIR__ . '/../DataFactories/Segment.php'; require_once __DIR__ . '/../DataFactories/Segment.php';
require_once __DIR__ . '/../DataFactories/Subscriber.php'; require_once __DIR__ . '/../DataFactories/Subscriber.php';
require_once __DIR__ . '/../DataFactories/Settings.php'; require_once __DIR__ . '/../DataFactories/Settings.php';
require_once __DIR__ . '/../DataFactories/ScheduledTask.php';
class SettingsInactiveSubscribersChangeCest { class SettingsInactiveSubscribersChangeCest {
@@ -22,6 +24,8 @@ class SettingsInactiveSubscribersChangeCest {
(new Subscriber())->withStatus('inactive')->withSegments([$segment])->create(); (new Subscriber())->withStatus('inactive')->withSegments([$segment])->create();
} }
(new Settings)->withDeactivateSubscriberAfter3Months()->withTrackingEnabled(); (new Settings)->withDeactivateSubscriberAfter3Months()->withTrackingEnabled();
$scheduled_tasks_factory = new ScheduledTask();
$scheduled_tasks_factory->deleteAll();
} }
function inactiveSubscribersSettingsChange(\AcceptanceTester $I) { function inactiveSubscribersSettingsChange(\AcceptanceTester $I) {
@@ -35,7 +39,7 @@ class SettingsInactiveSubscribersChangeCest {
$I->waitForText('Settings saved'); $I->waitForText('Settings saved');
$I->amOnMailPoetPage('Subscribers'); $I->amOnMailPoetPage('Subscribers');
// Subscribers are activated in background so we do a couple of reloads // Subscribers are activated in background so we do a couple of reloads
for ($i = 0; $i < 10; $i++) { for ($i = 0; $i < 15; $i++) {
try { try {
$I->wait(2); $I->wait(2);
$I->reloadPage(); $I->reloadPage();