Skip to content

Commit 5166629

Browse files
committed
minor #50091 Fix the semaphore tests (stof)
This PR was merged into the 6.3 branch. Discussion ---------- Fix the semaphore tests | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | n/a | License | MIT | Doc PR | n/a The save method of the store is not meant to return a boolean. And with the new `void` type in the phpdoc, phpunit complains about that. Commits ------- 93d0caa Fix the semaphore tests
2 parents 9289fd8 + 93d0caa commit 5166629

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/Symfony/Component/Semaphore/Tests/SemaphoreFactoryTest.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ public function testCreateSemaphore()
3535
$keys[] = $key;
3636

3737
return true;
38-
}))
39-
->willReturn(true);
38+
}));
4039

4140
$logger = $this->createMock(LoggerInterface::class);
4241
$factory = new SemaphoreFactory($store);
@@ -64,8 +63,7 @@ public function testCreateSemaphoreFromKey()
6463
$keys[] = $key;
6564

6665
return true;
67-
}))
68-
->willReturn(true);
66+
}));
6967

7068
$logger = $this->createMock(LoggerInterface::class);
7169
$factory = new SemaphoreFactory($store);

0 commit comments

Comments
 (0)