Codeception / Codeception/module-amqp

AMQP pushToQueue declares queue with default parameters

Đang mở
#3 4 bình luận 4 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
PHP
Star
4
Fork
9
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

https://github.com/Codeception/Codeception/blob/2.2.10/src/Codeception/Module/AMQP.php#L153

```php
$this->connection->channel()->queue_declare($queue);
```

If the queue is already declared it can cause error like this:
```
[PhpAmqpLib\Exception\AMQPProtocolChannelException] PRECONDITION_FAILED - inequivalent arg 'durable' for queue 'TEST.start' in vhost 'test_vhost': received 'false' but current is 'true'

Scenario Steps:

8. $I->pushToQueue("TEST.start","test") at tests/functional/startCest.php:53

```

I think that `queue_declare` call can be removed from `pushToQueue`, because separate `declareQueue` method was introduced in 2.2.9.
(Probably it is best to make this change in master branch)

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Start with src/Codeception/Module/AMQP.php around line 153 and compare pushToQueue with the separate declareQueue method introduced in 2.2.9. Review tests/functional/startCest.php around line 53 and verify that pushing to an already-declared queue no longer triggers an inequivalent durable-parameter error.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
php
Lĩnh vực
backend, testing-qa
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.