[
https://issues.apache.org/jira/browse/ARTEMIS-4037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17617177#comment-17617177
]
ASF subversion and git services commented on ARTEMIS-4037:
----------------------------------------------------------
Commit ea04426bcd2b3dab266abd17c2568524a5e3b6b5 in activemq-artemis's branch
refs/heads/main from Justin Bertram
[ https://gitbox.apache.org/repos/asf?p=activemq-artemis.git;h=ea04426bcd ]
ARTEMIS-4037 refactor MQTTRetainMessageManagerTest
Commit 5a42de5fa6ee1b96f6f3e404f5a3d11a702e1776 called my attention to
this test. It really needs to be refactored because:
- It belongs in the integration-tests module rather than the MQTT
protocol module.
- It is using a lot of non-standard components (e.g.
EmbeddedJMSResource, Awaitility, etc.).
- It is overly complicated (e.g. using its own MqttClientService).
This commit resolves all those problems. The new implementation is quite
a bit different but still equivalent. I reverted the original fix from
ARTEMIS-2476 and the test still fails.
> Use random alphanumeric strings for MQTTRetainMessageManagerTest
> ----------------------------------------------------------------
>
> Key: ARTEMIS-4037
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4037
> Project: ActiveMQ Artemis
> Issue Type: Task
> Reporter: Domenico Francesco Bruscino
> Assignee: Domenico Francesco Bruscino
> Priority: Major
> Fix For: 2.27.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
