Add integration tests for PMD, and fix issues with error reporting.

Review Request #11557 — Created March 26, 2021 and submitted

Information

ReviewBot
release-3.0.x

Reviewers

This updates the PMD test suite to add both real-world simulation and
integration tests. The tests have been expanded to test for files
without errors, invalid ruleset configurations, and errors processing
source files (syntax errors).

This uncovered a handful of issues, mostly in the processing failure or
invalid ruleset cases, which have all been fixed. This code is more
bullet-proof, and generates more helpful error messages in the review
request.

We're now using the JSON output, rather than the CSV output, which is
more useful and safer to parse. There shouldn't be any compatibility
issues with this.

We're also explicitly turning off report caching, since we don't want to
preserve that information anywhere.

All unit tests pass on Python 2.7 and 3.x.

Summary ID
Add integration tests for PMD, and fix issues with error reporting.
This updates the PMD test suite to add both real-world simulation and integration tests. The tests have been expanded to test for files without errors, invalid ruleset configurations, and errors processing source files (syntax errors). This uncovered a handful of issues, mostly in the processing failure or invalid ruleset cases, which have all been fixed. This code is more bullet-proof, and generates more helpful error messages in the review request. We're now using the JSON output, rather than the CSV output, which is more useful and safer to parse. There shouldn't be any compatibility issues with this. We're also explicitly turning off report caching, since we don't want to preserve that information anywhere.
95a65ab7da647bca7930d4702f8a659189e1d971
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (d2c32c3)