pashko/apache-poi
1
0
Fork 0
mirror of https://github.com/apache/poi.git synced 2026-02-27 20:40:08 +08:00
Code Issues Packages Projects Releases Wiki Activity

[bug-69715] update tests

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1926446 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
PJ Fanning 2025-06-15 14:22:41 +00:00
parent 47f2d72d35
commit d8b3934646
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
poi/src/test/java/org/apache/poi/util/DefaultTempFileCreationStrategyTest.java
View File

@ -27,6 +27,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import org.apache.commons.io.FileUtils;
import org.junit.jupiter.api.AfterEach;
@ -69,7 +70,7 @@ class DefaultTempFileCreationStrategyTest {
try {
assertTrue(Files.isDirectory(dir.toPath()), "File is not a directory: " + dir);
DefaultTempFileCreationStrategy testStrategy = new DefaultTempFileCreationStrategy(dir);
checkGetFile(testStrategy);
checkGetFileAndPath(testStrategy, dir.toPath());
} finally {
// Clean up the directory after the test
FileUtils.deleteDirectory(dir);
@ -84,7 +85,7 @@ class DefaultTempFileCreationStrategyTest {
assertTrue(dir.delete(), "directory not deleted: " + dir);
try {
DefaultTempFileCreationStrategy testStrategy = new DefaultTempFileCreationStrategy(dir);
checkGetFile(testStrategy);
checkGetFileAndPath(testStrategy, dir.toPath());
} finally {
// Clean up the directory after the test
FileUtils.deleteDirectory(dir);
@ -104,8 +105,18 @@ class DefaultTempFileCreationStrategyTest {
}
private static void checkGetFile(DefaultTempFileCreationStrategy strategy) throws IOException {
checkGetFileAndPath(strategy, null);
}
private static void checkGetFileAndPath(DefaultTempFileCreationStrategy strategy,
Path path) throws IOException {
File file = strategy.createTempFile("POITest", ".tmp");
try {
if (path != null) {
assertTrue(file.toPath().startsWith(path),
"File path does not start with expected path: " + path);
}
assertTrue(file.getParentFile().exists(),
"Failed for " + file.getParentFile());