-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
WIP Reintroduces Gradle ITs in main tree #4579
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gastaldi ! I added a small question inline.
The native tests are not really being invoked. Still investigating |
Looking into this I realize there is a problem with the generated gradle projects. I'll open a PR soon |
Also there seems to be a general issue with the integration tests and gradle that I am looking into. |
I have updated #4600 to get tests working in gradle projects |
So I suppose this one should be rebased? |
With all the Gradle changes that have gone in, I would say definitely |
I'm getting the following error after rebasing, any clues?
|
I've never seen that one before... |
@gastaldi CI is failing, can you have a look? |
Depends on #4870 |
Finally green! However, the Build error in Windows seems unrelated. Flaky tests? |
I restarted the Windows test, let's see... |
Looks like it's green now. Thanks for restarting the build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@gsmet OK to merge? |
This would be great to have. I could really use some gradle integration tests for a ClassLoader change I want to explore. |
Requires #4937 |
I don't know exactly what happened between all green and all red but we now have this sort of failures that look related:
|
That's because the quakus-gradle-plugin cannot be resolved from the quarkus-maven-plugin because it uses a different local repository and that local repository ( I am still investigating on how to solve that, but it probably won't be in time for 0.27.0 |
@gastaldi want to give it one more attempt with a rebase onto the latest master? |
Integration tests for the Gradle plugin should be executed using Gradle as the build tool, therefore it's recommended to be in the same project as the Gradle plugin lives. Closing this PR now. |
Fixes #3174 and closes #2783 and closes #1623