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

Rework async send tests part1 #30511

Draft
wants to merge 11 commits into
base: integration
Choose a base branch
from

Conversation

sakerl
Copy link
Member

@sakerl sakerl commented Jan 9, 2025

For #30510

Reworking first set of tests in JMS1AsyncSend. Includes:

  • testJMS1AsyncSendException
  • testJMS1AsyncSendNoDestination
  • testJMS1AsyncSendNullListener
  • testJMS1AsyncSendUnidentifiedProducerUnidentifiedDestination

@sakerl sakerl self-assigned this Jan 9, 2025
@LibbyBot
Copy link

LibbyBot commented Jan 9, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 1 infrastructure code files were changed.

  • Please describe in a separate comment how you tested your changes.

sakerl added 4 commits January 9, 2025 14:26
testJMS1AsyncSendUnidentifiedProducerUnidentifiedDestination
testJMS1AsyncSendUnidentifiedProducerUnidentifiedDestination
@sakerl sakerl force-pushed the reworkAsyncSendTestsPart1 branch from 992e602 to b00a920 Compare January 9, 2025 14:27
Not strictly necessary for tests that aren't receiving messages, but the previously reused connection was started before the tests ran.
@sakerl
Copy link
Member Author

sakerl commented Jan 13, 2025

!build (view Open Liberty Personal Build - ⏳InProgress)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • Please describe in a separate comment how you tested your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants