From 3105cfbfb817444ebbfec8a726b8526e7ff38632 Mon Sep 17 00:00:00 2001 From: Pamela Dragosh Date: Mon, 2 Dec 2019 14:38:05 -0500 Subject: Clear some simple xacml sonar Missing assertions in JUnit test cases. Use replace vs replaceAll Swapping arguments Issue-ID: POLICY-2079 Change-Id: I5accd462eab3d4f6b3321e375d4ba1084caf4b8c Signed-off-by: Pamela Dragosh --- .../pdpx/main/comm/listeners/XacmlPdpUpdateListener.java | 2 +- .../test/java/org/onap/policy/pdpx/main/XacmlStateTest.java | 4 ++-- .../pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java | 9 ++++++--- .../policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java | 13 ++++++++++--- .../java/org/onap/policy/pdpx/main/startstop/TestMain.java | 11 ++++++++--- 5 files changed, 27 insertions(+), 12 deletions(-) (limited to 'main/src') diff --git a/main/src/main/java/org/onap/policy/pdpx/main/comm/listeners/XacmlPdpUpdateListener.java b/main/src/main/java/org/onap/policy/pdpx/main/comm/listeners/XacmlPdpUpdateListener.java index 64ffdeda..1e275c15 100644 --- a/main/src/main/java/org/onap/policy/pdpx/main/comm/listeners/XacmlPdpUpdateListener.java +++ b/main/src/main/java/org/onap/policy/pdpx/main/comm/listeners/XacmlPdpUpdateListener.java @@ -34,7 +34,7 @@ import org.slf4j.LoggerFactory; public class XacmlPdpUpdateListener extends ScoListener { - private static final Logger LOGGER = LoggerFactory.getLogger(XacmlPdpStateChangeListener.class); + private static final Logger LOGGER = LoggerFactory.getLogger(XacmlPdpUpdateListener.class); private final XacmlState state; diff --git a/main/src/test/java/org/onap/policy/pdpx/main/XacmlStateTest.java b/main/src/test/java/org/onap/policy/pdpx/main/XacmlStateTest.java index 6d7b3c96..b09fe1a1 100644 --- a/main/src/test/java/org/onap/policy/pdpx/main/XacmlStateTest.java +++ b/main/src/test/java/org/onap/policy/pdpx/main/XacmlStateTest.java @@ -171,8 +171,8 @@ public class XacmlStateTest { assertEquals(SUBGROUP, status.getPdpSubgroup()); status = state.updateInternalState(req, "Failed to load policy: failLoadPolicy1: null"); - assertEquals(status.getResponse().getResponseMessage(), "Failed to load policy: failLoadPolicy1: null"); - assertEquals(status.getResponse().getResponseStatus(), PdpResponseStatus.FAIL); + assertEquals("Failed to load policy: failLoadPolicy1: null", status.getResponse().getResponseMessage()); + assertEquals(PdpResponseStatus.FAIL, status.getResponse().getResponseStatus()); assertEquals(GROUP, status.getPdpGroup()); } diff --git a/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java b/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java index a1f50771..3478ef3a 100644 --- a/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java +++ b/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java @@ -20,6 +20,7 @@ package org.onap.policy.pdpx.main.comm; +import static org.assertj.core.api.Assertions.assertThatCode; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertSame; import static org.mockito.Matchers.any; @@ -170,9 +171,11 @@ public class XacmlPdpHearbeatPublisherTest { // create a plain listener to test the "real" makeTimer() method publisher = new XacmlPdpHearbeatPublisher(client, state); - publisher.start(); - publisher.restart(100L); - publisher.terminate(); + assertThatCode( () -> { + publisher.start(); + publisher.restart(100L); + publisher.terminate(); + }).doesNotThrowAnyException(); } private class MyPublisher extends XacmlPdpHearbeatPublisher { diff --git a/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java b/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java index c05e0999..9f35f530 100644 --- a/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java +++ b/main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java @@ -20,6 +20,7 @@ package org.onap.policy.pdpx.main.comm; +import static org.assertj.core.api.Assertions.assertThatCode; import static org.mockito.Mockito.when; import org.junit.Before; @@ -54,18 +55,24 @@ public class XacmlPdpPapRegistrationTest { @Test public void testPdpRegistration_SendOk() throws TopicSinkClientException { - reg.pdpRegistration(status); + assertThatCode(() -> + reg.pdpRegistration(status) + ).doesNotThrowAnyException(); } @Test public void testPdpRegistration_SendFail() throws TopicSinkClientException { when(client.send(status)).thenReturn(false); - reg.pdpRegistration(status); + assertThatCode(() -> + reg.pdpRegistration(status) + ).doesNotThrowAnyException(); } @Test public void testPdpRegistration_SendEx() throws TopicSinkClientException { when(client.send(status)).thenThrow(new IllegalStateException()); - reg.pdpRegistration(status); + assertThatCode(() -> + reg.pdpRegistration(status) + ).doesNotThrowAnyException(); } } diff --git a/main/src/test/java/org/onap/policy/pdpx/main/startstop/TestMain.java b/main/src/test/java/org/onap/policy/pdpx/main/startstop/TestMain.java index 8b6889d6..5e46b669 100644 --- a/main/src/test/java/org/onap/policy/pdpx/main/startstop/TestMain.java +++ b/main/src/test/java/org/onap/policy/pdpx/main/startstop/TestMain.java @@ -21,6 +21,7 @@ package org.onap.policy.pdpx.main.startstop; +import static org.assertj.core.api.Assertions.assertThatCode; import static org.assertj.core.api.Assertions.assertThatThrownBy; import org.junit.After; @@ -51,6 +52,7 @@ public class TestMain extends CommonRest { CommonRest.stopMain(); } + @Override @Before public void setUp() { main = null; @@ -59,6 +61,7 @@ public class TestMain extends CommonRest { /** * Shuts "main" down. */ + @Override @After public void tearDown() { if (main != null) { @@ -69,9 +72,11 @@ public class TestMain extends CommonRest { @Test public void testMain() throws PolicyXacmlPdpException { final String[] xacmlPdpConfigParameters = {"-c", CONFIG_FILE}; - main = new Main(xacmlPdpConfigParameters); - main.shutdown(); - main = null; + assertThatCode(() -> { + main = new Main(xacmlPdpConfigParameters); + main.shutdown(); + main = null; + }).doesNotThrowAnyException(); } @Test -- cgit 1.2.3-korg