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 e005b89f6a SonarQube bug fixes
e005b89f6a is described below
commit e005b89f6a16363ef0527224da97a837942cd9fb
Author: James Bognar <[email protected]>
AuthorDate: Wed Feb 4 16:18:54 2026 -0500
SonarQube bug fixes
---
.../java/org/apache/juneau/commons/io/Console.java | 1 +
.../juneau/commons/io/StringBuilderWriter.java | 8 +++++--
.../juneau/csv/annotation/CsvConfigAnnotation.java | 8 +++++--
.../apache/juneau/encoders/IdentityEncoder.java | 4 +++-
.../html/annotation/HtmlConfigAnnotation.java | 4 +++-
.../annotation/MsgPackConfigAnnotation.java | 4 +++-
.../annotation/PlainTextConfigAnnotation.java | 8 +++++--
.../juneau/examples/rest/UtilityBeansResource.java | 4 +++-
.../microservice/BasicMicroserviceListener.java | 12 +++++++---
.../jetty/BasicJettyMicroserviceListener.java | 20 ++++++++++++----
.../apache/juneau/rest/client/ResponseContent.java | 8 +++++--
.../org/apache/juneau/rest/client/RestClient.java | 4 +++-
.../apache/juneau/http/entity/BasicHttpEntity.java | 8 +++++--
.../apache/juneau/http/resource/BasicResource.java | 4 +++-
.../juneau/http/response/BasicHttpException.java | 4 +++-
.../juneau/http/response/BasicHttpResponse.java | 4 +++-
.../rest/mock/MockHttpClientConnectionManager.java | 28 ++++++++++++++++------
.../apache/juneau/rest/mock/MockHttpSession.java | 4 +++-
.../apache/juneau/rest/mock/MockRestClient.java | 12 +++++++---
.../juneau/rest/mock/MockServletRequest.java | 8 +++++--
.../juneau/rest/mock/MockServletResponse.java | 12 +++++++---
.../annotation/AppliedAnnotationObject_Test.java | 1 +
.../juneau/commons/inject/BeanCreator2_Test.java | 8 +++++++
.../commons/reflect/AnnotationInfo_Test.java | 1 +
.../commons/reflect/AnnotationProvider_Test.java | 3 +++
.../commons/reflect/ConstructorInfo_Test.java | 3 +++
.../juneau/commons/reflect/ElementInfo_Test.java | 7 ++++++
.../commons/reflect/ExecutableInfo_Test.java | 2 +-
.../juneau/commons/reflect/MethodInfo_Test.java | 4 ++++
.../juneau/commons/reflect/ParameterInfo_Test.java | 1 +
.../commons/reflect/ReflectionUtils_Test.java | 1 +
.../juneau/commons/reflect/Visibility_Test.java | 6 +++--
.../apache/juneau/commons/utils/IoUtils_Test.java | 1 +
33 files changed, 162 insertions(+), 45 deletions(-)
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/Console.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/Console.java
index 85cf95bf52..f21bf94612 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/Console.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/Console.java
@@ -32,6 +32,7 @@ public class Console {
* Constructor.
*/
public Console() {
+ // No-op: Intentional empty constructor
}
/**
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/StringBuilderWriter.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/StringBuilderWriter.java
index cea31e3846..dc5ae5b2fd 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/StringBuilderWriter.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/io/StringBuilderWriter.java
@@ -179,10 +179,14 @@ public class StringBuilderWriter extends Writer {
}
@Override /* Overridden from Writer */
- public void close() throws IOException {}
+ public void close() throws IOException {
+ // No-op: StringBuilder does not require closing
+ }
@Override /* Overridden from Writer */
- public void flush() {}
+ public void flush() {
+ // No-op: StringBuilder does not require flushing
+ }
@Override /* Overridden from Object */
public String toString() {
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/csv/annotation/CsvConfigAnnotation.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/csv/annotation/CsvConfigAnnotation.java
index 76f2e3d144..34c31092a1 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/csv/annotation/CsvConfigAnnotation.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/csv/annotation/CsvConfigAnnotation.java
@@ -42,7 +42,9 @@ public class CsvConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<CsvConfig> ai,
CsvParser.Builder b) {}
+ public void apply(AnnotationInfo<CsvConfig> ai,
CsvParser.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
/**
@@ -60,6 +62,8 @@ public class CsvConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<CsvConfig> ai,
CsvSerializer.Builder b) {}
+ public void apply(AnnotationInfo<CsvConfig> ai,
CsvSerializer.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
}
\ No newline at end of file
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/encoders/IdentityEncoder.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/encoders/IdentityEncoder.java
index 7286b6cc22..a169ac004e 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/encoders/IdentityEncoder.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/encoders/IdentityEncoder.java
@@ -41,7 +41,9 @@ public class IdentityEncoder extends Encoder {
public static final IdentityEncoder INSTANCE = new IdentityEncoder();
/** Constructor. */
- public IdentityEncoder() {}
+ public IdentityEncoder() {
+ // No-op: Intentional empty constructor
+ }
/**
* Returns <code>[<js>"identity"</js>]</code>.
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/annotation/HtmlConfigAnnotation.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/annotation/HtmlConfigAnnotation.java
index cd7d89f077..d4d7348c32 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/annotation/HtmlConfigAnnotation.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/annotation/HtmlConfigAnnotation.java
@@ -45,7 +45,9 @@ public class HtmlConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<HtmlConfig> ai,
HtmlParser.Builder b) {}
+ public void apply(AnnotationInfo<HtmlConfig> ai,
HtmlParser.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
/**
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/annotation/MsgPackConfigAnnotation.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/annotation/MsgPackConfigAnnotation.java
index aee5aa8cde..bed51e69a7 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/annotation/MsgPackConfigAnnotation.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/annotation/MsgPackConfigAnnotation.java
@@ -45,7 +45,9 @@ public class MsgPackConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<MsgPackConfig> ai,
MsgPackParser.Builder b) {}
+ public void apply(AnnotationInfo<MsgPackConfig> ai,
MsgPackParser.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
/**
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/plaintext/annotation/PlainTextConfigAnnotation.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/plaintext/annotation/PlainTextConfigAnnotation.java
index c6e31c227c..2f2fd3d385 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/plaintext/annotation/PlainTextConfigAnnotation.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/plaintext/annotation/PlainTextConfigAnnotation.java
@@ -42,7 +42,9 @@ public class PlainTextConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<PlainTextConfig> ai,
PlainTextParser.Builder b) {}
+ public void apply(AnnotationInfo<PlainTextConfig> ai,
PlainTextParser.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
/**
@@ -60,6 +62,8 @@ public class PlainTextConfigAnnotation {
}
@Override
- public void apply(AnnotationInfo<PlainTextConfig> ai,
PlainTextSerializer.Builder b) {}
+ public void apply(AnnotationInfo<PlainTextConfig> ai,
PlainTextSerializer.Builder b) {
+ // No-op: Annotation applier with no work to do
+ }
}
}
\ No newline at end of file
diff --git
a/juneau-examples/juneau-examples-rest/src/main/java/org/apache/juneau/examples/rest/UtilityBeansResource.java
b/juneau-examples/juneau-examples-rest/src/main/java/org/apache/juneau/examples/rest/UtilityBeansResource.java
index fa00e53575..085dc57850 100644
---
a/juneau-examples/juneau-examples-rest/src/main/java/org/apache/juneau/examples/rest/UtilityBeansResource.java
+++
b/juneau-examples/juneau-examples-rest/src/main/java/org/apache/juneau/examples/rest/UtilityBeansResource.java
@@ -78,7 +78,9 @@ public class UtilityBeansResource extends BasicRestObject {
public boolean isCurrent;
/** Default constructor. */
- public Address() {}
+ public Address() {
+ // No-op: Test helper class - intentionally empty for
testing
+ }
}
@SuppressWarnings("unused")
diff --git
a/juneau-microservice/juneau-microservice-core/src/main/java/org/apache/juneau/microservice/BasicMicroserviceListener.java
b/juneau-microservice/juneau-microservice-core/src/main/java/org/apache/juneau/microservice/BasicMicroserviceListener.java
index 88684d2617..82ddcf7898 100644
---
a/juneau-microservice/juneau-microservice-core/src/main/java/org/apache/juneau/microservice/BasicMicroserviceListener.java
+++
b/juneau-microservice/juneau-microservice-core/src/main/java/org/apache/juneau/microservice/BasicMicroserviceListener.java
@@ -28,11 +28,17 @@ import org.apache.juneau.config.event.*;
public class BasicMicroserviceListener implements MicroserviceListener {
@Override /* Overridden from MicroserviceListener */
- public void onConfigChange(Microservice microservice, ConfigEvents
events) {}
+ public void onConfigChange(Microservice microservice, ConfigEvents
events) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from MicroserviceListener */
- public void onStart(Microservice microservice) {}
+ public void onStart(Microservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from MicroserviceListener */
- public void onStop(Microservice microservice) {}
+ public void onStop(Microservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
}
\ No newline at end of file
diff --git
a/juneau-microservice/juneau-microservice-jetty/src/main/java/org/apache/juneau/microservice/jetty/BasicJettyMicroserviceListener.java
b/juneau-microservice/juneau-microservice-jetty/src/main/java/org/apache/juneau/microservice/jetty/BasicJettyMicroserviceListener.java
index f85d744a6f..c3dda0b21d 100644
---
a/juneau-microservice/juneau-microservice-jetty/src/main/java/org/apache/juneau/microservice/jetty/BasicJettyMicroserviceListener.java
+++
b/juneau-microservice/juneau-microservice-jetty/src/main/java/org/apache/juneau/microservice/jetty/BasicJettyMicroserviceListener.java
@@ -28,17 +28,27 @@ import org.apache.juneau.microservice.*;
public class BasicJettyMicroserviceListener extends BasicMicroserviceListener
implements JettyMicroserviceListener {
@Override /* Overridden from JettyMicroserviceListener */
- public void onCreateServer(JettyMicroservice microservice) {}
+ public void onCreateServer(JettyMicroservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from JettyMicroserviceListener */
- public void onPostStartServer(JettyMicroservice microservice) {}
+ public void onPostStartServer(JettyMicroservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from JettyMicroserviceListener */
- public void onPostStopServer(JettyMicroservice microservice) {}
+ public void onPostStopServer(JettyMicroservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from JettyMicroserviceListener */
- public void onStartServer(JettyMicroservice microservice) {}
+ public void onStartServer(JettyMicroservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
@Override /* Overridden from JettyMicroserviceListener */
- public void onStopServer(JettyMicroservice microservice) {}
+ public void onStopServer(JettyMicroservice microservice) {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
}
\ No newline at end of file
diff --git
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/ResponseContent.java
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/ResponseContent.java
index 2eb873cf24..6ce29a75d2 100644
---
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/ResponseContent.java
+++
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/ResponseContent.java
@@ -59,7 +59,9 @@ public class ResponseContent implements HttpEntity {
private static final HttpEntity NULL_ENTITY = new HttpEntity() {
@Override
- public void consumeContent() throws IOException {}
+ public void consumeContent() throws IOException {
+ // No-op: Mock implementation - full functionality not
required
+ }
@Override
public InputStream getContent() throws IOException,
UnsupportedOperationException { return new ByteArrayInputStream(new byte[0]); }
@@ -83,7 +85,9 @@ public class ResponseContent implements HttpEntity {
public boolean isStreaming() { return false; }
@Override
- public void writeTo(OutputStream outstream) throws IOException
{}
+ public void writeTo(OutputStream outstream) throws IOException {
+ // No-op: Mock implementation - full functionality not
required
+ }
};
private final RestClient client;
diff --git
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
index e9b2aefe38..367701514b 100644
---
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
+++
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
@@ -7774,7 +7774,9 @@ public class RestClient extends BeanContextable
implements HttpClient, Closeable
/**
* Gets called add the end of the constructor call to perform any
post-initialization.
*/
- protected void init() {}
+ protected void init() {
+ // No-op: Intentional empty implementation for optional
override method
+ }
/**
* Returns <jk>true</jk> if empty request form-data parameter values
should be ignored.
diff --git
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/entity/BasicHttpEntity.java
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/entity/BasicHttpEntity.java
index 9f1f8722ef..3e0a9c2499 100644
---
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/entity/BasicHttpEntity.java
+++
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/entity/BasicHttpEntity.java
@@ -152,7 +152,9 @@ public class BasicHttpEntity implements HttpEntity {
}
@Override /* Overridden from HttpEntity */
- public void consumeContent() throws IOException {}
+ public void consumeContent() throws IOException {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
/**
* Creates a builder for this class initialized with the contents of
this bean.
@@ -383,7 +385,9 @@ public class BasicHttpEntity implements HttpEntity {
}
@Override /* Overridden from HttpEntity */
- public void writeTo(OutputStream outStream) throws IOException {}
+ public void writeTo(OutputStream outStream) throws IOException {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
/**
* Same as {@link #asBytes()} but wraps {@link IOException
IOExceptions} inside a {@link RuntimeException}.
diff --git
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/resource/BasicResource.java
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/resource/BasicResource.java
index d4aea076cb..9e252f45d9 100644
---
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/resource/BasicResource.java
+++
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/resource/BasicResource.java
@@ -182,7 +182,9 @@ public class BasicResource implements HttpResource {
}
@Override
- public void consumeContent() throws IOException {}
+ public void consumeContent() throws IOException {
+ // No-op: Intentional empty implementation for optional
interface method
+ }
/**
* Creates a builder for this class initialized with the contents of
this bean.
diff --git
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpException.java
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpException.java
index d10e74cfee..ae2ee2d931 100644
---
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpException.java
+++
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpException.java
@@ -424,7 +424,9 @@ public class BasicHttpException extends
BasicRuntimeException implements HttpRes
@SuppressWarnings("deprecation")
@Override /* Overridden from HttpMessage */
- public void setParams(HttpParams params) {}
+ public void setParams(HttpParams params) {
+ // No-op: Intentional empty implementation for deprecated
optional interface method
+ }
/**
* Sets the protocol version on the status line.
diff --git
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpResponse.java
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpResponse.java
index b556547b7b..90c6e0e994 100644
---
a/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpResponse.java
+++
b/juneau-rest/juneau-rest-common/src/main/java/org/apache/juneau/http/response/BasicHttpResponse.java
@@ -327,7 +327,9 @@ public class BasicHttpResponse implements HttpResponse {
@SuppressWarnings("deprecation")
@Override /* Overridden from HttpMessage */
- public void setParams(HttpParams params) {}
+ public void setParams(HttpParams params) {
+ // No-op: Intentional empty implementation for deprecated
optional interface method
+ }
/**
* Sets the protocol version on the status line.
diff --git
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpClientConnectionManager.java
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpClientConnectionManager.java
index 4e5651aa12..6bccf3d94f 100644
---
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpClientConnectionManager.java
+++
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpClientConnectionManager.java
@@ -45,13 +45,19 @@ class MockHttpClientConnectionManager implements
HttpClientConnectionManager {
private ConnectionRequest cr;
@Override /* Overridden from HttpClientConnectionManager */
- public void closeExpiredConnections() {}
+ public void closeExpiredConnections() {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnectionManager */
- public void closeIdleConnections(long idletime, TimeUnit tunit) {}
+ public void closeIdleConnections(long idletime, TimeUnit tunit) {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnectionManager */
- public void connect(HttpClientConnection conn, HttpRoute route, int
connectTimeout, HttpContext context) throws IOException {}
+ public void connect(HttpClientConnection conn, HttpRoute route, int
connectTimeout, HttpContext context) throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from Object */
public boolean equals(Object o) {
@@ -84,7 +90,9 @@ class MockHttpClientConnectionManager implements
HttpClientConnectionManager {
}
@Override /* Overridden from HttpClientConnectionManager */
- public void releaseConnection(HttpClientConnection conn, Object
newState, long validDuration, TimeUnit timeUnit) {}
+ public void releaseConnection(HttpClientConnection conn, Object
newState, long validDuration, TimeUnit timeUnit) {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnectionManager */
public ConnectionRequest requestConnection(HttpRoute route, Object
state) {
@@ -92,11 +100,17 @@ class MockHttpClientConnectionManager implements
HttpClientConnectionManager {
}
@Override /* Overridden from HttpClientConnectionManager */
- public void routeComplete(HttpClientConnection conn, HttpRoute route,
HttpContext context) throws IOException {}
+ public void routeComplete(HttpClientConnection conn, HttpRoute route,
HttpContext context) throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnectionManager */
- public void shutdown() {}
+ public void shutdown() {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnectionManager */
- public void upgrade(HttpClientConnection conn, HttpRoute route,
HttpContext context) throws IOException {}
+ public void upgrade(HttpClientConnection conn, HttpRoute route,
HttpContext context) throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
}
\ No newline at end of file
diff --git
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpSession.java
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpSession.java
index fabc62bfbe..d3063d9548 100644
---
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpSession.java
+++
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockHttpSession.java
@@ -105,7 +105,9 @@ public class MockHttpSession implements HttpSession {
}
@Override /* Overridden from HttpSession */
- public void invalidate() {}
+ public void invalidate() {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpSession */
public boolean isNew() { return isNew; }
diff --git
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockRestClient.java
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockRestClient.java
index 2d73e16cd0..911e3539d4 100644
---
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockRestClient.java
+++
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockRestClient.java
@@ -1908,7 +1908,9 @@ public class MockRestClient extends RestClient implements
HttpClientConnection {
}
@Override /* Overridden from HttpClientConnection */
- public void flush() throws IOException {}
+ public void flush() throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from RestClient */
public MockRestRequest formPost(Object url) throws RestCallException {
@@ -2157,10 +2159,14 @@ public class MockRestClient extends RestClient
implements HttpClientConnection {
}
@Override /* Overridden from HttpClientConnection */
- public void setSocketTimeout(int timeout) {}
+ public void setSocketTimeout(int timeout) {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpClientConnection */
- public void shutdown() throws IOException {}
+ public void shutdown() throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
/**
* Attempts to unwrap the request to find the underlying RestRequest
object.
diff --git
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletRequest.java
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletRequest.java
index 5b747f1b2d..d9bd541ad6 100644
---
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletRequest.java
+++
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletRequest.java
@@ -634,10 +634,14 @@ public class MockServletRequest implements
HttpServletRequest {
}
@Override /* Overridden from HttpServletRequest */
- public void login(String username, String password) throws
ServletException {}
+ public void login(String username, String password) throws
ServletException {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpServletRequest */
- public void logout() throws ServletException {}
+ public void logout() throws ServletException {
+ // No-op: Mock implementation - full functionality not required
+ }
/**
* Fluent setter.
diff --git
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletResponse.java
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletResponse.java
index 5a932e5939..5c7478d988 100644
---
a/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletResponse.java
+++
b/juneau-rest/juneau-rest-mock/src/main/java/org/apache/juneau/rest/mock/MockServletResponse.java
@@ -58,7 +58,9 @@ public class MockServletResponse implements
HttpServletResponse {
private Map<String,String[]> headerMap = map();
@Override /* Overridden from HttpServletResponse */
- public void addCookie(Cookie cookie) {}
+ public void addCookie(Cookie cookie) {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpServletResponse */
public void addDateHeader(String name, long date) {
@@ -94,7 +96,9 @@ public class MockServletResponse implements
HttpServletResponse {
}
@Override /* Overridden from HttpServletResponse */
- public void flushBuffer() throws IOException {}
+ public void flushBuffer() throws IOException {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpServletResponse */
public int getBufferSize() { return bufferSize; }
@@ -155,7 +159,9 @@ public class MockServletResponse implements
HttpServletResponse {
public boolean isCommitted() { return false; }
@Override /* Overridden from HttpServletResponse */
- public void reset() {}
+ public void reset() {
+ // No-op: Mock implementation - full functionality not required
+ }
@Override /* Overridden from HttpServletResponse */
public void resetBuffer() {}
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 f0fc2dd56f..260c941882 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
@@ -106,6 +106,7 @@ class AppliedAnnotationObject_Test extends TestBase {
// Test classes for targeting
//------------------------------------------------------------------------------------------------------------------
+ @SuppressWarnings("java:S1186")
public static class TC {
public String field1;
public int field2;
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 d575f097ca..4b5096df2c 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
@@ -56,6 +56,7 @@ class BeanCreator2_Test extends TestBase {
//====================================================================================================
// Type not in bean store - used for testing unresolvable dependencies
+ @SuppressWarnings("java:S1186")
public static class UnresolvableType {
public UnresolvableType() {}
}
@@ -89,6 +90,7 @@ class BeanCreator2_Test extends TestBase {
}
// Bean with no-arg constructor
+ @SuppressWarnings("java:S1186")
public static class SimpleBean {
public String value;
public SimpleBean() {}
@@ -109,6 +111,7 @@ class BeanCreator2_Test extends TestBase {
}
// Bean with injected fields
+ @SuppressWarnings("java:S1186")
public static class BeanWithInjectedFields {
@Inject TestService service;
@Inject AnotherService another;
@@ -120,6 +123,7 @@ class BeanCreator2_Test extends TestBase {
}
// Bean with injected method
+ @SuppressWarnings("java:S1186")
public static class BeanWithInjectedMethod {
private TestService service;
@@ -221,11 +225,13 @@ class BeanCreator2_Test extends TestBase {
}
// Bean classes for type specification testing (used across multiple
nested test classes)
+ @SuppressWarnings("java:S1186")
public static class ParentBean {
public ParentBean() {}
}
// Child bean for type specification testing
+ @SuppressWarnings("java:S1186")
public static class ChildBean extends ParentBean {
public ChildBean() {}
}
@@ -561,6 +567,7 @@ class BeanCreator2_Test extends TestBase {
}
// Bean with PostConstruct method
+ @SuppressWarnings("java:S1186")
public static class B03_BeanWithPostConstruct {
private TestService service;
boolean postConstructCalled = false;
@@ -3037,6 +3044,7 @@ class BeanCreator2_Test extends TestBase {
}
// Bean that implements BeanInterface for fallback testing
+ @SuppressWarnings("java:S1186")
public static class M05_ConcreteBeanInterface implements
BeanInterface {
public M05_ConcreteBeanInterface() {}
}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationInfo_Test.java
index 1488a631bf..287656c4aa 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationInfo_Test.java
@@ -887,6 +887,7 @@ class AnnotationInfo_Test extends TestBase {
String value() default "default";
}
+ @SuppressWarnings("java:S1186")
public static class MethodAnnotationTestClass {
@TestAnnotationMultiTarget("method")
public void testMethod() {}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationProvider_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationProvider_Test.java
index 83f7a542e1..4b64d72e31 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationProvider_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/AnnotationProvider_Test.java
@@ -51,6 +51,7 @@ class AnnotationProvider_Test extends TestBase {
}
@TestAnnotation("class")
+ @SuppressWarnings("java:S1186")
public static class TestClass {
@MultiTargetAnnotation(1)
public String field1;
@@ -76,11 +77,13 @@ class AnnotationProvider_Test extends TestBase {
void matchingMethod(String param);
}
+ @SuppressWarnings("java:S1186")
public static class MatchingMethodParent {
@MultiTargetAnnotation(20)
public void matchingMethod(String param) {}
}
+ @SuppressWarnings("java:S1186")
public static class MatchingMethodChild extends MatchingMethodParent
implements MatchingMethodInterface {
@MultiTargetAnnotation(30)
@Override
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 e981ca2272..90bd22353f 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
@@ -107,6 +107,7 @@ class ConstructorInfo_Test extends TestBase {
@Retention(RUNTIME)
public static @interface DeprecatedAnnotation {}
+ @SuppressWarnings("java:S1186")
public static class DeprecatedClass {
@Deprecated
public DeprecatedClass() {}
@@ -120,6 +121,7 @@ class ConstructorInfo_Test extends TestBase {
public ExceptionClass() throws Exception {}
}
+ @SuppressWarnings("java:S1186")
public static class EqualsTestClass {
public EqualsTestClass() {}
public EqualsTestClass(String param) {}
@@ -857,6 +859,7 @@ class ConstructorInfo_Test extends TestBase {
// Inner class for testing outer parameter
// Note: This is a non-static inner class, so it requires an outer
instance
+ @SuppressWarnings("java:S1186")
static class OuterClass {
public OuterClass() {}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ElementInfo_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ElementInfo_Test.java
index 6386e0e081..6e1c8072fc 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ElementInfo_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ElementInfo_Test.java
@@ -31,6 +31,7 @@ class ElementInfo_Test extends TestBase {
// Test classes with various modifiers
//-----------------------------------------------------------------------------------------------------------------
+ @SuppressWarnings("java:S1186")
public static class PublicClass {
public PublicClass() {}
public void publicMethod() {}
@@ -45,23 +46,27 @@ class ElementInfo_Test extends TestBase {
private int privateField;
}
+ @SuppressWarnings("java:S1186")
protected static class ProtectedClass {
protected ProtectedClass() {}
protected void protectedMethod() {}
protected int protectedField;
}
+ @SuppressWarnings("java:S1186")
static class PackagePrivateClass {
PackagePrivateClass() {}
void packagePrivateMethod() {}
int packagePrivateField;
}
+ @SuppressWarnings("java:S1186")
public static class StaticClass {
public static void staticMethod() {}
public static int staticField;
}
+ @SuppressWarnings("java:S1186")
public static class FinalClass {
public final void finalMethod() {}
public final int finalField = 0;
@@ -75,6 +80,7 @@ class ElementInfo_Test extends TestBase {
void interfaceMethod();
}
+ @SuppressWarnings("java:S1186")
public static class SynchronizedClass {
public synchronized void synchronizedMethod() {}
}
@@ -83,6 +89,7 @@ class ElementInfo_Test extends TestBase {
public native void nativeMethod();
}
+ @SuppressWarnings("java:S1186")
public static class StrictClass {
public void strictMethod() {}
}
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 cfe8de3147..31ae3cf23c 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
@@ -886,7 +886,7 @@ class ExecutableInfo_Test extends TestBase {
}
// Test classes for comprehensive toString() testing
- @SuppressWarnings("unused")
+ @SuppressWarnings({"unused", "java:S1186"})
static class ToStringTestClass {
public ToStringTestClass() {}
private ToStringTestClass(int i) {} // NOSONAR(java:S1186):
Unused test method/constructor
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 bcfec93271..d496e9fae1 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
@@ -102,6 +102,7 @@ class MethodInfo_Test extends TestBase {
}
static MethodInfo a_m = ofm(A1.class, "m"); // NOSONAR(java:UNKNOWN):
Field initialization
+ @SuppressWarnings("java:S1186")
public static class EqualsTestClass {
public void method1() {}
public void method2(String param) {}
@@ -268,6 +269,7 @@ class MethodInfo_Test extends TestBase {
}
// Test classes for compareTo tie-breaker testing
+ @SuppressWarnings("java:S1186")
public static class CompareToParent {
public CompareToParent getInstance() {
return new CompareToParent();
@@ -275,6 +277,7 @@ class MethodInfo_Test extends TestBase {
public void method(String param) {}
}
+ @SuppressWarnings("java:S1186")
public static class CompareToChild extends CompareToParent {
@Override
public CompareToChild getInstance() {
@@ -918,6 +921,7 @@ class MethodInfo_Test extends TestBase {
}
// Test method classes
+ @SuppressWarnings("java:S1186")
public static class TestMethodClass {
public void method1(TestService service) {}
public void
method2(@org.apache.juneau.annotation.Named("service1") TestService service) {}
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 7b66853f96..f3f92b6804 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
@@ -250,6 +250,7 @@ class ParameterInfo_Test extends TestBase {
}
// Constructor hierarchy tests
+ @SuppressWarnings("java:S1186")
public static class EqualsTestClass {
public void method(String param1, int param2) {}
}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ReflectionUtils_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ReflectionUtils_Test.java
index fd9d0dbef3..4946801455 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ReflectionUtils_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/ReflectionUtils_Test.java
@@ -25,6 +25,7 @@ import org.junit.jupiter.api.*;
class ReflectionUtils_Test extends TestBase {
+ @SuppressWarnings("java:S1186")
public static class TestClass {
public String field;
public TestClass() {}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/Visibility_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/Visibility_Test.java
index b60b9ae4cd..b03a1b847e 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/Visibility_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/reflect/Visibility_Test.java
@@ -29,26 +29,28 @@ class Visibility_Test extends TestBase {
// Test classes with various visibility modifiers
//-----------------------------------------------------------------------------------------------------------------
+ @SuppressWarnings("java:S1186")
public static class PublicClass {
public PublicClass() {}
public void publicMethod() {}
public int publicField;
}
+ @SuppressWarnings({"java:S1186", "unused"})
private static class PrivateClass {
private PrivateClass() {}
- @SuppressWarnings("unused")
private void privateMethod() {}
- @SuppressWarnings("unused")
private int privateField;
}
+ @SuppressWarnings("java:S1186")
protected static class ProtectedClass {
protected ProtectedClass() {}
protected void protectedMethod() {}
protected int protectedField;
}
+ @SuppressWarnings("java:S1186")
static class PackagePrivateClass {
PackagePrivateClass() {}
void packagePrivateMethod() {}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/commons/utils/IoUtils_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/commons/utils/IoUtils_Test.java
index ef18025075..64ca73ea36 100644
---
a/juneau-utest/src/test/java/org/apache/juneau/commons/utils/IoUtils_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/commons/utils/IoUtils_Test.java
@@ -1059,6 +1059,7 @@ class IoUtils_Test extends TestBase {
}
}
+ @SuppressWarnings("java:S1186")
public static class ThrowingTestReader extends Reader {
public ThrowingTestReader() {
}
