-
-
Notifications
You must be signed in to change notification settings - Fork 198
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
Add integration tests to AMQPMessageConsumptionTaskITest class #1130
Conversation
Welcome to the Microcks community! 💖 Thanks and congrats 🎉 for opening your first pull request here! Be sure to follow the pull request template or please update it accordingly. Hope you have a great time there! |
I will add more tests to another destination types ( |
Signed-off-by: Matheus Cruz <matheuscruz.dev@gmail.com>
Hi @lbroudoux, I think you can review it now! Was necessary to send some message to the exchange asynchronously because it:
In your experience, do you have another way to solve this? |
Signed-off-by: Matheus Cruz <matheuscruz.dev@gmail.com>
Hello! Thank you very much for the PR! Looks good and it sounds like you onboarded very quickly!
Unfortunately no... I don't see other ways to overcome that. I've done the same kind of things in the tests for the Microcks Testcontainers modules... |
You are now a Microcks community contributor! 💖 Thanks and congrats 🚀 on merging your first pull request! We are delighted and very proud of you! 👏 📢 If you're using Microcks in your organization, please add your company name to this list. 🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact. If you need to know why and how to add yourself to the list, please read the blog post "Join the Microcks Adopters list and Empower the vibrant open source Community 🙌" Kudos and please keep going, we need you 🙌 |
Thank you very much @mcruzdev! |
Description
microcks-async-minion
Related issue(s)
Fixes #1128