GitHub user feuillemorte opened a pull request:

    https://github.com/apache/activemq-artemis/pull/2260

    Added several tests in artemis-cli//MessageSerializerTest

    Hello!
    
    I wrote some tests in MessageSerializerTest file, please review it.
    
    Please, advice me how to fix these tests:
    `testMulticastTopicToAnycastQueueBothAddress`
    `testAnycastQueueToMulticastTopicBothAddress`
    
    I want to import/export messages using topic subscription. And I don't know 
how to understand that my consumer started to listen messages. I tried to use 
this method:
    
https://github.com/apache/activemq-artemis/blob/c8792186e1ac5a92e8b72dcbd41e5d6731eff11f/artemis-cli/src/test/java/org/apache/activemq/cli/test/MessageSerializerTest.java#L324-L343
    to catch some logs like "wait until 10 messages are consumed" but I get 
back only empty list.... 
    also this consumer
    ```java
    MessageConsumer consumer = 
session.createConsumer(getTopicDestination(address));
    ```
    doesn't log anything
    
    now I put "sleep" before sending messages and it works fine for me, but 
it's not a good way to do it.
    
    Thank you!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/feuillemorte/activemq-artemis 
message_serializer_tests

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/2260.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2260
    
----
commit b3dde26e0001bfad4af993fa40be4a69438dd031
Author: feuillemorte <github@...>
Date:   2018-08-22T14:59:15Z

    Added several tests to artemis-cli//MessageSerializerTest

----


---

Reply via email to