Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release of 1.10.4 of junit-jupiter module contains wrong classes - REPUBLISHED AS 1.10.5 #1111

Closed
michael-simons opened this issue Jan 2, 2019 · 7 comments

Comments

@michael-simons
Copy link
Contributor

michael-simons commented Jan 2, 2019

I was investigating why #1019 still fails me even if #1020 was merged.

The test in the project works (I have checked it several times), so I tried to build super plain reproducers (for Maven and Gradle), which both fail with 1.10.4.

So I finally ended up debugging and make my way through the delomboked sources and in 1.10.4 TestcontainersExtension still only implements BeforeEachCallback and not BeforeAllCallback.

Please see the sources from central:
https://search.maven.org/remotecontent?filepath=org/testcontainers/junit-jupiter/1.10.4/junit-jupiter-1.10.4-sources.jar

I think one solution would be just drafting a new release in time. Being able to use the shared container in a BeforeAll seems to be crucial (at least from my perspective).

Thanks.

@bsideup
Copy link
Member

bsideup commented Jan 2, 2019

@michael-simons thanks a lot for the investigation!
We had a major issue with Bintray and publishing, and it seems that wrong binaries were published by Bintray 😱

We will work on a new release process ASAP, directly to the Maven Central (without Bintray in the middle), such issues are too critical 😑

/cc @jbaruch

@bsideup bsideup pinned this issue Jan 2, 2019
@rnorth
Copy link
Member

rnorth commented Jan 2, 2019

We're running a new release right now, 1.10.5, using the Bintray->Maven Central model.

Hopefully this will fix whatever's gone wrong here without taking the time of reverting to direct Maven Central publication.

@rnorth
Copy link
Member

rnorth commented Jan 2, 2019

https://dl.bintray.com/testcontainers/releases/org/testcontainers/junit-jupiter/1.10.5/junit-jupiter-1.10.5-sources.jar seems to contain the correct code:

...
class TestcontainersExtension implements BeforeEachCallback, BeforeAllCallback, TestInstancePostProcessor {
...

bintrayPublish has also succeeded on this release, instead of timing out the build: https://travis-ci.org/testcontainers/testcontainers-java/jobs/474365588

@rnorth
Copy link
Member

rnorth commented Jan 2, 2019

Synced 1.10.5 to Maven Central, and verified that the resultant JAR is correct. I'll put out some messages to prompt people to upgrade.

I have no idea what happened with the 1.10.4 publication, but we'll have to try and figure it out.

Thanks so much for noticing and raising the issue to our attention, @michael-simons.

@michael-simons
Copy link
Contributor Author

I can confirm things regarding the JUnit 5 extension work now as expected (i.e. michael-simons/bootiful-music@8759a48)

I didn't check other artefacts, so I leave this issue for you to close. From my side, it's fixed. Thanks everyone.

@rnorth rnorth changed the title Release of 1.10.4 of junit-jupiter module contains wrong classes Release of 1.10.4 of junit-jupiter module contains wrong classes - REPUBLISHED AS 1.10.5 Jan 2, 2019
@rnorth
Copy link
Member

rnorth commented Jan 2, 2019

For anyone seeing this or a similar problem, please use 1.10.5 which was released to resolve this.

I'll keep this ticket open while we try and figure out what went wrong with 1.10.4 publication. It likely relates to either/both of issues we had when publishing artifacts to Bintray, or when syncing those artifacts to Maven Central.

Sorry for any inconvenience.

@bsideup bsideup unpinned this issue Jan 12, 2019
@rnorth rnorth closed this as completed Jan 21, 2019
@rnorth
Copy link
Member

rnorth commented Jan 21, 2019

Closed, as there's probably been enough time for people to see this ticket and act on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants