-
Notifications
You must be signed in to change notification settings - Fork 120
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
Add junit compilation test integration #148
base: main
Are you sure you want to change the base?
Conversation
This surely is interesting, but I don't see the value of using an annotation vs. defining this code in a java method. It seems like the main benefit is that you can specify the files directly via strings and omit the The downsides that initially come to my mind are:
|
@ronshapiro Yes, you are right. This project name is 'compile-testing', but it is only care about the compile result not the compile itself. For example
Use |
I have create [pull request](google/compile-testing#148) to google compile-test project. But before it merged, I reserve the copyright.
@ronshapiro And for your 4 points
I can add support if anyone want.
Although I can add support, I think is a bad usage. You should extract source code into new file instead of embed in your test code. But if you really need it, I can support it.
I can't understand your use case. You can get
Yes, it is. But it's not a problem as you can't use |
I have create [pull request](google/compile-testing#148) to google compile-test project. But before it merged, I reserve the copyright.
I have create [pull request](google/compile-testing#148) to google compile-test project. But before it merged, I reserve the copyright.
No activity since 2 years here... Is there anyone to accept this PR ? |
Add support to quick build compile-period or compilation test in JUnit.
@Compile
on your test method.@Compiled
on your test method.