This is an automated email from the ASF dual-hosted git repository.
reta pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cxf.git
The following commit(s) were added to refs/heads/main by this push:
new 9b3fc63488 Bump net.sourceforge.pmd:pmd-java from 7.10.0 to 7.11.0
(#2297)
9b3fc63488 is described below
commit 9b3fc63488f5a8c93dd3f113a2aa1a777d01aa58
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
AuthorDate: Sat Mar 1 18:48:46 2025 -0500
Bump net.sourceforge.pmd:pmd-java from 7.10.0 to 7.11.0 (#2297)
* Bump net.sourceforge.pmd:pmd-java from 7.10.0 to 7.11.0
Bumps [net.sourceforge.pmd:pmd-java](https://github.com/pmd/pmd) from
7.10.0 to 7.11.0.
- [Release notes](https://github.com/pmd/pmd/releases)
-
[Changelog](https://github.com/pmd/pmd/blob/main/docs/render_release_notes.rb)
-
[Commits](https://github.com/pmd/pmd/compare/pmd_releases/7.10.0...pmd_releases/7.11.0)
---
updated-dependencies:
- dependency-name: net.sourceforge.pmd:pmd-java
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <[email protected]>
* Fix PMD violations
---------
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot]
<49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Andriy Redko <[email protected]>
---
parent/pom.xml | 2 +-
.../binding/corba/types/AbstractStartEndEventProducer.java | 1 +
.../corba/types/CorbaOctetSequenceEventProducer.java | 1 +
.../cxf/systest/http/HTTPConduitIoExceptionsTest.java | 4 ++--
.../cxf/systest/servlet/NoSpringServletClientTest.java | 14 +++++++-------
5 files changed, 12 insertions(+), 10 deletions(-)
diff --git a/parent/pom.xml b/parent/pom.xml
index 627953f4db..57d44b71b6 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -435,7 +435,7 @@
<dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-java</artifactId>
- <version>7.10.0</version>
+ <version>7.11.0</version>
</dependency>
</dependencies>
<configuration>
diff --git
a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/AbstractStartEndEventProducer.java
b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/AbstractStartEndEventProducer.java
index 43938b2d4d..3a28b0fb3b 100644
---
a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/AbstractStartEndEventProducer.java
+++
b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/AbstractStartEndEventProducer.java
@@ -29,6 +29,7 @@ import javax.xml.stream.events.Namespace;
import org.apache.cxf.service.model.ServiceInfo;
import org.omg.CORBA.ORB;
+@SuppressWarnings("PMD.FinalFieldCouldBeStatic")
public abstract class AbstractStartEndEventProducer implements
CorbaTypeEventProducer {
diff --git
a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/CorbaOctetSequenceEventProducer.java
b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/CorbaOctetSequenceEventProducer.java
index da3deb43e4..cd6010f2e4 100644
---
a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/CorbaOctetSequenceEventProducer.java
+++
b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/types/CorbaOctetSequenceEventProducer.java
@@ -25,6 +25,7 @@ import javax.xml.stream.XMLStreamConstants;
import javax.xml.stream.events.Attribute;
import javax.xml.stream.events.Namespace;
+@SuppressWarnings("PMD.FinalFieldCouldBeStatic")
public class CorbaOctetSequenceEventProducer implements CorbaTypeEventProducer
{
int state;
diff --git
a/systests/transports/src/test/java/org/apache/cxf/systest/http/HTTPConduitIoExceptionsTest.java
b/systests/transports/src/test/java/org/apache/cxf/systest/http/HTTPConduitIoExceptionsTest.java
index 6cc0f3681b..9a7b7993b9 100644
---
a/systests/transports/src/test/java/org/apache/cxf/systest/http/HTTPConduitIoExceptionsTest.java
+++
b/systests/transports/src/test/java/org/apache/cxf/systest/http/HTTPConduitIoExceptionsTest.java
@@ -82,7 +82,7 @@ public class HTTPConduitIoExceptionsTest extends
AbstractBusClientServerTestBase
exception.expect(WebServiceException.class);
exception.expectCause(new TypeSafeMatcher<Throwable>() {
- private final String message = "HTTP response '503: Service
Unavailable' when "
+ private static final String message = "HTTP response '503: Service
Unavailable' when "
+ "communicating with http://localhost:"; + BadServer.PORT +
"/Mortimer";
@Override
@@ -109,7 +109,7 @@ public class HTTPConduitIoExceptionsTest extends
AbstractBusClientServerTestBase
exception.expect(WebServiceException.class);
exception.expectCause(new TypeSafeMatcher<Throwable>() {
- private final String message = "HTTP response '404: Not Found'
when "
+ private static final String message = "HTTP response '404: Not
Found' when "
+ "communicating with http://localhost:"; + BadServer.PORT +
"/Mortimer";
@Override
diff --git
a/systests/transports/src/test/java/org/apache/cxf/systest/servlet/NoSpringServletClientTest.java
b/systests/transports/src/test/java/org/apache/cxf/systest/servlet/NoSpringServletClientTest.java
index f793358734..175450fce0 100644
---
a/systests/transports/src/test/java/org/apache/cxf/systest/servlet/NoSpringServletClientTest.java
+++
b/systests/transports/src/test/java/org/apache/cxf/systest/servlet/NoSpringServletClientTest.java
@@ -51,10 +51,10 @@ import static org.junit.Assert.fail;
public class NoSpringServletClientTest extends AbstractServletTest {
private static final String PORT = NoSpringServletServer.PORT;
+ private static final String SERVICE_URL = "http://localhost:"; + PORT +
"/soap/";
private static Bus serverBus;
private static NoSpringServletServer server;
private final QName portName = new
QName("http://apache.org/hello_world_soap_http";, "SoapPort");
- private final String serviceURL = "http://localhost:"; + PORT + "/soap/";
@BeforeClass
public static void startServers() throws Exception {
@@ -73,7 +73,7 @@ public class NoSpringServletClientTest extends
AbstractServletTest {
@Test
public void testBasicConnection() throws Exception {
- SOAPService service = new SOAPService(new URL(serviceURL +
"Greeter?wsdl"));
+ SOAPService service = new SOAPService(new URL(SERVICE_URL +
"Greeter?wsdl"));
Greeter greeter = service.getPort(portName, Greeter.class);
try {
String reply = greeter.greetMe("test");
@@ -90,7 +90,7 @@ public class NoSpringServletClientTest extends
AbstractServletTest {
@Test
public void testHelloService() throws Exception {
JaxWsProxyFactoryBean cpfb = new JaxWsProxyFactoryBean();
- String address = serviceURL + "Hello";
+ String address = SERVICE_URL + "Hello";
cpfb.setServiceClass(Hello.class);
cpfb.setAddress(address);
Hello hello = (Hello) cpfb.create();
@@ -133,7 +133,7 @@ public class NoSpringServletClientTest extends
AbstractServletTest {
@Test
public void testGetServiceList() throws Exception {
try (CloseableHttpClient client = HttpClients.createDefault()) {
- final HttpGet method = new HttpGet(serviceURL + "/services");
+ final HttpGet method = new HttpGet(SERVICE_URL + "/services");
try (CloseableHttpResponse res = client.execute(method)) {
HTMLDocumentImpl doc = parse(res.getEntity().getContent());
@@ -144,9 +144,9 @@ public class NoSpringServletClientTest extends
AbstractServletTest {
s.add(l.getHref());
}
assertEquals("There should be 3 links for the service", 3,
links.size());
- assertTrue(s.contains(serviceURL + "Greeter?wsdl"));
- assertTrue(s.contains(serviceURL + "Hello?wsdl"));
- assertTrue(s.contains(serviceURL + "?wsdl"));
+ assertTrue(s.contains(SERVICE_URL + "Greeter?wsdl"));
+ assertTrue(s.contains(SERVICE_URL + "Hello?wsdl"));
+ assertTrue(s.contains(SERVICE_URL + "?wsdl"));
assertEquals("text/html", getContentType(res));
}
}
