X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=dom%2Fmdsal-dom-spi%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fmdsal%2Fdom%2Fspi%2Fstore%2FSnapshotBackedWriteTransactionTest.java;h=ae8ce6c3f5603ab04326f529154fc55ccd0beddb;hb=refs%2Fchanges%2F79%2F97379%2F60;hp=07e5b0c8e3c96661109011a162ece8a9ee064fa5;hpb=0a8bae948f989800cbd6002ac161babe5a59bd12;p=mdsal.git diff --git a/dom/mdsal-dom-spi/src/test/java/org/opendaylight/mdsal/dom/spi/store/SnapshotBackedWriteTransactionTest.java b/dom/mdsal-dom-spi/src/test/java/org/opendaylight/mdsal/dom/spi/store/SnapshotBackedWriteTransactionTest.java index 07e5b0c8e3..ae8ce6c3f5 100644 --- a/dom/mdsal-dom-spi/src/test/java/org/opendaylight/mdsal/dom/spi/store/SnapshotBackedWriteTransactionTest.java +++ b/dom/mdsal-dom-spi/src/test/java/org/opendaylight/mdsal/dom/spi/store/SnapshotBackedWriteTransactionTest.java @@ -9,9 +9,10 @@ package org.opendaylight.mdsal.dom.spi.store; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; -import static org.mockito.Matchers.any; -import static org.mockito.Matchers.eq; -import static org.mockito.Matchers.same; +import static org.mockito.ArgumentMatchers.any; +import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.ArgumentMatchers.same; +import static org.mockito.Mockito.doAnswer; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.doThrow; @@ -26,8 +27,8 @@ import org.junit.Test; import org.opendaylight.mdsal.dom.spi.store.SnapshotBackedWriteTransaction.TransactionReadyPrototype; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; -import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeModification; -import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeSnapshot; +import org.opendaylight.yangtools.yang.data.tree.api.DataTreeModification; +import org.opendaylight.yangtools.yang.data.tree.api.DataTreeSnapshot; public class SnapshotBackedWriteTransactionTest { @@ -37,8 +38,8 @@ public class SnapshotBackedWriteTransactionTest { mock(TransactionReadyPrototype.class); private static final DOMStoreThreePhaseCommitCohort DOM_STORE_THREE_PHASE_COMMIT_COHORT = mock(DOMStoreThreePhaseCommitCohort.class); - private static final NormalizedNode NORMALIZED_NODE = mock(NormalizedNode.class); - private static final Optional> NORMALIZED_NODE_OPTIONAL = Optional.of(NORMALIZED_NODE); + private static final NormalizedNode NORMALIZED_NODE = mock(NormalizedNode.class); + private static final Optional NORMALIZED_NODE_OPTIONAL = Optional.of(NORMALIZED_NODE); private static SnapshotBackedWriteTransaction snapshotBackedWriteTransaction; @Before @@ -54,24 +55,24 @@ public class SnapshotBackedWriteTransactionTest { doReturn(DOM_STORE_THREE_PHASE_COMMIT_COHORT) .when(TRANSACTION_READY_PROTOTYPE) .transactionReady(any(),any(), any()); - doReturn(NORMALIZED_NODE_OPTIONAL).when(DATA_TREE_MODIFICATION).readNode(YangInstanceIdentifier.EMPTY); + doReturn(NORMALIZED_NODE_OPTIONAL).when(DATA_TREE_MODIFICATION).readNode(YangInstanceIdentifier.empty()); snapshotBackedWriteTransaction = new SnapshotBackedWriteTransaction<>(new Object(), false, DATA_TREE_SNAPSHOT, TRANSACTION_READY_PROTOTYPE); } @Test public void basicTest() throws Exception { - snapshotBackedWriteTransaction.write(YangInstanceIdentifier.EMPTY, NORMALIZED_NODE); + snapshotBackedWriteTransaction.write(YangInstanceIdentifier.empty(), NORMALIZED_NODE); verify(DATA_TREE_MODIFICATION).write(any(), any()); - snapshotBackedWriteTransaction.merge(YangInstanceIdentifier.EMPTY, NORMALIZED_NODE); + snapshotBackedWriteTransaction.merge(YangInstanceIdentifier.empty(), NORMALIZED_NODE); verify(DATA_TREE_MODIFICATION).merge(any(), any()); - snapshotBackedWriteTransaction.delete(YangInstanceIdentifier.EMPTY); + snapshotBackedWriteTransaction.delete(YangInstanceIdentifier.empty()); verify(DATA_TREE_MODIFICATION).delete(any()); assertEquals(NORMALIZED_NODE_OPTIONAL, - snapshotBackedWriteTransaction.readSnapshotNode(YangInstanceIdentifier.EMPTY)); + snapshotBackedWriteTransaction.readSnapshotNode(YangInstanceIdentifier.empty())); verify(DATA_TREE_MODIFICATION).readNode(any()); assertTrue(snapshotBackedWriteTransaction.addToStringAttributes( @@ -98,20 +99,26 @@ public class SnapshotBackedWriteTransactionTest { @Test(expected = IllegalArgumentException.class) public void writeWithException() throws Exception { - doThrow(TestException.class).when(DATA_TREE_MODIFICATION).write(any(), any()); - snapshotBackedWriteTransaction.write(YangInstanceIdentifier.EMPTY, NORMALIZED_NODE); + doAnswer(inv -> { + throw new TestException(); + }).when(DATA_TREE_MODIFICATION).write(any(), any()); + snapshotBackedWriteTransaction.write(YangInstanceIdentifier.empty(), NORMALIZED_NODE); } @Test(expected = IllegalArgumentException.class) public void mergeWithException() throws Exception { - doThrow(TestException.class).when(DATA_TREE_MODIFICATION).merge(any(), any()); - snapshotBackedWriteTransaction.merge(YangInstanceIdentifier.EMPTY, NORMALIZED_NODE); + doAnswer(inv -> { + throw new TestException(); + }).when(DATA_TREE_MODIFICATION).merge(any(), any()); + snapshotBackedWriteTransaction.merge(YangInstanceIdentifier.empty(), NORMALIZED_NODE); } @Test(expected = IllegalArgumentException.class) public void deleteWithException() throws Exception { - doThrow(TestException.class).when(DATA_TREE_MODIFICATION).delete(any()); - snapshotBackedWriteTransaction.delete(YangInstanceIdentifier.EMPTY); + doAnswer(inv -> { + throw new TestException(); + }).when(DATA_TREE_MODIFICATION).delete(any()); + snapshotBackedWriteTransaction.delete(YangInstanceIdentifier.empty()); } private static final class TestException extends Exception {