Skip to content

Commit 290ccdd

Browse files
Test baseline with all six types (E_DEPRECATED, E_NOTICE, E_WARNING, E_USER_DEPRECATED, E_USER_NOTICE, E_USER_WARNING) of issues that can be ignored
1 parent 50d62bd commit 290ccdd

File tree

5 files changed

+107
-13
lines changed

5 files changed

+107
-13
lines changed

.php-cs-fixer.dist.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
->in(__DIR__ . '/tests/unit')
1717
// DeprecatedPhpFeatureTest.php must not use declare(strict_types=1);
1818
->notName('DeprecatedPhpFeatureTest.php')
19+
// UseBaselineTest.php must not use declare(strict_types=1);
20+
->notName('UseBaselineTest.php')
1921
// Issue5795Test.php contains required whitespace that would be cleaned up
2022
->notName('Issue5795Test.php')
2123
->notName('*.phpt');
Lines changed: 20 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,23 @@
11
<?xml version="1.0"?>
22
<files version="1">
3-
<file path="tests/Test.php">
4-
<line number="20" hash="a1022fb62c4705938dd2c6df5ff35b2621f9e97d">
5-
<issue><![CDATA[deprecation]]></issue>
6-
</line>
7-
</file>
3+
<file path="tests/UseBaselineTest.php">
4+
<line number="23" hash="dba225b56a963ec1cb91eff6711e51d9e3df6c6e">
5+
<issue><![CDATA[strlen(): Passing null to parameter #1 ($string) of type string is deprecated]]></issue>
6+
</line>
7+
<line number="29" hash="b8af300020a6744a3774f463a41339c21bef5648">
8+
<issue><![CDATA[Only variables should be assigned by reference]]></issue>
9+
</line>
10+
<line number="31" hash="1144713f7825a9ca87b378ce41789cf994e627e6">
11+
<issue><![CDATA[Undefined variable $b]]></issue>
12+
</line>
13+
<line number="33" hash="a1022fb62c4705938dd2c6df5ff35b2621f9e97d">
14+
<issue><![CDATA[deprecation]]></issue>
15+
</line>
16+
<line number="34" hash="fff8be75c2fbcbc4d395247e58fbbe6541189cf0">
17+
<issue><![CDATA[notice]]></issue>
18+
</line>
19+
<line number="35" hash="a5b91c0a182bedb089007e5bc0d0f462637bc904">
20+
<issue><![CDATA[warning]]></issue>
21+
</line>
22+
</file>
823
</files>

tests/end-to-end/_files/baseline/use-baseline/tests/Test.php renamed to tests/end-to-end/_files/baseline/use-baseline/tests/UseBaselineTest.php

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<?php declare(strict_types=1);
1+
<?php
22
/*
33
* This file is part of PHPUnit.
44
*
@@ -10,14 +10,29 @@
1010
namespace PHPUnit\TestFixture\Baseline;
1111

1212
use const E_USER_DEPRECATED;
13+
use const E_USER_NOTICE;
14+
use const E_USER_WARNING;
15+
use function strlen;
1316
use function trigger_error;
1417
use PHPUnit\Framework\TestCase;
1518

16-
final class Test extends TestCase
19+
final class UseBaselineTest extends TestCase
1720
{
1821
public function testOne(): void
1922
{
23+
strlen(null);
24+
25+
$f = static function (): void
26+
{
27+
};
28+
29+
$a = &$f();
30+
31+
$a = $b;
32+
2033
trigger_error('deprecation', E_USER_DEPRECATED);
34+
trigger_error('notice', E_USER_NOTICE);
35+
trigger_error('warning', E_USER_WARNING);
2136

2237
$this->assertTrue(true);
2338
}

tests/end-to-end/baseline/ignore-baseline.phpt

Lines changed: 67 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ $_SERVER['argv'][] = '--configuration';
77
$_SERVER['argv'][] = __DIR__ . '/../_files/baseline/use-baseline/phpunit.xml';
88
$_SERVER['argv'][] = '--ignore-baseline';
99
$_SERVER['argv'][] = '--display-deprecations';
10+
$_SERVER['argv'][] = '--display-notices';
11+
$_SERVER['argv'][] = '--display-warnings';
1012

1113
require_once __DIR__ . '/../../bootstrap.php';
1214

@@ -17,19 +19,79 @@ PHPUnit %s by Sebastian Bergmann and contributors.
1719
Runtime: %s
1820
Configuration: %s
1921

20-
D 1 / 1 (100%)
22+
W 1 / 1 (100%)
2123

2224
Time: %s, Memory: %s
2325

26+
1 test triggered 1 PHP warning:
27+
28+
1) %sUseBaselineTest.php:31
29+
Undefined variable $b
30+
31+
Triggered by:
32+
33+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
34+
%sUseBaselineTest.php:21
35+
36+
--
37+
38+
1 test triggered 1 warning:
39+
40+
1) %sUseBaselineTest.php:35
41+
warning
42+
43+
Triggered by:
44+
45+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
46+
%sUseBaselineTest.php:21
47+
48+
--
49+
50+
1 test triggered 1 PHP notice:
51+
52+
1) %sUseBaselineTest.php:29
53+
Only variables should be assigned by reference
54+
55+
Triggered by:
56+
57+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
58+
%sUseBaselineTest.php:21
59+
60+
--
61+
62+
1 test triggered 1 notice:
63+
64+
1) %sUseBaselineTest.php:34
65+
notice
66+
67+
Triggered by:
68+
69+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
70+
%sUseBaselineTest.php:21
71+
72+
--
73+
74+
1 test triggered 1 PHP deprecation:
75+
76+
1) %sUseBaselineTest.php:23
77+
strlen(): Passing null to parameter #1 ($string) of type string is deprecated
78+
79+
Triggered by:
80+
81+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
82+
%sUseBaselineTest.php:21
83+
84+
--
85+
2486
1 test triggered 1 deprecation:
2587

26-
1) %sTest.php:%d
88+
1) %sUseBaselineTest.php:33
2789
deprecation
2890

2991
Triggered by:
3092

31-
* PHPUnit\TestFixture\Baseline\Test::testOne
32-
%sTest.php:%d
93+
* PHPUnit\TestFixture\Baseline\UseBaselineTest::testOne
94+
%sUseBaselineTest.php:21
3395

3496
OK, but there were issues!
35-
Tests: 1, Assertions: 1, Deprecations: 1.
97+
Tests: 1, Assertions: 1, Warnings: 2, Deprecations: 2, Notices: 2.

tests/end-to-end/baseline/use-baseline.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,4 @@ Time: %s, Memory: %s
2121

2222
OK (1 test, 1 assertion)
2323

24-
1 issue was ignored by baseline.
24+
6 issues were ignored by baseline.

0 commit comments

Comments
 (0)