X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fconfig%2Fconfig-persister-file-xml-adapter%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fconfig%2Fpersist%2Fstorage%2Ffile%2Fxml%2FFileStorageAdapterTest.java;h=0b9e1b3a09b6c1628be5c608c570573c8ac5c5a5;hp=2bfe70adb811d2c5575663540e4a194179f78dae;hb=f43b01b81319959b1907e3e04537f5169e7f33d8;hpb=aacafb25b08def6ea9f2537356da418805c80ba9 diff --git a/opendaylight/config/config-persister-file-xml-adapter/src/test/java/org/opendaylight/controller/config/persist/storage/file/xml/FileStorageAdapterTest.java b/opendaylight/config/config-persister-file-xml-adapter/src/test/java/org/opendaylight/controller/config/persist/storage/file/xml/FileStorageAdapterTest.java index 2bfe70adb8..0b9e1b3a09 100644 --- a/opendaylight/config/config-persister-file-xml-adapter/src/test/java/org/opendaylight/controller/config/persist/storage/file/xml/FileStorageAdapterTest.java +++ b/opendaylight/config/config-persister-file-xml-adapter/src/test/java/org/opendaylight/controller/config/persist/storage/file/xml/FileStorageAdapterTest.java @@ -8,49 +8,57 @@ package org.opendaylight.controller.config.persist.storage.file.xml; -import static junit.framework.Assert.assertFalse; +import static junit.framework.Assert.assertTrue; import static org.custommonkey.xmlunit.XMLAssert.assertXMLEqual; import static org.hamcrest.CoreMatchers.is; import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertThat; +import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.mock; +import com.google.common.collect.Sets; import java.io.File; import java.io.FileNotFoundException; import java.io.IOException; import java.nio.file.Files; +import java.nio.charset.StandardCharsets; import java.util.List; import java.util.SortedSet; import java.util.TreeSet; - -import junit.framework.Assert; - +import org.junit.After; import org.junit.Before; import org.junit.Test; -import org.mockito.Mockito; import org.opendaylight.controller.config.persist.api.ConfigSnapshotHolder; import org.opendaylight.controller.config.persist.test.PropertiesProviderTest; -import com.google.common.base.Charsets; - public class FileStorageAdapterTest { private static int i; private File file; private static final String NON_EXISTENT_DIRECTORY = "./nonExistentDir/"; private static final String NON_EXISTENT_FILE = "nonExistent.txt"; + private XmlFileStorageAdapter storage; @Before public void setUp() throws Exception { file = Files.createTempFile("testFilePersist", ".txt").toFile(); - if (!file.exists()) + file.deleteOnExit(); + if (!file.exists()) { return; - com.google.common.io.Files.write("", file, Charsets.UTF_8); + } + com.google.common.io.Files.write("", file, StandardCharsets.UTF_8); i = 1; + storage = new XmlFileStorageAdapter(); + } + + @After + public void tearDown() throws Exception { + storage.reset(); } @Test public void testNewFile() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); PropertiesProviderTest pp = new PropertiesProviderTest(); pp.addProperty("fileStorage",NON_EXISTENT_DIRECTORY+NON_EXISTENT_FILE); pp.addProperty("numberOfBackups",Integer.toString(Integer.MAX_VALUE)); @@ -71,12 +79,12 @@ public class FileStorageAdapterTest { storage.persistConfig(holder); - Assert.assertEquals(storage.toString().replace("\\","/"),"XmlFileStorageAdapter [storage="+NON_EXISTENT_DIRECTORY+NON_EXISTENT_FILE+"]"); + assertEquals(storage.toString().replace("\\","/"),"XmlFileStorageAdapter [storage="+NON_EXISTENT_DIRECTORY+NON_EXISTENT_FILE+"]"); delete(new File(NON_EXISTENT_DIRECTORY)); } + @Test public void testFileAdapter() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); PropertiesProviderTest pp = new PropertiesProviderTest(); pp.addProperty("fileStorage",file.getPath()); pp.addProperty("numberOfBackups",Integer.toString(Integer.MAX_VALUE)); @@ -97,7 +105,7 @@ public class FileStorageAdapterTest { storage.persistConfig(holder); - assertEquals(27, com.google.common.io.Files.readLines(file, Charsets.UTF_8).size()); + assertEquals(29, com.google.common.io.Files.readLines(file, StandardCharsets.UTF_8).size()); List lastConf = storage.loadLastConfigs(); assertEquals(1, lastConf.size()); ConfigSnapshotHolder configSnapshotHolder = lastConf.get(0); @@ -109,7 +117,7 @@ public class FileStorageAdapterTest { storage.setNumberOfBackups(Integer.MAX_VALUE); List last = storage.loadLastConfigs(); - Assert.assertEquals(createCaps(), last.get(0).getCapabilities()); + assertEquals(createCaps(), last.get(0).getCapabilities()); } private SortedSet createCaps() { @@ -124,8 +132,6 @@ public class FileStorageAdapterTest { @Test public void testFileAdapterOneBackup() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); - PropertiesProviderTest pp = new PropertiesProviderTest(); pp.addProperty("fileStorage",file.getPath()); pp.addProperty("numberOfBackups",Integer.toString(Integer.MAX_VALUE)); @@ -146,7 +152,7 @@ public class FileStorageAdapterTest { storage.persistConfig(holder); - assertEquals(27, com.google.common.io.Files.readLines(file, Charsets.UTF_8).size()); + assertEquals(29, com.google.common.io.Files.readLines(file, StandardCharsets.UTF_8).size()); List lastConf = storage.loadLastConfigs(); assertEquals(1, lastConf.size()); @@ -154,9 +160,69 @@ public class FileStorageAdapterTest { assertXMLEqual("2", configSnapshotHolder.getConfigSnapshot()); } + @Test + public void testWithFeatures() throws Exception { + PropertiesProviderTest pp = new PropertiesProviderTest(); + pp.addProperty("fileStorage",file.getPath()); + pp.addProperty("numberOfBackups",Integer.toString(Integer.MAX_VALUE)); + storage.instantiate(pp); + + final ConfigSnapshotHolder holder = new ConfigSnapshotHolder() { + @Override + public String getConfigSnapshot() { + return createConfig(); + } + + @Override + public SortedSet getCapabilities() { + return createCaps(); + } + }; + final FeatureListProvider mock = mock(FeatureListProvider.class); + + doReturn(Sets.newHashSet("f1-11", "f2-22")).when(mock).listFeatures(); + storage.setFeaturesService(mock); + storage.persistConfig(holder); + + assertEquals(20, com.google.common.io.Files.readLines(file, StandardCharsets.UTF_8).size()); + + List lastConf = storage.loadLastConfigs(); + assertEquals(1, lastConf.size()); + ConfigSnapshotHolder configSnapshotHolder = lastConf.get(0); + assertXMLEqual("1", configSnapshotHolder.getConfigSnapshot()); + assertEquals(Sets.newHashSet("f1-11", "f2-22"), storage.getPersistedFeatures()); + } + + @Test + public void testNoFeaturesStored() throws Exception { + PropertiesProviderTest pp = new PropertiesProviderTest(); + pp.addProperty("fileStorage",file.getPath()); + pp.addProperty("numberOfBackups",Integer.toString(Integer.MAX_VALUE)); + storage.instantiate(pp); + + com.google.common.io.Files.write("\n" + + "\n" + + " \n" + + " \n" + + " \n" + + " cap12\n" + + " \n" + + " \n" + + " 1\n" + + " \n" + + " \n" + + " \n" + + "", file, StandardCharsets.UTF_8); + + List lastConf = storage.loadLastConfigs(); + assertEquals(1, lastConf.size()); + ConfigSnapshotHolder configSnapshotHolder = lastConf.get(0); + assertXMLEqual("1", configSnapshotHolder.getConfigSnapshot()); + assertTrue(storage.getPersistedFeatures().isEmpty()); + } + @Test public void testFileAdapterOnlyTwoBackups() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); storage.setFileStorage(file); storage.setNumberOfBackups(2); final ConfigSnapshotHolder holder = new ConfigSnapshotHolder() { @@ -175,60 +241,68 @@ public class FileStorageAdapterTest { storage.persistConfig(holder); storage.persistConfig(holder); - List readLines = com.google.common.io.Files.readLines(file, Charsets.UTF_8); - assertEquals(27, readLines.size()); + List readLines = com.google.common.io.Files.readLines(file, StandardCharsets.UTF_8); + assertEquals(29, readLines.size()); List lastConf = storage.loadLastConfigs(); assertEquals(1, lastConf.size()); ConfigSnapshotHolder configSnapshotHolder = lastConf.get(0); assertXMLEqual("3", configSnapshotHolder.getConfigSnapshot()); assertFalse(readLines.contains(holder.getConfigSnapshot())); + assertTrue(storage.getPersistedFeatures().isEmpty()); } @Test public void testNoLastConfig() throws Exception { File file = Files.createTempFile("testFilePersist", ".txt").toFile(); - if (!file.exists()) + file.deleteOnExit(); + if (!file.exists()) { return; - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); - storage.setFileStorage(file); + } + try (XmlFileStorageAdapter storage = new XmlFileStorageAdapter()) { + storage.setFileStorage(file); - List elementOptional = storage.loadLastConfigs(); - assertThat(elementOptional.size(), is(0)); + List elementOptional = storage.loadLastConfigs(); + assertThat(elementOptional.size(), is(0)); + } } @Test(expected = NullPointerException.class) public void testNoProperties() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); - storage.loadLastConfigs(); + try (XmlFileStorageAdapter storage = new XmlFileStorageAdapter()) { + storage.loadLastConfigs(); + } } @Test(expected = NullPointerException.class) public void testNoProperties2() throws Exception { - XmlFileStorageAdapter storage = new XmlFileStorageAdapter(); - storage.persistConfig(new ConfigSnapshotHolder() { - @Override - public String getConfigSnapshot() { - return Mockito.mock(String.class); - } - - @Override - public SortedSet getCapabilities() { - return new TreeSet<>(); - } - } ); + try (XmlFileStorageAdapter storage = new XmlFileStorageAdapter()) { + storage.persistConfig(new ConfigSnapshotHolder() { + @Override + public String getConfigSnapshot() { + return mock(String.class); + } + + @Override + public SortedSet getCapabilities() { + return new TreeSet<>(); + } + }); + } } static String createConfig() { return "" + i++ + ""; } - private void delete(File f) throws IOException { + private void delete(final File f) throws IOException { if (f.isDirectory()) { - for (File c : f.listFiles()) + for (File c : f.listFiles()) { delete(c); + } } - if (!f.delete()) + if (!f.delete()) { throw new FileNotFoundException("Failed to delete file: " + f); + } } }