Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clean up after messaging createSharedDurableConsumerTest2 test #1718

Merged

Conversation

aubi
Copy link
Contributor

@aubi aubi commented Dec 23, 2024

Fixes Issue
There was an issue with the createSharedDurableConsumerTest2 test. There was a suspicion, that it will be an issue with cleanup.

12-23-2024 12:27:43:  SVR: Create shared Durable Subscription and 1st JMSConsumer with message selector
12-23-2024 12:27:43:  SVR: Create 2nd JMSConsumer with message selector
12-23-2024 12:27:43:  SVR: Send 3 messages to Topic
12-23-2024 12:27:43:  SVR: Message 1 sent
12-23-2024 12:27:43:  SVR: Message 2 sent
12-23-2024 12:27:43:  SVR: Message 3 sent
12-23-2024 12:27:43:  SVR: Receive TextMessage from consumer1
12-23-2024 12:27:43:  SVR: Check the value in TextMessage
12-23-2024 12:27:43:  SVR-ERROR: TextMessage is incorrect expected Message 3, received For new topic
12-23-2024 12:27:43:  SVR-ERROR: Test case throws exception: createSharedDurableConsumerTest2 failed
12-23-2024 12:27:43:  SVR-ERROR: Exception at:
12-23-2024 12:27:43:  SVR-ERROR: java.lang.Exception: createSharedDurableConsumerTest2 failed
        at com.sun.ts.tests.jms.core20.jmscontexttopictests.Client.createSharedDurableConsumerTest2(Client.java:1373)
        at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
        at java.base/java.lang.reflect.Method.invoke(Method.java:580)

After doing the cleanup (influencing another run with different vehicle), this test passes

[INFO] Tests run: 1578, Failures: 0, Errors: 0, Skipped: 0

Describe the change
I used the same cleanup as it is used in many other tests.

CC @alwin-joseph @anajosep @arjantijms @cesarhernandezgt @dblevins @m0mus @edbratt @gurunrao @jansupol @jgallimore @kazumura @kwsutter @LanceAndersen @bhatpmk @RohitKumarJain @shighbar @gthoman @brideck @OndroMih @dmatej
@starksm64 @scottmarlow

@arjantijms arjantijms added this to the 11.0 milestone Dec 24, 2024
@arjantijms arjantijms merged commit 5d3d593 into jakartaee:main Dec 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants