WW-5759 chore(core): address Sonar findings on the accessor resolution change

Make the non-null target contract of ultimateClass explicit, drop throws clauses the new
tests do not need and record the values the probe setters receive.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: lukaszlenart

core/src/main/java/org/apache/struts2/interceptor/parameter/StrutsParameterAuthorizer.java

@@ -39,6 +39,7 @@
 import java.lang.reflect.Modifier;
 import java.util.Arrays;
 import java.util.List;
+import java.util.Objects;
 import java.util.Optional;
 
 import static java.lang.String.format;
@@ -371,6 +372,7 @@ protected StrutsParameter getParameterAnnotation(AnnotatedElement element) {
     }
 
     protected Class<?> ultimateClass(Object target) {
+        Objects.requireNonNull(target, "target");
         if (proxyService.isProxy(target)) {
             return proxyService.ultimateTargetClass(target);
         }

core/src/test/java/org/apache/struts2/interceptor/parameter/ParametersInterceptorTest.java

@@ -324,7 +324,7 @@ public void testModelDrivenAnnotatedNestedPropertyOnActionIsAllowlisted() throws
      * OGNL resolves {@code uRange} to the JavaBeans setter {@code setuRange} before {@code setURange}, so that is the
      * method which has to carry the annotation. The model property alongside proves the parameters were applied.
      */
-    public void testModelDrivenJavaBeansStyleSetterOnActionRequiresAnnotation() throws Exception {
+    public void testModelDrivenJavaBeansStyleSetterOnActionRequiresAnnotation() {
         loadButSet(Map.of(StrutsConstants.STRUTS_PARAMETERS_REQUIRE_ANNOTATIONS, "true"));
         ParametersInterceptor pi = createParametersInterceptor();
 
@@ -349,7 +349,7 @@ public void testModelDrivenJavaBeansStyleSetterOnActionRequiresAnnotation() thro
      * A nested parameter goes through the getter OGNL resolves, {@code getURL} for {@code uRL}, which
      * introspection names property {@code URL}.
      */
-    public void testModelDrivenUppercaseGetterOnActionRequiresAnnotation() throws Exception {
+    public void testModelDrivenUppercaseGetterOnActionRequiresAnnotation() {
         loadButSet(Map.of(StrutsConstants.STRUTS_PARAMETERS_REQUIRE_ANNOTATIONS, "true"));
         ParametersInterceptor pi = createParametersInterceptor();
 
@@ -372,7 +372,7 @@ public void testModelDrivenUppercaseGetterOnActionRequiresAnnotation() throws Ex
     /**
      * The annotation counts only on the setter OGNL actually invokes for the parameter name as written.
      */
-    public void testAnnotationOnlyCountsOnTheSetterOgnlInvokes() throws Exception {
+    public void testAnnotationOnlyCountsOnTheSetterOgnlInvokes() {
         loadButSet(Map.of(StrutsConstants.STRUTS_PARAMETERS_REQUIRE_ANNOTATIONS, "true"));
         ParametersInterceptor pi = createParametersInterceptor();
 
@@ -395,7 +395,7 @@ public void testAnnotationOnlyCountsOnTheSetterOgnlInvokes() throws Exception {
      * Names differing from an accessor's property only in case resolve to no accessor on either object, so they
      * take the "declared on neither" route - which is safe only as long as OGNL does not bind them either.
      */
-    public void testModelDrivenCaseVariantsBindNothingOnAction() throws Exception {
+    public void testModelDrivenCaseVariantsBindNothingOnAction() {
         loadButSet(Map.of(StrutsConstants.STRUTS_PARAMETERS_REQUIRE_ANNOTATIONS, "true"));
         ParametersInterceptor pi = createParametersInterceptor();
 
@@ -1238,16 +1238,16 @@ public static class JavaBeansNamedAction {
         public String getxCoord() { return xCoord; }
 
         // NO @StrutsParameter - OGNL prefers it to the annotated setURange for "uRange"
-        public void setuRange(String uRange) { invoked.add("setuRange"); }
+        public void setuRange(String uRange) { invoked.add("setuRange=" + uRange); }
 
         @StrutsParameter
-        public void setURange(String uRange) { invoked.add("setURange"); }
+        public void setURange(String uRange) { invoked.add("setURange=" + uRange); }
 
         @StrutsParameter
-        public void setvRange(String vRange) { invoked.add("setvRange"); }
+        public void setvRange(String vRange) { invoked.add("setvRange=" + vRange); }
 
         // NO @StrutsParameter - the only setter OGNL considers for "VRange"
-        public void setVRange(String vRange) { invoked.add("setVRange"); }
+        public void setVRange(String vRange) { invoked.add("setVRange=" + vRange); }
 
         public List<String> getInvoked() { return invoked; }
     }