Claudenw commented on PR #399: URL: https://github.com/apache/creadur-rat/pull/399#issuecomment-2487822037
It is related to RAT-259 and RAT-14. The issue was that the `ReporterTest` integration test switched out the log file but did not switch it back. And being a static the change impacted other tests. The other problem is that the integration test is not really an integration test. I should probably move it to `apache-rat` from `apache-rat-core` and make it run after the packaging. This would make the most sense. There is some work to be done to clean up this area of testing. The tests work but as they are the "integration test" label is a bit of a misnomer. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@creadur.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
