Skip to content

Commit d361e20

Browse files
committed
minor #15782 [Mercure] Remove wrong parameter in MockHub (elliotbruneel)
This PR was submitted for the 5.3 branch but it was merged into the 4.4 branch instead. Discussion ---------- [Mercure] Remove wrong parameter in MockHub There is no parameter in `MockHub` to specify the hub name (`default`) `MockHub` constructor : https://github.com/symfony/mercure/blob/main/src/MockHub.php#L30-L35 ```php /** * `@param` (callable(Update): string) $publisher */ public function __construct( string $url, TokenProviderInterface $jwtProvider, callable $publisher, TokenFactoryInterface $jwtFactory = null, string $publicUrl = null ) ``` <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/releases for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `5.x` for features of unreleased versions). --> Commits ------- 28cbc5d Remove wrong parameter in MockHub
2 parents d3974e2 + 28cbc5d commit d361e20

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mercure.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -576,7 +576,7 @@ You can instead make use of the `MockHub`::
576576
{
577577
public function testPublishing()
578578
{
579-
$hub = new MockHub('default', 'https://internal/.well-known/mercure', new StaticTokenProvider('foo'), function(Update $update): string {
579+
$hub = new MockHub('https://internal/.well-known/mercure', new StaticTokenProvider('foo'), function(Update $update): string {
580580
// $this->assertTrue($update->isPrivate());
581581

582582
return 'id';

0 commit comments

Comments
 (0)