[famod]

This fixes e.g. MariaDBContainer.withInitScript(...) in Quarkus continuous testing, which is using a hierarchical classloading setup.

Btw, for comparison, MountableFile (e.g. MariaDBContainer.withConfigurationOverride(...)) does work already:

testcontainers-java/core/src/main/java/org/testcontainers/utility/MountableFile.java

Line 128 in 14eff07

classLoadersToSearch.add(Thread.currentThread().getContextClassLoader());

[famod]

FYI @stuartwdouglas @geoand 🔝

[geoand]

👌

[famod]

@rnorth @bsideup @kiview can you have a look please? Thanks!

[kiview]

@famod Thanks for the ping, we'll have a look at it 🙂

[kiview]

@famod Can you please explain how to reproduce the issue with Quarkus continuous testing, or share a reproducer?

[bsideup]

I don't think we need Quarkus to test this, just a simple test should be added to the PR that ensures that the code takes context's classloader into the consideration 👍

[famod]

I'll have a look. As I have a lot on my plate: Is there any ETA for the next TC release?

[kiview]

@famod We are currently in the process of preparing the next release. So having the test could help with getting this PR into the upcoming release.