Code Review / openflowplugin.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Make methods static
[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 fa304791a092fc6b21b83a4796c3576f635c0a1f..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,6 +37,7 @@ 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}.
@@ -48,21 +49,21 @@ public class MultipartRequestInputFactoryTest {
 
     @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 =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestDescCase.class);
+        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 =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -79,14 +80,11 @@ 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 =
@@ -106,25 +104,26 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputAggregate() throws Exception {
+    public void testMakeMultipartRequestInputAggregate() {
         MultipartType mpType = MultipartType.OFPMPAGGREGATE;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestAggregateCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestAggregateCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputTable() throws Exception {
+    public void testMakeMultipartRequestInputTable() {
         MultipartType mpType = MultipartType.OFPMPTABLE;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestTableCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(), MultipartRequestTableCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputPortStats() throws Exception {
+    public void testMakeMultipartRequestInputPortStats() {
         final MultipartType mpType = MultipartType.OFPMPPORTSTATS;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -137,7 +136,7 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputQueue() throws Exception {
+    public void testMakeMultipartRequestInputQueue() {
         final MultipartType mpType = MultipartType.OFPMPQUEUE;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -151,7 +150,7 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputGroup() throws Exception {
+    public void testMakeMultipartRequestInputGroup() {
         final MultipartType mpType = MultipartType.OFPMPGROUP;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -164,25 +163,27 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputGroupDesc() throws Exception {
+    public void testMakeMultipartRequestInputGroupDesc() {
         MultipartType mpType = MultipartType.OFPMPGROUPDESC;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupDescCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestGroupDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputGroupFeatures() throws Exception {
+    public void testMakeMultipartRequestInputGroupFeatures() {
         MultipartType mpType = MultipartType.OFPMPGROUPFEATURES;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestGroupFeaturesCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestGroupFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputMeter() throws Exception {
+    public void testMakeMultipartRequestInputMeter() {
         final MultipartType mpType = MultipartType.OFPMPMETER;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -195,7 +196,7 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputMeterConfig() throws Exception {
+    public void testMakeMultipartRequestInputMeterConfig() {
         final MultipartType mpType = MultipartType.OFPMPMETERCONFIG;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -209,16 +210,17 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputMeterFeatures() throws Exception {
+    public void testMakeMultipartRequestInputMeterFeatures() {
         MultipartType mpType = MultipartType.OFPMPMETERFEATURES;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestMeterFeaturesCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestMeterFeaturesCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputTableFeatures() throws Exception {
+    public void testMakeMultipartRequestInputTableFeatures() {
         final MultipartType mpType = MultipartType.OFPMPTABLEFEATURES;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
@@ -232,21 +234,23 @@ public class MultipartRequestInputFactoryTest {
     }
 
     @Test
-    public void testMakeMultipartRequestInputPortDesc() throws Exception {
+    public void testMakeMultipartRequestInputPortDesc() {
         MultipartType mpType = MultipartType.OFPMPPORTDESC;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestPortDescCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestPortDescCase.class);
     }
 
     @Test
-    public void testMakeMultipartRequestInputExperimenter() throws Exception {
+    public void testMakeMultipartRequestInputExperimenter() {
         MultipartType mpType = MultipartType.OFPMPEXPERIMENTER;
         final MultipartRequestInput mpRqInput =
             (MultipartRequestInput) MultipartRequestInputFactory.makeMultipartRequest(xid, ofVersion, mpType, false);
         checkHeader(mpRqInput, mpType);
-        checkEmptyBody(mpRqInput.getMultipartRequestBody(), MultipartRequestExperimenterCase.class);
+        checkEmptyBody(mpRqInput.getMultipartRequestBody().implementedInterface(),
+                MultipartRequestExperimenterCase.class);
     }
 
     private void checkHeader(MultipartRequestInput mpRqInput, MultipartType mpType) {
@@ -256,10 +260,9 @@ public class MultipartRequestInputFactoryTest {
         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);
     }
 }
OpenFlow Plugin