Change kafka testcontainer artifact name

[gastaldi]

Summary

Quarkus BOM updated testcontainers to new mayor version 2.0.2 - quarkusio/quarkus#51314
In this version artfiact org.testcontainers:kafka is renamed to org.testcontainers:testcontainers-kafka
Please select the relevant options.

• Bug fix (non-breaking change which fixes an issue)
• Dependency update
• Refactoring
• Backport
• New scenario (non-breaking change which adds functionality)
• This change requires a documentation update
• This change requires execution against OCP (use run tests phrase in comment)
• This change requires execution with OCP on Aarch64 (use run arm tests phrase in comment)

Checklist:

• Methods and classes used in PR scenarios are meaningful
• Commits are well encapsulated and follow the best practices

[mocenas]

LGTM

[gastaldi]

I guess it requires quarkus-qe/quarkus-test-framework#1771 to be merged/released first?

[mocenas]

I guess it requires quarkus-qe/quarkus-test-framework#1771 to be merged/released first?

yep it does

[fedinskiy]

Now the error is different.
I seems, that changing import junit.framework.AssertionFailedError to org.opentest4j.AssertionFailedError[1] in service-discovery/stork/src/test/java/io/quarkus/ts/stork/AbstractCommonTestCases.java should help
UPD: same for service-discovery/stork-custom/src/test/java/io/quarkus/ts/stork/custom/StorkCustomServiceDiscoveryAndLoadBalancerIT.java
[1] see link for AssertionFailedError https://docs.junit.org/5.0.1/api/org/junit/jupiter/api/Assertions.html

[fedinskiy]

I took the liberty to push the commit into the branch myself. @mocenas @gastaldi please confirm, that you're okay with that

[gastaldi]

@fedinskiy I haven't seen your latest comment in time, so I pushed a fix now

[michalvavrik]

I fixed compilation failures because this was blocking me in testing my utils PR.

[michalvavrik]

LGTM; if validation build passes, I'll merge it and whatever follow-up may be necessary can be detected from the next daily build.