diff options
author | 2020-06-10 22:16:48 +0000 | |
---|---|---|
committer | 2020-06-10 22:16:48 +0000 | |
commit | 6d11b115301763cc66d011f00236ccedd68ba820 (patch) | |
tree | 9516f0be13aed09be0370fd4dbdc95788c4bcc26 /controlloop/common/eventmanager/src | |
parent | dcc7cac5f17dacf9a58b18b518574391cdc73925 (diff) | |
parent | fbd144a2442fa7273c8b1d920cd2f7611b2aefcd (diff) |
Merge "reduce sonar issue - drool-applications"
Diffstat (limited to 'controlloop/common/eventmanager/src')
2 files changed, 17 insertions, 17 deletions
diff --git a/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopEventManagerTest.java b/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopEventManagerTest.java index db16c4528..2d9fb7e1d 100644 --- a/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopEventManagerTest.java +++ b/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopEventManagerTest.java @@ -647,8 +647,8 @@ public class ControlLoopEventManagerTest { VirtualControlLoopEvent onsetEvent = getOnsetEvent(); ControlLoopEventManager manager = makeManager(onsetEvent); - assertTrue(0 == manager.getControlLoopTimeout(null)); - assertTrue(120 == manager.getControlLoopTimeout(120)); + assertEquals(0, manager.getControlLoopTimeout(null)); + assertEquals(120, manager.getControlLoopTimeout(120)); InputStream is = new FileInputStream(new File(TEST_YAML)); final String yamlString = IOUtils.toString(is, StandardCharsets.UTF_8); @@ -673,8 +673,8 @@ public class ControlLoopEventManagerTest { assertNotNull(notification); assertEquals(ControlLoopNotificationType.ACTIVE, notification.getNotification()); - assertTrue(0 == manager.getControlLoopTimeout(null)); - assertTrue(120 == manager.getControlLoopTimeout(120)); + assertEquals(0, manager.getControlLoopTimeout(null)); + assertEquals(120, manager.getControlLoopTimeout(120)); } @Test @@ -690,8 +690,8 @@ public class ControlLoopEventManagerTest { assertNotNull(notification); assertEquals(ControlLoopNotificationType.ACTIVE, notification.getNotification()); - assertTrue(0 == manager.getControlLoopTimeout(null)); - assertTrue(120 == manager.getControlLoopTimeout(120)); + assertEquals(0, manager.getControlLoopTimeout(null)); + assertEquals(120, manager.getControlLoopTimeout(120)); } @Test diff --git a/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopOperationManagerTest.java b/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopOperationManagerTest.java index 4cb365a76..b64d9484e 100644 --- a/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopOperationManagerTest.java +++ b/controlloop/common/eventmanager/src/test/java/org/onap/policy/controlloop/eventmanager/ControlLoopOperationManagerTest.java @@ -226,7 +226,7 @@ public class ControlLoopOperationManagerTest { // PolicyResult result = manager.onResponse(dmaapResponse); logger.debug("{}", manager); - assertTrue(result == null); + assertNull(result); assertFalse(manager.isOperationComplete()); assertTrue(manager.isOperationRunning()); // @@ -239,7 +239,7 @@ public class ControlLoopOperationManagerTest { dmaapResponse.setBody(outputBody); result = manager.onResponse(dmaapResponse); logger.debug("{}", manager); - assertTrue(result.equals(PolicyResult.FAILURE)); + assertEquals(PolicyResult.FAILURE, result); assertFalse(manager.isOperationComplete()); assertFalse(manager.isOperationRunning()); // @@ -268,7 +268,7 @@ public class ControlLoopOperationManagerTest { // result = manager.onResponse(dmaapResponse); logger.debug("{}", manager); - assertTrue(result == null); + assertNull(result); assertFalse(manager.isOperationComplete()); assertTrue(manager.isOperationRunning()); // @@ -281,15 +281,15 @@ public class ControlLoopOperationManagerTest { dmaapResponse.setBody(outputBody); result = manager.onResponse(dmaapResponse); logger.debug("{}", manager); - assertTrue(result.equals(PolicyResult.FAILURE)); + assertEquals(PolicyResult.FAILURE, result); // // Should be complete now // assertTrue(manager.isOperationComplete()); assertFalse(manager.isOperationRunning()); assertNotNull(manager.getOperationResult()); - assertTrue(manager.getOperationResult().equals(PolicyResult.FAILURE_RETRIES)); - assertTrue(manager.getHistory().size() == 2); + assertEquals(PolicyResult.FAILURE_RETRIES, manager.getOperationResult()); + assertEquals(2, manager.getHistory().size()); } @Test @@ -354,7 +354,7 @@ public class ControlLoopOperationManagerTest { // PolicyResult result = manager.onResponse(dmaapResponse); logger.debug("{}", manager); - assertTrue(result == null); + assertNull(result); assertFalse(manager.isOperationComplete()); assertTrue(manager.isOperationRunning()); // @@ -364,8 +364,8 @@ public class ControlLoopOperationManagerTest { logger.debug("{}", manager); assertTrue(manager.isOperationComplete()); assertFalse(manager.isOperationRunning()); - assertTrue(manager.getHistory().size() == 1); - assertTrue(manager.getOperationResult().equals(PolicyResult.FAILURE_TIMEOUT)); + assertEquals(1, manager.getHistory().size()); + assertEquals(PolicyResult.FAILURE_TIMEOUT, manager.getOperationResult()); // // Now we are going to Fail the previous request // @@ -381,8 +381,8 @@ public class ControlLoopOperationManagerTest { // assertTrue(manager.isOperationComplete()); assertFalse(manager.isOperationRunning()); - assertTrue(manager.getHistory().size() == 1); - assertTrue(manager.getOperationResult().equals(PolicyResult.FAILURE_TIMEOUT)); + assertEquals(1, manager.getHistory().size()); + assertEquals(PolicyResult.FAILURE_TIMEOUT, manager.getOperationResult()); } @Test |