#1373 [java] JUnitAssertionsShouldIncludeMessage: is no longer compatible with TestNG

[Thrawn]

The JUnit ruleset was previously compatible with TestNG, but became incompatible when version 5.3.0 changed the JUnitAssertionsShouldIncludeMessage rule to require a String as the first argument to assertEquals.

Is it feasible to adjust this to allow either the first or last argument to be a String?

[Thrawn]

This results in false positives on TestNG assertions. If TestNG is not supported, then it should be ignored entirely. However, it would be valuable to support it properly.

[Andreas Dangel]

The rule will detect now whether testng is in use (via imports) and won't trigger anymore.
I've created a feature request to add TestNG rules -> [#1447]

Commit: https://github.com/pmd/pmd/commit/2452db15f431c10ab8729a7f18df768ad09ce5e1