[java] Remove FPs in UnnecessaryBoxing

[oowekyala]

Describe the PR

This fixes many FPs of the rule, and improves some poorly tested parts of PolyResolution (related to conversion contexts). This last change also helps UnnecessaryCast.

Related issues

• Fix [java] UnnecessaryBoxing, but explicit conversion is necessary #4903
• Fix [java] UnnecessaryBoxing false positive in PMD 7.0.0 in lambda #4924

Ready?

• Added unit tests for fixed bug/feature
• Passing all unit tests
• Complete build ./mvnw clean verify passes (checked automatically by github actions)
• Added (in-code) documentation (if needed)

[pmd-test]

	1 Message
📖	Compared to master: This changeset changes 6 violations, introduces 9 new violations, 0 new errors and 0 new configuration errors, removes 136 violations, 0 errors and 0 configuration errors. Download full report as build artifact
✅	Compared to master: This changeset changes 6 violations, introduces 9 new violations, 0 new errors and 0 new configuration errors, removes 140 violations, 0 errors and 0 configuration errors. Download full report as build artifact
✅	Compared to master: This changeset changes 6 violations, introduces 9 new violations, 0 new errors and 0 new configuration errors, removes 142 violations, 0 errors and 0 configuration errors. Download full report as build artifact
✅	Compared to master: This changeset changes 0 violations, introduces 6 new violations, 0 new errors and 0 new configuration errors, removes 252 violations, 0 errors and 0 configuration errors. Download full report as build artifact
✅	Compared to master: This changeset changes 85 violations, introduces 6 new violations, 0 new errors and 0 new configuration errors, removes 70 violations, 0 errors and 0 configuration errors. Download full report as build artifact

Generated by 🚫 Danger

[oowekyala]

Stuff to do:

• new FNs about unboxing

return value == ((Long)obj).longValue();
assertThat(testBean.getMyFloat().floatValue() == 1.1f).isTrue();

• Add test for this TN

public boolean valid(Double ratio) {
return Double.valueOf(42).equals(ratio);
}

• current FP I just noticed: https://github.com/spring-projects/spring-framework/blob/23bb745ea49ae3d5ae58638e465c54a3595c5f78/spring-beans/src/test/java/org/springframework/beans/BeanWrapperGenericsTests.java#L286. Here removing the explicit conversion would box 5 to (Integer) 5 instead of (Long) 5 and the test would fail.
• New FN https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/io/ObjectStreamClass.java#L643
• New FN https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/lang/Boolean.java#L241
• New FN https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/lang/Long.java#L1154
• New FN https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/time/format/DateTimeFormatterBuilder.java#L4293
• New FN https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/jdk/internal/org/objectweb/asm/util/ASMifier.java#L1409
• Add test for new TP in UnnecessaryCast: https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/util/stream/LongPipeline.java#L414

[adangel]

Thanks!