Add blueprint wiring for neutron renderer 98/38998/8
authorAlexis de Talhouët <adetalhouet@inocybe.com>
Tue, 17 May 2016 17:41:00 +0000 (13:41 -0400)
committerAlexis de Talhouët <adetalhouet@inocybe.com>
Fri, 1 Jul 2016 03:19:24 +0000 (03:19 +0000)
Change-Id: I00fcd2f7d661ec38802974fc39da4a4009b37d43
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
19 files changed:
features/pom.xml
features/src/main/features/features.xml
netvirt/renderers/neutron/src/main/config/default-config.xml [deleted file]
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/Constants.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/DelegatingDataTreeListener.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/MdsalHelper.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronNetworkChangeListener.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronNetworkDataProcessor.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronPortChangeListener.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronPortDataProcessor.java
netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProvider.java [deleted file]
netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModule.java [deleted file]
netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactory.java [deleted file]
netvirt/renderers/neutron/src/main/resources/org/opendaylight/blueprint/netvirt.xml [new file with mode: 0644]
netvirt/renderers/neutron/src/main/yang/netvirt-neutron.yang [deleted file]
netvirt/renderers/neutron/src/test/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronPortDataProcessorTest.java
netvirt/renderers/neutron/src/test/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProviderTest.java [deleted file]
netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactoryTest.java [deleted file]
netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleTest.java [deleted file]

index 899593b5407591fc4bd5e6232dee825bcdea136a..821a09ceb2b0c63419cdde19b16e24d2e28f79fa 100644 (file)
@@ -241,13 +241,6 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <artifactId>neutron</artifactId>
       <version>${project.version}</version>
     </dependency>
-    <dependency>
-      <groupId>${project.groupId}</groupId>
-      <artifactId>neutron</artifactId>
-      <version>${project.version}</version>
-      <type>xml</type>
-      <classifier>config</classifier>
-    </dependency>
     <dependency>
       <groupId>org.opendaylight.dlux</groupId>
       <artifactId>features-dlux</artifactId>
index 187238100db320b45984e2648500b2105afaea53..a44b706359b6992fc00d560b4e24bf3f6d0ea1dd 100644 (file)
@@ -75,7 +75,6 @@
     <bundle>mvn:org.opendaylight.netvirt/utils.mdsal-utils/{{VERSION}}</bundle>
     <bundle>mvn:org.opendaylight.ovsdb/utils.mdsal-utils/{{VERSION}}</bundle>
     <bundle>mvn:org.opendaylight.netvirt/neutron/{{VERSION}}</bundle>
-    <configfile finalname="etc/opendaylight/karaf/netvirt-neutron-default-config.xml">mvn:org.opendaylight.netvirt/neutron/{{VERSION}}/xml/config</configfile>
   </feature>
 
   <feature name='odl-netvirt-it' version='${project.version}' description='OpenDaylight :: netvirt :: IT'>
diff --git a/netvirt/renderers/neutron/src/main/config/default-config.xml b/netvirt/renderers/neutron/src/main/config/default-config.xml
deleted file mode 100644 (file)
index 7cb1144..0000000
+++ /dev/null
@@ -1,29 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!-- vi: set et smarttab sw=4 tabstop=4: -->
-<!--
-Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
-
-This program and the accompanying materials are made available under the
-terms of the Eclipse Public License v1.0 which accompanies this distribution,
-and is available at http://www.eclipse.org/legal/epl-v10.html
--->
-<snapshot>
-  <required-capabilities>
-      <capability>urn:opendaylight:params:xml:ns:yang:netvirt:neutron?module=netvirt-neutron&amp;revision=2016-03-08</capability>
-      <capability>urn:opendaylight:params:xml:ns:yang:controller:md:sal:binding?module=opendaylight-md-sal-binding&amp;revision=2013-10-28</capability>
-  </required-capabilities>
-  <configuration>
-    <data xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
-      <modules xmlns="urn:opendaylight:params:xml:ns:yang:controller:config">
-        <module>
-          <type xmlns:prefix="urn:opendaylight:params:xml:ns:yang:netvirt:neutron">prefix:netvirt-neutron</type>
-          <name>netvirt-neutron</name>
-          <broker>
-            <type xmlns:binding="urn:opendaylight:params:xml:ns:yang:controller:md:sal:binding">binding:binding-broker-osgi-registry</type>
-            <name>binding-osgi-broker</name>
-          </broker>
-        </module>
-      </modules>
-    </data>
-  </configuration>
-</snapshot>
index a47ce6694e3fe430e0cd3896358c810389477f3f..f4bfd00d6c9f59a670d97f1fe10e99840647cdd5 100644 (file)
@@ -9,15 +9,18 @@
 package org.opendaylight.netvirt.netvirt.renderers.neutron;
 
 import com.google.common.collect.ImmutableBiMap;
-import com.google.common.collect.ImmutableBiMap.Builder;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.*;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.NetworkTypeBase;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.NetworkTypeFlat;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.NetworkTypeGre;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.NetworkTypeVlan;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.common.rev151227.NetworkTypeVxlan;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeBase;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeComputeNova;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeDhcp;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeFloatingIp;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeRouter;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeRouterGateway;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeRouterInterfaceDistributed;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.PortTypeRouter;
 
 /**
  * A collection of configuration constants
index fd35781193f45a62ca8fe0a0bffc1ee4b1967a5a..51e5e272bce3ce5fe3102d523ac54f99a3cd8ca4 100644 (file)
@@ -28,7 +28,6 @@ import org.slf4j.LoggerFactory;
  */
 public class DelegatingDataTreeListener<T extends DataObject> implements AutoCloseable, ClusteredDataTreeChangeListener<T> {
     private static final Logger LOG = LoggerFactory.getLogger(DelegatingDataTreeListener.class);
-    protected NeutronProvider provider;
     protected DataBroker db;
     private static final ThreadFactory threadFactory = new ThreadFactoryBuilder()
         .setNameFormat("NV-NeutronDTL-%d").build();
@@ -36,9 +35,8 @@ public class DelegatingDataTreeListener<T extends DataObject> implements AutoClo
     private final DataProcessor<T> dataProcessor;
     private ListenerRegistration<DelegatingDataTreeListener<T>> listenerRegistration;
 
-    public DelegatingDataTreeListener(NeutronProvider provider, DataProcessor<T> dataProcessor,
+    public DelegatingDataTreeListener(DataProcessor<T> dataProcessor,
                                       DataBroker db, DataTreeIdentifier<T> treeId) {
-        this.provider = Preconditions.checkNotNull(provider, "provider can not be null!");
         this.dataProcessor = Preconditions.checkNotNull(dataProcessor, "Data processor can not be null!");
         registerListener(Preconditions.checkNotNull(db, "Data broker can not be null!"),
                 Preconditions.checkNotNull(treeId, "Tree identifier can not be null!"));
index 18923a681364b4e16aacfabbfc280a634a314e53..3e52e542bc0b1e77fff47e25e707449af31c64a7 100644 (file)
@@ -19,11 +19,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.Por
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.ports.Port;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.ports.PortKey;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class MdsalHelper {
-    private static final Logger LOG = LoggerFactory.getLogger(MdsalHelper.class);
     public static InstanceIdentifier<Port> createPortInstanceIdentifier(Uuid portUuid) {
         return InstanceIdentifier.create(Ports.class)
                 .child(Port.class, new PortKey(portUuid));
index afbe7a9427de5b01ba39f254e98db5e3e47b6793..79df9472743fd3987866470536ef6a46a88a1a8a 100644 (file)
@@ -21,11 +21,10 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 public class NeutronNetworkChangeListener extends DelegatingDataTreeListener<Network> {
     /**
      * {@link NeutronNetworkChangeListener} constructor.
-     * @param provider Neutron Provider
      * @param db MdSal {@link DataBroker}
      */
-    public NeutronNetworkChangeListener(final NeutronProvider provider, final DataBroker db) {
-        super(provider, new NeutronNetworkDataProcessor(provider, db), db,
+    public NeutronNetworkChangeListener(final DataBroker db) {
+        super(new NeutronNetworkDataProcessor(db), db,
                 new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION,
                         InstanceIdentifier.create(Neutron.class).child(Networks.class).child(Network.class)));
     }
index e83a6c16d58609bf0db7eda3d0eaf24799347fc5..bed87c64ceb46eaeae05f5f8e6528e9bfab45380 100644 (file)
@@ -28,15 +28,12 @@ import org.slf4j.LoggerFactory;
 public class NeutronNetworkDataProcessor implements DataProcessor<Network> {
     private static final Logger LOG = LoggerFactory.getLogger(NeutronNetworkDataProcessor.class);
     private final MdsalUtils mdsalUtils;
-    private final NeutronProvider provider;
 
     /**
      *
-     * @param provider - Neutron provider
      * @param db - mdsal
      */
-    public NeutronNetworkDataProcessor(final NeutronProvider provider, DataBroker db) {
-        this.provider = Preconditions.checkNotNull(provider, "Provider can not be null!");
+    public NeutronNetworkDataProcessor(DataBroker db) {
         mdsalUtils = new MdsalUtils(db);
     }
 
index 8ac6eddd39f24df69886d3ff19099a18b871cb4b..83ffac2700f9ca195743a80355fb36f9d29f23b8 100644 (file)
@@ -21,11 +21,10 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 public class NeutronPortChangeListener extends DelegatingDataTreeListener<Port> {
     /**
      * {@link NeutronPortChangeListener} constructor.
-     * @param provider Neutron Provider
      * @param db MdSal {@link DataBroker}
      */
-    public NeutronPortChangeListener(final NeutronProvider provider, final DataBroker db) {
-        super(provider, new NeutronPortDataProcessor(provider, db), db,
+    public NeutronPortChangeListener(final DataBroker db) {
+        super(new NeutronPortDataProcessor(db), db,
                 new DataTreeIdentifier<>(LogicalDatastoreType.CONFIGURATION,
                         InstanceIdentifier.create(Neutron.class).child(Ports.class).child(Port.class)));
     }
index 6d5e76bb2dbd9777eb79202af27dabefee0f84ed..54d16b69ea642bb38093fd005bf5f3201480d7cc 100644 (file)
@@ -35,15 +35,12 @@ import org.slf4j.LoggerFactory;
 public class NeutronPortDataProcessor implements DataProcessor<Port> {
     private static final Logger LOG = LoggerFactory.getLogger(NeutronPortDataProcessor.class);
     private MdsalUtils mdsalUtils;
-    private final NeutronProvider provider;
 
     /**
      *
-     * @param provider - Neutron provider
      * @param db - mdsal
      */
-    public NeutronPortDataProcessor(final NeutronProvider provider, DataBroker db) {
-        this.provider = Preconditions.checkNotNull(provider, "Provider can not be null!");
+    public NeutronPortDataProcessor(DataBroker db) {
         mdsalUtils = new MdsalUtils(db);
     }
 
diff --git a/netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProvider.java b/netvirt/renderers/neutron/src/main/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProvider.java
deleted file mode 100644 (file)
index 41f33f3..0000000
+++ /dev/null
@@ -1,70 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * 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.netvirt.netvirt.renderers.neutron;
-
-import com.google.common.base.Optional;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipState;
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
-import org.opendaylight.controller.md.sal.common.api.clustering.Entity;
-import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipService;
-import org.opendaylight.controller.sal.binding.api.BindingAwareBroker.ProviderContext;
-import org.opendaylight.controller.sal.binding.api.BindingAwareProvider;
-import org.osgi.framework.BundleContext;
-
-public class NeutronProvider implements BindingAwareProvider, AutoCloseable {
-    private static final Logger LOG = LoggerFactory.getLogger(NeutronProvider.class);
-    private BundleContext bundleContext = null;
-    private static DataBroker dataBroker = null;
-    private static EntityOwnershipService entityOwnershipService;
-    private static final Entity ownerInstanceEntity = new Entity(Constants.NETVIRT_NEUTRON_OWNER_ENTITY_TYPE,
-            Constants.NETVIRT_NEUTRON_OWNER_ENTITY_TYPE);
-    private NeutronPortChangeListener neutronPortChangeListener;
-    private NeutronNetworkChangeListener neutronNetworkChangeListener;
-
-    public NeutronProvider(BundleContext bundleContext, EntityOwnershipService eos) {
-        LOG.info("Netvirt NeutronProvider: bundleContext: {}", bundleContext);
-        this.bundleContext = bundleContext;
-        entityOwnershipService = eos;
-    }
-
-    public NeutronProvider() {
-        LOG.info("Netvirt NeutronProvider created");
-    }
-
-    public static boolean isMasterProviderInstance() {
-        if (entityOwnershipService != null) {
-            Optional<EntityOwnershipState> state = entityOwnershipService.getOwnershipState(ownerInstanceEntity);
-            return state.isPresent() && state.get().isOwner();
-        }
-        return false;
-    }
-
-
-    @Override
-    public void onSessionInitiated(ProviderContext providerContext) {
-        dataBroker = providerContext.getSALService(DataBroker.class);
-        LOG.info("Netvirt NeutronProvider: onSessionInitiated dataBroker: {}", dataBroker);
-
-        neutronPortChangeListener = new NeutronPortChangeListener(this, dataBroker);
-        neutronNetworkChangeListener = new NeutronNetworkChangeListener(this, dataBroker);
-    }
-
-    @Override
-    public void close() throws Exception {
-        if (neutronPortChangeListener != null) {
-            neutronPortChangeListener.close();
-        }
-        if (neutronNetworkChangeListener != null) {
-            neutronNetworkChangeListener.close();
-        }
-        LOG.info("Netvirt NeutronProvider Closed");
-    }
-
-}
diff --git a/netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModule.java b/netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModule.java
deleted file mode 100644 (file)
index 52cc310..0000000
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * 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.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308;
-
-import org.opendaylight.netvirt.netvirt.renderers.neutron.NeutronProvider;
-
-public class NeutronModule extends org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308.AbstractNeutronModule {
-    public NeutronModule(org.opendaylight.controller.config.api.ModuleIdentifier identifier, org.opendaylight.controller.config.api.DependencyResolver dependencyResolver) {
-        super(identifier, dependencyResolver);
-    }
-
-    public NeutronModule(org.opendaylight.controller.config.api.ModuleIdentifier identifier, org.opendaylight.controller.config.api.DependencyResolver dependencyResolver, org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308.NeutronModule oldModule, java.lang.AutoCloseable oldInstance) {
-        super(identifier, dependencyResolver, oldModule, oldInstance);
-    }
-
-    @Override
-    public void customValidation() {
-        // add custom validation form module attributes here.
-    }
-
-    @Override
-    public java.lang.AutoCloseable createInstance() {
-        NeutronProvider provider = new NeutronProvider();
-        getBrokerDependency().registerProvider(provider);
-        return provider;
-    }
-
-}
diff --git a/netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactory.java b/netvirt/renderers/neutron/src/main/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactory.java
deleted file mode 100644 (file)
index 98cd54a..0000000
+++ /dev/null
@@ -1,19 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * terms of the Eclipse Public License v1.0 which accompanies this distribution,
- * and is available at http://www.eclipse.org/legal/epl-v10.html
- */
-/*
-* Generated file
-*
-* Generated from: yang module name: neutron yang module local name: neutron
-* Generated by: org.opendaylight.controller.config.yangjmxgenerator.plugin.JMXGenerator
-*
-* Do not modify this file unless it is present under src/main directory
-*/
-package org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308;
-public class NeutronModuleFactory extends org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308.AbstractNeutronModuleFactory {
-
-}
diff --git a/netvirt/renderers/neutron/src/main/resources/org/opendaylight/blueprint/netvirt.xml b/netvirt/renderers/neutron/src/main/resources/org/opendaylight/blueprint/netvirt.xml
new file mode 100644 (file)
index 0000000..c9d1574
--- /dev/null
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<blueprint xmlns="http://www.osgi.org/xmlns/blueprint/v1.0.0"
+    xmlns:odl="http://opendaylight.org/xmlns/blueprint/v1.0.0"
+    odl:use-default-for-reference-types="true">
+
+    <reference id="dataBroker"
+        interface="org.opendaylight.controller.md.sal.binding.api.DataBroker"
+        odl:type="default" />
+
+    <bean id="neutronPortChangeListener"
+        class="org.opendaylight.netvirt.netvirt.renderers.neutron.NeutronPortChangeListener"
+        destroy-method="close">
+        <argument ref="dataBroker" />
+    </bean>
+
+    <bean id="neutronNetworkChangeListener"
+        class="org.opendaylight.netvirt.netvirt.renderers.neutron.NeutronNetworkChangeListener"
+        destroy-method="close">
+        <argument ref="dataBroker" />
+    </bean>
+</blueprint>
\ No newline at end of file
diff --git a/netvirt/renderers/neutron/src/main/yang/netvirt-neutron.yang b/netvirt/renderers/neutron/src/main/yang/netvirt-neutron.yang
deleted file mode 100644 (file)
index 85f7888..0000000
+++ /dev/null
@@ -1,36 +0,0 @@
-module netvirt-neutron {
-    yang-version 1;
-    namespace "urn:opendaylight:params:xml:ns:yang:netvirt:neutron";
-    prefix "neutron";
-
-    import config { prefix config; revision-date 2013-04-05; }
-    import opendaylight-md-sal-binding { prefix md-sal-binding; revision-date 2013-10-28;}
-    import opendaylight-entity-ownership-service {prefix eos; revision-date 2015-08-10;}
-
-    description
-        "Service definition for NetVirt Neutron renderer";
-
-    revision "2016-03-08" {
-        description
-            "Initial revision";
-    }
-
-    identity netvirt-neutron {
-        base config:module-type;
-        config:java-name-prefix Neutron;
-    }
-
-    augment "/config:modules/config:module/config:configuration" {
-        case netvirt-neutron {
-            when "/config:modules/config:module/config:type = 'netvirt-neutron'";
-            container broker {
-                uses config:service-ref {
-                    refine type {
-                        mandatory true;
-                        config:required-identity md-sal-binding:binding-broker-osgi-registry;
-                    }
-                }
-            }
-        }
-    }
-}
index d3295615ee0168e0199653e57ed500dba1a8a0c3..f63f1d6556b936d208935469c9a883500fb3fd85 100644 (file)
@@ -1,9 +1,12 @@
 package org.opendaylight.netvirt.netvirt.renderers.neutron;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
+import com.google.common.base.Optional;
 import java.util.ArrayList;
 import java.util.List;
 import org.junit.Test;
@@ -16,11 +19,10 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.Uuid;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.port.attributes.FixedIps;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.port.attributes.FixedIpsBuilder;
-import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.ports.attributes.Ports;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.ports.attributes.ports.Port;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.ports.attributes.ports.PortBuilder;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.ports.attributes.Ports;
-import com.google.common.base.Optional;
+import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
 /**
  * Unit test for {@link NeutronPortDataProcessor}
@@ -37,7 +39,6 @@ public class NeutronPortDataProcessorTest extends AbstractDataBrokerTest {
         InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.ports.Port> portIid =
                 MdsalHelper.createPortInstanceIdentifier(uuid);
 
-        org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.ports.Port netvirtPort = null;
         Optional<org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.ports.rev151227.ports.Port> data =
             getDataBroker()
             .newReadOnlyTransaction()
@@ -71,7 +72,7 @@ public class NeutronPortDataProcessorTest extends AbstractDataBrokerTest {
         if (!initialized) {
             session = mock(ProviderContext.class);
             when(session.getSALService(DataBroker.class)).thenReturn(getDataBroker());
-            neutronPortDataProcessor = new NeutronPortDataProcessor(new NeutronProvider(), session.getSALService(DataBroker.class));
+            neutronPortDataProcessor = new NeutronPortDataProcessor(getDataBroker());
             initialized = true;
         }
     }
diff --git a/netvirt/renderers/neutron/src/test/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProviderTest.java b/netvirt/renderers/neutron/src/test/java/org/opendaylight/netvirt/netvirt/renderers/neutron/NeutronProviderTest.java
deleted file mode 100644 (file)
index dbcc924..0000000
+++ /dev/null
@@ -1,38 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * 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.netvirt.netvirt.renderers.neutron;
-
-import org.junit.Test;
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
-import org.opendaylight.controller.sal.binding.api.BindingAwareBroker;
-import org.opendaylight.controller.sal.binding.api.BindingAwareBroker.ProviderContext;
-
-import static org.mockito.Mockito.mock;
-
-public class NeutronProviderTest {
-    @Test
-    public void testOnSessionInitiated() {
-
-        NeutronProvider provider = new NeutronProvider();
-
-        // ensure no exceptions
-        // currently this method is empty
-
-        //TODO
-        //provider.onSessionInitiated(mock(BindingAwareBroker.ProviderContext.class));
-    }
-
-    @Test
-    public void testClose() throws Exception {
-        NeutronProvider provider = new NeutronProvider();
-
-        // ensure no exceptions
-        // currently this method is empty
-        provider.close();
-    }
-}
diff --git a/netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactoryTest.java b/netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleFactoryTest.java
deleted file mode 100644 (file)
index 76bf797..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * 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.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308;
-
-import org.junit.Test;
-
-public class NeutronModuleFactoryTest {
-    @Test
-    public void testFactoryConstructor() {
-        // ensure no exceptions on construction
-        new NeutronModuleFactory();
-    }
-}
diff --git a/netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleTest.java b/netvirt/renderers/neutron/src/test/java/org/opendaylight/yang/gen/v1/urn/opendaylight/params/xml/ns/yang/netvirt/neutron/rev160308/NeutronModuleTest.java
deleted file mode 100644 (file)
index 46b2228..0000000
+++ /dev/null
@@ -1,54 +0,0 @@
-/*
- * Copyright (c) 2016 Red Hat, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * 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.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.neutron.rev160308;
-
-import org.junit.Test;
-import org.opendaylight.controller.config.api.DependencyResolver;
-import org.opendaylight.controller.config.api.JmxAttribute;
-import org.opendaylight.controller.config.api.ModuleIdentifier;
-import org.opendaylight.controller.sal.binding.api.BindingAwareBroker;
-import org.opendaylight.netvirt.netvirt.renderers.neutron.NeutronProvider;
-
-import javax.management.ObjectName;
-
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-public class NeutronModuleTest {
-    @Test
-    public void testCustomValidation() {
-        NeutronModule module = new NeutronModule(mock(ModuleIdentifier.class), mock(DependencyResolver.class));
-
-        // ensure no exceptions on validation
-        // currently this method is empty
-        module.customValidation();
-    }
-
-    @Test
-    public void testCreateInstance() throws Exception {
-        // configure mocks
-        DependencyResolver dependencyResolver = mock(DependencyResolver.class);
-        BindingAwareBroker broker = mock(BindingAwareBroker.class);
-        when(dependencyResolver.resolveInstance(eq(BindingAwareBroker.class), any(ObjectName.class), any(JmxAttribute.class))).thenReturn(broker);
-
-        // create instance of module with injected mocks
-        NeutronModule module = new NeutronModule(mock(ModuleIdentifier.class), dependencyResolver);
-
-        // getInstance calls resolveInstance to get the broker dependency and then calls createInstance
-        AutoCloseable closeable = module.getInstance();
-
-        // verify that the module registered the returned provider with the broker
-        verify(broker).registerProvider((NeutronProvider)closeable);
-
-        // ensure no exceptions on close
-        closeable.close();
-    }
-}