Mark potentially-static methods as static 46/91846/2
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 4 Aug 2020 11:14:40 +0000 (13:14 +0200)
committerHema Gopalakrishnan <hema.gopalkrishnan@ericsson.com>
Wed, 12 Aug 2020 18:37:40 +0000 (18:37 +0000)
Eclipse is pointing out a large number of warnings around methods
not being static. Fix these up.

Change-Id: Id8ff43ef4556b8e21aa01296e95af36747425489
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
30 files changed:
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/ericmatches/EricMatchInfoHelper.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/nxmatches/NxMatchInfoHelper.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/packet/IPv4.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/utils/clustering/EntityOwnershipUtils.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionLearnTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionLoadIpToSpaTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionLoadMacToShaTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionMoveSourceDestinationEthTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionMoveSourceDestinationIpTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionMoveSourceDestinationIpv6Test.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionNxLoadInPortTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionNxLoadMetadataTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetDestinationIpTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetFieldEthernetDestinationTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetFieldEthernetSourceTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetIcmpTypeTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetIcmpv6TypeTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetSourceIpTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetSourceIpv6Test.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetTcpDestinationPortTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetTcpSourcePortTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetUdpDestinationPortTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/actions/ActionSetUdpSourcePortTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/instructions/InstructionApplyActionsTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/instructions/InstructionClearActionsTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/instructions/InstructionGotoTableTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/instructions/InstructionWriteActionsTest.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/interfaces/testutils/FlowAssertTestUtils.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/interfaces/testutils/TestIMdsalApiManager.java
mdsalutil/mdsalutil-api/src/test/java/org/opendaylight/genius/mdsalutil/matches/MatchInPortTest.java

index 3032cc638b5513b8c9b9f8576582c222de244694..66bba53e74f7665e4a03f4d7c4f071fadb728e40 100644 (file)
@@ -63,7 +63,7 @@ public abstract class EricMatchInfoHelper<T extends DataObject, B extends Builde
         }
     }
 
-    private GeneralAugMatchNodesNodeTableFlow generalAugMatchBuilder(
+    private static GeneralAugMatchNodesNodeTableFlow generalAugMatchBuilder(
             GeneralAugMatchNodesNodeTableFlow existingAugmentations, EricAugMatchNodesNodeTableFlow ericAugMatch,
             Class<? extends ExtensionKey> extentionKey) {
         Map<ExtensionListKey, ExtensionList> extensions = null;
index 6ab62448c5914a7ed43986b02242496b90964d22..09b87ee54d86d03c535c4a16f18971acddcee40a 100644 (file)
@@ -65,7 +65,7 @@ public abstract class NxMatchInfoHelper<T extends DataObject, B extends Builder<
         }
     }
 
-    private GeneralAugMatchNodesNodeTableFlow generalAugMatchBuilder(
+    private static GeneralAugMatchNodesNodeTableFlow generalAugMatchBuilder(
             GeneralAugMatchNodesNodeTableFlow existingAugmentations, NxAugMatchNodesNodeTableFlow nxAugMatch,
             Class<? extends ExtensionKey> extentionKey) {
 
index 02feff1037aadd8a3efd27524818632e77990817..60575afa46438c278a1fd277cd0d0bc0a6b10745 100644 (file)
@@ -435,7 +435,7 @@ public class IPv4 extends Packet {
      * in IPv4 Header.
      * @return short
      */
-    private short generateId() {
+    private static short generateId() {
         Random randomgen = new Random();
         return (short) randomgen.nextInt(Short.MAX_VALUE + 1);
     }
index 64133a85631c48decfb7e97a70af021bbe120591..ba2b2af28ce624a871f9acd3a6f18ad6d933ad02 100644 (file)
@@ -147,7 +147,7 @@ public class EntityOwnershipUtils {
     }
 
     // see GENIUS-237 why we do this optimization instead of using Entity.toString()
-    private String getEntityIdentifierString(String entityType, String entityName) {
+    private static String getEntityIdentifierString(String entityType, String entityName) {
         return new StringBuilder(entityType.length() + entityName.length()).append(entityType).append(entityName)
                 .toString();
     }
index 3fa030ae08bf6f077947c46e076c2226059f1bff..da87578f7655f812dbcf7e2db20acbfb27143015 100644 (file)
@@ -41,7 +41,7 @@ public class ActionLearnTest {
     private static final int FIN_IDLE_TIMEOUT = 8;
     private static final int FIN_HARD_TIMEOUT = 9;
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionLearnNodesNodeTableFlowApplyActionsCase);
         NxActionLearnNodesNodeTableFlowApplyActionsCase nxAction =
             (NxActionLearnNodesNodeTableFlowApplyActionsCase) action.getAction();
@@ -118,7 +118,7 @@ public class ActionLearnTest {
         verifyAction(buildActionLearn().buildAction());
     }
 
-    private ActionLearn buildActionLearn() {
+    private static ActionLearn buildActionLearn() {
         return new ActionLearn(IDLE_TIMEOUT, HARD_TIMEOUT, PRIORITY, COOKIE, FLAGS, TABLE_ID, FIN_IDLE_TIMEOUT,
             FIN_HARD_TIMEOUT,
             Arrays.asList(
index d08b62684e280a694241fcdbc636dffab5bb0889..47a71e43a7f2757b539236fed176e13bc30b3662 100644 (file)
@@ -28,14 +28,14 @@ import org.opendaylight.yangtools.yang.common.Uint64;
 public class ActionLoadIpToSpaTest {
     private static final String IP_ADDRESS = "1.2.3.4";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionLoadIpToSpa(IP_ADDRESS).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegLoadNodesNodeTableFlowApplyActionsCase);
         NxActionRegLoadNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegLoadNodesNodeTableFlowApplyActionsCase) action.getAction();
index 26fdd8632452da384e562606e4c2624d7072e02f..8249af3c5b58860ec52ebbd33dd2f54728814a22 100644 (file)
@@ -29,14 +29,14 @@ import org.opendaylight.yangtools.yang.common.Uint64;
 public class ActionLoadMacToShaTest {
     private static final String MAC_ADDRESS = "11:22:33:44:55:66";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionLoadMacToSha(new MacAddress(MAC_ADDRESS)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegLoadNodesNodeTableFlowApplyActionsCase);
         NxActionRegLoadNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegLoadNodesNodeTableFlowApplyActionsCase) action.getAction();
@@ -54,7 +54,7 @@ public class ActionLoadMacToShaTest {
     @Test
     public void generateAction() {
         ActionInfo actionInfo = new ActionLoadMacToSha(new MacAddress(MAC_ADDRESS));
-        assertEquals(("MacAddress{_value=" + MAC_ADDRESS + "}"),
+        assertEquals("MacAddress{_value=" + MAC_ADDRESS + "}",
                 ((ActionLoadMacToSha) actionInfo).getAddress().toString());
 
     }
index f8fced838339b4a916a94b09ddbb127ad834c1b7..13c7310a42cd5977cac2b5f15f1702074e0c2f23 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yangtools.yang.common.Empty;
  * Test class for {@link ActionMoveSourceDestinationEth}.
  */
 public class ActionMoveSourceDestinationEthTest {
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionMoveSourceDestinationEth().buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegMoveNodesNodeTableFlowApplyActionsCase);
         NxActionRegMoveNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegMoveNodesNodeTableFlowApplyActionsCase) action.getAction();
index f74404b124e86ec411f110d5e0d2004cdc8477e4..1093cf543bcad58f0f9a1c9c049064a8245d58c5 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yangtools.yang.common.Empty;
  * Test class for {@link ActionMoveSourceDestinationIp}.
  */
 public class ActionMoveSourceDestinationIpTest {
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionMoveSourceDestinationIp().buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegMoveNodesNodeTableFlowApplyActionsCase);
         NxActionRegMoveNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegMoveNodesNodeTableFlowApplyActionsCase) action.getAction();
index cd7b31dc7783046c79601b1b35dafeed51fbb657..ad76801f096f3156fceddc80123e752efebdfd9d 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yangtools.yang.common.Empty;
  * Test class for {@link ActionMoveSourceDestinationIpv6}.
  */
 public class ActionMoveSourceDestinationIpv6Test {
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionMoveSourceDestinationIpv6().buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegMoveNodesNodeTableFlowApplyActionsCase);
         NxActionRegMoveNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegMoveNodesNodeTableFlowApplyActionsCase) action.getAction();
index b4b1049c57b5ba2fad9cbe35f02132609097e492..2f163119408d16c3b4c71d89db3fd351d378360e 100644 (file)
@@ -35,7 +35,7 @@ public class ActionNxLoadInPortTest {
         verifyAction(new ActionNxLoadInPort(VALUE).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegLoadNodesNodeTableFlowApplyActionsCase);
         NxActionRegLoadNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegLoadNodesNodeTableFlowApplyActionsCase) action.getAction();
index 7a62dd09630b39e734053c56b8b38fa74af40d23..79516f694c352daead224897a3a9e17b9f6cbbd4 100644 (file)
@@ -38,7 +38,7 @@ public class ActionNxLoadMetadataTest {
         verifyAction(new ActionNxLoadMetadata(VALUE, START, END).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof NxActionRegLoadNodesNodeTableFlowApplyActionsCase);
         NxActionRegLoadNodesNodeTableFlowApplyActionsCase actionCase =
             (NxActionRegLoadNodesNodeTableFlowApplyActionsCase) action.getAction();
index cf4daee3123c8530608cc1e77b8dc05038cddc26..1f05cbf6de455f7062fa0244c104b1f170e0d26d 100644 (file)
@@ -26,7 +26,7 @@ public class ActionSetDestinationIpTest {
     private static final String IP_ADDRESS = "1.2.3.4";
     private static final String IP_MASK = "32";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
@@ -35,7 +35,7 @@ public class ActionSetDestinationIpTest {
         verifyAction(new ActionSetDestinationIp(new Ipv4Prefix(IP_ADDRESS + "/" + IP_MASK)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer3Match());
@@ -47,7 +47,7 @@ public class ActionSetDestinationIpTest {
     @Test
     public void generateAction() {
         ActionInfo actionInfo = new ActionSetDestinationIp(IP_ADDRESS);
-        assertEquals(("Ipv4Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}"),
+        assertEquals("Ipv4Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}",
                 ((ActionSetDestinationIp) actionInfo).getDestination().toString());
     }
 }
index 1c062466e5a4c7e41cafadc3b408b7d76a5f0ffc..ac800d62940e594524249c9c290202a51a2b71ab 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.acti
 public class ActionSetFieldEthernetDestinationTest {
     private static final String MAC_ADDRESS = "11:22:33:44:55:66";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetFieldEthernetDestination(new MacAddress(MAC_ADDRESS)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getEthernetMatch());
@@ -44,7 +44,7 @@ public class ActionSetFieldEthernetDestinationTest {
     public void generateAction() {
         ActionInfo actionInfo = new ActionSetFieldEthernetDestination(new MacAddress(MAC_ADDRESS));
         actionInfo.buildAction();
-        assertEquals(("MacAddress{_value=" + MAC_ADDRESS + "}"),
+        assertEquals("MacAddress{_value=" + MAC_ADDRESS + "}",
                 ((ActionSetFieldEthernetDestination) actionInfo).getDestination().toString());
     }
 }
index 454eb5cf61a6b80ba70c7e664f84cfa58d1659aa..9077e05fc4356290f60057a002b5891be0e29ea6 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.acti
 public class ActionSetFieldEthernetSourceTest {
     private static final String MAC_ADDRESS = "11:22:33:44:55:66";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetFieldEthernetSource(new MacAddress(MAC_ADDRESS)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getEthernetMatch());
@@ -44,7 +44,7 @@ public class ActionSetFieldEthernetSourceTest {
     public void generateAction() {
         ActionInfo actionInfo = new ActionSetFieldEthernetSource(new MacAddress(MAC_ADDRESS));
         actionInfo.buildAction();
-        assertEquals(("MacAddress{_value=" + MAC_ADDRESS + "}"),
+        assertEquals("MacAddress{_value=" + MAC_ADDRESS + "}",
                 ((ActionSetFieldEthernetSource) actionInfo).getSource().toString());
     }
 }
index acbc2475bbeef8e2e802d7ff0dc927273169bf61..da92b35e6f55b4606f79ae0d95a9e41f1a3391cc 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetIcmpTypeTest {
     private static final short TYPE = (short) 2;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetIcmpType(TYPE).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getIcmpv4Match());
index 70fb4a0169c2fcd1d2d9c463ac96682f199927f1..6e3a5c03325b6f90591917eaa8177d3a25c050e0 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetIcmpv6TypeTest {
     private static final short TYPE = (short) 129;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetIcmpv6Type(TYPE).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getIcmpv6Match());
index f987361705f3cc0a6ffdcc82becb3a86e6d53897..d83332b324eafdbdad30ab3e22c928f45f5dc59a 100644 (file)
@@ -26,7 +26,7 @@ public class ActionSetSourceIpTest {
     private static final String IP_ADDRESS = "1.2.3.4";
     private static final String IP_MASK = "32";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
@@ -35,7 +35,7 @@ public class ActionSetSourceIpTest {
         verifyAction(new ActionSetSourceIp(new Ipv4Prefix(IP_ADDRESS + "/" + IP_MASK)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer3Match());
@@ -47,7 +47,7 @@ public class ActionSetSourceIpTest {
     @Test
     public void generateAction() {
         ActionInfo actionInfo = new ActionSetSourceIp(IP_ADDRESS);
-        assertEquals(("Ipv4Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}"),
+        assertEquals("Ipv4Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}",
                ((ActionSetSourceIp) actionInfo).getSource().toString());
     }
 }
index ec3cca6ca010f1a7940722075eaebaf61ee32331..11edb1d32d499144aff45d08c9fce4b77d61462d 100644 (file)
@@ -26,7 +26,7 @@ public class ActionSetSourceIpv6Test {
     private static final String IP_ADDRESS = "2001:db8::1";
     private static final String IP_MASK = "128";
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
@@ -35,7 +35,7 @@ public class ActionSetSourceIpv6Test {
         verifyAction(new ActionSetSourceIpv6(new Ipv6Prefix(IP_ADDRESS + "/" + IP_MASK)).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer3Match());
@@ -47,7 +47,7 @@ public class ActionSetSourceIpv6Test {
     @Test
     public void generateAction() {
         ActionInfo actionInfo = new ActionSetSourceIpv6(IP_ADDRESS);
-        assertEquals(("Ipv6Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}"),
+        assertEquals("Ipv6Prefix{_value=" + IP_ADDRESS + "/" + IP_MASK + "}",
                 ((ActionSetSourceIpv6) actionInfo).getSource().toString());
     }
 }
index 44f643a861f63fdd991ace16aee2c158510319b6..c796aa93b7ba752135b3107036e1418614eb8adf 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetTcpDestinationPortTest {
     private static final int PORT = 22;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetTcpDestinationPort(PORT).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer4Match());
index 2087d3ecd37d2fbb066363bbec3645d011234565..9d4b1706ed818cc52344ab82908caecad8915ff2 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetTcpSourcePortTest {
     private static final int PORT = 22;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetTcpSourcePort(PORT).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer4Match());
index f1c90f3cd7c07d8684f0fb41536e606159cf9c7c..158dd65fdb6f4461a0f49e8d150a26898239506c 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetUdpDestinationPortTest {
     private static final int PORT = 22;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetUdpDestinationPort(PORT).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer4Match());
index 64de9fd0478ce5f711c061c971e1c90b55036527..e301acd7455d08df3e1a820ffaba6fabbb139771 100644 (file)
@@ -24,14 +24,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 public class ActionSetUdpSourcePortTest {
     private static final int PORT = 22;
 
-    private XtendBeanGenerator generator = new XtendBeanGenerator();
+    private final XtendBeanGenerator generator = new XtendBeanGenerator();
 
     @Test
     public void actionInfoTest() {
         verifyAction(new ActionSetUdpSourcePort(PORT).buildAction());
     }
 
-    private void verifyAction(Action action) {
+    private static void verifyAction(Action action) {
         assertTrue(action.getAction() instanceof SetFieldCase);
         SetFieldCase actionCase = (SetFieldCase) action.getAction();
         assertNotNull(actionCase.getSetField().getLayer4Match());
index 8cb530756820be50914bb2663d932ce2db6db959..cb77caea59a02beeeec1e00121b078c301989dc3 100644 (file)
@@ -35,7 +35,7 @@ public class InstructionApplyActionsTest {
         verifyInstructionInfo(INSTRUCTION_INFO);
     }
 
-    private void verifyInstructionInfo(InstructionInfo instructionInfo) {
+    private static void verifyInstructionInfo(InstructionInfo instructionInfo) {
         Instruction instruction = instructionInfo.buildInstruction(2);
         assertEquals(2, instruction.key().getOrder().intValue());
         assertTrue(instruction.getInstruction() instanceof ApplyActionsCase);
index 717640055876f462634de180fdc1dc73a515d04d..c0ecb246968efe5d7488d3af428c691a9dff7af5 100644 (file)
@@ -28,7 +28,7 @@ public class InstructionClearActionsTest {
         verifyInstructionInfo(INSTRUCTION_INFO);
     }
 
-    private void verifyInstructionInfo(InstructionInfo instructionInfo) {
+    private static void verifyInstructionInfo(InstructionInfo instructionInfo) {
         Instruction instruction = instructionInfo.buildInstruction(2);
         assertEquals(2, instruction.key().getOrder().intValue());
         assertTrue(instruction.getInstruction() instanceof ClearActionsCase);
index 6fa8b4ac7add81368fdc43d121267c2cf15a84d9..7f4f7c373a96346c3b33470b713b9266d5eb683f 100644 (file)
@@ -28,7 +28,7 @@ public class InstructionGotoTableTest {
         verifyInstructionInfo(INSTRUCTION_INFO);
     }
 
-    private void verifyInstructionInfo(InstructionInfo instructionInfo) {
+    private static void verifyInstructionInfo(InstructionInfo instructionInfo) {
         Instruction instruction = instructionInfo.buildInstruction(2);
         assertEquals(2, instruction.key().getOrder().intValue());
         assertTrue(instruction.getInstruction() instanceof GoToTableCase);
index d2b0e35606772683494eb2f0966b4d3195b0bdbf..aadde89088b0ef93f47c8c3f6252075bcfe9c359 100644 (file)
@@ -35,7 +35,7 @@ public class InstructionWriteActionsTest {
         verifyInstructionInfo(INSTRUCTION_INFO);
     }
 
-    private void verifyInstructionInfo(InstructionInfo instructionInfo) {
+    private static void verifyInstructionInfo(InstructionInfo instructionInfo) {
         Instruction instruction = instructionInfo.buildInstruction(2);
         assertEquals(2, instruction.key().getOrder().intValue());
         assertTrue(instruction.getInstruction() instanceof WriteActionsCase);
index 6688aa35ed7c4c38128d7b4c31231f649093a152..b056668368b98f0340881d1cc5e82a53ea74f60f 100644 (file)
@@ -36,7 +36,7 @@ public class FlowAssertTestUtils {
         assertEqualBeans(rebuildFlow(expected), rebuildFlow(actual));
     }
 
-    private Flow rebuildFlow(Flow flow) {
+    private static Flow rebuildFlow(Flow flow) {
         FlowBuilder flowBuilder = new FlowBuilder(flow);
         InstructionsBuilder instructionsBuilder = new InstructionsBuilder();
         if (flow.getInstructions() != null) {
index d85564c0f42bcffdf90cdc755818192c3f2a0646..5533ccb85a69d192865ce516200a7dca3de7b714 100644 (file)
@@ -170,7 +170,7 @@ public abstract class TestIMdsalApiManager implements IMdsalApiManager {
         }
     }
 
-    private List<FlowEntity> sortFlows(Iterable<FlowEntity> flowsToSort) {
+    private static List<FlowEntity> sortFlows(Iterable<FlowEntity> flowsToSort) {
         List<FlowEntity> sortedFlows = Lists.newArrayList(flowsToSort);
         sortedFlows.sort((flow1, flow2) -> ComparisonChain.start()
                 .compare(flow1.getTableId(), flow2.getTableId())
index e59b9ae38c1e891cd8cae66de750215dd9160bf6..4551188f69b816b3b187ad020eceb22912c8fb89 100644 (file)
@@ -21,12 +21,11 @@ public class MatchInPortTest {
             .testEquals();
     }
 
-    private MatchInPort newMatchInPort1() {
+    private static MatchInPort newMatchInPort1() {
         return new MatchInPort(Uint64.ONE, 123);
     }
 
-    private MatchInPort newMatchInPort2() {
+    private static MatchInPort newMatchInPort2() {
         return new MatchInPort(Uint64.valueOf(10), 456);
     }
-
 }