-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Introduce Ant build support for the JUnit Platform #512
Comments
|
Nope. Not that I'm aware of. With that in mind, I'll reword the subject of this issue and make it "up for grabs". 😉 |
|
As users of JUnit5 we can say. Later we're going to migrate to gradle (or maven). But at this moment we find it difficult to do both things at the same time - migrate to Junit5 and to gradle (or maven). So it would be very nice if you implement this feature. |
|
It's been a long time since I've used Ant. I believe it shouldn't be too hard to write an adapter for |
|
Our team has been using ANT to run the junit tests. Now that I am upgrading to junit 5 to try out dynamic tests with the @testfactory annotation, and the ANT task is failing with the following exception Is there any other tool apart from ANT that I can try out?? Any help is appreciated. Thanks and Regards |
|
@vvelrath You could use the Console Launcher standalone JAR from within ANT as a work-around. For example, configure an ANT java task to try out |
|
Console Launcher is not an option for us since it would require non-trivial changes on a huge amount of build scripts or even project structures. Let alone interoperability with other task like e.g. code coverage determination. |
|
Support for JUnit5 in Ant is currently being implemented. More details here https://www.mail-archive.com/dev@ant.apache.org/msg46487.html and there's a issue in Ant's issue tracker that you can keep a watch on to be notified when the feature is available https://bz.apache.org/bugzilla/show_bug.cgi?id=61796 |
|
This is great news, @jaikiran Did you see the PR by @dhilario https://github.com/junit-team/junit5/pull/624/files ? If I remember correctly, it was almost ready to run. |
|
Hi @sormuras, I hadn't seen that PR before. I had a look at it now. The approach I followed in my effort to build this integration is a bit different than what's there in the PR you pointed to. In the approach that I followed, the new The approach in the linked PR is different to this. It's neither wrong nor bad and has its own pros. It seems to act as a wrapper over JUnit 5 constructs and more closer to it and uses Ant mostly like a pass-through mechanism for constructing the test requests. Again, this isn't a bad thing - I'm just noting the difference in the approach. To give a better picture of how the proposed [1] https://home.apache.org/~jaikiran/temp_workspace/manual/Tasks/junitlauncher.html |
|
@jaikiran, that's looking quite good! So, your work will be the official Ant support for the JUnit Platform -- is that correct? If so, I'll close this issue. |
Yes, that's correct. We currently have a dev discussion going on and a PR [1] open which will be reviewed and merged, depending on some of our upcoming Ant release schedule. [1] apache/ant#60 |
|
That's great news! Thanks for the feedback, @jaikiran. I am therefore closing this issue since an upcoming release of Ant will provide JUnit Platform support out of the box. |
|
@jaikiran, I couldn't find your Twitter handle... that I wanted to include in this tweet: https://twitter.com/sam_brannen/status/964476615160942592 |
|
@sbrannen, I don't have a twitter account :) Thanks for tweeting it to a wider audience! |
|
@sbrannen, more of a FYI - we just released Ant 1.10.3 today which now has support for JUnit 5 https://jaitechwriteups.blogspot.in/2018/03/ant-1103-released-with-junit-5-support.html |
|
Awesome! 👏 Thanks for letting us know and sharing the link. I'll tweet about it ASAP. 👍 |
|
@marcphilipp, sure I can take a look at that one this coming week and send a PR |
This question came up after a JUnit5 talk of mine:
Is anybody working on a JUnit5-Ant integration?
The text was updated successfully, but these errors were encountered: