Skip to content

Commit f0d136f

Browse files
committed
fix: some error formatter tests and also for old PHP 7.1
This reverts commit 4b6111b.
1 parent c73cf73 commit f0d136f

File tree

3 files changed

+15
-12
lines changed

3 files changed

+15
-12
lines changed

tests/ErrorFormatter/GithubErrorFormatterTest.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,7 @@ public function dataFormatterOutputProvider(): iterable
6464
Line folder with unicode 😃/file name with "spaces" and unicode 😃.php
6565
' . $dashLine . '
6666
4 Foo
67-
' . $dashLine . '
68-
67+
' . $dashLine . $this->getEndTable() . '
6968
7069
[ERROR] Found 1 error
7170
@@ -83,8 +82,7 @@ public function dataFormatterOutputProvider(): iterable
8382
-- ---------------------
8483
first generic error
8584
-- ---------------------
86-
87-
85+
' . $this->getEndTable() . '
8886
[ERROR] Found 1 error
8987
9088
::error ::first generic error
@@ -132,8 +130,7 @@ public function dataFormatterOutputProvider(): iterable
132130
first generic error
133131
second generic error
134132
-- ----------------------
135-
136-
133+
' . $this->getEndTable() . '
137134
[ERROR] Found 2 errors
138135
139136
::error ::first generic error

tests/ErrorFormatter/TableErrorFormatterTest.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,7 @@ public function dataFormatterOutputProvider(): iterable
6363
Line folder with unicode 😃/file name with "spaces" and unicode 😃.php
6464
' . $dashLine . '
6565
4 Foo
66-
' . $dashLine . '
67-
66+
' . $dashLine . $this->getEndTable() . '
6867
6968
[ERROR] Found 1 error
7069
@@ -81,8 +80,7 @@ public function dataFormatterOutputProvider(): iterable
8180
-- ---------------------
8281
first generic error
8382
-- ---------------------
84-
85-
83+
' . $this->getEndTable() . '
8684
[ERROR] Found 1 error
8785
8886
',
@@ -125,8 +123,7 @@ public function dataFormatterOutputProvider(): iterable
125123
first generic error
126124
second generic error
127125
-- ----------------------
128-
129-
126+
' . $this->getEndTable() . '
130127
[ERROR] Found 2 errors
131128
132129
',

tests/ErrorFormatterTestCase.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,15 @@ abstract class ErrorFormatterTestCase extends \CodeLts\CliTools\Tests\AbstractTe
4949
/** @var Output|null */
5050
private $output = null;
5151

52+
protected function getEndTable(): string
53+
{
54+
if (PHP_MAJOR_VERSION === 7 && PHP_MINOR_VERSION === 1) {
55+
return '';
56+
}
57+
58+
return "\n";
59+
}
60+
5261
protected function getTableDecoration(): string
5362
{
5463
if (PHP_MAJOR_VERSION === 7 && PHP_MINOR_VERSION === 1) {

0 commit comments

Comments
 (0)