Skip to content

Commit b324a52

Browse files
sebastianbergmannsebastianbergmann
committed
Return specialized strings from asString() for test suites used to group data providers and repeated tests
1 parent 3efa099 commit b324a52

28 files changed

Lines changed: 221 additions & 68 deletions

‎src/Event/Events/TestSuite/Finished.php‎

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,31 @@ public function testSuite(): TestSuite
4747
*/
4848
public function asString(): string
4949
{
50+
$prefix = 'Test Suite Finished';
51+
$unit = 'test';
52+
53+
if ($this->testSuite->isForTestMethodWithDataProvider()) {
54+
$prefix = 'Test Suite for Test Method with Data Provider Finished';
55+
$unit = 'data set';
56+
} elseif ($this->testSuite->isForRepeatedTestMethod()) {
57+
$prefix = 'Test Suite for Repeated Test Method Finished';
58+
$unit = 'repetition';
59+
}
60+
61+
$count = $this->testSuite->count();
62+
$plural = '';
63+
64+
if ($count !== 1) {
65+
$plural = 's';
66+
}
67+
5068
return sprintf(
51-
'Test Suite Finished (%s, %d test%s)',
69+
'%s (%s, %d %s%s)',
70+
$prefix,
5271
$this->testSuite->name(),
53-
$this->testSuite->count(),
54-
$this->testSuite->count() !== 1 ? 's' : '',
72+
$count,
73+
$unit,
74+
$plural,
5575
);
5676
}
5777
}

‎src/Event/Events/TestSuite/Started.php‎

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,31 @@ public function testSuite(): TestSuite
4747
*/
4848
public function asString(): string
4949
{
50+
$prefix = 'Test Suite Started';
51+
$unit = 'test';
52+
53+
if ($this->testSuite->isForTestMethodWithDataProvider()) {
54+
$prefix = 'Test Suite for Test Method with Data Provider Started';
55+
$unit = 'data set';
56+
} elseif ($this->testSuite->isForRepeatedTestMethod()) {
57+
$prefix = 'Test Suite for Repeated Test Method Started';
58+
$unit = 'repetition';
59+
}
60+
61+
$count = $this->testSuite->count();
62+
$plural = '';
63+
64+
if ($count !== 1) {
65+
$plural = 's';
66+
}
67+
5068
return sprintf(
51-
'Test Suite Started (%s, %d test%s)',
69+
'%s (%s, %d %s%s)',
70+
$prefix,
5271
$this->testSuite->name(),
53-
$this->testSuite->count(),
54-
$this->testSuite->count() !== 1 ? 's' : '',
72+
$count,
73+
$unit,
74+
$plural,
5575
);
5676
}
5777
}

‎tests/end-to-end/cli/filter/filter-empty-method-portion-keeps-all-providers.phpt‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,18 +28,18 @@ Test Suite Sorted
2828
Test Suite Filtered (2 tests)
2929
Test Runner Execution Started (2 tests)
3030
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
31-
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 1 test)
31+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 1 data set)
3232
Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3333
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3434
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3535
Test Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
36-
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 1 test)
37-
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB, 1 test)
36+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 1 data set)
37+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB, 1 data set)
3838
Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB#1)
3939
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB#1)
4040
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB#1)
4141
Test Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB#1)
42-
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB, 1 test)
42+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testB, 1 data set)
4343
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
4444
Test Runner Execution Finished
4545
Test Runner Finished

‎tests/end-to-end/cli/filter/filter-explicit-regex-keeps-all-providers.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Test Suite Sorted
2828
Test Suite Filtered (2 tests)
2929
Test Runner Execution Started (2 tests)
3030
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
31-
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 tests)
31+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 data sets)
3232
Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
3333
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
3434
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
@@ -37,7 +37,7 @@ Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::
3737
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3838
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3939
Test Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
40-
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 tests)
40+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 data sets)
4141
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
4242
Test Runner Execution Finished
4343
Test Runner Finished

‎tests/end-to-end/cli/filter/filter-skips-data-provider-for-non-matching-method.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ Test Suite Sorted
2525
Test Suite Filtered (2 tests)
2626
Test Runner Execution Started (2 tests)
2727
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
28-
Test Suite Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 tests)
28+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 data sets)
2929
Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
3030
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
3131
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#0)
@@ -34,7 +34,7 @@ Test Preparation Started (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::
3434
Test Prepared (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3535
Test Passed (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
3636
Test Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA#1)
37-
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 tests)
37+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest::testA, 2 data sets)
3838
Test Suite Finished (PHPUnit\TestFixture\DataProviderSkipWhenFilteredTest, 2 tests)
3939
Test Runner Execution Finished
4040
Test Runner Finished

‎tests/end-to-end/cli/test-id-filter/test-id-filter-by-id-with-special-characters.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,12 @@ Test Suite Sorted
2525
Test Suite Filtered (1 test)
2626
Test Runner Execution Started (1 test)
2727
Test Suite Started (PHPUnit\TestFixture\TestIdFilter\BarTest, 1 test)
28-
Test Suite Started (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters, 1 test)
28+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters, 1 data set)
2929
Test Preparation Started (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters#total ($100))
3030
Test Prepared (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters#total ($100))
3131
Test Passed (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters#total ($100))
3232
Test Finished (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters#total ($100))
33-
Test Suite Finished (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters, 1 test)
33+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\TestIdFilter\BarTest::testWithSpecialCharacters, 1 data set)
3434
Test Suite Finished (PHPUnit\TestFixture\TestIdFilter\BarTest, 1 test)
3535
Test Runner Execution Finished
3636
Test Runner Finished

‎tests/end-to-end/event/data-provider-external.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ Test Runner Started
2525
Test Suite Sorted
2626
Test Runner Execution Started (2 tests)
2727
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderExternalTest, 2 tests)
28-
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess, 2 tests)
28+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess, 2 data sets)
2929
Test Preparation Started (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#0)
3030
Test Prepared (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#0)
3131
Test Passed (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#0)
@@ -34,7 +34,7 @@ Test Preparation Started (PHPUnit\TestFixture\Event\DataProviderExternalTest::te
3434
Test Prepared (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#1)
3535
Test Passed (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#1)
3636
Test Finished (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess#1)
37-
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess, 2 tests)
37+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\Event\DataProviderExternalTest::testSuccess, 2 data sets)
3838
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderExternalTest, 2 tests)
3939
Test Runner Execution Finished
4040
Test Runner Finished

‎tests/end-to-end/event/data-provider-in-parent-class.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,12 @@ Test Runner Started
2222
Test Suite Sorted
2323
Test Runner Execution Started (1 test)
2424
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderInParentTest, 1 test)
25-
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething, 1 test)
25+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething, 1 data set)
2626
Test Preparation Started (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething#0)
2727
Test Prepared (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething#0)
2828
Test Passed (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething#0)
2929
Test Finished (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething#0)
30-
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething, 1 test)
30+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\Event\DataProviderInParentTest::testSomething, 1 data set)
3131
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderInParentTest, 1 test)
3232
Test Runner Execution Finished
3333
Test Runner Finished

‎tests/end-to-end/event/data-provider-invalid-argument-name.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Test Runner Started
2222
Test Suite Sorted
2323
Test Runner Execution Started (2 tests)
2424
Test Suite Started (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest, 2 tests)
25-
Test Suite Started (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess, 2 tests)
25+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess, 2 data sets)
2626
Test Preparation Started (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess#0)
2727
Test Prepared (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess#0)
2828
Test Passed (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess#0)
@@ -32,7 +32,7 @@ Test Prepared (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::t
3232
Test Errored (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess#1)
3333
Unknown named parameter $value3
3434
Test Finished (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess#1)
35-
Test Suite Finished (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess, 2 tests)
35+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest::testSuccess, 2 data sets)
3636
Test Suite Finished (PHPUnit\TestFixture\Event\InvalidParameterNameDataProviderTest, 2 tests)
3737
Test Runner Execution Finished
3838
Test Runner Finished

‎tests/end-to-end/event/data-provider.phpt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Test Runner Started
2222
Test Suite Sorted
2323
Test Runner Execution Started (2 tests)
2424
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderTest, 2 tests)
25-
Test Suite Started (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess, 2 tests)
25+
Test Suite for Test Method with Data Provider Started (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess, 2 data sets)
2626
Test Preparation Started (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#0)
2727
Test Prepared (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#0)
2828
Test Passed (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#0)
@@ -31,7 +31,7 @@ Test Preparation Started (PHPUnit\TestFixture\Event\DataProviderTest::testSucces
3131
Test Prepared (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#1)
3232
Test Passed (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#1)
3333
Test Finished (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess#1)
34-
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess, 2 tests)
34+
Test Suite for Test Method with Data Provider Finished (PHPUnit\TestFixture\Event\DataProviderTest::testSuccess, 2 data sets)
3535
Test Suite Finished (PHPUnit\TestFixture\Event\DataProviderTest, 2 tests)
3636
Test Runner Execution Finished
3737
Test Runner Finished

0 commit comments

Comments
 (0)