Skip to content

Commit 91e7776

Browse files
committed
test: remove workaround to expose upstream bug
1 parent fb5fc17 commit 91e7776

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

tests/Unit/ReportPrinterSprintfBugTest.php

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,10 @@
77
use PHPUnit\Framework\TestCase;
88

99
/**
10-
* Validates that escaping a percent sign as `%%` works around the Codeception 5
11-
* upstream ArgumentCountError in `Codeception\Reporter\ReportPrinter` using the
12-
* `--report` cli flag.
10+
* Exposes the upstream Codeception 5 ArgumentCountError in `Codeception\Reporter\ReportPrinter`
11+
* using the `--report` cli flag when a test name contains a `%`.
12+
*
13+
* @see https://github.com/Codeception/Codeception/pull/6927
1314
*/
1415
class ReportPrinterSprintfBugTest extends TestCase
1516
{
@@ -27,8 +28,8 @@ public function testEscapedPercentInName(int $value): void
2728
public static function provideEscapedPercentInNameCases(): iterable
2829
{
2930
return [
30-
// The %% is rendered as % by the Codeception ReportPrinter sprintf
31-
'100%% safely escaped' => [1],
31+
// This test is expected to fail with `ArgumentCountError` due to a bug in Codeception ReportPrinter
32+
'100% coverage' => [1],
3233
];
3334
}
3435
}

0 commit comments

Comments
 (0)