diff options
author | Pamela Dragosh <pdragosh@research.att.com> | 2019-12-02 14:38:05 -0500 |
---|---|---|
committer | Pamela Dragosh <pdragosh@research.att.com> | 2019-12-02 15:09:09 -0500 |
commit | 3105cfbfb817444ebbfec8a726b8526e7ff38632 (patch) | |
tree | cda90d42e538a87d92d6e79de27ff6fe253d5717 /applications/common/src/test | |
parent | e53be640b3a4c37dbd841c81b5c467082986c05b (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 'applications/common/src/test')
-rw-r--r-- | applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/operationshistory/CountRecentOperationsPipTest.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/operationshistory/CountRecentOperationsPipTest.java b/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/operationshistory/CountRecentOperationsPipTest.java index b01fa708..ac8a0c29 100644 --- a/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/operationshistory/CountRecentOperationsPipTest.java +++ b/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/operationshistory/CountRecentOperationsPipTest.java @@ -18,6 +18,7 @@ package org.onap.policy.pdp.xacml.application.common.operationshistory; +import static org.assertj.core.api.Assertions.assertThatCode; import static org.junit.Assert.assertEquals; import static org.mockito.Matchers.any; import static org.mockito.Matchers.eq; @@ -167,7 +168,9 @@ public class CountRecentOperationsPipTest { @Test public void testConfigure_DbException() throws Exception { properties.put("javax.persistence.jdbc.url", "invalid"); - pipEngine.configure("issuer", properties); + assertThatCode(() -> + pipEngine.configure("issuer", properties) + ).doesNotThrowAnyException(); } @Test |