Fix sonar complains 38/53538/2
authorClaudio D. Gasparini <claudio.gasparini@pantheon.tech>
Sun, 19 Mar 2017 13:10:31 +0000 (14:10 +0100)
committerClaudio D. Gasparini <claudio.gasparini@pantheon.tech>
Mon, 20 Mar 2017 09:17:44 +0000 (10:17 +0100)
- Remove unused imports
- Throw exception
- Name convention
- Comparison
- private constructor on util
- replace explicit type per <>
- migrate to java stream api

Change-Id: I224eb8dfe776130b0f45468be4791239daf878af
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>
73 files changed:
bgp/bmp-impl/src/main/java/org/opendaylight/protocol/bmp/impl/app/BmpRibInWriter.java
bgp/bmp-impl/src/main/java/org/opendaylight/protocol/bmp/impl/session/BmpSessionImpl.java
bgp/bmp-impl/src/test/java/org/opendaylight/controller/config/yang/bmp/impl/AbstractBmpModuleTest.java
bgp/bmp-impl/src/test/java/org/opendaylight/controller/config/yang/bmp/impl/BmpMonitorImplModuleTest.java
bgp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/session/BmpDispatcherImplTest.java
bgp/bmp-mock/src/main/java/org/opendaylight/protocol/bmp/mock/BmpMock.java
bgp/bmp-mock/src/main/java/org/opendaylight/protocol/bmp/mock/BmpMockArguments.java
bgp/bmp-mock/src/main/java/org/opendaylight/protocol/bmp/mock/BmpMockSession.java
bgp/bmp-mock/src/test/java/org/opendaylight/protocol/bmp/mock/BmpMockSessionTest.java
bgp/bmp-parser-impl/src/test/java/org/opendaylight/protocol/bmp/parser/message/TestUtil.java
bgp/bmp-spi/src/test/java/org/opendaylight/protocol/bmp/spi/registry/AbstractBmpExtensionProviderActivatorTest.java
bgp/cli/src/main/java/org/opendaylight/protocol/bgp/cli/AbstractStatsCommandProvider.java
bgp/cli/src/main/java/org/opendaylight/protocol/bgp/cli/utils/BgpCliUtils.java
bgp/inet/src/test/java/org/opendaylight/protocol/bgp/inet/Ipv6BgpPrefixSidParserTest.java
bgp/inet/src/test/java/org/opendaylight/protocol/bgp/inet/codec/BGPParserTest.java
bgp/inet/src/test/java/org/opendaylight/protocol/bgp/inet/codec/Ipv4NlriParserTest.java
bgp/inet/src/test/java/org/opendaylight/protocol/bgp/inet/codec/nexthop/NextHopParserSerializerTest.java
bgp/labeled-unicast/src/main/java/org/opendaylight/protocol/bgp/labeled/unicast/OriginatorSrgbTlvParser.java
bgp/labeled-unicast/src/test/java/org/opendaylight/protocol/bgp/labeled/unicast/BgpPrefixSidTlvsTest.java
bgp/openconfig-state/src/test/java/org/opendaylight/protocol/bgp/state/NeighborUtilTest.java
bgp/parser-impl/src/main/java/org/opendaylight/protocol/bgp/parser/impl/message/update/BgpPrefixSidAttributeParser.java
bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/AddPathCapabilityHandlerTest.java
bgp/parser-impl/src/test/java/org/opendaylight/protocol/bgp/parser/impl/BGPParserTest.java
bgp/parser-spi/src/main/java/org/opendaylight/protocol/bgp/parser/spi/pojo/SimpleAttributeRegistry.java
bgp/parser-spi/src/main/java/org/opendaylight/protocol/bgp/parser/spi/pojo/SimpleBGPExtensionProviderContext.java
bgp/parser-spi/src/test/java/org/opendaylight/protocol/bgp/parser/spi/pojo/SimpleRegistryTest.java
bgp/path-selection-mode/src/main/java/org/opendaylight/protocol/bgp/mode/impl/BestPathStateImpl.java
bgp/path-selection-mode/src/main/java/org/opendaylight/protocol/bgp/mode/impl/add/AddPathAbstractRouteEntry.java
bgp/path-selection-mode/src/test/java/org/opendaylight/protocol/bgp/mode/impl/AbstractRouteEntryTest.java
bgp/path-selection-mode/src/test/java/org/opendaylight/protocol/bgp/mode/impl/base/BaseBestPathTest.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/AbstractBGPSessionNegotiator.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/AdjRibOutListener.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/CachingImportPolicy.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/LocRibWriter.java
bgp/rib-impl/src/test/java/org/opendaylight/controller/config/yang/bgp/rib/impl/BGPPeerModuleTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/BGPSessionImplTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/BgpPeerRpcTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/FSMTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/SynchronizationAndExceptionTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/config/AbstractConfig.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/config/RibImplTest.java
bgp/rib-mock/src/main/java/org/opendaylight/protocol/bgp/rib/mock/BGPMock.java
bgp/rib-spi/src/test/java/org/opendaylight/protocol/bgp/rib/spi/AbstractRIBSupportTest.java
bgp/topology-provider/src/main/java/org/opendaylight/bgpcep/bgp/topology/provider/LinkstateTopologyBuilder.java
concepts/src/test/java/org/opendaylight/protocol/concepts/BandwidthTest.java
data-change-counter/src/main/java/org/opendaylight/controller/config/yang/bgpcep/data/change/counter/DataChangeCounterImplModule.java
data-change-counter/src/test/java/org/opendaylight/controller/config/yang/bgpcep/data/change/counter/DataChangeCounterImplModuleTest.java
pcep/impl/src/main/java/org/opendaylight/protocol/pcep/impl/AbstractPCEPSessionNegotiator.java
pcep/impl/src/main/java/org/opendaylight/protocol/pcep/impl/AbstractSessionNegotiator.java
pcep/impl/src/main/java/org/opendaylight/protocol/pcep/impl/PCEPByteToMessageDecoder.java
pcep/impl/src/main/java/org/opendaylight/protocol/pcep/impl/PCEPPeerRegistry.java
pcep/impl/src/main/java/org/opendaylight/protocol/pcep/impl/PCEPSessionNegotiator.java
pcep/impl/src/test/java/org/opendaylight/protocol/pcep/impl/AbstractPCEPSessionTest.java
pcep/pcc-mock/src/main/java/org/opendaylight/protocol/pcep/pcc/mock/Main.java
pcep/pcc-mock/src/main/java/org/opendaylight/protocol/pcep/pcc/mock/PCCTunnelManagerImpl.java
pcep/pcc-mock/src/main/java/org/opendaylight/protocol/pcep/pcc/mock/protocol/PCCDispatcherImpl.java
pcep/pcc-mock/src/main/java/org/opendaylight/protocol/pcep/pcc/mock/spi/MsgBuilderUtil.java
pcep/pcc-mock/src/test/java/org/opendaylight/protocol/pcep/pcc/mock/PCCTunnelManagerImplTest.java
pcep/testtool/src/test/java/org/opendaylight/protocol/pcep/testtool/PCCMock.java
pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/TopologyNodeState.java
pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/config/PCEPTopologyDeployer.java
pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/config/PCEPTopologyDeployerImpl.java
pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/config/PCEPTopologyProviderBean.java
programming/impl/src/main/java/org/opendaylight/bgpcep/programming/impl/ProgrammingServiceImpl.java
programming/impl/src/test/java/org/opendaylight/bgpcep/programming/impl/ProgrammingServiceImplTest.java
rsvp/spi/src/main/java/org/opendaylight/protocol/rsvp/parser/spi/pojo/SimpleRSVPExtensionProviderContext.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/pojo/SimpleEROSubobjectRegistryTest.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/pojo/SimpleRROSubobjectRegistryTest.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/pojo/SimpleXROSubobjectRegistryTest.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/subobjects/EROSubobjectListParserTest.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/subobjects/RROSubobjectListParserTest.java
rsvp/spi/src/test/java/org/opendaylight/protocol/rsvp/parser/spi/subobjects/XROSubobjectListParserTest.java
util/src/test/java/org/opendaylight/protocol/util/ByteArrayTest.java

index a58583416c3e912b257c67789a5736257a929db6..9f69f93b68c4780817ac93daaa6380dec2e76455 100644 (file)
@@ -116,12 +116,11 @@ final class BmpRibInWriter {
      * @param tableTypes
      * @param yangTableRootIId
      * @param tx
-     * @param registry
      * @return
      */
     private ImmutableMap.Builder<TablesKey, TableContext> createTableInstance(final Set<TablesKey> tableTypes,
-            final YangInstanceIdentifier yangTableRootIId, final DOMDataWriteTransaction tx, final RIBExtensionConsumerContext ribExtensions,
-            final BindingCodecTree tree) {
+        final YangInstanceIdentifier yangTableRootIId, final DOMDataWriteTransaction tx,
+        final RIBExtensionConsumerContext ribExtensions, final BindingCodecTree tree) {
 
         final ImmutableMap.Builder<TablesKey, TableContext> tb = ImmutableMap.builder();
         for (final TablesKey k : tableTypes) {
index c4655f0cc715ed68de818d97419ce8c1fb2c9ee2..b819f39b677c7f07c14990654499beb53d478c0b 100644 (file)
@@ -120,8 +120,7 @@ public final class BmpSessionImpl extends SimpleChannelInboundHandler<Notificati
             }
             break;
         case IDLE:
-            new IllegalStateException("Received message " + msg + " while BMP Session " + this + " was not active.");
-            break;
+            throw new IllegalStateException("Received message " + msg + " while BMP Session " + this + " was not active.");
         default:
             break;
         }
index 50da138f0c537059d87ce021807a6afc3d429dc3..4eec08ae79dce670c2428456d1b3ce4457353edc 100644 (file)
@@ -15,8 +15,6 @@ import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.mock;
 import org.junit.Before;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.config.manager.impl.AbstractConfigTest;
 import org.opendaylight.protocol.bgp.parser.spi.AddressFamilyRegistry;
 import org.opendaylight.protocol.bgp.parser.spi.BGPExtensionProviderContext;
index b043f708081a41f1fce86d9c6526654174b81b77..19a4d9a9b4c2789b40d7a270481983c7b1bccf4d 100644 (file)
@@ -29,8 +29,6 @@ import org.junit.Test;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.config.api.jmx.CommitStatus;
 import org.opendaylight.controller.config.manager.impl.factoriesresolver.HardcodedModuleFactoriesResolver;
 import org.opendaylight.controller.config.util.ConfigTransactionJMXClient;
index e538dd0d9fee883f8c2ce3f497e8eecc12c0323d..d48310e4682ba86488e7ebf1d28c7c1ea57ae69d 100644 (file)
@@ -9,9 +9,10 @@
 package org.opendaylight.protocol.bmp.impl.session;
 
 import static org.junit.Assert.assertTrue;
+import static org.mockito.Matchers.any;
+import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.timeout;
 import static org.mockito.Mockito.verify;
-import static org.mockito.internal.verification.VerificationModeFactory.times;
 import static org.opendaylight.protocol.util.CheckUtil.checkEquals;
 import static org.opendaylight.protocol.util.CheckUtil.waitFutureSuccess;
 
@@ -25,14 +26,12 @@ import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mock;
-import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
 import org.opendaylight.protocol.bgp.parser.impl.BGPActivator;
 import org.opendaylight.protocol.bgp.parser.spi.BGPExtensionProviderContext;
 import org.opendaylight.protocol.bgp.parser.spi.pojo.SimpleBGPExtensionProviderContext;
 import org.opendaylight.protocol.bmp.api.BmpDispatcher;
 import org.opendaylight.protocol.bmp.api.BmpSession;
-import org.opendaylight.protocol.bmp.api.BmpSessionFactory;
 import org.opendaylight.protocol.bmp.api.BmpSessionListenerFactory;
 import org.opendaylight.protocol.bmp.impl.BmpDispatcherImpl;
 import org.opendaylight.protocol.bmp.parser.BmpActivator;
@@ -57,13 +56,13 @@ public class BmpDispatcherImplTest {
     @Before
     public void setUp() throws Exception {
         MockitoAnnotations.initMocks(this);
-        Mockito.doNothing().when(this.mockedSession).handlerRemoved(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).handlerAdded(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).channelRegistered(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).channelActive(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).channelInactive(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).channelUnregistered(Mockito.any(ChannelHandlerContext.class));
-        Mockito.doNothing().when(this.mockedSession).channelReadComplete(Mockito.any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).handlerRemoved(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).handlerAdded(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).channelRegistered(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).channelActive(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).channelInactive(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).channelUnregistered(any(ChannelHandlerContext.class));
+        doNothing().when(this.mockedSession).channelReadComplete(any(ChannelHandlerContext.class));
 
         this.bgpActivator = new BGPActivator();
         final BGPExtensionProviderContext context = new SimpleBGPExtensionProviderContext();
@@ -99,9 +98,9 @@ public class BmpDispatcherImplTest {
 
         final Channel clientChannel = futureClient.channel();
         checkEquals(()-> assertTrue(clientChannel.isActive()));
-        verify(this.mockedSession, timeout(500).times(2)).handlerAdded(Mockito.any(ChannelHandlerContext.class));
-        verify(this.mockedSession, timeout(500).times(2)).channelRegistered(Mockito.any(ChannelHandlerContext.class));
-        verify(this.mockedSession, timeout(500).times(2)).channelActive(Mockito.any(ChannelHandlerContext.class));
+        verify(this.mockedSession, timeout(500).times(2)).handlerAdded(any(ChannelHandlerContext.class));
+        verify(this.mockedSession, timeout(500).times(2)).channelRegistered(any(ChannelHandlerContext.class));
+        verify(this.mockedSession, timeout(500).times(2)).channelActive(any(ChannelHandlerContext.class));
         waitFutureSuccess(clientChannel.close());
         waitFutureSuccess(serverChannel.close());
     }
index e838967001fdb1674af99f6f45480b375317be83..c129b1802e635b484f2b3143400c72d93bf29fca 100644 (file)
@@ -9,15 +9,11 @@
 package org.opendaylight.protocol.bmp.mock;
 
 import com.google.common.net.InetAddresses;
-import io.netty.channel.Channel;
 import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import org.opendaylight.protocol.bgp.parser.impl.BGPActivator;
 import org.opendaylight.protocol.bgp.parser.spi.BGPExtensionProviderContext;
 import org.opendaylight.protocol.bgp.parser.spi.pojo.SimpleBGPExtensionProviderContext;
-import org.opendaylight.protocol.bmp.api.BmpSession;
-import org.opendaylight.protocol.bmp.api.BmpSessionFactory;
-import org.opendaylight.protocol.bmp.api.BmpSessionListenerFactory;
 import org.opendaylight.protocol.bmp.parser.BmpActivator;
 import org.opendaylight.protocol.bmp.spi.registry.BmpExtensionProviderActivator;
 import org.opendaylight.protocol.bmp.spi.registry.BmpExtensionProviderContext;
index 7874e1db02ab8762c321394733b59a7c108403a7..005612ce8b49a4b7318981bf63d85ef6c45fea42 100644 (file)
@@ -16,10 +16,8 @@ import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import net.sourceforge.argparse4j.ArgumentParsers;
 import net.sourceforge.argparse4j.impl.Arguments;
-import net.sourceforge.argparse4j.inf.Argument;
 import net.sourceforge.argparse4j.inf.ArgumentParser;
 import net.sourceforge.argparse4j.inf.ArgumentParserException;
-import net.sourceforge.argparse4j.inf.ArgumentType;
 import net.sourceforge.argparse4j.inf.Namespace;
 import org.opendaylight.protocol.util.ArgumentsInput;
 
index 887d201aabb8a908d099aedd7c8baf53ff5a9e46..f338be57659774fed44c0011cb7e7adfa494b169 100644 (file)
@@ -12,7 +12,6 @@ import static org.opendaylight.protocol.util.Ipv4Util.incrementIpv4Address;
 import static org.opendaylight.protocol.util.Ipv4Util.incrementIpv4Prefix;
 
 import io.netty.channel.Channel;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.SimpleChannelInboundHandler;
index 62c311fada214d270f484b0b17a339e4a2510185..ea46754df98b891ab8a71ade0333848796cc784a 100644 (file)
@@ -23,8 +23,6 @@ import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.util.InetSocketAddressUtil;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bmp.message.rev150512.InitiationMessage;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bmp.message.rev150512.PeerUp;
index fbb056eecfa64a41310a82ee99fcb2d155851a56..b10855dff20e34ec1650924cd389b6d99bc9e0d3 100644 (file)
@@ -307,7 +307,7 @@ public final class TestUtil {
             .setMultiExitDisc(new MultiExitDiscBuilder().setMed(123L).build())
             .setOrigin(new OriginBuilder().setValue(BgpOrigin.Igp).build())
             .setOriginatorId(new OriginatorIdBuilder().setOriginator(IPV4_ADDRESS_12).build())
-            .setUnrecognizedAttributes(new ArrayList<UnrecognizedAttributes>());
+            .setUnrecognizedAttributes(new ArrayList<>());
         return attribBuilder.build();
     }
 
index 63e55ce77fa95c1eab58d4a77c5ab1aa8fbf56fb..9aa0ec1e3a60f85d9c28c1bb9fe481cf4d6c44fa 100644 (file)
@@ -32,7 +32,7 @@ public class AbstractBmpExtensionProviderActivatorTest {
     private static class SimpleAbstractBmpExtensionProviderActivator extends AbstractBmpExtensionProviderActivator {
         @Override
         protected List<AutoCloseable> startImpl(final BmpExtensionProviderContext context) {
-            final List<AutoCloseable> reg = new ArrayList<AutoCloseable>();
+            final List<AutoCloseable> reg = new ArrayList<>();
             reg.add(context.registerBmpMessageParser(1, new SimpleBmpMessageRegistry()));
             return reg;
         }
index c8e81b8fde9c4049604740270ce11b710ec7c666..5caf7156cc26e9b51646380c721d8ced9f50ada3 100644 (file)
@@ -29,9 +29,7 @@ public abstract class AbstractStatsCommandProvider extends OsgiCommandSupport {
     @Override
     protected final Object doExecute() throws Exception {
         final PrintStream out = this.session.getConsole();
-        getObjectNames().forEach(objectName -> {
-            onExecution(out, getRuntimeMXBean(objectName), objectName);
-        });
+        getObjectNames().forEach(objectName -> onExecution(out, getRuntimeMXBean(objectName), objectName));
         return null;
     }
 
index 3b933af5bd3f019e2be3a34c28c861030efb3183..2954a95648f7a40bdda8c0d4e184b1f6814b0bb9 100644 (file)
@@ -21,6 +21,9 @@ public final class BgpCliUtils {
     private static final String PEER_PREFERENCES_LABEL = "PeerPreferences.";
     private static final String SPEAKER_PREFERENCES_LABEL = "SpeakerPreferences.";
 
+    private BgpCliUtils() {
+        throw new UnsupportedOperationException();
+    }
     public static void displayAll(final ObjectName objectName, final BgpSessionState bgpSessionState, final PrintStream printStream) {
         if (bgpSessionState == null) {
             printStream.println(String.format("No BgpSessionState found for [%s]", objectName));
index da00cf72ba196c0c1d7f909e97be05509cd6404c..579706ffec8e28d8c90b1f3778e2e9bfb496cd75 100644 (file)
@@ -17,7 +17,6 @@ import org.junit.Test;
 import org.opendaylight.protocol.bgp.inet.codec.Ipv6BgpPrefixSidParser;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev150305.update.attributes.bgp.prefix.sid.bgp.prefix.sid.tlvs.bgp.prefix.sid.tlv.Ipv6SidTlvBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.path.attributes.attributes.bgp.prefix.sid.bgp.prefix.sid.tlvs.BgpPrefixSidTlv;
-import org.opendaylight.yangtools.yang.binding.DataContainer;
 
 public final class Ipv6BgpPrefixSidParserTest {
 
index 4e88e9160787f7f3ddbed2755be80c9ce9c2d640..546b3676d2f0066d10ef44a012fe8f978004bf5b 100644 (file)
@@ -157,7 +157,7 @@ public class BGPParserTest {
         assertNull(message.getNlri());
 
         // attributes
-        final List<AsNumber> asNumbers = new ArrayList<AsNumber>();
+        final List<AsNumber> asNumbers = new ArrayList<>();
         asNumbers.add(new AsNumber(65001L));
         final List<Segments> asPath = Lists.newArrayList();
         asPath.add(new SegmentsBuilder().setAsSequence(asNumbers).build());
index bbf8d4abc6ca81d8ee93194233593ed023b76afb..7cb98b90b84976d2a2818e91b9e26e71d3b37cf9 100644 (file)
@@ -95,7 +95,7 @@ public class Ipv4NlriParserTest {
                 new DestinationIpv4Builder().setIpv4Prefixes(this.prefixes).build()).build();
         this.ip4caseAD = new DestinationIpv4CaseBuilder().setDestinationIpv4(new DestinationIpv4Builder().setIpv4Prefixes(this.prefixes).build()).build();
 
-        final ArrayList<Ipv4Prefixes> fakePrefixes = new ArrayList<Ipv4Prefixes>(this.prefixes);
+        final ArrayList<Ipv4Prefixes> fakePrefixes = new ArrayList<>(this.prefixes);
         fakePrefixes.add(new Ipv4PrefixesBuilder().setPrefix(wrongPrefix).build());
         this.ip4caseWDWrong = new org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev150305.update.attributes.mp.unreach.nlri.withdrawn.routes.destination.type.DestinationIpv4CaseBuilder().setDestinationIpv4(
                 new DestinationIpv4Builder().setIpv4Prefixes(fakePrefixes).build()).build();
index 806f13cc46f360f7e97f47e6c68597bef83fd673..786d45d11a403525856732faf5f0a77b32f90e60 100644 (file)
@@ -29,7 +29,6 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.type
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.next.hop.c.next.hop.Ipv6NextHopCaseBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.next.hop.c.next.hop.ipv4.next.hop._case.Ipv4NextHopBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.next.hop.c.next.hop.ipv6.next.hop._case.Ipv6NextHopBuilder;
-import org.opendaylight.yangtools.yang.binding.DataContainer;
 
 public class NextHopParserSerializerTest {
 
index caaa0aa58963cafd4a6700fe298ef61eaa8aa272..63da734e7438a8edcbf12791d8823f9a2819f806 100644 (file)
@@ -35,7 +35,7 @@ final class OriginatorSrgbTlvParser implements BgpPrefixSidTlvParser, BgpPrefixS
 
     private static List<SrgbValue> parseSrgbs(final ByteBuf buffer) {
         Preconditions.checkState(buffer.readableBytes() % SRGB_LENGTH == 0, "Number of SRGBs doesn't fit available bytes.");
-        final List<SrgbValue> ret = new ArrayList<SrgbValue>();
+        final List<SrgbValue> ret = new ArrayList<>();
         while (buffer.isReadable()) {
             ret.add(new SrgbValueBuilder().setBase(new Srgb((long) buffer.readUnsignedMedium())).setRange(new Srgb((long) buffer.readUnsignedMedium())).build());
         }
index e0be13d7c9607a7e34e22d404ab8beb428e4325b..a6fbc624de1d878c9fe402a5a30a1344afae7668 100644 (file)
@@ -42,7 +42,7 @@ public final class BgpPrefixSidTlvsTest {
     @Test
     public void testOriginatorParser() {
         final OriginatorSrgbTlvParser parser = new OriginatorSrgbTlvParser();
-        final List<SrgbValue> list = new ArrayList<SrgbValue>();
+        final List<SrgbValue> list = new ArrayList<>();
 
         final Srgb srgb1 = new Srgb(1L);
         final Srgb srgb2 = new Srgb(2L);
index e4163a784f1b021f2f9b26e4d9749bdcad00fc97..a81c7c25fd4155f0bf979e3d479a5ca3e4a2303c 100644 (file)
@@ -23,8 +23,6 @@ import org.junit.Test;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.bgp.openconfig.spi.BGPTableTypeRegistryConsumer;
 import org.opendaylight.protocol.bgp.rib.spi.State;
 import org.opendaylight.protocol.bgp.rib.spi.state.BGPAfiSafiState;
index f93232594bf427701e612c92b2a2a92f657dee58..84468742cd2aa1a23bf56589a070eda1b228083b 100644 (file)
@@ -51,7 +51,7 @@ public final class BgpPrefixSidAttributeParser implements AttributeParser, Attri
     @Override
     public void parseAttribute(final ByteBuf buffer, final AttributesBuilder builder) throws BGPDocumentedException, BGPParsingException {
         final BgpPrefixSidBuilder sid = new BgpPrefixSidBuilder();
-        final List<BgpPrefixSidTlvs> tlvList = new ArrayList<BgpPrefixSidTlvs>();
+        final List<BgpPrefixSidTlvs> tlvList = new ArrayList<>();
         while (buffer.isReadable()) {
             final BgpPrefixSidTlv tlv = this.reg.parseBgpPrefixSidTlv(buffer.readUnsignedByte(), buffer);
             tlvList.add(new BgpPrefixSidTlvsBuilder().setBgpPrefixSidTlv(tlv).build());
index cf0098c563ed4710548eb48260dd5e092d99f8e0..00de9ed556f158bde6092ed9beb6cfbb74354ae1 100644 (file)
@@ -64,7 +64,7 @@ public class AddPathCapabilityHandlerTest {
 
     @Test
     public void testCapabilityHandler() throws BGPDocumentedException, BGPParsingException {
-        final List<AddressFamilies> family = new ArrayList<AddressFamilies>();
+        final List<AddressFamilies> family = new ArrayList<>();
         family.add(new AddressFamiliesBuilder().setAfi(this.afi).setSafi(this.safi).setSendReceive(SendReceive.forValue(1)).build());
 
         final CParameters capabilityToSerialize = new CParametersBuilder().addAugmentation(CParameters1.class, new CParameters1Builder().setAddPathCapability(
@@ -96,7 +96,7 @@ public class AddPathCapabilityHandlerTest {
     @Test
     public void testSendReceiveIgnored() throws BGPDocumentedException, BGPParsingException {
         final CParameters capabilityToSerialize = new CParametersBuilder().addAugmentation(CParameters1.class, new CParameters1Builder().setAddPathCapability(
-            new AddPathCapabilityBuilder().setAddressFamilies(new ArrayList<AddressFamilies>()).build()).build()).build();
+            new AddPathCapabilityBuilder().setAddressFamilies(new ArrayList<>()).build()).build()).build();
 
         final ByteBuf bytes = this.parseWrongBytes.copy();
         final AddPathCapabilityHandler handler = new AddPathCapabilityHandler(this.afiRegistry, this.safiRegistry);
@@ -106,7 +106,7 @@ public class AddPathCapabilityHandlerTest {
 
     @Test(expected=IllegalArgumentException.class)
     public void testUnhandledAfi() {
-        final List<AddressFamilies> family = new ArrayList<AddressFamilies>();
+        final List<AddressFamilies> family = new ArrayList<>();
         family.add(new AddressFamiliesBuilder().setAfi(this.afi).setSafi(this.safi).setSendReceive(SendReceive.forValue(2)).build());
 
         final CParameters capabilityToSerialize = new CParametersBuilder().addAugmentation(CParameters1.class, new CParameters1Builder().setAddPathCapability(
@@ -119,7 +119,7 @@ public class AddPathCapabilityHandlerTest {
 
     @Test(expected=IllegalArgumentException.class)
     public void testUnhandledSafi() {
-        final List<AddressFamilies> family = new ArrayList<AddressFamilies>();
+        final List<AddressFamilies> family = new ArrayList<>();
         family.add(new AddressFamiliesBuilder().setAfi(this.afi).setSafi(this.safi).setSendReceive(SendReceive.forValue(3)).build());
 
         final CParameters capabilityToSerialize = new CParametersBuilder().addAugmentation(CParameters1.class, new CParameters1Builder().setAddPathCapability(
@@ -132,7 +132,7 @@ public class AddPathCapabilityHandlerTest {
 
     @Test(expected=IllegalArgumentException.class)
     public void testUnhandledSendReceive() {
-        final List<AddressFamilies> family = new ArrayList<AddressFamilies>();
+        final List<AddressFamilies> family = new ArrayList<>();
         family.add(new AddressFamiliesBuilder().setAfi(this.afi).setSafi(this.safi).setSendReceive(SendReceive.forValue(4)).build());
 
         final CParameters capabilityToSerialize = new CParametersBuilder().addAugmentation(CParameters1.class, new CParameters1Builder().setAddPathCapability(
index 5c5ee8b93e947d0ec03a1eaf1a63bcef15e4a227..9437776eda2a75542d6c9c8f76292ebd5bd2867a 100644 (file)
@@ -68,7 +68,7 @@ public class BGPParserTest {
     /**
      * Used by other tests as well
      */
-    static final List<byte[]> inputBytes = new ArrayList<byte[]>();
+    static final List<byte[]> inputBytes = new ArrayList<>();
 
     private static int COUNTER = 7;
 
@@ -161,7 +161,7 @@ public class BGPParserTest {
         assertNull(message.getWithdrawnRoutes());
 
         // attributes
-        final List<AsNumber> asNumbers = new ArrayList<AsNumber>();
+        final List<AsNumber> asNumbers = new ArrayList<>();
         asNumbers.add(new AsNumber(65002L));
         final List<Segments> asPath = Lists.newArrayList();
         asPath.add(new SegmentsBuilder().setAsSequence(asNumbers).build());
@@ -283,7 +283,7 @@ public class BGPParserTest {
         assertNull(message.getWithdrawnRoutes());
 
         // attributes
-        final List<AsNumber> asNumbers = new ArrayList<AsNumber>();
+        final List<AsNumber> asNumbers = new ArrayList<>();
         asNumbers.add(new AsNumber(30L));
         final List<Segments> asPath = Lists.newArrayList();
         asPath.add(new SegmentsBuilder().setAsSequence(asNumbers).build());
index 85152dc46ba4cd7ad0aba520fee8b4320acbe34d..3a05df0c5ce28e11a464eda622d30aa61003b608 100644 (file)
@@ -58,7 +58,7 @@ final class SimpleAttributeRegistry implements AttributeRegistry {
     private final HandlerRegistry<DataContainer, AttributeParser, AttributeSerializer> handlers = new HandlerRegistry<>();
     private final Map<AbstractRegistration, AttributeSerializer> serializers = new LinkedHashMap<>();
     private final AtomicReference<Iterable<AttributeSerializer>> roSerializers =
-        new AtomicReference<Iterable<AttributeSerializer>>(this.serializers.values());
+        new AtomicReference<>(this.serializers.values());
     private final List<UnrecognizedAttributes> unrecognizedAttributes = new ArrayList<>();
 
 
index 20ad4f5adf66627819daefb834e973754ec71b38..fbdd8b8034bbaa8ec65cce9f1c27372b9c7f24d0 100644 (file)
@@ -66,7 +66,7 @@ public class SimpleBGPExtensionProviderContext extends SimpleBGPExtensionConsume
         this.maximumCachedObjects = maximumCachedObjects;
 
         final Cache<Object, Object> cache = CacheBuilder.newBuilder().maximumSize(maximumCachedObjects).build();
-        this.cacheRef = new AtomicReference<Cache<Object,Object>>(cache);
+        this.cacheRef = new AtomicReference<>(cache);
     }
 
     @Override
index f8306d6637d08eb1586d5dc91a177da95196a70e..513e0e8ece29655e253269e18b167371442a41a8 100644 (file)
@@ -39,7 +39,6 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.open.message.BgpParameters;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.path.attributes.AttributesBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.path.attributes.attributes.bgp.prefix.sid.bgp.prefix.sid.tlvs.BgpPrefixSidTlv;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.BgpTableType;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.update.attributes.MpReachNlri;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.update.attributes.MpReachNlriBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.update.attributes.MpUnreachNlri;
index 2a98f61fca9a5b7d4820beb0daf951d33aa23f59..e2f69e21c60699872344daa421cf8ce662bb5698 100644 (file)
@@ -17,7 +17,6 @@ import com.google.common.collect.ImmutableList;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
-import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutionException;
 import javax.annotation.concurrent.NotThreadSafe;
 import org.opendaylight.protocol.bgp.mode.api.BestPathState;
index 497175d1abc600c7d3998bb801fcd95936f45c82..012d196d40695e7e3829ecbadaadedd5ff208d6a 100644 (file)
@@ -282,9 +282,7 @@ public abstract class AddPathAbstractRouteEntry extends AbstractRouteEntry {
         this.bestPath.forEach(oldBest -> {
             final Optional<AddPathBestPath> present = newBestPathList.stream()
                 .filter(newBest -> newBest.getPathId() == oldBest.getPathId() && newBest.getRouteKey() == oldBest.getRouteKey()).findAny();
-            if(present.isPresent()) {
-                this.bestPathRemoved.remove(oldBest);
-            }
+            present.ifPresent(addPathBestPath -> this.bestPathRemoved.remove(oldBest));
         });
     }
 }
index d40a0a7159c2f95bd0e156bee8b46dd26f0821cd..ae78af67554472c573ebc22b0f27a4b231a81a63 100644 (file)
@@ -21,8 +21,6 @@ import java.util.Map;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction;
 import org.opendaylight.protocol.bgp.rib.spi.ExportPolicyPeerTracker;
index 00290de53b563ddce5dfeee940bc4c22a855a0c6..c6053ef8d94157a159c629b333a88db5427f59fc 100644 (file)
@@ -51,7 +51,7 @@ public class BaseBestPathTest {
     public void testHashCodeAndEqual() throws Exception {
         assertTrue(this.baseBestPath.equals(this.baseBestPathCopy) && this.baseBestPathCopy.equals(this.baseBestPath));
         assertTrue(this.baseBestPath.hashCode() == this.baseBestPathCopy.hashCode());
-        assertFalse(this.baseBestPath.equals(PATH_ID));
+        assertTrue(this.baseBestPath.getPathId() == PATH_ID);
     }
 
     @Test
index 0af5d690c19ba2fdb944bf38c5f03d6b6207e194..cef181ab081f9f491a793992a493e7be62a0ee32 100644 (file)
@@ -11,7 +11,6 @@ package org.opendaylight.protocol.bgp.rib.impl;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import io.netty.channel.Channel;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.ChannelInboundHandlerAdapter;
index 7edaa8903cfef1eb22c65ae7acd81ea8c94c93b4..6bb66697a1b0dc56de9dcaecaebd33c7392e1e3a 100644 (file)
@@ -82,7 +82,7 @@ final class AdjRibOutListener implements ClusteredDOMDataTreeChangeListener, Pre
 
     static AdjRibOutListener create(@Nonnull final PeerId peerId, @Nonnull final TablesKey tablesKey, @Nonnull final YangInstanceIdentifier ribId,
         @Nonnull final CodecsRegistry registry, @Nonnull final RIBSupport support, @Nonnull final DOMDataTreeChangeService service,
-        @Nonnull final ChannelOutputLimiter session, @Nonnull final boolean mpSupport, @Nonnull final LongAdder routeCounter
+        @Nonnull final ChannelOutputLimiter session, final boolean mpSupport, @Nonnull final LongAdder routeCounter
     ) {
         return new AdjRibOutListener(peerId, tablesKey, ribId, registry, support, service, session, mpSupport, routeCounter);
     }
index c9cfb93cc0603371c80c33d5951a602bbeb83580..4fd885ac7f068492060d5517cba0b5c01ba6d58f 100644 (file)
@@ -77,7 +77,7 @@ final class CachingImportPolicy extends AbstractImportPolicy {
          * and thus are on the path to create a new cache entry.
          */
         final ContainerNode interned = this.interner.intern(attributes);
-        if (interned != attributes) {
+        if (!interned.equals(attributes)) {
             final ContainerNode retry = this.cache.get(interned);
             if (retry != null) {
                 return unmaskNull(retry);
index ec785e68f7b7498d48e567e153bbb8ed18f15d1b..6e280fefab30c9b80584ca24a42e15c3f443146e 100644 (file)
@@ -88,10 +88,10 @@ final class LocRibWriter implements AutoCloseable, TotalPrefixesCounter, TotalPa
         final PathSelectionMode pathSelectionMode) {
         this.chain = Preconditions.checkNotNull(chain);
         this.target = Preconditions.checkNotNull(target);
-        final NodeIdentifierWithPredicates tableKey = RibSupportUtils.toYangTablesKey(tablesKey);
+        this.tableKey  = RibSupportUtils.toYangTablesKey(tablesKey);
         this.localTablesKey = tablesKey;
-        this.tableKey = tableKey;
-        this.locRibTarget = YangInstanceIdentifier.create(target.node(LocRib.QNAME).node(Tables.QNAME).node(tableKey).getPathArguments());
+        this.locRibTarget = YangInstanceIdentifier.create(target.node(LocRib.QNAME).node(Tables.QNAME)
+            .node(this.tableKey).getPathArguments());
         this.ourAs = Preconditions.checkNotNull(ourAs);
         this.service = Preconditions.checkNotNull(service);
         this.ribSupport = registry.getRIBSupportContext(tablesKey).getRibSupport();
index 366494ec1774565bdc499a86f8a5d0404efdcfa4..ae1d1972fe79e0f2530b07530b65835402f73b0e 100755 (executable)
@@ -164,7 +164,7 @@ public class BGPPeerModuleTest extends AbstractRIBImplModuleTest {
         mxBean2.setAddressFamily(AddPathImplModuleTest.createAddressFamily(transaction, "add-path-inst-2"));
         mxBean2.setSendReceive(SendReceive.Receive);
 
-        final List<ObjectName> ret = new ArrayList<ObjectName>();
+        final List<ObjectName> ret = new ArrayList<>();
         ret.add(name1);
         ret.add(name2);
         return ret;
index a1dd4586005aa56574805c19aa5781abdfe1729a..0245ce31450156de3d34d8d8d4807cd5912d3305 100644 (file)
@@ -35,8 +35,6 @@ import org.mockito.Matchers;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.config.yang.bgp.rib.impl.BgpSessionState;
 import org.opendaylight.protocol.bgp.parser.BGPDocumentedException;
 import org.opendaylight.protocol.bgp.parser.BGPError;
index 29d343afd6d3af2baf34703459ae3c96d7220123..a7ad6bcc1202afcfa8f7a14bfa537c4c2bfb5e46 100644 (file)
@@ -29,7 +29,7 @@ import org.opendaylight.yangtools.yang.common.RpcResult;
 
 public class BgpPeerRpcTest {
 
-    private final Set<TablesKey> supportedFamilies = new HashSet<TablesKey>();
+    private final Set<TablesKey> supportedFamilies = new HashSet<>();
     private final BGPSessionImpl session = Mockito.mock(BGPSessionImpl.class);
     private final BgpPeerRpc rpc = new BgpPeerRpc(this.session, this.supportedFamilies);
     private final PeerRef peer = Mockito.mock(PeerRef.class);
index ef09a18db15effb8cfdf253d764ee47be51e389d..ec44b2aa25b9c850055205d7d3c44f8a454a581f 100644 (file)
@@ -36,8 +36,6 @@ import org.junit.Test;
 import org.mockito.Matchers;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.bgp.parser.BGPError;
 import org.opendaylight.protocol.bgp.parser.BgpExtendedMessageUtil;
 import org.opendaylight.protocol.bgp.parser.BgpTableTypeImpl;
@@ -138,7 +136,7 @@ public class FSMTest {
         final BGPPeerRegistry peerRegistry = new StrictBGPPeerRegistry();
         peerRegistry.addPeer(new IpAddress(new Ipv4Address(peerAddress.getHostAddress())), new SimpleSessionListener(), prefs);
 
-        this.clientSession = new BGPClientSessionNegotiator(new DefaultPromise<BGPSessionImpl>(GlobalEventExecutor.INSTANCE), this.speakerListener, peerRegistry);
+        this.clientSession = new BGPClientSessionNegotiator(new DefaultPromise<>(GlobalEventExecutor.INSTANCE), this.speakerListener, peerRegistry);
 
         this.classicOpen = new OpenBuilder().setMyAsNumber(30).setHoldTimer(3).setVersion(new ProtocolVersion((short) 4)).setBgpParameters(
             tlvs).setBgpIdentifier(new Ipv4Address("1.1.1.2")).build();
@@ -146,7 +144,7 @@ public class FSMTest {
 
     @Test
     public void testDenyPeer() {
-        this.clientSession = new BGPClientSessionNegotiator(new DefaultPromise<BGPSessionImpl>(GlobalEventExecutor.INSTANCE), this.speakerListener, new StrictBGPPeerRegistry());
+        this.clientSession = new BGPClientSessionNegotiator(new DefaultPromise<>(GlobalEventExecutor.INSTANCE), this.speakerListener, new StrictBGPPeerRegistry());
         this.clientSession.channelActive(null);
         assertEquals(1, this.receivedMsgs.size());
         assertTrue(this.receivedMsgs.get(0) instanceof Notify);
index b2ccca18684b50678c32607083d1800013a8003c..c2d0f11a96be434a58ca731536f29f25e2d34811 100644 (file)
@@ -40,8 +40,6 @@ import org.junit.Test;
 import org.mockito.Matchers;
 import org.mockito.Mock;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataBroker;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataTreeChangeService;
index 22c8d543ecb9a7540712b42fc86dfbda1a821bea..34b11b6e0483ae94a3bb9426c81ae50f1c56a209 100644 (file)
@@ -24,8 +24,6 @@ import org.junit.Before;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.md.sal.common.api.data.TransactionChainListener;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataTreeChangeService;
index 968956288bd244b2468dc1689cd4b7ab509238b6..b78940127fca6238b918b530d24859cd4ab15a48 100644 (file)
@@ -23,8 +23,6 @@ import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mock;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataBroker;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataTreeChangeService;
 import org.opendaylight.controller.sal.core.api.model.SchemaService;
@@ -32,7 +30,6 @@ import org.opendaylight.mdsal.binding.dom.codec.api.BindingCodecTreeFactory;
 import org.opendaylight.mdsal.binding.generator.impl.GeneratedClassLoadingStrategy;
 import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonService;
 import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider;
-import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceRegistration;
 import org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier;
 import org.opendaylight.protocol.bgp.parser.BgpTableTypeImpl;
 import org.opendaylight.protocol.bgp.rib.impl.RIBImpl;
index e50006587441d9578c119c0da1f73f3ce9ed95dc..ea17327e361f1016a501ee7c2ff3bc984d18afd9 100644 (file)
@@ -61,7 +61,7 @@ public final class BGPMock implements Closeable {
 
                 final byte[] body = ByteArray.cutBytes(b, 1);
 
-                messages.add(registry.parseMessage(Unpooled.copiedBuffer(body)));
+                messages.add(registry.parseMessage(Unpooled.copiedBuffer(body), null));
             }
         } catch (final BGPDocumentedException | BGPParsingException e) {
             LOG.warn("Failed to parse message {}", e.getMessage(), e);
index 1f73b04aef952397888ecc1e83c01b4af03d0578..4b6cba2f0bc20f7e304c25b50bee87f4d7acd5ab 100644 (file)
@@ -21,8 +21,6 @@ import org.junit.Before;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.controller.md.sal.binding.impl.BindingToNormalizedNodeCodec;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction;
index 30f16db62cbd9babfe6c30219b2fbd5b5c89ab06..aabf76a6975278698e2e44bd8c16937e881a9614 100644 (file)
@@ -245,12 +245,7 @@ public class LinkstateTopologyBuilder extends AbstractTopologyBuilder<LinkstateR
         }
 
         private void addTp(final TerminationPoint tp, final LinkId link, final boolean isRemote) {
-            TpHolder h = this.tps.get(tp.getTpId());
-            if (h == null) {
-                h = new TpHolder(tp);
-                this.tps.put(tp.getTpId(), h);
-            }
-
+            final TpHolder h = this.tps.computeIfAbsent(tp.getTpId(), k -> new TpHolder(tp));
             h.addLink(link, isRemote);
         }
 
@@ -276,7 +271,7 @@ public class LinkstateTopologyBuilder extends AbstractTopologyBuilder<LinkstateR
             LOG.debug("Node {} is advertized", nb.getNodeId());
         }
 
-        private Object getNodeId() {
+        private NodeId getNodeId() {
             return this.nb.getNodeId();
         }
     }
index 3453972e79cf10399ec7d9759367fa5a56ccad6f..220c559e7c9f6a56215e2541dc964bc7521ad157 100644 (file)
@@ -49,7 +49,7 @@ public class BandwidthTest {
 
     @Test
     public void testHashCode() {
-        final Set<Bandwidth> set = new HashSet<Bandwidth>();
+        final Set<Bandwidth> set = new HashSet<>();
 
         set.add(this.b1);
         assertEquals(1, set.size());
index a936f019c5e65f4d638aeacf38d261fa15119c6c..e8e8edc0971b38f4d190de5a3edca0548879e586 100755 (executable)
@@ -45,7 +45,7 @@ public class DataChangeCounterImplModule extends org.opendaylight.controller.con
         final InstanceIdentifier<Topology> topoIId = InstanceIdentifier.builder(NetworkTopology.class)
                 .child(Topology.class, new TopologyKey(new TopologyId(getTopologyName()))).build();
         final ListenerRegistration<TopologyDataChangeCounter> registration = getDataProviderDependency().registerDataTreeChangeListener(
-                new DataTreeIdentifier<Topology>(LogicalDatastoreType.OPERATIONAL, topoIId), counter);
+            new DataTreeIdentifier<>(LogicalDatastoreType.OPERATIONAL, topoIId), counter);
         return new DataChangeCounterCloseable(counter, registration);
     }
 
index 5b11c4656420168571d4cbf957a220331d7260f7..7b408e0f62c33f1242114fd2e43fa214fb891422 100644 (file)
@@ -148,7 +148,7 @@ public class DataChangeCounterImplModuleTest extends AbstractConfigTest {
 
         @Override
         public Set<Class<? extends AbstractServiceInterface>> getImplementedServiceIntefaces() {
-            final java.util.Set<Class<? extends org.opendaylight.controller.config.api.annotations.AbstractServiceInterface>> serviceIfcs2 = new java.util.HashSet<Class<? extends org.opendaylight.controller.config.api.annotations.AbstractServiceInterface>>();
+            final java.util.Set<Class<? extends org.opendaylight.controller.config.api.annotations.AbstractServiceInterface>> serviceIfcs2 = new java.util.HashSet<>();
             return java.util.Collections.unmodifiableSet(serviceIfcs2);
         }
 
index 508e2c18b7db65c8528fb9d7d1f620ccaa7b4d34..b0e1ee5a43ba8aa0ed57b3baff90f895e60d723f 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.protocol.pcep.impl;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import io.netty.channel.Channel;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.handler.ssl.SslHandler;
 import io.netty.util.concurrent.Promise;
index b3b8163b60176117edf54bfa2e5d97a11909156b..2fdbc724555c0732b30e293bb2e1bde7aafe628f 100644 (file)
@@ -9,7 +9,6 @@ package org.opendaylight.protocol.pcep.impl;
 
 import com.google.common.base.Preconditions;
 import io.netty.channel.Channel;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.ChannelInboundHandlerAdapter;
index 261e2aaab910965e81c1ca0a0050fb3b791a5a1a..184ecfb907b879eb2613c4bb8d27366940d54def 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.protocol.pcep.impl;
 import com.google.common.base.Preconditions;
 import io.netty.buffer.ByteBuf;
 import io.netty.buffer.ByteBufUtil;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.handler.codec.ByteToMessageDecoder;
index 49c3ca41c495cebe5303c24a6efeabb3acfad713..73c82493c121ebd034d6f6d4a4ba6023eeb8c530 100644 (file)
@@ -14,7 +14,6 @@ import com.google.common.cache.CacheBuilder;
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.Map;
-import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 import javax.annotation.concurrent.GuardedBy;
index 46004e0faddec1c4e452ade8ac0f52000afa3e0d..e8c8967cb3e73fc43dd1a6da8e8b1dc1c2b315d5 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.protocol.pcep.impl;
 import com.google.common.base.Optional;
 import com.google.common.primitives.UnsignedBytes;
 import io.netty.channel.Channel;
-import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelFutureListener;
 import io.netty.util.concurrent.Promise;
 import java.net.InetSocketAddress;
index c65164e22f2b2b63970188b1a2e2f0098c3ec5ff..59ad5969e898cfc8fb1115bff593de881617efe5 100644 (file)
@@ -29,8 +29,6 @@ import java.util.concurrent.TimeUnit;
 import org.junit.Before;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.util.InetSocketAddressUtil;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.message.rev131007.Keepalive;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.message.rev131007.KeepaliveBuilder;
index 4721e82baf5c1a58a34feca8f7ce499ede758df4..121904533e96577b21bf9e10a5888289641238fe 100755 (executable)
@@ -12,7 +12,6 @@ import ch.qos.logback.classic.Level;
 import ch.qos.logback.classic.LoggerContext;
 import com.google.common.base.Optional;
 import com.google.common.base.Preconditions;
-import com.google.common.base.Predicate;
 import com.google.common.collect.Iterables;
 import com.google.common.collect.Lists;
 import com.google.common.net.InetAddresses;
index f63a8e6a069bd770f51b61411b93b2503c386b95..a8280f14017e8ff6aca50ec533ac6dbfff12971e 100644 (file)
@@ -22,7 +22,6 @@ import com.google.common.base.Preconditions;
 import com.google.common.net.InetAddresses;
 import io.netty.util.Timeout;
 import io.netty.util.Timer;
-import io.netty.util.TimerTask;
 import java.math.BigInteger;
 import java.net.InetAddress;
 import java.util.Collections;
index 6d74dc714d52718e0d83794adf33d0511deb4152..787d59e934ea4e2621595b81be858b9849cfd526 100755 (executable)
@@ -58,14 +58,14 @@ public final class PCCDispatcherImpl implements PCCDispatcher, AutoCloseable {
     }
 
     @Override
-    public Future<PCEPSession> createClient(@Nonnull final InetSocketAddress remoteAddress, @Nonnull final long reconnectTime,
+    public Future<PCEPSession> createClient(@Nonnull final InetSocketAddress remoteAddress, final long reconnectTime,
         @Nonnull final PCEPSessionListenerFactory listenerFactory, @Nonnull final PCEPSessionNegotiatorFactory negotiatorFactory,
         @Nullable final KeyMapping keys, @Nonnull final InetSocketAddress localAddress) {
         return createClient(remoteAddress, reconnectTime, listenerFactory, negotiatorFactory, keys, localAddress, BigInteger.ONE);
     }
 
     @Override
-    public Future<PCEPSession> createClient(@Nonnull final InetSocketAddress remoteAddress, @Nonnull final long reconnectTime,
+    public Future<PCEPSession> createClient(@Nonnull final InetSocketAddress remoteAddress, final long reconnectTime,
         @Nonnull final PCEPSessionListenerFactory listenerFactory, @Nonnull final PCEPSessionNegotiatorFactory negotiatorFactory,
         @Nullable final KeyMapping keys, @Nonnull final InetSocketAddress localAddress, @Nonnull final BigInteger dbVersion) {
         final Bootstrap b = new Bootstrap();
index c75f75bf354e403f55d328a9d3a44711f55a2efd..ce23797b0099183417dbe0cfcdbfe22d232ca869 100644 (file)
@@ -162,7 +162,7 @@ public final class MsgBuilderUtil {
         return Optional.of(tlvs);
     }
 
-    public static Pcerr createErrorMsg(@Nonnull final PCEPErrors e, @Nonnull final long srpId) {
+    public static Pcerr createErrorMsg(@Nonnull final PCEPErrors e, final long srpId) {
         final PcerrMessageBuilder msgBuilder = new PcerrMessageBuilder();
         return new PcerrBuilder().setPcerrMessage(
             msgBuilder
index 12b630728b8b2b337879e19838609a6c2560ccd7..f059f769ad085e3f0167921b05a9aa78cebdadfe 100644 (file)
@@ -25,8 +25,6 @@ import org.junit.Test;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.pcep.pcc.mock.api.PCCSession;
 import org.opendaylight.protocol.pcep.pcc.mock.api.PCCTunnelManager;
 import org.opendaylight.protocol.pcep.spi.PCEPErrors;
index 00a4d3e9c85bfea0779199cf02cfe40950125233..211c7bb07318aac016c441e696970cadbd27d482 100644 (file)
@@ -14,8 +14,6 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.ExecutionException;
 import org.opendaylight.protocol.pcep.PCEPCapability;
-import org.opendaylight.protocol.pcep.PCEPSessionListener;
-import org.opendaylight.protocol.pcep.PCEPSessionListenerFactory;
 import org.opendaylight.protocol.pcep.PCEPSessionNegotiatorFactory;
 import org.opendaylight.protocol.pcep.PCEPSessionProposalFactory;
 import org.opendaylight.protocol.pcep.impl.BasePCEPSessionProposalFactory;
index 09bab98c5738f2536d522b1664cccd22ea455b72..122f3dd5beb9138037c53003c4675a174252b346 100644 (file)
@@ -73,12 +73,7 @@ final class TopologyNodeState implements AutoCloseable, TransactionChainListener
     }
 
     public synchronized void cleanupExcept(final Collection<String> values) {
-        final Iterator<String> it = this.metadata.keySet().iterator();
-        while (it.hasNext()) {
-            if (!values.contains(it.next())) {
-                it.remove();
-            }
-        }
+        this.metadata.keySet().removeIf(s -> !values.contains(s));
     }
 
     public synchronized void released(final boolean persist) {
index f98ece68c24b49df6c480a86a115a85fa59b7e47..6ad5771ae3ee49f0b2452e53cf8ce6cf69b259d8 100644 (file)
@@ -8,7 +8,6 @@
 package org.opendaylight.bgpcep.pcep.topology.provider.config;
 
 import com.google.common.base.Optional;
-import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
index 7794bc54088d7a0f4f38e8de356432ade1c90f65..4d08045ee5151e65ccb618bed37f1321fd381727 100644 (file)
@@ -41,12 +41,12 @@ public class PCEPTopologyDeployerImpl implements PCEPTopologyDeployer, AutoClose
             LOG.warn("Topology Provider {} already exist. New instance won't be created", topologyId);
             return;
         }
-        final PCEPTopologyProviderBean PCEPTopologyProviderBean = (PCEPTopologyProviderBean) this.container
+        final PCEPTopologyProviderBean pcepTopologyProviderBean = (PCEPTopologyProviderBean) this.container
             .getComponentInstance(PCEPTopologyProviderBean.class.getSimpleName());
-        this.pcepTopologyServices.put(topologyId, PCEPTopologyProviderBean);
+        this.pcepTopologyServices.put(topologyId, pcepTopologyProviderBean);
         final PCEPTopologyConfigDependencies configDependencies = new PCEPTopologyConfigDependencies(inetSocketAddress,
             keys, schedulerDependency, topologyId, runtime, rpcTimeout);
-        PCEPTopologyProviderBean.start(configDependencies);
+        pcepTopologyProviderBean.start(configDependencies);
     }
 
     @Override
index cc43d1ce64da174cff5ed3566f0f0f1cdded5701..5c26e6fffb6d68b3d50db1bc43e0c9142c8b8f2d 100644 (file)
@@ -74,7 +74,7 @@ public final class PCEPTopologyProviderBean implements PCEPTopologyProviderDepen
         try {
             this.pcepTopoProviderCSS = new PCEPTopologyProviderBeanCSS(configDependencies);
         } catch (final Exception e) {
-            LOG.debug("Failed to create PCEPTopologyProvider {}", configDependencies.getTopologyId().getValue());
+            LOG.debug("Failed to create PCEPTopologyProvider {}", configDependencies.getTopologyId().getValue(), e);
         }
     }
 
index 834fae584fd5bf50f015656c6197577a4253dcab..5e8e0dea19151df7de17b804faa2678400239fdb 100644 (file)
@@ -426,7 +426,7 @@ public final class ProgrammingServiceImpl implements AutoCloseable, ClusterSingl
             try {
                 this.csspReg.close();
             } catch (final Exception e) {
-                LOG.debug("Failed to close Instruction Scheduler service");
+                LOG.debug("Failed to close Instruction Scheduler service", e);
             }
         }
         if (this.serviceRegistration != null) {
index 65efb3533179c86d0ddf215745f8d253e7207b7e..889b400b51c5b2318dd0df0c59a2247247aef014 100644 (file)
@@ -33,7 +33,6 @@ import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mock;
 import org.mockito.Mockito;
-import org.mockito.MockitoAnnotations;
 import org.opendaylight.bgpcep.programming.NanotimeUtil;
 import org.opendaylight.bgpcep.programming.spi.Instruction;
 import org.opendaylight.bgpcep.programming.spi.SchedulerException;
index c8767df248569e840de597a56467f8ee9ecac761..34cf490e7c534681edbb95177997fafc5fda5b8c 100644 (file)
@@ -54,7 +54,7 @@ public class SimpleRSVPExtensionProviderContext extends SimpleRSVPExtensionConsu
 
     public SimpleRSVPExtensionProviderContext(final int maximumCachedObjects) {
         final Cache<Object, Object> cache = CacheBuilder.newBuilder().maximumSize(maximumCachedObjects).build();
-        this.cacheRef = new AtomicReference<Cache<Object,Object>>(cache);
+        this.cacheRef = new AtomicReference<>(cache);
     }
 
     @Override
index 417f62764c06f43aa8ef59c7cfa63984c47c0f73..bd9afc90c9659493c10c9c5d81d9674abca4398c 100644 (file)
@@ -19,8 +19,6 @@ import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.EROSubobjectParser;
 import org.opendaylight.protocol.rsvp.parser.spi.EROSubobjectSerializer;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
index 273965e2ee2d9fb70eb4f039e7d317864ff8f0f2..9d116824bce59cd8ad4a1c3a6b7b892fa7b49b05 100644 (file)
@@ -19,8 +19,6 @@ import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.RROSubobjectParser;
 import org.opendaylight.protocol.rsvp.parser.spi.RROSubobjectSerializer;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
index 396f1da7b639c74c6603a7ab1112427818acdaf8..71bb0b0e6671bfd4fbff74e111ef2eac90c32cec 100644 (file)
@@ -19,8 +19,6 @@ import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
 import org.opendaylight.protocol.rsvp.parser.spi.XROSubobjectParser;
 import org.opendaylight.protocol.rsvp.parser.spi.XROSubobjectSerializer;
index d00575f309059e0ec8a93497ce4c17036cc84ad3..6d6cf4ab5a2470e60e2b31bd3efbebe6190b7521 100644 (file)
@@ -18,8 +18,6 @@ import java.util.List;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.EROSubobjectRegistry;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev150820.AttributeFilter;
index ffbaeec6f4dfbc0a20a9b2f0875d0cc9db73262b..6912aac3911b1e7f43af2680f852fbcbb02e5c8b 100644 (file)
@@ -15,8 +15,6 @@ import java.util.List;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.RROSubobjectRegistry;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev150820.RsvpTeObject;
index ec82d97b4ced7983d6210d807159445f98e4b2c5..7c2cc8bf1480d2d042f1308a47664b2a9b7fc1b9 100644 (file)
@@ -15,8 +15,6 @@ import java.util.List;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
-import org.mockito.invocation.InvocationOnMock;
-import org.mockito.stubbing.Answer;
 import org.opendaylight.protocol.rsvp.parser.spi.RSVPParsingException;
 import org.opendaylight.protocol.rsvp.parser.spi.XROSubobjectRegistry;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev150820.RsvpTeObject;
index fac0ba28a3dac9f22cb29de08f1d06f51ee41290..5a096e33f1c500410b74feacaaea1c6605313a09 100644 (file)
@@ -115,8 +115,7 @@ public class ByteArrayTest {
         ByteArray.cutBytes(this.before, 0);
     }
 
-    private final byte[] inBytes = { (byte) 0x03, (byte) 0xFF, (byte) 0x01, (byte) 0x80 };
-    final BitSet inBitSet = new BitSet();
+    private final BitSet inBitSet = new BitSet();
 
     @Before
     public void generateBitSet() {