From 427cbbb9e25f843bcf06d75904f6294ac07180be Mon Sep 17 00:00:00 2001 From: Taka Cho Date: Thu, 2 Jul 2020 11:56:18 -0400 Subject: drools-pdp sonar fix most of changes are assert statement Issue-ID: POLICY-2616 Change-Id: I3bd3076115d28d8cfabcbe0859cc4b21085e1394 Signed-off-by: Taka Cho --- .../policy/drools/protocol/coders/ProtocolCoderToolsetTest.java | 6 +++--- .../java/org/onap/policy/drools/system/PolicyEngineManagerTest.java | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'policy-management/src') diff --git a/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java b/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java index 7a5ed2c0..6b497fa3 100644 --- a/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java +++ b/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java @@ -176,9 +176,9 @@ public class ProtocolCoderToolsetTest { tripleDecoded = (Triple) coderToolset.decode(tripleEncoded); - Assert.assertTrue(tripleDecoded.first().equals(triple.first())); - Assert.assertTrue(tripleDecoded.second().equals(triple.second())); - Assert.assertTrue(tripleDecoded.third().equals(triple.third())); + Assert.assertEquals(tripleDecoded.first(), triple.first()); + Assert.assertEquals(tripleDecoded.second(), triple.second()); + Assert.assertEquals(tripleDecoded.third(), triple.third()); coderFilters.getFilter().setRule("[?($.third =~ /.*v3.*/)]"); } diff --git a/policy-management/src/test/java/org/onap/policy/drools/system/PolicyEngineManagerTest.java b/policy-management/src/test/java/org/onap/policy/drools/system/PolicyEngineManagerTest.java index a999c718..f4876a95 100644 --- a/policy-management/src/test/java/org/onap/policy/drools/system/PolicyEngineManagerTest.java +++ b/policy-management/src/test/java/org/onap/policy/drools/system/PolicyEngineManagerTest.java @@ -519,7 +519,7 @@ public class PolicyEngineManagerTest { verify(prov1).beforeConfigure(mgr, properties); verify(prov2).beforeConfigure(mgr, properties); - assertTrue(mgr.getProperties() == properties); + assertSame(properties, mgr.getProperties()); assertEquals(sources, mgr.getSources()); assertEquals(sinks, mgr.getSinks()); @@ -553,7 +553,7 @@ public class PolicyEngineManagerTest { (prov, flag) -> when(prov.afterConfigure(mgr)).thenReturn(flag), () -> mgr.configure(properties), prov -> verify(prov).beforeConfigure(mgr, properties), - () -> assertTrue(mgr.getProperties() == properties), + () -> assertSame(properties, mgr.getProperties()), prov -> verify(prov).afterConfigure(mgr)); } -- cgit 1.2.3-korg