Fix sonar issues
- Move string literals to the left side of comparison
- Fix a typo
Change-Id: I3e59cde5d73d99d918b51fa5bcaa7c737ed7f525
diff --git a/apps/test/distributed-primitives/src/main/java/org/onosproject/distributedprimitives/cli/ValueTestCommand.java b/apps/test/distributed-primitives/src/main/java/org/onosproject/distributedprimitives/cli/ValueTestCommand.java
index aa17ca4..afb39ad 100644
--- a/apps/test/distributed-primitives/src/main/java/org/onosproject/distributedprimitives/cli/ValueTestCommand.java
+++ b/apps/test/distributed-primitives/src/main/java/org/onosproject/distributedprimitives/cli/ValueTestCommand.java
@@ -60,15 +60,15 @@
.withSerializer(Serializer.using(KryoNamespaces.BASIC))
.build()
.asAtomicValue();
- if (operation.equals("get")) {
+ if ("get".equals(operation)) {
print("%s", atomicValue.get());
- } else if (operation.equals("set")) {
+ } else if ("set".equals(operation)) {
atomicValue.set(value1);
- } else if (operation.equals("compareAndSet")) {
+ } else if ("compareAndSet".equals(operation)) {
print("%b", atomicValue.compareAndSet(value1, value2));
- } else if (operation.equals("getAndSet")) {
+ } else if ("getAndSet".equals(operation)) {
print("%s", atomicValue.getAndSet(value1));
- } else if (operation.equals("destroy")) {
+ } else if ("destroy".equals(operation)) {
atomicValue.destroy();
} else {
print("Error, unknown operation %s", operation);