From 32c5f8d8e44298052e472bddd634a0fb684721f9 Mon Sep 17 00:00:00 2001 From: Marek Gradzki Date: Wed, 22 Nov 2017 16:35:20 +0100 Subject: Cleanup various Sonar warings Unused code, imports, etc. Change-Id: Icb19d6a877d5ddbfd85caa86c4e62880e4db3c44 Signed-off-by: Marek Gradzki --- .../src/test/java/io/fd/honeycomb/data/impl/DataBrokerTest.java | 7 +++---- .../io/fd/honeycomb/data/impl/PersistingDataTreeAdapterTest.java | 3 +-- 2 files changed, 4 insertions(+), 6 deletions(-) (limited to 'infra/data-impl/src/test') diff --git a/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/DataBrokerTest.java b/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/DataBrokerTest.java index f1bc3105a..4a0fb11a9 100644 --- a/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/DataBrokerTest.java +++ b/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/DataBrokerTest.java @@ -22,9 +22,9 @@ import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; import static org.mockito.MockitoAnnotations.initMocks; -import io.fd.honeycomb.data.ReadableDataManager; -import io.fd.honeycomb.data.ModifiableDataManager; import io.fd.honeycomb.data.DataModification; +import io.fd.honeycomb.data.ModifiableDataManager; +import io.fd.honeycomb.data.ReadableDataManager; import java.util.Map; import org.junit.Before; import org.junit.Test; @@ -36,7 +36,6 @@ import org.opendaylight.controller.md.sal.dom.api.DOMDataBrokerExtension; import org.opendaylight.controller.md.sal.dom.api.DOMDataChangeListener; import org.opendaylight.controller.md.sal.dom.api.DOMDataReadOnlyTransaction; import org.opendaylight.controller.md.sal.dom.api.DOMDataReadWriteTransaction; -import org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; public class DataBrokerTest { @@ -69,7 +68,7 @@ public class DataBrokerTest { @Test public void testNewWriteOnlyTransaction() { - final DOMDataWriteTransaction writeTx = broker.newWriteOnlyTransaction(); + broker.newWriteOnlyTransaction(); // verify that write transactions use config snapshot verify(confiDataTree).newModification(); diff --git a/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/PersistingDataTreeAdapterTest.java b/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/PersistingDataTreeAdapterTest.java index ed3529530..7e135a259 100644 --- a/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/PersistingDataTreeAdapterTest.java +++ b/infra/data-impl/src/test/java/io/fd/honeycomb/data/impl/PersistingDataTreeAdapterTest.java @@ -135,8 +135,7 @@ public class PersistingDataTreeAdapterTest { public void testPersisterCreateFile() throws Exception { // Delete to test file creation Files.delete(tmpPersistFile); - final PersistingDataTreeAdapter.JsonPersister jsonPersister = - new PersistingDataTreeAdapter.JsonPersister(tmpPersistFile, schemaService); + new PersistingDataTreeAdapter.JsonPersister(tmpPersistFile, schemaService); assertTrue(Files.exists(tmpPersistFile)); } } \ No newline at end of file -- cgit 1.2.3-korg