Skip to content

[Bug]: SolaceContainer misses permissions when using non-default VPN … - #10450

Merged
eddumelendez merged 5 commits into
testcontainers:mainfrom
infoShare:feature/solace_queues_support
Jul 9, 2025
Merged

[Bug]: SolaceContainer misses permissions when using non-default VPN …#10450
eddumelendez merged 5 commits into
testcontainers:mainfrom
infoShare:feature/solace_queues_support

Conversation

@infoShare

@infoShare infoShare commented Jul 3, 2025

Copy link
Copy Markdown
Contributor

Solace queues support and permissions for #10370

Tomasz Forys
[Bug]: SolaceContainer misses permissions when using non-default VPN #…
cc1e951 
…10370

- queues support and permissions on solace container
- JUnits coverage
- Adding missing javadocs
@infoShare
infoShare requested a review from a team July 3, 2025 17:05
@eddumelendez

eddumelendez commented Jul 8, 2025

Copy link
Copy Markdown
Member

Thanks for the contribution, @infoShare! Can you run ./gradlew :solace:spotlessApply, please?

@infoShare

infoShare commented Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for the contribution, @infoShare! Can you run ./gradlew :solace:spotlessApply, please?

Done, it was executes earlier but forgot to uncheck "Organize imports" on my IDE.

eddumelendez
eddumelendez requested changes Jul 9, 2025
View reviewed changes
Comment thread modules/solace/src/test/java/org/testcontainers/solace/SolaceContainerSMFTest.java Outdated
Comment thread modules/solace/src/test/java/org/testcontainers/solace/SolaceContainerSMFTest.java Outdated
Comment thread modules/solace/src/main/java/org/testcontainers/solace/SolaceContainer.java Outdated
Comment thread modules/solace/src/main/java/org/testcontainers/solace/SolaceContainer.java Outdated
@eddumelendez eddumelendez added this to the next milestone Jul 9, 2025
@eddumelendez
eddumelendez merged commit c46aec2 into