Skip to content

Commit 4e6c2ad

Browse files
Refactor data provider
1 parent 42c19c1 commit 4e6c2ad

File tree

1 file changed

+53
-48
lines changed

1 file changed

+53
-48
lines changed

tests/unit/Util/PHP/DefaultJobRunnerTest.php

Lines changed: 53 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
*/
1010
namespace PHPUnit\Util\PHP;
1111

12+
use Generator;
1213
use PHPUnit\Framework\Attributes\CoversClass;
1314
use PHPUnit\Framework\Attributes\DataProvider;
1415
use PHPUnit\Framework\Attributes\Small;
@@ -21,84 +22,88 @@
2122
#[Small]
2223
final class DefaultJobRunnerTest extends TestCase
2324
{
24-
public static function provider(): array
25+
public static function provider(): Generator
2526
{
26-
return [
27-
'output to stdout' => [
28-
new Result('test', ''),
29-
new Job(
30-
<<<'EOT'
27+
yield 'output to stdout' => [
28+
new Result('test', ''),
29+
new Job(
30+
<<<'EOT'
3131
<?php declare(strict_types=1);
3232
fwrite(STDOUT, 'test');
3333

3434
EOT
35-
),
36-
],
37-
'output to stderr' => [
38-
new Result('', 'test'),
39-
new Job(
40-
<<<'EOT'
35+
),
36+
];
37+
38+
yield 'output to stderr' => [
39+
new Result('', 'test'),
40+
new Job(
41+
<<<'EOT'
4142
<?php declare(strict_types=1);
4243
fwrite(STDERR, 'test');
4344

4445
EOT
45-
),
46-
],
47-
'output to stdout and stderr' => [
48-
new Result('test-stdout', 'test-stderr'),
49-
new Job(
50-
<<<'EOT'
46+
),
47+
];
48+
49+
yield 'output to stdout and stderr' => [
50+
new Result('test-stdout', 'test-stderr'),
51+
new Job(
52+
<<<'EOT'
5153
<?php declare(strict_types=1);
5254
fwrite(STDOUT, 'test-stdout');
5355
fwrite(STDERR, 'test-stderr');
5456

5557
EOT
56-
),
57-
],
58-
'stderr redirected to stdout' => [
59-
new Result('test', ''),
60-
new Job(
61-
<<<'EOT'
58+
),
59+
];
60+
61+
yield 'stderr redirected to stdout' => [
62+
new Result('test', ''),
63+
new Job(
64+
<<<'EOT'
6265
<?php declare(strict_types=1);
6366
fwrite(STDERR, 'test');
6467

6568
EOT,
66-
redirectErrors: true,
67-
),
68-
],
69-
'configured environment variables' => [
70-
new Result('test', ''),
71-
new Job(
72-
<<<'EOT'
69+
redirectErrors: true,
70+
),
71+
];
72+
73+
yield 'configured environment variables' => [
74+
new Result('test', ''),
75+
new Job(
76+
<<<'EOT'
7377
<?php declare(strict_types=1);
7478
print getenv('test');
7579

7680
EOT,
77-
environmentVariables: ['test' => 'test'],
78-
),
79-
],
80-
'arguments' => [
81-
new Result('test', ''),
82-
new Job(
83-
<<<'EOT'
81+
environmentVariables: ['test' => 'test'],
82+
),
83+
];
84+
85+
yield 'arguments' => [
86+
new Result('test', ''),
87+
new Job(
88+
<<<'EOT'
8489
<?php declare(strict_types=1);
8590
print $argv[1];
8691

8792
EOT,
88-
arguments: ['test'],
89-
),
90-
],
91-
'input from stdin' => [
92-
new Result('test', ''),
93-
new Job(
94-
<<<'EOT'
93+
arguments: ['test'],
94+
),
95+
];
96+
97+
yield 'input from stdin' => [
98+
new Result('test', ''),
99+
new Job(
100+
<<<'EOT'
95101
<?php declare(strict_types=1);
96102
print file_get_contents('php://stdin');
97103

98104
EOT,
99-
input: 'test',
100-
),
101-
],
105+
input: 'test',
106+
),
102107
];
103108
}
104109

0 commit comments

Comments
 (0)