jbertram opened a new pull request, #4255:
URL: https://github.com/apache/activemq-artemis/pull/4255
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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
