Merge "Remove deprecated properties for CDS and VFC"
diff --git a/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java b/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
index eeb382a..a19006e 100644
--- a/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
+++ b/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
@@ -187,7 +187,7 @@
@Test
public void getSequenceNumber() {
ControlLoopMetricsFeature feature = new ControlLoopMetricsFeature();
- assertEquals(feature.getSequenceNumber(), ControlLoopMetricsFeature.FEATURE_SEQUENCE_PRIORITY);
+ assertEquals(ControlLoopMetricsFeature.FEATURE_SEQUENCE_PRIORITY, feature.getSequenceNumber());
}
@Test
diff --git a/controlloop/m2/adapters/src/test/java/org/onap/policy/m2/adapters/VirtualOnsetAdapterTest.java b/controlloop/m2/adapters/src/test/java/org/onap/policy/m2/adapters/VirtualOnsetAdapterTest.java
index 0ecbda6..c19a80e 100644
--- a/controlloop/m2/adapters/src/test/java/org/onap/policy/m2/adapters/VirtualOnsetAdapterTest.java
+++ b/controlloop/m2/adapters/src/test/java/org/onap/policy/m2/adapters/VirtualOnsetAdapterTest.java
@@ -21,6 +21,7 @@
package org.onap.policy.m2.adapters;
import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
import org.junit.Test;
@@ -39,15 +40,15 @@
VirtualControlLoopEvent virtualControlLoopEvent = new VirtualControlLoopEvent();
VirtualOnsetAdapter virtualOnsetAdapter =
VirtualOnsetAdapter.class.cast(OnsetAdapter.get(virtualControlLoopEvent));
- assertTrue(virtualOnsetAdapter != null);
+ assertNotNull(virtualOnsetAdapter);
ControlLoopNotification notification = virtualOnsetAdapter.createNotification(virtualControlLoopEvent);
- assertTrue(notification != null);
+ assertNotNull(notification);
// we want an exact class match, so 'instanceOf' is not being used
assertEquals(VirtualControlLoopNotification.class, notification.getClass());
ControlLoopEvent controlLoopEvent = new ControlLoopEvent() {};
notification = virtualOnsetAdapter.createNotification(controlLoopEvent);
- assertTrue(notification != null);
+ assertNotNull(notification);
}
}
diff --git a/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmHealthCheckOperation.java b/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmHealthCheckOperation.java
index 42e06f9..cc0df45 100644
--- a/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmHealthCheckOperation.java
+++ b/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmHealthCheckOperation.java
@@ -28,7 +28,6 @@
import org.onap.policy.common.utils.coder.CoderException;
import org.onap.policy.controlloop.ControlLoopEvent;
import org.onap.policy.controlloop.ControlLoopException;
-import org.onap.policy.controlloop.VirtualControlLoopEvent;
import org.onap.policy.controlloop.policy.Policy;
import org.onap.policy.controlloop.policy.PolicyResult;
import org.onap.policy.guard.PolicyGuardResponse;
@@ -88,7 +87,6 @@
if (!(object instanceof AppcLcmDmaapWrapper)) {
if (object instanceof PolicyGuardResponse) {
incomingGuardMessage((PolicyGuardResponse) object);
- return;
}
// ignore this message (not sure why we even got it)
return;
diff --git a/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmOperation.java b/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmOperation.java
index 6a2518f..668860e 100644
--- a/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmOperation.java
+++ b/controlloop/m2/appclcm/src/main/java/org/onap/policy/m2/appclcm/AppcLcmOperation.java
@@ -245,7 +245,7 @@
* the lock, but it has now became available.
*/
public void lockAvailable() {
- if (this.state == LCM_WAIT_FOR_LOCK) {
+ if (LCM_WAIT_FOR_LOCK.equals(this.state)) {
// we have the lock -- invoke 'quardQuery()',
// go to the appropriate state, and mark the transaction as modified
guardQuery();
@@ -265,7 +265,7 @@
* obtained.
*/
public void lockUnavailable() {
- if (this.state == LCM_WAIT_FOR_LOCK) {
+ if (LCM_WAIT_FOR_LOCK.equals(this.state)) {
try {
setErrorStatus("Already processing event with this target");
} catch (ControlLoopException e) {
@@ -314,7 +314,6 @@
payload = setRebootPayload();
break;
default:
- payload = null;
break;
}
@@ -485,7 +484,7 @@
void incomingGuardMessage(PolicyGuardResponse response) {
// this message is only meaningful if we are waiting for a
// 'guard' response -- ignore it, if this isn't the case
- if (this.state == LCM_GUARD_PENDING) {
+ if (LCM_GUARD_PENDING.equals(this.state)) {
if ("Deny".equals(response.getResult())) {
// this is a guard failure
logger.error("LCM operation denied by 'Guard'");
@@ -510,10 +509,8 @@
if (! (object instanceof AppcLcmDmaapWrapper)) {
if (object instanceof PolicyGuardResponse) {
incomingGuardMessage((PolicyGuardResponse)object);
- return;
} else if (object instanceof ControlLoopEvent) {
incomingAbatedEvent((ControlLoopEvent) object);
- return;
}
// ignore this message (not sure why we even got it)
return;