Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commita8ea11b

Browse files
committed
bug#37770 [String] We cannot have a "provides" function in test cases (derrabus)
This PR was merged into the 5.1 branch.Discussion----------[String] We cannot have a "provides" function in test cases| Q | A| ------------- | ---| Branch? | 5.1| Bug fix? | yes| New feature? | no| Deprecations? | no| Tickets | Part of#37564| License | MIT| Doc PR | N/ABecause of a change in PHPUnit 9.3 (seesebastianbergmann/phpunit#3936), we cannot have define a method named `provides` in test cases. And since php is case-insensitive regarding method calls, the method `provideS` used by the String component's `FunctionTest` will cause a fatal error. I have renamed it to work around that issue.cc@fancywebCommits-------46e2a0c [String] We cannot have a "provides" function in test cases.
2 parents1382001 +46e2a0c commita8ea11b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

‎src/Symfony/Component/String/Tests/FunctionsTest.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,14 @@
2020
finalclass FunctionsTestextends TestCase
2121
{
2222
/**
23-
* @dataProviderprovideS
23+
* @dataProviderprovideStrings
2424
*/
2525
publicfunctiontestS(AbstractString$expected,string$input)
2626
{
2727
$this->assertEquals($expected,s($input));
2828
}
2929

30-
publicfunctionprovideS()
30+
publicfunctionprovideStrings():array
3131
{
3232
return [
3333
[newUnicodeString('foo'),'foo'],

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp