From 05506483dff28c491522beeaaea0f6904077e3ed Mon Sep 17 00:00:00 2001 From: Jim Hahn Date: Mon, 29 Jun 2020 14:13:14 -0400 Subject: Fix issues in xacml-pdp for new sonar rules Addressed issues reported due to updates to the sonar rules: - invoke only one method in a junit lambda - complete the assertion - use assertThat().exists(), canRead(), etc. Issue-ID: POLICY-2681 Change-Id: I2935f4d0295244ab3b11a57a3bc401c64b695fd1 Signed-off-by: Jim Hahn --- .../xacml/pdp/application/naming/NamingPdpApplicationTest.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'applications/naming') diff --git a/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java b/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java index 0077d0b9..85b182cd 100644 --- a/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java +++ b/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java @@ -176,7 +176,7 @@ public class NamingPdpApplicationTest { LOGGER.info("Decision {}", decision.getKey()); assertThat(decision.getKey()).isNotNull(); - assertThat(decision.getKey().getPolicies().size()).isEqualTo(0); + assertThat(decision.getKey().getPolicies()).isEmpty(); } @Test @@ -194,7 +194,7 @@ public class NamingPdpApplicationTest { // There is no default policy // assertThat(response).isNotNull(); - assertThat(response.getPolicies().size()).isEqualTo(0); + assertThat(response.getPolicies()).isEmpty(); // // Ask for VNF // @@ -204,7 +204,7 @@ public class NamingPdpApplicationTest { // response = makeDecision(); assertThat(response).isNotNull(); - assertThat(response.getPolicies().size()).isEqualTo(1); + assertThat(response.getPolicies()).hasSize(1); // // Validate it // @@ -227,7 +227,7 @@ public class NamingPdpApplicationTest { LOGGER.info("Decision Returned Policy {}", entrySet.getKey()); assertThat(entrySet.getValue()).isInstanceOf(Map.class); Map policyContents = (Map) entrySet.getValue(); - assertThat(policyContents.containsKey("properties")).isTrue(); + assertThat(policyContents).containsKey("properties"); assertThat(policyContents.get("properties")).isInstanceOf(Map.class); Map policyProperties = (Map) policyContents.get("properties"); -- cgit 1.2.3-korg