Skip to content

Commit 003e616

Browse files
minor symfony#26037 [Lock] Fix strategy tests (Valouleloup)
This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes symfony#26037). Discussion ---------- [Lock] Fix strategy tests | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Not sure if it is a mistake, but in Phpunit tests need to begin with ` test ` to be played, right ? Commits ------- afbb8be Fix lock strategy tests
2 parents 18510f8 + afbb8be commit 003e616

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/Symfony/Component/Lock/Tests/Strategy/ConsensusStrategyTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public function testMet($success, $failure, $total, $isMet)
8282
/**
8383
* @dataProvider provideIndeterminate
8484
*/
85-
public function canBeMet($success, $failure, $total, $isMet)
85+
public function testCanBeMet($success, $failure, $total, $isMet)
8686
{
8787
$this->assertSame($isMet, $this->strategy->canBeMet($failure, $total));
8888
}

src/Symfony/Component/Lock/Tests/Strategy/UnanimousStrategyTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public function testMet($success, $failure, $total, $isMet)
8282
/**
8383
* @dataProvider provideIndeterminate
8484
*/
85-
public function canBeMet($success, $failure, $total, $isMet)
85+
public function testCanBeMet($success, $failure, $total, $isMet)
8686
{
8787
$this->assertSame($isMet, $this->strategy->canBeMet($failure, $total));
8888
}

0 commit comments

Comments
 (0)