Skip to content

Commit 54341b3

Browse files
committed
MCP-826: Use RabbitMQ for all consumers if AMQP is set in deployment config
- Fix static tests array_merge in loop
1 parent 39dd6a0 commit 54341b3

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

dev/tests/integration/testsuite/Magento/Framework/MessageQueue/Consumer/DeprecatedConfigTest.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
namespace Magento\Framework\MessageQueue\Consumer;
77

88
use Magento\Framework\MessageQueue\Consumer\Config\ConsumerConfigItem\Handler\Iterator as HandlerIterator;
9+
use Magento\TestModuleMessageQueueConfiguration\AsyncHandler;
10+
use Magento\TestModuleMessageQueueConfiguration\SyncHandler;
911

1012
/**
1113
* Test access to consumer configuration declared in deprecated queue.xml configs using Consumer\ConfigInterface.
@@ -40,9 +42,9 @@ public function testGetConsumerMultipleHandlersFromCommunicationConfig()
4042
$this->assertInstanceOf(HandlerIterator::class, $handlers);
4143
$this->assertCount(2, $handlers);
4244
$this->assertEquals('methodWithBoolParam', $handlers[0]->getMethod());
43-
$this->assertEquals(\Magento\TestModuleMessageQueueConfiguration\AsyncHandler::class, $handlers[0]->getType());
45+
$this->assertEquals(AsyncHandler::class, $handlers[0]->getType());
4446
$this->assertEquals('methodWithMixedParam', $handlers[1]->getMethod());
45-
$this->assertEquals(\Magento\TestModuleMessageQueueConfiguration\AsyncHandler::class, $handlers[1]->getType());
47+
$this->assertEquals(AsyncHandler::class, $handlers[1]->getType());
4648
}
4749

4850
public function testGetConsumerCustomHandler()
@@ -61,7 +63,7 @@ public function testGetConsumerCustomHandler()
6163
$this->assertInstanceOf(HandlerIterator::class, $handlers);
6264
$this->assertCount(1, $handlers);
6365
$this->assertEquals('methodWithMixedParam', $handlers[0]->getMethod());
64-
$this->assertEquals(\Magento\TestModuleMessageQueueConfiguration\AsyncHandler::class, $handlers[0]->getType());
66+
$this->assertEquals(AsyncHandler::class, $handlers[0]->getType());
6567
}
6668

6769
public function testGetConsumerCustomConnectionSync()
@@ -80,7 +82,7 @@ public function testGetConsumerCustomConnectionSync()
8082
$this->assertInstanceOf(HandlerIterator::class, $handlers);
8183
$this->assertCount(1, $handlers);
8284
$this->assertEquals('methodWithBoolParam', $handlers[0]->getMethod());
83-
$this->assertEquals(\Magento\TestModuleMessageQueueConfiguration\SyncHandler::class, $handlers[0]->getType());
85+
$this->assertEquals(SyncHandler::class, $handlers[0]->getType());
8486
}
8587

8688
public function testGetConsumerCustomConsumerAndMaxMessages()

lib/internal/Magento/Framework/Amqp/Bulk/Exchange.php

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@
55
*/
66
namespace Magento\Framework\Amqp\Bulk;
77

8-
use Magento\Framework\MessageQueue\Bulk\ExchangeInterface;
9-
use PhpAmqpLib\Message\AMQPMessage;
108
use Magento\Framework\Communication\ConfigInterface as CommunicationConfigInterface;
9+
use Magento\Framework\MessageQueue\Bulk\ExchangeInterface;
1110
use Magento\Framework\MessageQueue\Publisher\ConfigInterface as PublisherConfig;
11+
use PhpAmqpLib\Message\AMQPMessage;
1212

1313
/**
1414
* Used to send messages in bulk in AMQP queue.
@@ -76,10 +76,12 @@ public function enqueue($topic, array $envelopes)
7676
$exchange = $publisher->getConnection()->getExchange();
7777

7878
foreach ($envelopes as $envelope) {
79-
$msg = new AMQPMessage(
80-
$envelope->getBody(),
81-
array_merge(['delivery_mode' => 2], $envelope->getProperties())
82-
);
79+
$properties =[];
80+
$properties['delivery_mode'] = 2;
81+
foreach ($envelope->getProperties() as $key =>$value) {
82+
$properties[$key] = $value;
83+
}
84+
$msg = new AMQPMessage($envelope->getBody(), $properties);
8385
$channel->batch_basic_publish($msg, $exchange, $topic);
8486
}
8587
$channel->publish_batch();

lib/internal/Magento/Framework/Amqp/ConnectionTypeResolver.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ public function __construct(DeploymentConfig $deploymentConfig)
4040
}
4141

4242
/**
43-
*
4443
* @inheritdoc
4544
* @since 103.0.0
4645
*/

lib/internal/Magento/Framework/Data/Argument/Interpreter/Constant.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
class Constant implements InterpreterInterface
1414
{
1515
/**
16-
*
1716
* @inheritdoc
1817
* @return mixed
1918
* @throws \InvalidArgumentException

0 commit comments

Comments
 (0)