aboutsummaryrefslogtreecommitdiffstats
path: root/main/src/test
diff options
context:
space:
mode:
authorPamela Dragosh <pdragosh@research.att.com>2019-12-02 14:38:05 -0500
committerPamela Dragosh <pdragosh@research.att.com>2019-12-02 15:09:09 -0500
commit3105cfbfb817444ebbfec8a726b8526e7ff38632 (patch)
treecda90d42e538a87d92d6e79de27ff6fe253d5717 /main/src/test
parente53be640b3a4c37dbd841c81b5c467082986c05b (diff)
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 <pdragosh@research.att.com>
Diffstat (limited to 'main/src/test')
-rw-r--r--main/src/test/java/org/onap/policy/pdpx/main/XacmlStateTest.java4
-rw-r--r--main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpHearbeatPublisherTest.java9
-rw-r--r--main/src/test/java/org/onap/policy/pdpx/main/comm/XacmlPdpPapRegistrationTest.java13
-rw-r--r--main/src/test/java/org/onap/policy/pdpx/main/startstop/TestMain.java11
4 files changed, 26 insertions, 11 deletions
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