Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit da4db91

Browse files
committed
minor #30864 [Messenger] fixing unused variable names (weaverryan)
This PR was merged into the 4.3-dev branch. Discussion ---------- [Messenger] fixing unused variable names | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | https://github.com/symfony/symfony/pull/30757/files#r272061245 | License | MIT | Doc PR | n/a Typo on the variable name - variables aren't needed anyways. Cheers! Commits ------- f7ec2d3 fixing unused variable names
2 parents 4dfb741 + f7ec2d3 commit da4db91

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/Symfony/Component/Messenger/Tests/Transport/AmqpExt/AmqpExtIntegrationTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,9 +177,9 @@ public function testItCountsMessagesInQueue()
177177

178178
$sender = new AmqpSender($connection, $serializer);
179179

180-
$sender->send($first = new Envelope(new DummyMessage('First')));
181-
$sender->send($second = new Envelope(new DummyMessage('Second')));
182-
$sender->send($second = new Envelope(new DummyMessage('Third')));
180+
$sender->send(new Envelope(new DummyMessage('First')));
181+
$sender->send(new Envelope(new DummyMessage('Second')));
182+
$sender->send(new Envelope(new DummyMessage('Third')));
183183

184184
sleep(1); // give amqp a moment to have the messages ready
185185
$this->assertSame(3, $connection->countMessagesInQueue());

0 commit comments

Comments
 (0)