Code Review / openflowplugin.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Fix odlparent 3 Checkstyle issues
[openflowplugin.git] / openflowplugin-impl / src / test / java / org / opendaylight / openflowplugin / impl / common / MultipartRequestInputFactoryTest.java
diff --git a/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/common/MultipartRequestInputFactoryTest.java b/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/common/MultipartRequestInputFactoryTest.java
index 8eb0f9ff7d19fcbf23a6ec20330339a48a60f1fd..fa304791a092fc6b21b83a4796c3576f635c0a1f 100644 (file)
--- a/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/common/MultipartRequestInputFactoryTest.java
+++ b/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/common/MultipartRequestInputFactoryTest.java
@@ -43,7 +43,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
  */
 public class MultipartRequestInputFactoryTest {
 
-    private long xid = 42L;
+    private final long xid = 42L;
     private short ofVersion;
 
     @Before
@@ -56,8 +56,8 @@ public class MultipartRequestInputFactoryTest {
     public void testMakeMultipartRequestInput_DESC() throws Exception {
         MultipartType mpType = MultipartType.OFPMPDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestDescCase.class);
     }
 
@@ -65,8 +65,8 @@ public class MultipartRequestInputFactoryTest {
     public void testMakeMultipartRequestInput_FLOW_13() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPFLOW;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestFlowCase);
 
@@ -80,8 +80,8 @@ public class MultipartRequestInputFactoryTest {
         Assert.assertNull(mpRq.getMatchV10());
 
         final String EMPTY_MATCH_DUMP =
-                "Match [_type=class org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.OxmMatchType, " +
-                        "augmentation=[]]";
+                "Match [_type=class org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.OxmMatchType, "
+                        + "augmentation=[]]";
         Assert.assertEquals(EMPTY_MATCH_DUMP, String.valueOf(mpRq.getMatch()));
     }
 
@@ -90,8 +90,8 @@ public class MultipartRequestInputFactoryTest {
         final MultipartType mpType = MultipartType.OFPMPFLOW;
         ofVersion = OFConstants.OFP_VERSION_1_0;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestFlowCase);
 
@@ -106,29 +106,29 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_AGGREGATE() throws Exception {
+    public void testMakeMultipartRequestInputAggregate() throws Exception {
         MultipartType mpType = MultipartType.OFPMPAGGREGATE;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestAggregateCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_TABLE() throws Exception {
+    public void testMakeMultipartRequestInputTable() throws Exception {
         MultipartType mpType = MultipartType.OFPMPTABLE;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestTableCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_PORTSTATS() throws Exception {
+    public void testMakeMultipartRequestInputPortStats() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPPORTSTATS;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestPortStatsCase);
 
@@ -137,11 +137,11 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_QUEUE() throws Exception {
+    public void testMakeMultipartRequestInputQueue() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPQUEUE;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestQueueCase);
 
@@ -151,11 +151,11 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUP() throws Exception {
+    public void testMakeMultipartRequestInputGroup() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPGROUP;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestGroupCase);
 
@@ -164,29 +164,29 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUPDESC() throws Exception {
+    public void testMakeMultipartRequestInputGroupDesc() throws Exception {
         MultipartType mpType = MultipartType.OFPMPGROUPDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUPFEATURES() throws Exception {
+    public void testMakeMultipartRequestInputGroupFeatures() throws Exception {
         MultipartType mpType = MultipartType.OFPMPGROUPFEATURES;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_METER() throws Exception {
+    public void testMakeMultipartRequestInputMeter() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPMETER;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestMeterCase);
 
@@ -195,68 +195,71 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_METERCONFIG() throws Exception {
+    public void testMakeMultipartRequestInputMeterConfig() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPMETERCONFIG;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestMeterConfigCase);
 
-        MultipartRequestMeterConfig mpRq = ((MultipartRequestMeterConfigCase) mpRqBody).getMultipartRequestMeterConfig();
+        MultipartRequestMeterConfig mpRq =
+                ((MultipartRequestMeterConfigCase) mpRqBody).getMultipartRequestMeterConfig();
         Assert.assertEquals(OFConstants.OFPM_ALL, mpRq.getMeterId().getValue());
     }
 
     @Test
-    public void testMakeMultipartRequestInput_METERFEATURES() throws Exception {
+    public void testMakeMultipartRequestInputMeterFeatures() throws Exception {
         MultipartType mpType = MultipartType.OFPMPMETERFEATURES;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestMeterFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_TABLEFEATURES() throws Exception {
+    public void testMakeMultipartRequestInputTableFeatures() throws Exception {
         final MultipartType mpType = MultipartType.OFPMPTABLEFEATURES;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         final MultipartRequestBody mpRqBody = mpRqInput.getMultipartRequestBody();
         Assert.assertTrue(mpRqBody instanceof MultipartRequestTableFeaturesCase);
 
-        MultipartRequestTableFeatures mpRq = ((MultipartRequestTableFeaturesCase) mpRqBody).getMultipartRequestTableFeatures();
+        MultipartRequestTableFeatures mpRq =
+                ((MultipartRequestTableFeaturesCase) mpRqBody).getMultipartRequestTableFeatures();
         Assert.assertNotNull(mpRq);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_PORTDESC() throws Exception {
+    public void testMakeMultipartRequestInputPortDesc() throws Exception {
         MultipartType mpType = MultipartType.OFPMPPORTDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestPortDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_EXPERIMENTER() throws Exception {
+    public void testMakeMultipartRequestInputExperimenter() throws Exception {
         MultipartType mpType = MultipartType.OFPMPEXPERIMENTER;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
         checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestExperimenterCase.class);
     }
 
-    private void checkHeader(MultipartRequestInput mpRqInput, MultipartType mpType, short ofVersion) {
+    private void checkHeader(MultipartRequestInput mpRqInput, MultipartType mpType) {
         Assert.assertFalse(mpRqInput.getFlags().isOFPMPFREQMORE());
         Assert.assertEquals(ofVersion, mpRqInput.getVersion().shortValue());
         Assert.assertEquals(mpType, mpRqInput.getType());
         Assert.assertEquals(xid, mpRqInput.getXid().longValue());
     }
 
-    private void checkEmptyBody(MultipartRequestBody mpRqBody, Class<? extends MultipartRequestBody> expectedMpRqBodyClass) throws Exception {
+    private void checkEmptyBody(MultipartRequestBody mpRqBody, Class<? extends
+            MultipartRequestBody> expectedMpRqBodyClass) throws Exception {
         Assert.assertTrue(expectedMpRqBodyClass.isAssignableFrom(mpRqBody.getImplementedInterface()));
         final String expectedDump = expectedMpRqBodyClass.getSimpleName() + " [augmentation=[]]";
         Assert.assertEquals(expectedDump, String.valueOf(mpRqBody));
     }
-}
\ No newline at end of file
+}
OpenFlow Plugin