This is an automated email from the ASF dual-hosted git repository.
jamesbognar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/juneau.git
The following commit(s) were added to refs/heads/master by this push:
new 3f387249a1 SonarQube bug fixes
3f387249a1 is described below
commit 3f387249a1bfe600f8fb3298f6f62f3854c845c8
Author: James Bognar <[email protected]>
AuthorDate: Fri Feb 6 08:23:10 2026 -0500
SonarQube bug fixes
---
AI.md | 4 ++++
.../main/java/org/apache/juneau/commons/reflect/ReflectionMap.java | 1 +
.../main/java/org/apache/juneau/commons/utils/CollectionUtils.java | 5 ++++-
.../src/main/java/org/apache/juneau/commons/utils/Utils.java | 1 +
.../juneau-marshall/src/main/java/org/apache/juneau/BeanSession.java | 1 +
.../src/main/java/org/apache/juneau/collections/JsonList.java | 1 +
.../src/main/java/org/apache/juneau/collections/JsonMap.java | 1 +
.../src/main/java/org/apache/juneau/msgpack/MsgPackOutputStream.java | 1 +
.../src/main/java/org/apache/juneau/swap/BuilderSwap.java | 4 ++--
.../java/org/apache/juneau/examples/core/config/store/SqlStore.java | 1 +
.../src/main/java/org/apache/juneau/rest/arg/PathRemainderArg.java | 1 +
.../org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java | 1 +
.../test/java/org/apache/juneau/BeanProxyInvocationHandler_Test.java | 1 +
juneau-utest/src/test/java/org/apache/juneau/PojoExamplesTest.java | 1 +
juneau-utest/src/test/java/org/apache/juneau/TestUtils.java | 1 +
.../java/org/apache/juneau/a/rttests/JsonMaps_RoundTripTest.java | 1 +
.../apache/juneau/bean/html5/Html5Template_ComboRoundTripTest.java | 2 +-
.../juneau/commons/annotation/AppliedAnnotationObject_Test.java | 1 +
.../java/org/apache/juneau/commons/inject/BeanCreator2_Test.java | 1 +
.../test/java/org/apache/juneau/commons/reflect/ClassInfo_Test.java | 2 +-
.../java/org/apache/juneau/commons/reflect/ConstructorInfo_Test.java | 2 +-
.../java/org/apache/juneau/commons/reflect/ExecutableInfo_Test.java | 2 +-
.../test/java/org/apache/juneau/commons/reflect/MethodInfo_Test.java | 2 +-
.../java/org/apache/juneau/commons/reflect/ParameterInfo_Test.java | 2 +-
juneau-utest/src/test/java/org/apache/juneau/cp/BeanStore_Test.java | 2 +-
.../test/java/org/apache/juneau/http/remote/RrpcInterface_Test.java | 2 +-
.../src/test/java/org/apache/juneau/junit/bct/BctConfig_Test.java | 2 +-
.../java/org/apache/juneau/junit/bct/PropertyExtractors_Test.java | 1 +
.../src/test/java/org/apache/juneau/mstat/ThrownStore_Test.java | 1 +
.../org/apache/juneau/rest/test/client/ThirdPartyProxy_Test.java | 1 +
.../test/java/org/apache/juneau/serializer/UriResolution_Test.java | 1 +
.../src/test/java/org/apache/juneau/transform/AutoListSwapTest.java | 2 +-
.../src/test/java/org/apache/juneau/transform/AutoMapSwapTest.java | 2 +-
.../test/java/org/apache/juneau/transform/AutoNumberSwapTest.java | 2 +-
.../test/java/org/apache/juneau/transform/AutoObjectSwapTest.java | 2 +-
juneau-utest/src/test/java/org/apache/juneau/utils/MutatersTest.java | 1 +
.../src/test/java/org/apache/juneau/xml/InvalidXmlBeans_Test.java | 1 +
37 files changed, 44 insertions(+), 16 deletions(-)
diff --git a/AI.md b/AI.md
index bcf702c8a4..1cc30f8dc5 100644
--- a/AI.md
+++ b/AI.md
@@ -15,6 +15,10 @@ This document outlines the rules, guidelines, and best
practices that AI assista
- **"s"** means **"status"** - When the user sends just "s", give a status
update on what you're currently working on
- **"TODO-x"** means **"work on this TODO"** - When the user sends just
"TODO-3", "TODO-67", etc., start working on that specific TODO item from the
TODO.md file
+### Task Interpretation Commands
+- **"make a plan"** or **"come up with a plan"** - When the user asks to make
a plan for something, provide a summary of suggested changes only. **Do NOT
make actual code changes**. The plan should outline what needs to be done, but
implementation should wait for explicit user approval.
+- **"suppress warnings"** or **"suppress issues"** - When the user asks to
suppress warnings or issues that appear to refer to SonarLint/SonarQube issues,
add `@SuppressWarnings("java:Sxxx")` annotations to the appropriate class or
method level. Use the specific rule ID from the warning (e.g., `java:S100`,
`java:S115`, `java:S116`). Apply class-level suppressions when multiple
methods/fields are affected, or method-level suppressions for specific methods.
+
### Script Shortcut Commands
- **"start docs"** or **"start docusaurus"** - Runs
`scripts/start-docusaurus.py`
- **"revert staged"** - Runs `scripts/revert-staged.py`
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ReflectionMap.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ReflectionMap.java
index 672d5f8675..77f792f9d5 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ReflectionMap.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ReflectionMap.java
@@ -513,6 +513,7 @@ public class ReflectionMap<V> {
* @param c The class type of values (used for type safety, not stored).
* @return A new builder instance.
*/
+ @SuppressWarnings("java:S1172") // Parameter c is used for type
inference, not runtime behavior
public static <V> Builder<V> create(Class<V> c) {
return new Builder<>();
}
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/CollectionUtils.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/CollectionUtils.java
index 127936541e..73e3aa243b 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/CollectionUtils.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/CollectionUtils.java
@@ -1096,6 +1096,7 @@ public class CollectionUtils {
* @param type The element type class.
* @return An empty list.
*/
+ @SuppressWarnings("java:S1172") // Parameter type is used for type
inference, not runtime behavior
public static <T> List<T> liste(Class<T> type) {
return Collections.emptyList();
}
@@ -1107,6 +1108,7 @@ public class CollectionUtils {
* @param type The element type class.
* @return <jk>null</jk>.
*/
+ @SuppressWarnings("java:S1172") // Parameter type is used for type
inference, not runtime behavior
public static <T> List<T> listn(Class<T> type) {
return null;
}
@@ -1860,6 +1862,7 @@ public class CollectionUtils {
* @param type The component type class.
* @return <jk>null</jk>.
*/
+ @SuppressWarnings("java:S1172") // Parameter type is used for type
inference, not runtime behavior
public static <T> T[] na(Class<T> type) {
return null;
}
@@ -2286,7 +2289,7 @@ public class CollectionUtils {
* It must match the actual component type in the array.
* @return A new {@link ArrayList}
*/
- @SuppressWarnings("unchecked")
+ @SuppressWarnings({ "unchecked", "java:S1172" }) // Parameter
elementType is used for type inference, not runtime behavior
public static <E> List<E> toList(Object array, Class<E> elementType) {
var l = new ArrayList<E>(Array.getLength(array));
for (var i = 0; i < Array.getLength(array); i++)
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/Utils.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/Utils.java
index 93ff36076a..29c77add74 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/Utils.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/Utils.java
@@ -1484,6 +1484,7 @@ public class Utils {
* @param type The type class (unused, but helps with type inference).
* @return <jk>null</jk>.
*/
+ @SuppressWarnings("java:S1172") // Parameter type is used for type
inference, not runtime behavior
public static <T> T no(Class<T> type) {
return null;
}
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanSession.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanSession.java
index d43c207c58..6dfb06f907 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanSession.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanSession.java
@@ -1135,6 +1135,7 @@ public class BeanSession extends ContextSession {
* @param propertyNamer The property namer to use.
* @return The wrapped object.
*/
+ @SuppressWarnings("java:S1172") // Parameter propertyNamer is unused
but kept for API consistency
public final <T> BeanMap<T> toBeanMap(T o, PropertyNamer propertyNamer)
{
return this.toBeanMap(o, (Class<T>)o.getClass());
}
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonList.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonList.java
index 88aa8b080b..b1f4176df1 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonList.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonList.java
@@ -306,6 +306,7 @@ public class JsonList extends LinkedList<Object> {
* @return A new list or <jk>null</jk> if the input was <jk>null</jk>.
* @throws ParseException Malformed input encountered.
*/
+ @SuppressWarnings("java:S1172") // Parameter p is unused but kept for
API consistency
public static JsonList ofText(Reader in, Parser p) throws
ParseException {
return in == null ? null : new JsonList(in);
}
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonMap.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonMap.java
index 39e12b16df..09ef01a869 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonMap.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/collections/JsonMap.java
@@ -283,6 +283,7 @@ public class JsonMap extends LinkedHashMap<String,Object> {
* @return A new map or <jk>null</jk> if the input was <jk>null</jk>.
* @throws ParseException Malformed input encountered.
*/
+ @SuppressWarnings("java:S1172") // Parameter p is unused but kept for
API consistency
public static JsonMap ofText(Reader in, Parser p) throws ParseException
{
return in == null ? null : new JsonMap(in);
}
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackOutputStream.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackOutputStream.java
index cf56b75539..666cd7670e 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackOutputStream.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackOutputStream.java
@@ -81,6 +81,7 @@ public class MsgPackOutputStream extends OutputStream {
return count;
}
+ @SuppressWarnings("java:S1172") // Parameter out is unused but kept for
method signature consistency
private int writeUtf8To(CharSequence in, OutputStream out) {
var count = 0;
for (int i = 0, len = in.length(); i < len; i++) {
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/swap/BuilderSwap.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/swap/BuilderSwap.java
index 5365af2079..49780b90d5 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/swap/BuilderSwap.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/swap/BuilderSwap.java
@@ -49,7 +49,7 @@ public class BuilderSwap<T,B> {
* @param mVis Minimum method visibility.
* @return A new swap instance, or <jk>null</jk> if class wasn't a
builder class.
*/
- @SuppressWarnings("rawtypes")
+ @SuppressWarnings({ "rawtypes", "java:S1172" }) // Parameter mVis is
unused but kept for API consistency
public static BuilderSwap<?,?> findSwapFromBuilderClass(Class<?>
builderClass, Visibility cVis, Visibility mVis) {
var bci = info(builderClass);
if (bci.isNotPublic())
@@ -92,7 +92,7 @@ public class BuilderSwap<T,B> {
* @param mVis Minimum method visibility.
* @return A new swap instance, or <jk>null</jk> if class didn't have a
builder class.
*/
- @SuppressWarnings("rawtypes")
+ @SuppressWarnings({ "rawtypes", "java:S1172" }) // Parameter mVis is
unused but kept for API consistency
public static BuilderSwap<?,?> findSwapFromObjectClass(BeanContext bc,
Class<?> objectClass, Visibility cVis, Visibility mVis) {
var builderClass = Value.<Class<?>>empty();
MethodInfo objectCreateMethod;
diff --git
a/juneau-examples/juneau-examples-core/src/main/java/org/apache/juneau/examples/core/config/store/SqlStore.java
b/juneau-examples/juneau-examples-core/src/main/java/org/apache/juneau/examples/core/config/store/SqlStore.java
index fab473ccc5..5c25dfa73e 100644
---
a/juneau-examples/juneau-examples-core/src/main/java/org/apache/juneau/examples/core/config/store/SqlStore.java
+++
b/juneau-examples/juneau-examples-core/src/main/java/org/apache/juneau/examples/core/config/store/SqlStore.java
@@ -218,6 +218,7 @@ public class SqlStore extends ConfigStore {
}
// Reads the value from the database.
+ @SuppressWarnings("java:S1172") // Parameter name is unused in base
implementation, intended for override
protected String getDatabaseValue(String name) {
// Implement me!
return null;
diff --git
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathRemainderArg.java
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathRemainderArg.java
index 0a9c299204..3ef76da6f7 100644
---
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathRemainderArg.java
+++
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathRemainderArg.java
@@ -70,6 +70,7 @@ public class PathRemainderArg implements RestOpArg {
* @param pathMatcher Path matcher for the specified method (not used,
but included for BasicBeanStore compatibility).
* @return A new {@link PathRemainderArg}, or <jk>null</jk> if the
parameter is not annotated with {@link PathRemainder}.
*/
+ @SuppressWarnings("java:S1172") // Parameter pathMatcher is kept for
BasicBeanStore compatibility
public static PathRemainderArg create(ParameterInfo paramInfo,
AnnotationWorkList annotations, UrlPathMatcher pathMatcher) {
if (AP.has(PathRemainder.class, paramInfo))
return new PathRemainderArg(paramInfo, annotations);
diff --git
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
index a9f37daddb..a62d5b1b70 100644
---
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
+++
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
@@ -729,6 +729,7 @@ public class BasicSwaggerProviderSession {
piri.put(examplesKey, examples);
}
+ @SuppressWarnings("java:S1172") // Parameter type is unused but kept
for API consistency
private static void addParamExample(RestOpContext sm, JsonMap piri,
RestPartType in, Type type) {
var s = piri.getString(SWAGGER_example);
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/BeanProxyInvocationHandler_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/BeanProxyInvocationHandler_Test.java
index 85a2c0cd09..790133d5bf 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/BeanProxyInvocationHandler_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/BeanProxyInvocationHandler_Test.java
@@ -22,6 +22,7 @@ import java.lang.reflect.*;
import org.junit.jupiter.api.*;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class BeanProxyInvocationHandler_Test extends TestBase {
public interface TestInterface {
diff --git a/juneau-utest/src/test/java/org/apache/juneau/PojoExamplesTest.java
b/juneau-utest/src/test/java/org/apache/juneau/PojoExamplesTest.java
index 2a513e8d34..61b47438d7 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/PojoExamplesTest.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/PojoExamplesTest.java
@@ -24,6 +24,7 @@ import org.junit.jupiter.api.*;
/*
* Tests the BEAN_examples property and @Example annotation.
*/
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class PojoExamplesTest extends TestBase {
private final JsonParserSession session =
JsonParser.DEFAULT.getSession();
diff --git a/juneau-utest/src/test/java/org/apache/juneau/TestUtils.java
b/juneau-utest/src/test/java/org/apache/juneau/TestUtils.java
index 1e6f7cd80a..2b3041b8c3 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/TestUtils.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/TestUtils.java
@@ -182,6 +182,7 @@ public class TestUtils extends Utils {
assertEquals(expected, s.toString(actual));
}
+ @SuppressWarnings("java:S1172") // Parameter expectedType is unused but
kept for API consistency
public static <T extends Throwable> T assertThrowable(Class<? extends
Throwable> expectedType, String expectedSubstring, T t) {
var messages = getMessages(t);
assertTrue(messages.contains(expectedSubstring), fs("Expected
message to contain: {0}.\nActual:\n{1}", expectedSubstring, messages));
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/a/rttests/JsonMaps_RoundTripTest.java
b/juneau-utest/src/test/java/org/apache/juneau/a/rttests/JsonMaps_RoundTripTest.java
index 366c813f9e..3104ccf864 100755
---
a/juneau-utest/src/test/java/org/apache/juneau/a/rttests/JsonMaps_RoundTripTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/a/rttests/JsonMaps_RoundTripTest.java
@@ -30,6 +30,7 @@ import org.junit.jupiter.params.provider.*;
* Tests designed to serialize and parse objects to make sure we end up
* with the same objects for all serializers and parsers.
*/
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class JsonMaps_RoundTripTest extends RoundTripTest_Base {
//====================================================================================================
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/bean/html5/Html5Template_ComboRoundTripTest.java
b/juneau-utest/src/test/java/org/apache/juneau/bean/html5/Html5Template_ComboRoundTripTest.java
index a8d276a15f..b3011d6909 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/bean/html5/Html5Template_ComboRoundTripTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/bean/html5/Html5Template_ComboRoundTripTest.java
@@ -27,7 +27,7 @@ import org.apache.juneau.annotation.*;
/**
* Tests serialization of HTML5 templates.
*/
-@SuppressWarnings({})
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class Html5Template_ComboRoundTripTest extends ComboRoundTripTest_Base {
private static <T> ComboRoundTrip_Tester.Builder<T> tester(int index,
String label, Type type, T bean) {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/annotation/AppliedAnnotationObject_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/annotation/AppliedAnnotationObject_Test.java
index 260c941882..34a4488ae5 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/annotation/AppliedAnnotationObject_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/annotation/AppliedAnnotationObject_Test.java
@@ -26,6 +26,7 @@ import org.apache.juneau.*;
import org.apache.juneau.commons.reflect.*;
import org.junit.jupiter.api.*;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class AppliedAnnotationObject_Test extends TestBase {
private static final String CNAME =
AppliedAnnotationObject_Test.class.getName();
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/inject/BeanCreator2_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/inject/BeanCreator2_Test.java
index 4b5096df2c..37f11f66bd 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/inject/BeanCreator2_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/inject/BeanCreator2_Test.java
@@ -34,6 +34,7 @@ import org.junit.jupiter.api.*;
import org.apache.juneau.commons.annotation.*;
import java.util.logging.Level;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class BeanCreator2_Test extends TestBase {
private BasicBeanStore2 beanStore;
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ClassInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ClassInfo_Test.java
index 441133e31a..d1242c5d3d 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ClassInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ClassInfo_Test.java
@@ -40,7 +40,7 @@ import org.apache.juneau.commons.lang.*;
import org.apache.juneau.svl.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"java:S3008", "java:S5961", "java:S1186"})
+@SuppressWarnings({ "java:S3008", "java:S5961", "java:S1186", "java:S1172" })
// Unused parameters in tests are typically intentional
public class ClassInfo_Test extends TestBase {
@Documented
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ConstructorInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ConstructorInfo_Test.java
index 9012debe09..a4d395769d 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ConstructorInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ConstructorInfo_Test.java
@@ -34,7 +34,7 @@ import org.apache.juneau.annotation.Named;
import org.apache.juneau.commons.inject.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"java:S4144", "java:S1186"})
+@SuppressWarnings({ "java:S4144", "java:S1186", "java:S1172" }) // Unused
parameters in tests are typically intentional
class ConstructorInfo_Test extends TestBase {
private static void check(String expected, Object o) {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ExecutableInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ExecutableInfo_Test.java
index 8b106d1c5e..ec2eaa3785 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ExecutableInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ExecutableInfo_Test.java
@@ -32,7 +32,7 @@ import java.util.stream.*;
import org.apache.juneau.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"java:S3008", "java:S5961", "java:S1186"})
+@SuppressWarnings({ "java:S3008", "java:S5961", "java:S1186", "java:S1172" })
// Unused parameters in tests are typically intentional
class ExecutableInfo_Test extends TestBase {
private static void check(String expected, Object o) {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/MethodInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/MethodInfo_Test.java
index b9d2992b69..77181da414 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/MethodInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/MethodInfo_Test.java
@@ -33,7 +33,7 @@ import org.junit.jupiter.api.*;
import static org.apache.juneau.commons.utils.Utils.*;
-@SuppressWarnings({ "java:S1186", "java:S116" }) // Field names use
underscores for test data (e.g., a_m, c_a1)
+@SuppressWarnings({ "java:S1186", "java:S116", "java:S1172" }) // Field names
use underscores for test data; unused parameters in tests are typically
intentional
class MethodInfo_Test extends TestBase {
@Documented
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ParameterInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ParameterInfo_Test.java
index 3629be3ca3..cbb543f5d4 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ParameterInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ParameterInfo_Test.java
@@ -32,7 +32,7 @@ import org.apache.juneau.*;
import org.apache.juneau.annotation.Name;
import org.junit.jupiter.api.*;
-@SuppressWarnings({ "java:S117", "java:S5961", "java:S1186", "java:S116" }) //
Field names use underscores for test data (e.g., b_a1_a, b_b_a)
+@SuppressWarnings({ "java:S117", "java:S5961", "java:S1186", "java:S116",
"java:S1172" }) // Field names use underscores for test data; unused parameters
in tests are typically intentional
class ParameterInfo_Test extends TestBase {
private static String originalDisableParamNameDetection;
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/cp/BeanStore_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/cp/BeanStore_Test.java
index 3459508da7..e5203168af 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/cp/BeanStore_Test.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/cp/BeanStore_Test.java
@@ -32,7 +32,7 @@ import org.apache.juneau.annotation.Named;
import org.apache.juneau.commons.reflect.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"java:S4144","java:S5961", "java:S1186"})
+@SuppressWarnings({ "java:S4144", "java:S5961", "java:S1186", "java:S1172" })
// Unused parameters in tests are typically intentional
class BeanStore_Test extends TestBase {
@Documented
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/http/remote/RrpcInterface_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/http/remote/RrpcInterface_Test.java
index cb18336f26..22f87b39b5 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/http/remote/RrpcInterface_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/http/remote/RrpcInterface_Test.java
@@ -44,7 +44,7 @@ import org.junit.jupiter.params.*;
import org.junit.jupiter.params.provider.*;
import org.opentest4j.*;
-@SuppressWarnings("java:S1186")
+@SuppressWarnings({ "java:S1186", "java:S1172" }) // Unused parameters in
tests are typically intentional
class RrpcInterface_Test extends TestBase {
private static final Input[] INPUT = {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/junit/bct/BctConfig_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/junit/bct/BctConfig_Test.java
index 09cc7245c4..fc1671db79 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/junit/bct/BctConfig_Test.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/junit/bct/BctConfig_Test.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.*;
/**
* Unit tests for {@link BctConfig} annotation and extension.
*/
-@SuppressWarnings("java:S4144")
+@SuppressWarnings({ "java:S4144", "java:S1172" }) // Unused parameters in
tests are typically intentional
class BctConfig_Test extends TestBase {
//
====================================================================================================
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/junit/bct/PropertyExtractors_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/junit/bct/PropertyExtractors_Test.java
index 8a208f2ca0..3106f6e867 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/junit/bct/PropertyExtractors_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/junit/bct/PropertyExtractors_Test.java
@@ -30,6 +30,7 @@ import org.junit.jupiter.api.*;
* Unit tests for PropertyExtractors.
*/
@DisplayName("PropertyExtractors")
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class PropertyExtractors_Test extends TestBase {
private BasicBeanConverter converter;
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/mstat/ThrownStore_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/mstat/ThrownStore_Test.java
index b80c18ff88..274ce4c86e 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/mstat/ThrownStore_Test.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/mstat/ThrownStore_Test.java
@@ -28,6 +28,7 @@ import org.apache.juneau.cp.*;
import org.apache.juneau.rest.stats.*;
import org.junit.jupiter.api.*;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class ThrownStore_Test extends TestBase {
//------------------------------------------------------------------------------------------------------------------
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/rest/test/client/ThirdPartyProxy_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/rest/test/client/ThirdPartyProxy_Test.java
index 103670dea7..4e139a0c77 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/rest/test/client/ThirdPartyProxy_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/rest/test/client/ThirdPartyProxy_Test.java
@@ -47,6 +47,7 @@ import org.junit.jupiter.params.*;
import org.junit.jupiter.params.provider.*;
import org.opentest4j.*;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class ThirdPartyProxy_Test extends TestBase {
private static final Input[] INPUT = {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/serializer/UriResolution_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/serializer/UriResolution_Test.java
index 2009baea5b..6f9acae0a4 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/serializer/UriResolution_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/serializer/UriResolution_Test.java
@@ -35,6 +35,7 @@ import org.junit.jupiter.params.provider.*;
/**
* Superclass for tests that verify results against all supported content
types.
*/
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class UriResolution_Test extends TestBase {
private static class Tester {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoListSwapTest.java
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoListSwapTest.java
index 274eb9d3d5..779ba20f44 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoListSwapTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoListSwapTest.java
@@ -31,7 +31,7 @@ import org.apache.juneau.serializer.*;
import org.apache.juneau.swap.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"rawtypes"})
+@SuppressWarnings({ "rawtypes", "java:S1172" }) // Unused parameters in tests
are typically intentional
class AutoListSwapTest extends TestBase {
private static final List<String> STRINGLIST = l("foo");
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoMapSwapTest.java
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoMapSwapTest.java
index c5bcb2f326..038a647eae 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoMapSwapTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoMapSwapTest.java
@@ -32,7 +32,7 @@ import org.apache.juneau.serializer.*;
import org.apache.juneau.swap.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"rawtypes"})
+@SuppressWarnings({ "rawtypes", "java:S1172" }) // Unused parameters in tests
are typically intentional
class AutoMapSwapTest extends TestBase {
private static final Map<String,String> STRINGMAP = m("foo","bar");
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoNumberSwapTest.java
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoNumberSwapTest.java
index 3049068af1..9b595d3ad6 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoNumberSwapTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoNumberSwapTest.java
@@ -29,7 +29,7 @@ import org.apache.juneau.serializer.*;
import org.apache.juneau.swap.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"rawtypes", "java:S1186"})
+@SuppressWarnings({ "rawtypes", "java:S1186", "java:S1172" }) // Unused
parameters in tests are typically intentional
class AutoNumberSwapTest extends TestBase {
private static ObjectSwap find(Class<?> c) {
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoObjectSwapTest.java
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoObjectSwapTest.java
index 1831e474d4..73ed7e5fbc 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/transform/AutoObjectSwapTest.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/transform/AutoObjectSwapTest.java
@@ -32,7 +32,7 @@ import org.apache.juneau.serializer.*;
import org.apache.juneau.swap.*;
import org.junit.jupiter.api.*;
-@SuppressWarnings({"rawtypes"})
+@SuppressWarnings({ "rawtypes", "java:S1172" }) // Unused parameters in tests
are typically intentional
class AutoObjectSwapTest extends TestBase {
private static final Map<String,String> STRINGMAP = m("foo","bar");
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/utils/MutatersTest.java
b/juneau-utest/src/test/java/org/apache/juneau/utils/MutatersTest.java
index f67d26bd0a..2c3f38b5c3 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/utils/MutatersTest.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/utils/MutatersTest.java
@@ -22,6 +22,7 @@ import static org.junit.jupiter.api.Assertions.*;
import org.apache.juneau.*;
import org.junit.jupiter.api.*;
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class MutatersTest extends TestBase {
//-----------------------------------------------------------------------------------------------------------------
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/xml/InvalidXmlBeans_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/xml/InvalidXmlBeans_Test.java
index 35de089f0b..977ec90ed0 100644
--- a/juneau-utest/src/test/java/org/apache/juneau/xml/InvalidXmlBeans_Test.java
+++ b/juneau-utest/src/test/java/org/apache/juneau/xml/InvalidXmlBeans_Test.java
@@ -28,6 +28,7 @@ import org.junit.jupiter.params.provider.*;
/**
* Verifies that the correct error messages are displayed when you do
something wrong with the @Xml annotation.
*/
+@SuppressWarnings("java:S1172") // Unused parameters in tests are typically
intentional
class InvalidXmlBeans_Test extends TestBase {
private static final XmlSerializer
