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

Fix PHPStan warnings in tests

Browse Source 
[MAILPOET-5751]
This commit is contained in:
Jan Lysý
2023-11-30 10:01:49 +01:00
committed by Aschepikov
parent 1119e73056
commit 41a55262fe
28 changed files with 92 additions and 66 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
mailpoet/tests/integration/Automation/Engine/Control/StepRunLoggerTest.php
View File

@@ -126,15 +126,13 @@ class StepRunLoggerTest extends MailPoetTest {
$logger->logSuccess();
/** @var int $runs PHPStan thinks $runs === 0 from the previous assert */
$this->assertSame(1, $runs);
$this->assertSame(1, $runs); // @phpstan-ignore-line - PHPStan thinks $runs === 0 from the previous assert
$this->assertNotNull($lastLog);
$this->assertLogData($lastLog, ['step_key' => 'step-key', 'status' => 'complete', 'data' => '{"test":"value"}']);
$error = new Exception('test error');
$logger->logFailure($error);
/** @var int $runs PHPStan thinks $runs === 1 from the previous assert */
$this->assertSame(2, $runs);
$this->assertNotNull($lastLog);
$this->assertLogData($lastLog, ['step_key' => 'step-key', 'status' => 'failed', 'data' => '{"test":"value"}', 'error' => $error]);
@@ -159,8 +157,7 @@ class StepRunLoggerTest extends MailPoetTest {
$logger->logSuccess();
/** @var int $runs PHPStan thinks $runs === 0 from the previous assert */
$this->assertSame(1, $runs);
$this->assertSame(1, $runs); // @phpstan-ignore-line - PHPStan thinks $runs === 0 from the previous assert
$logs = $this->storage->getLogsForAutomationRun(1);
$this->assertCount(1, $logs);