From: Tony Tkacik <ttkacik@cisco.com>
Date: Tue, 24 Feb 2015 09:39:34 +0000 (+0000)
Subject: Merge "Bug 2731: Discard changes only when transaction exist."
X-Git-Tag: release/lithium~473^2~5
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=commitdiff_plain;h=5c008222efa5c0af49cf8a52881a6299b1e249dc;hp=-c

Merge "Bug 2731: Discard changes only when transaction exist."
---

5c008222efa5c0af49cf8a52881a6299b1e249dc
diff --combined opendaylight/netconf/config-netconf-connector/src/test/java/org/opendaylight/controller/netconf/confignetconfconnector/NetconfMappingTest.java
index 356223515a,fdae71a16b..d6b5e62b27
--- a/opendaylight/netconf/config-netconf-connector/src/test/java/org/opendaylight/controller/netconf/confignetconfconnector/NetconfMappingTest.java
+++ b/opendaylight/netconf/config-netconf-connector/src/test/java/org/opendaylight/controller/netconf/confignetconfconnector/NetconfMappingTest.java
@@@ -95,8 -95,8 +95,8 @@@ import org.opendaylight.controller.netc
  import org.opendaylight.controller.netconf.confignetconfconnector.osgi.YangStoreService;
  import org.opendaylight.controller.netconf.confignetconfconnector.transactions.TransactionProvider;
  import org.opendaylight.controller.netconf.impl.mapping.operations.DefaultCloseSession;
 +import org.opendaylight.controller.netconf.impl.osgi.AggregatedNetconfOperationServiceFactory;
  import org.opendaylight.controller.netconf.impl.osgi.NetconfOperationRouter;
 -import org.opendaylight.controller.netconf.impl.osgi.NetconfOperationServiceSnapshotImpl;
  import org.opendaylight.controller.netconf.mapping.api.HandlingPriority;
  import org.opendaylight.controller.netconf.mapping.api.NetconfOperation;
  import org.opendaylight.controller.netconf.mapping.api.NetconfOperationChainedExecution;
@@@ -141,9 -141,7 +141,9 @@@ public class NetconfMappingTest extend
      @Mock
      NetconfOperationRouter netconfOperationRouter;
      @Mock
 -    NetconfOperationServiceSnapshotImpl netconfOperationServiceSnapshot;
 +    AggregatedNetconfOperationServiceFactory netconfOperationServiceSnapshot;
 +    @Mock
 +    private AutoCloseable sessionCloseable;
  
      private TransactionProvider transactionProvider;
  
@@@ -159,12 -157,13 +159,12 @@@
  
          doReturn(getMbes()).when(this.yangStoreSnapshot).getModuleMXBeanEntryMap();
          doReturn(getModules()).when(this.yangStoreSnapshot).getModules();
 -        doNothing().when(netconfOperationServiceSnapshot).close();
  
          this.factory = new NetconfTestImplModuleFactory();
          this.factory2 = new DepTestImplModuleFactory();
          this.factory3 = new IdentityTestModuleFactory();
          factory4 = new TestImplModuleFactory();
 -
 +        doNothing().when(sessionCloseable).close();
  
          super.initConfigTransactionManagerImpl(new HardcodedModuleFactoriesResolver(mockedContext, this.factory, this.factory2,
                  this.factory3, factory4));
@@@ -377,7 -376,7 +377,7 @@@
  
          edit("netconfMessages/editConfig_none.xml");
          closeSession();
 -        verify(netconfOperationServiceSnapshot).close();
 +        verify(sessionCloseable).close();
          verifyNoMoreInteractions(netconfOperationRouter);
          verifyNoMoreInteractions(netconfOperationServiceSnapshot);
      }
@@@ -391,7 -390,7 +391,7 @@@
  
      private void closeSession() throws NetconfDocumentedException, ParserConfigurationException, SAXException,
              IOException {
 -        DefaultCloseSession closeOp = new DefaultCloseSession(NETCONF_SESSION_ID, netconfOperationServiceSnapshot);
 +        DefaultCloseSession closeOp = new DefaultCloseSession(NETCONF_SESSION_ID, sessionCloseable);
          executeOp(closeOp, "netconfMessages/closeSession.xml");
      }
  
@@@ -569,14 -568,14 +569,14 @@@
          commit();
      }
  
-     @Test(expected = NetconfDocumentedException.class)
+     @Test
      public void testEx2() throws Exception {
-         discard();
+         assertContainsElement(discard(), readXmlToElement("<ok xmlns=\"urn:ietf:params:xml:ns:netconf:base:1.0\"/>"));
      }
  
-     private void discard() throws ParserConfigurationException, SAXException, IOException, NetconfDocumentedException {
+     private Document discard() throws ParserConfigurationException, SAXException, IOException, NetconfDocumentedException {
          DiscardChanges discardOp = new DiscardChanges(transactionProvider, configRegistryClient, NETCONF_SESSION_ID);
-         executeOp(discardOp, "netconfMessages/discardChanges.xml");
+         return executeOp(discardOp, "netconfMessages/discardChanges.xml");
      }
  
      private void checkBinaryLeafEdited(final Document response) throws NodeTestException, SAXException, IOException {
@@@ -716,7 -715,7 +716,7 @@@
      }
  
      private Document get() throws NetconfDocumentedException, ParserConfigurationException, SAXException, IOException {
 -        Get getOp = new Get(yangStoreSnapshot, configRegistryClient, NETCONF_SESSION_ID);
 +        Get getOp = new Get(transactionProvider, yangStoreSnapshot, configRegistryClient, NETCONF_SESSION_ID);
          return executeOp(getOp, "netconfMessages/get.xml");
      }