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

chore(rabbitmq): refactor assertEntity in tests #2856

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

mmorel-35
Copy link
Contributor

What does this PR do?

Refactors assertions on entities in rabbitmq_test.go

This applies the assertions in each function and mutualizes duplicated code

Related issues

Related to #2855 (comment)

@mmorel-35 mmorel-35 requested a review from a team as a code owner October 29, 2024 12:09
Copy link

netlify bot commented Oct 29, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit e07422a
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/6721dc7cc5a1700008eb2ff7
😎 Deploy Preview https://deploy-preview-2856--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mdelapenya mdelapenya self-assigned this Oct 29, 2024
@mdelapenya mdelapenya added chore Changes that do not impact the existing functionality hacktoberfest Pull Requests accepted for Hacktoberfest. labels Oct 29, 2024
mdelapenya
mdelapenya previously approved these changes Oct 29, 2024
Copy link
Member

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for your efficiency and quickness 🙇

testing.go Outdated Show resolved Hide resolved
@mmorel-35 mmorel-35 force-pushed the rabbitmq/require-entity branch 2 times, most recently from e33a102 to 2e57db1 Compare October 30, 2024 07:12
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
testing.go Show resolved Hide resolved
Copy link
Member

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@mdelapenya mdelapenya merged commit 2ee47e4 into testcontainers:main Oct 30, 2024
122 checks passed
@mmorel-35 mmorel-35 deleted the rabbitmq/require-entity branch October 30, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not impact the existing functionality hacktoberfest Pull Requests accepted for Hacktoberfest.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants