Skip to content

fix(rabbitmq): add vhost as parameter to RabbitMqContainer - #656

Merged
alexanderankin merged 1 commit into
testcontainers:mainfrom
MorganTrench:rabbitmq-vhost
Jul 31, 2024
Merged

fix(rabbitmq): add vhost as parameter to RabbitMqContainer#656
alexanderankin merged 1 commit into
testcontainers:mainfrom
MorganTrench:rabbitmq-vhost

Conversation

@MorganTrench

@MorganTrenchMorganTrench commented Jul 26, 2024

Copy link
Copy Markdown
Contributor

Adds a vhost parameter to the RabbitMQContainer constructor that allows the RABBITMQ_DEFAULT_VHOSTenvironment variable to be modified.

Subsequently vhost is then also used inside the get_connection_params method for the pika connection parameters, which is used to test if the container is ready.

@codecov

codecovBot commented Jul 26, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@068c431). Learn more about missing BASE report.

Additional details and impacted files
@@ Coverage Diff @@## main #656 +/- ##
=======================================
Coverage ? 77.59% =======================================
Files ? 12 Lines ? 598 Branches ? 89 =======================================
Hits ? 464 Misses ? 108 Partials ? 26 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alexanderankinalexanderankin changed the title feat(rabbitmq): add vhost as parameter to RabbitMqContainerfix(rabbitmq): add vhost as parameter to RabbitMqContainerJul 26, 2024
@alexanderankinalexanderankin added the 👥 community feat feature but its a community module so we wont bump tc core for it label Jul 26, 2024
@alexanderankin
alexanderankin merged commit fa2081a into testcontainers:mainJul 31, 2024
alexanderankin pushed a commit that referenced this pull request Aug 14, 2024
🤖 I have created a release *beep* *boop*
---
##
[4.8.0](testcontainers-v4.7.2...testcontainers-v4.8.0)
(2024-08-14)
### Features
* Adding support for Cassandra and Scylla
([#167](#167))
([2d8bc11](2d8bc11))
* **compose:** ability to retain volumes when using context manager
([#659](#659))
([e1e3d13](e1e3d13))
* **compose:** add ability to get docker compose config
([#669](#669))
([8c28a86](8c28a86))
* **core:** add ability to do OR & AND for waitforlogs
([#661](#661))
([b1453e8](b1453e8))
* **new:** Added AWS Lambda module
([#655](#655))
([9161cb6](9161cb6))
* refactor network setup
([#678](#678))
([d5de0aa](d5de0aa))
### Bug Fixes
* Add Db2 support
([#673](#673))
([1e43923](1e43923))
* bring back cassandra driver bc otherwise how does it get installed for
cassandra module test run?
([#680](#680))
([71c3a1a](71c3a1a))
* **rabbitmq:** add `vhost` as parameter to RabbitMqContainer
([#656](#656))
([fa2081a](fa2081a))
* **selenium:** add Arg/Options to api of selenium container
([#654](#654))
([e02c1b3](e02c1b3)),
closes
[#652](#652)
---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👥 community featfeature but its a community module so we wont bump tc core for it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@MorganTrench@alexanderankin