Code Review / openflowplugin.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Merge "Add INFO.yaml for openflowplugin"
[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..82237057ff38192d8e586a9247f8ae17c838f240 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
@@ -5,7 +5,6 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.openflowplugin.impl.common;
 
 import org.junit.Assert;
@@ -13,6 +12,7 @@ import org.junit.Before;
 import org.junit.Test;
 import org.opendaylight.openflowplugin.api.OFConstants;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.common.types.rev130731.MultipartType;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.OxmMatchType;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.MultipartRequestInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.multipart.request.MultipartRequestBody;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.multipart.request.multipart.request.body.MultipartRequestAggregateCase;
@@ -37,36 +37,37 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.multipart.request.multipart.request.body.multipart.request.port.stats._case.MultipartRequestPortStats;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.multipart.request.multipart.request.body.multipart.request.queue._case.MultipartRequestQueue;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.multipart.request.multipart.request.body.multipart.request.table.features._case.MultipartRequestTableFeatures;
+import org.opendaylight.yangtools.yang.binding.DataContainer;
 
 /**
  * Test for {@link MultipartRequestInputFactory}.
  */
 public class MultipartRequestInputFactoryTest {
 
-    private long xid = 42L;
+    private final long xid = 42L;
     private short ofVersion;
 
     @Before
 
-    public void setUp() throws Exception {
+    public void setUp() {
         ofVersion = OFConstants.OFP_VERSION_1_3;
     }
 
     @Test
-    public void testMakeMultipartRequestInput_DESC() throws Exception {
+    public void testMakeMultipartRequestInput_DESC() {
         MultipartType mpType = MultipartType.OFPMPDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestDescCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(), MultipartRequestDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_FLOW_13() throws Exception {
+    public void testMakeMultipartRequestInput_FLOW_13() {
         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);
 
@@ -79,19 +80,16 @@ public class MultipartRequestInputFactoryTest {
         Assert.assertNotNull(mpRq.getMatch());
         Assert.assertNull(mpRq.getMatchV10());
 
-        final String EMPTY_MATCH_DUMP =
-                "Match [_type=class org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.OxmMatchType, " +
-                        "augmentation=[]]";
-        Assert.assertEquals(EMPTY_MATCH_DUMP, String.valueOf(mpRq.getMatch()));
+        Assert.assertEquals(OxmMatchType.class, mpRq.getMatch().getType());
     }
 
     @Test
-    public void testMakeMultipartRequestInput_FLOW_10() throws Exception {
+    public void testMakeMultipartRequestInput_FLOW_10() {
         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 +104,30 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_AGGREGATE() throws Exception {
+    public void testMakeMultipartRequestInputAggregate() {
         MultipartType mpType = MultipartType.OFPMPAGGREGATE;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestAggregateCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestAggregateCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_TABLE() throws Exception {
+    public void testMakeMultipartRequestInputTable() {
         MultipartType mpType = MultipartType.OFPMPTABLE;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestTableCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(), MultipartRequestTableCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_PORTSTATS() throws Exception {
+    public void testMakeMultipartRequestInputPortStats() {
         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 +136,11 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_QUEUE() throws Exception {
+    public void testMakeMultipartRequestInputQueue() {
         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 +150,11 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUP() throws Exception {
+    public void testMakeMultipartRequestInputGroup() {
         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 +163,31 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUPDESC() throws Exception {
+    public void testMakeMultipartRequestInputGroupDesc() {
         MultipartType mpType = MultipartType.OFPMPGROUPDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupDescCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestGroupDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_GROUPFEATURES() throws Exception {
+    public void testMakeMultipartRequestInputGroupFeatures() {
         MultipartType mpType = MultipartType.OFPMPGROUPFEATURES;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupFeaturesCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestGroupFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_METER() throws Exception {
+    public void testMakeMultipartRequestInputMeter() {
         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 +196,73 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInput_METERCONFIG() throws Exception {
+    public void testMakeMultipartRequestInputMeterConfig() {
         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() {
         MultipartType mpType = MultipartType.OFPMPMETERFEATURES;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestMeterFeaturesCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestMeterFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_TABLEFEATURES() throws Exception {
+    public void testMakeMultipartRequestInputTableFeatures() {
         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() {
         MultipartType mpType = MultipartType.OFPMPPORTDESC;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestPortDescCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestPortDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInput_EXPERIMENTER() throws Exception {
+    public void testMakeMultipartRequestInputExperimenter() {
         MultipartType mpType = MultipartType.OFPMPEXPERIMENTER;
         final MultipartRequestInput mpRqInput =
-                MultipartRequestInputFactory.makeMultipartRequestInput(xid, ofVersion, mpType);
-        checkHeader(mpRqInput, mpType, ofVersion);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestExperimenterCase.class);
+            (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
+        checkHeader(mpRqInput, mpType);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                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 {
-        Assert.assertTrue(expectedMpRqBodyClass.isAssignableFrom(mpRqBody.getImplementedInterface()));
-        final String expectedDump = expectedMpRqBodyClass.getSimpleName() + " [augmentation=[]]";
-        Assert.assertEquals(expectedDump, String.valueOf(mpRqBody));
+    private static void checkEmptyBody(Class<? extends DataContainer> mpRqBody, Class<? extends
+            MultipartRequestBody> expectedMpRqBodyClass) {
+        Assert.assertTrue(expectedMpRqBodyClass.isAssignableFrom(mpRqBody));
+        Assert.assertEquals(expectedMpRqBodyClass, mpRqBody);
     }
-}
\ No newline at end of file
+}
OpenFlow Plugin