-
Notifications
You must be signed in to change notification settings - Fork 29
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
PDE junit tests -> build with acceptance tests #140
Comments
I'm looking at groovy-eclipse : /org.eclipse.jdt.groovy.core.tests.builder/src/org/eclipse/jdt/core/groovy/tests/builder/BasicGroovyBuildTests.java I think that's a pretty good starting point to work from. |
So the plugin does not have test right now, right? In order to avoid regression, mainly for syntax issues, you should create a test project with a bunch of less files splitted into a structured system. It may hold regular tests for complicated treatments and issue specific ones. Example: I am not sure if the deployment is fast enough to allow running all tests for each small modification, but at least it could help to avoid regression for each release without messing with junit, mockito, etc. @vtst |
@jackphel @FagnerMartinsBrack Would you write test case for framework of your interest? Having coverage test to catch regressions like #188 make sense, when CI is present #206 |
I would have done that at the time I posted this (or at least helped to do so). Right now it is not feasible due to the fact I am drowned in 2 projects. I am not sure if this issue was closed because it will not be done or because it will be done, another issue requesting documentation for contributing was closed too, so I suppose the author is not willing to implement this. |
Yep, the author is [yet] not social enough for such great open-source undertaking. |
eclipse is suggesting to me to run junit plugin tests
A brief google search turns up a possible test configuration
I think tests would give the project a breath of fresh air towards a vigorous future.
The text was updated successfully, but these errors were encountered: