Remove empty _afters
MAILPOET-5145
This commit is contained in:
committed by
John Oleksowicz
parent
f90e869402
commit
e7f766aedf
@ -586,10 +586,6 @@ class ServicesTest extends \MailPoetTest {
|
||||
expect($response->errors[0]['message'])->equals('Sending of congratulatory email failed.');
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
public function testItRespondsWithCorrectMessageIfKeyDoesntSupportMSS() {
|
||||
$bridge = $this->make(
|
||||
new Bridge(),
|
||||
|
@ -54,8 +54,4 @@ class SetupTest extends \MailPoetTest {
|
||||
$hookName = 'mailpoet_setup_reset';
|
||||
expect(WPHooksHelper::isActionDone($hookName))->true();
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -49,8 +49,4 @@ class TagsTest extends \MailPoetTest {
|
||||
$result = $this->testee->create([]);
|
||||
$this->assertEquals(400, $result->status);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -155,8 +155,4 @@ class TriggerHandlerTest extends \MailPoetTest {
|
||||
$this->testee->processTrigger($trigger, [$segmentSubject]);
|
||||
$this->assertEmpty($this->automationRunStorage->getAutomationRunsForAutomation($automation1));
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -220,10 +220,6 @@ class AutomationRunLogTest extends \MailPoetTest {
|
||||
expect(count($error['trace']))->greaterThan(0);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
private function getLogsForAction($callback = null) {
|
||||
if ($callback === null) {
|
||||
$callback = function() {
|
||||
|
@ -74,8 +74,4 @@ class AutomationTest extends \MailPoetTest {
|
||||
Automation::STATUS_ALL
|
||||
);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -42,8 +42,4 @@ class AutomationRunLogStorageTest extends \MailPoetTest {
|
||||
expect($errors['trace'])->array();
|
||||
expect(count($errors['trace']))->greaterThan(0);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -170,10 +170,6 @@ class AutomationStatisticsStorageTest extends \MailPoetTest {
|
||||
$this->assertEquals(1, $stats->getEntered());
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
private function createRun(Automation $automation, string $status) {
|
||||
$run = AutomationRun::fromArray([
|
||||
'automation_id' => $automation->getId(),
|
||||
|
@ -173,8 +173,4 @@ class AutomationStorageTest extends \MailPoetTest {
|
||||
}
|
||||
return $automation;
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -334,10 +334,6 @@ class CronHelperTest extends \MailPoetTest {
|
||||
expect($this->cronHelper->validatePingResponse('something else'))->false();
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
private function getDeamonTestData() {
|
||||
return [
|
||||
'token' => 'some_token',
|
||||
|
@ -276,10 +276,6 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
$daemon->ping();
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
private function createWorkersFactoryMock(array $workers = []) {
|
||||
$worker = $this->makeEmpty(SimpleWorker::class, [
|
||||
'process' => null,
|
||||
|
@ -89,10 +89,6 @@ class DaemonTest extends \MailPoetTest {
|
||||
$this->wp->removeFilter('mailpoet_cron_get_execution_limit', $limitCallback);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
|
||||
private function createWorkersFactoryMock(array $workers = []) {
|
||||
return $this->make(WorkersFactory::class, $workers + [
|
||||
'createScheduleWorker' => $this->createSimpleWorkerMock(),
|
||||
|
@ -72,8 +72,4 @@ class SupervisorTest extends \MailPoetTest {
|
||||
$daemon = $this->supervisor->checkDaemon();
|
||||
expect($daemon['status'])->equals(CronHelper::DAEMON_STATUS_ACTIVE);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -61,8 +61,4 @@ class PremiumKeyCheckTest extends \MailPoetTest {
|
||||
$this->premiumKey
|
||||
);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -114,8 +114,4 @@ class SendingServiceKeyCheckTest extends \MailPoetTest {
|
||||
]
|
||||
);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -125,8 +125,4 @@ class MailerTest extends \MailPoetTest {
|
||||
// send method should return true
|
||||
expect($mailerTask->send('Newsletter', 'Subscriber'))->equals(['response' => true]);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -54,8 +54,4 @@ class WooCommerceSyncTest extends \MailPoetTest {
|
||||
);
|
||||
expect($worker->processTaskStrategy($task, microtime(true)))->equals(true);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -339,9 +339,5 @@ class WordpressMailerTest extends \MailPoetTest {
|
||||
$mock->method('buildMailer')->willReturn($mailerMock);
|
||||
return $mock;
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
// phpcs:enable
|
||||
|
@ -144,8 +144,4 @@ class AutomationPutEndpointTest extends AutomationTest {
|
||||
}
|
||||
return $data;
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -86,8 +86,4 @@ class AutomationsCreateFromTemplateTest extends AutomationTest {
|
||||
$this->assertInstanceOf(Automation::class, $createdAutomation);
|
||||
$this->assertSame($createdAutomation->getId(), $response['data']['id']);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -72,8 +72,4 @@ class AutomationsDeleteEndpointTest extends AutomationTest {
|
||||
$automation = $this->automationStorage->getAutomation($this->automation->getId());
|
||||
$this->assertNull($automation);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -111,8 +111,4 @@ class AutomationsDuplicateEndpointTest extends AutomationTest {
|
||||
$this->assertInstanceOf(Automation::class, $automation);
|
||||
$this->assertTrue($automation->equals($expectedAutomation));
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -461,8 +461,4 @@ class BridgeTest extends \MailPoetTest {
|
||||
private function getPremiumKeyState() {
|
||||
return $this->settings->get(Bridge::PREMIUM_KEY_STATE_SETTING_NAME);
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -262,8 +262,4 @@ class SubscriberActionsTest extends \MailPoetTest {
|
||||
$this->entityManager->flush();
|
||||
return $newsletter;
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
@ -100,8 +100,4 @@ class ConflictResolverTest extends \MailPoetTest {
|
||||
// it should not dequeue select2 script
|
||||
expect(in_array('select2', $wp_scripts->queue))->true(); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps
|
||||
}
|
||||
|
||||
public function _after() {
|
||||
parent::_after();
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user