Skip to content
Snippets Groups Projects
Commit 5d23537e authored by felmer's avatar felmer
Browse files

SP-724, BIS-428: fix test

SVN: 29456
parent 92aed7e5
No related branches found
No related tags found
No related merge requests found
...@@ -54,10 +54,20 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase ...@@ -54,10 +54,20 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase
private static final File TEST_HDF5_CONTAINER = new File( private static final File TEST_HDF5_CONTAINER = new File(
"../openbis-common/resource/test-data/HDF5ContainerBasedHierarchicalContentNodeTest/thumbnails.h5"); "../openbis-common/resource/test-data/HDF5ContainerBasedHierarchicalContentNodeTest/thumbnails.h5");
private IHierarchicalContent content; public static File[] getUnzippedFiles()
{
File[] tempFiles = ZipBasedHierarchicalContent.TEMP_FOLDER.listFiles(new FilenameFilter()
{
@Override
public boolean accept(File file, String name)
{
return name.startsWith(ZipBasedHierarchicalContent.TEMP_FILE_PREFIX);
}
});
return tempFiles;
}
@BeforeMethod public static void removeUnzippedFiles()
public void setUpContent() throws Exception
{ {
File[] unzippedFiles = getUnzippedFiles(); File[] unzippedFiles = getUnzippedFiles();
for (File unzippedFile : unzippedFiles) for (File unzippedFile : unzippedFiles)
...@@ -65,6 +75,15 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase ...@@ -65,6 +75,15 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase
FileUtilities.delete(unzippedFile); FileUtilities.delete(unzippedFile);
} }
} }
private IHierarchicalContent content;
@BeforeMethod
public void setUpContent() throws Exception
{
removeUnzippedFiles();
}
@AfterMethod @AfterMethod
public void closeContent() public void closeContent()
...@@ -129,19 +148,8 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase ...@@ -129,19 +148,8 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase
HDF5ContainerBasedHierarchicalContentNodeTest.assertH5ExampleContent(hdf5Node); HDF5ContainerBasedHierarchicalContentNodeTest.assertH5ExampleContent(hdf5Node);
} content.close();
assertEquals("[]", Arrays.asList(getUnzippedFiles()).toString());
private File[] getUnzippedFiles()
{
File[] tempFiles = ZipBasedHierarchicalContent.TEMP_FOLDER.listFiles(new FilenameFilter()
{
@Override
public boolean accept(File file, String name)
{
return name.startsWith(ZipBasedHierarchicalContent.TEMP_FILE_PREFIX);
}
});
return tempFiles;
} }
private void assertDirectoryNode(String expectedPath, String expectedName, IHierarchicalContentNode node) private void assertDirectoryNode(String expectedPath, String expectedName, IHierarchicalContentNode node)
...@@ -228,7 +236,9 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase ...@@ -228,7 +236,9 @@ public class ZipBasedHierarchicalContentTest extends AbstractFileSystemTestCase
{ {
path += "/"; path += "/";
} }
zipOutputStream.putNextEntry(new ZipEntry(path)); ZipEntry entry = new ZipEntry(path);
entry.setTime(file.lastModified());
zipOutputStream.putNextEntry(entry);
} catch (IOException ex) } catch (IOException ex)
{ {
throw CheckedExceptionTunnel.wrapIfNecessary(ex); throw CheckedExceptionTunnel.wrapIfNecessary(ex);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment