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

Avoid calling MiqQueue.messaging_client and stub it as missing #23331

Merged

Conversation

jrafanie
Copy link
Member

@jrafanie jrafanie commented Feb 5, 2025

This is a rails 7.1 related change. The code path changes in rails 7.1 so we need to make sure we're not actually trying to call the real queue messaging client in this test.

extracted from #23225

@jrafanie jrafanie requested a review from Fryguy as a code owner February 5, 2025 21:47
@jrafanie jrafanie mentioned this pull request Feb 5, 2025
54 tasks
@Fryguy Fryguy merged commit acf143a into ManageIQ:master Feb 6, 2025
8 checks passed
@Fryguy Fryguy self-assigned this Feb 6, 2025
@jrafanie jrafanie deleted the rails71-queue-messaging-client-stub branch February 6, 2025 19:54
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