Fix most bgp-parser-impl checkstyle

[bgpcep.git] / bgp / parser-impl / src / test / java / org / opendaylight / protocol / bgp / parser / impl / OpenTest.java

diff --git a/bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/OpenTest.java b/bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/OpenTest.java
index eeace784b216ce060b8986db2a6091bd372893cd..29f0fe8a0ff74218167d64cf7c8a60475518614b 100644 (file)
--- a/bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/OpenTest.java
+++ b/bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/OpenTest.java
@@ -39,17 +39,24 @@ public class OpenTest {
     @Test
     public void testSerializeOpen() throws Exception {
         final List<OptionalCapabilities> optionalCapas = Lists.newArrayList();
-        final OptionalCapabilities optionalCapabilitiesBuilder= new OptionalCapabilitiesBuilder().setCParameters(new CParametersBuilder()
-            .setAs4BytesCapability(new As4BytesCapabilityBuilder().setAsNumber(new AsNumber(1000L)).build()).addAugmentation(
-                CParameters1.class, new CParameters1Builder().setGracefulRestartCapability(new GracefulRestartCapabilityBuilder()
-                    .setRestartFlags(new GracefulRestartCapability.RestartFlags(false)).setRestartTime(0).setTables
-                        (Collections.emptyList()).build()).build()).build()).build();
+        final OptionalCapabilities optionalCapabilitiesBuilder = new OptionalCapabilitiesBuilder()
+                .setCParameters(new CParametersBuilder()
+                    .setAs4BytesCapability(new As4BytesCapabilityBuilder().setAsNumber(new AsNumber(1000L)).build())
+                    .addAugmentation(CParameters1.class, new CParameters1Builder()
+                        .setGracefulRestartCapability(new GracefulRestartCapabilityBuilder()
+                            .setRestartFlags(new GracefulRestartCapability.RestartFlags(false)).setRestartTime(0)
+                            .setTables(Collections.emptyList()).build())
+                        .build())
+                    .build())
+                .build();
         optionalCapas.add(optionalCapabilitiesBuilder);
-        final List<BgpParameters> tlvs = Lists.newArrayList(new BgpParametersBuilder().setOptionalCapabilities(optionalCapas).build());
-        final Open open = new OpenBuilder().setBgpIdentifier(new Ipv4Address("127.0.0.1")).setMyAsNumber(30).setHoldTimer(3).setVersion(
-            new ProtocolVersion((short) 4)).setBgpParameters(tlvs).build();
+        final List<BgpParameters> tlvs = Lists.newArrayList(new BgpParametersBuilder()
+            .setOptionalCapabilities(optionalCapas).build());
+        final Open open = new OpenBuilder().setBgpIdentifier(new Ipv4Address("127.0.0.1")).setMyAsNumber(30)
+                .setHoldTimer(3).setVersion(new ProtocolVersion((short) 4)).setBgpParameters(tlvs).build();
         final ByteBuf msg = Unpooled.buffer();
-        new BGPOpenMessageParser(ServiceLoaderBGPExtensionProviderContext.getSingletonInstance().getParameterRegistry()).serializeMessage(open, msg);
+        new BGPOpenMessageParser(ServiceLoaderBGPExtensionProviderContext.getSingletonInstance().getParameterRegistry())
+            .serializeMessage(open, msg);
         final byte[] temp = ByteArray.cutBytes(ByteArray.getAllBytes(msg), 19);
         final Open openResult = new BGPOpenMessageParser(ServiceLoaderBGPExtensionProviderContext.getSingletonInstance()
             .getParameterRegistry()).parseMessageBody(Unpooled.copiedBuffer(temp), temp.length, null);
@@ -58,14 +65,17 @@ public class OpenTest {
         assertEquals(open.getHoldTimer(), openResult.getHoldTimer());
         assertEquals(open.getMyAsNumber(), openResult.getMyAsNumber());
         boolean grace = false;
-        if(openResult.getBgpParameters().get(0).getOptionalCapabilities().get(0).getCParameters().getAs4BytesCapability() != null ) {
+        if (openResult.getBgpParameters().get(0).getOptionalCapabilities().get(0).getCParameters()
+                .getAs4BytesCapability() != null) {
             grace = true;
         }
-        assertEquals(open.getBgpParameters().get(0).getOptionalCapabilities().get(0).getCParameters().getAs4BytesCapability(),
-            openResult.getBgpParameters().get(0).getOptionalCapabilities().get(!grace ? 1 : 0).getCParameters().getAs4BytesCapability());
         assertEquals(open.getBgpParameters().get(0).getOptionalCapabilities().get(0).getCParameters()
-            .augmentation(CParameters1.class)
-            .getGracefulRestartCapability(), openResult.getBgpParameters().get(0).getOptionalCapabilities().get(grace ? 1 : 0).getCParameters()
+            .getAs4BytesCapability(),
+            openResult.getBgpParameters().get(0).getOptionalCapabilities().get(grace ? 0 : 1).getCParameters()
+            .getAs4BytesCapability());
+        assertEquals(open.getBgpParameters().get(0).getOptionalCapabilities().get(0).getCParameters()
+            .augmentation(CParameters1.class).getGracefulRestartCapability(),
+            openResult.getBgpParameters().get(0).getOptionalCapabilities().get(grace ? 1 : 0).getCParameters()
                 .augmentation(CParameters1.class).getGracefulRestartCapability());
 
     }