testcontainers / testcontainers-scala

Docker containers for testing in scala
MIT License
634 stars 127 forks source link

bugfix: It is now possible to pass params into RabbitMQ Def #244

Closed LazyBun closed 1 year ago

LazyBun commented 1 year ago

It was not possible to properly configure RabbitMQContainer when using it with munit due to the missing parameters in RabbitMQContainer.Def. I added them, along with a simple test to see if everything works properly.

LazyBun commented 1 year ago

Looking into the test issue :eyes:

LazyBun commented 1 year ago

Seems fixed. Let me know if there is anything else that needs to be improved before merging this one

dimafeng commented 1 year ago

That looks great! Thank you so much!

LazyBun commented 1 year ago

@dimafeng would it be possible to release a new version with my change? :)

dimafeng commented 1 year ago

Sorry for the delay! I'll release it today