-
Notifications
You must be signed in to change notification settings - Fork 83
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
Option to manipulate the pom-file before calling the maven invoker #855
Conversation
…complete and failing) tests
… Also adds a new repairstep to manipulate the POM-file of the cloned project.
It seems that evosuite.org is currently timeouting, which seems to lead to the build erroring due to taking too long. |
…ogs and exceeding the allowed log length
Do we have a flaky test?
|
I believe the failing test might be because our test is relying on a repository that was recently updated, and where they specifically mention that they updated their checkstyle version 3 days ago. nosan/embedded-cassandra |
This PR should now be done. |
LGTM but CI is red. How to fix this? |
…t the correct plugin.
The NPERepair test was failing, and I assume it's because I updated the version of maven-repair it is running. Updated the test accordingly now and if it doesn't work I'll revert the update (the version is only relevant for NPEFixSafe) and see if that works. |
One of our tests for checkstyle is currently faulty due to the repository recently starting to use checkstyle. I've ignored the test for now (it was passing before) but if we want to keep a test we should find a new repo that has the same desirable traits as it used to have. @monperrus It should be done now. |
Thanks! |
To be able to load snapshots and similar experimental tools, we will need to be able to tell repairnator to use non-maven-central repos. Currently, I am not aware of any way to load a custom pom-model into the invoker, and thus adding these repos into the pom before calling the maven-invoker seems to be the most simple way of doing this.
I put this into a separate PR since I will be adding some tests and messing with the configuration files, and it is a separate feature to NPEFixSafe, although at the moment necessary.