Skip to content

Commit eee5d04

Browse files
committed
[CodeQuality] Use atLeast() instead of exactly() in DecorateWillReturnMapWithExpectsMockRector
1 parent 404733e commit eee5d04

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎rules-tests/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector/Fixture/fixture.php.inc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ final class SomeTest extends \PHPUnit\Framework\TestCase
2626
public function test()
2727
{
2828
$someMock = $this->createMock(\stdClass::class);
29-
$someMock->expects($this->exactly(2))->method('some')->willReturnMap([1, 2]);
29+
$someMock->expects($this->atLeast(2))->method('some')->willReturnMap([1, 2]);
3030
}
3131
}
3232

‎rules/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ protected function setUp(): void
7070
{
7171
$this->someMock = $this->createMock(SomeClass::class);
7272
73-
$this->someMock->expects($this->exactly(2))
73+
$this->someMock->expects($this->atLeast(2))
7474
->method("someMethod")
7575
->willReturnMap([
7676
["arg1", "arg2", "result1"],
@@ -166,7 +166,7 @@ private function refactorExpression(Expression $expression): bool
166166
new Identifier('expects'),
167167
[new Arg(new MethodCall(
168168
new Variable('this'),
169-
new Identifier('exactly'),
169+
new Identifier('atLeast'),
170170
[new Arg(new Int_($mapCount))]
171171
))]
172172
);

‎rules/CodeQuality/Rector/MethodCall/AssertTrueFalseToSpecificMethodRector.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ final class AssertTrueFalseToSpecificMethodRector extends AbstractRector
3232
'array_search' => ['array_search', 'assertContains', 'assertNotContains'],
3333
'in_array' => ['in_array', 'assertContains', 'assertNotContains'],
3434
'empty' => ['empty', 'assertEmpty', 'assertNotEmpty'],
35-
'file_exists' => ['file_exists', 'assertFileExists', 'assertFileNotExists'],
35+
'file_exists' => ['file_exists', 'assertFileExists', 'assertFileNotExist'],
3636
'is_dir' => ['is_dir', 'assertDirectoryExists', 'assertDirectoryNotExists'],
3737
'is_infinite' => ['is_infinite', 'assertInfinite', 'assertFinite'],
3838
'is_null' => ['is_null', 'assertNull', 'assertNotNull'],

0 commit comments

Comments
 (0)