citerus / dddsample-core

This is the new home of the original DDD Sample app (previously hosted at sf.net)..
MIT License
4.99k stars 1.47k forks source link

Bugfix: Add missing registration of MessageConsumers to JMS queues. #62

Closed orende closed 1 year ago

orende commented 1 year ago

Why

The recent refactoring of the messaging package (converting it from XML-based bean definitions to POJOs) accidentally left out the part where the message consumers are registered on JMS queues. Since leaving this out does not cause any errors during startup and since there are no tests that test the JMS integration, this bug was undetected until now.

What

Adds registration of message consumers to the JMS queues converted from the old bean XML-configuration. Also updates the example SOAP message with values that I used to test this locally instead of it's previous invalid values.

Let me know if you want me to add some automated integration tests to verify this as well.