20-FixResourceManagementIssues

This commit is contained in:
Adrian-Devries 2025-04-24 12:54:06 +02:00
parent 64b84508ed
commit 12905e116b
2 changed files with 17 additions and 9 deletions

View file

@ -30,6 +30,7 @@ import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.List;
import java.util.stream.Stream;
import static de.kosit.validationtool.impl.Helper.ASSERTIONS;
import static org.assertj.core.api.Assertions.assertThat;
@ -57,13 +58,15 @@ public class CommandlineApplicationTest {
@After
public void cleanup() throws IOException {
Files.list(Paths.get("")).filter(p -> p.getFileName().toString().endsWith("-report.xml")).forEach(path -> {
try ( Stream<Path> stream = Files.list(Paths.get("")) ) {
stream.filter(p -> p.getFileName().toString().endsWith("-report.xml")).forEach(path -> {
try {
Files.delete(path);
} catch (final IOException e) {
log.error("Error deleting file", e);
}
});
}
CommandLine.deactivate();
}
@ -186,7 +189,9 @@ public class CommandlineApplicationTest {
Paths.get(Simple.SIMPLE_VALID).toString() };
CommandLineApplication.mainProgram(args);
assertThat(CommandLine.getErrorOutput()).contains(RESULT_OUTPUT);
assertThat(Files.list(this.output).filter(f -> f.toString().endsWith(".html")).count()).isPositive();
try ( Stream<Path> stream = Files.list(this.output) ) {
assertThat(stream.filter(f -> f.toString().endsWith(".html")).count()).isPositive();
}
}
@Test

View file

@ -30,6 +30,7 @@ import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.util.stream.Collectors;
import java.util.stream.Stream;
import static org.assertj.core.api.Assertions.assertThat;
@ -64,6 +65,8 @@ public class ExtractHtmlActionTest {
assertThat(this.action.isSkipped(b)).isFalse();
this.action.check(b);
assertThat(b.isStopped()).isFalse();
assertThat(Files.list(this.tmpDirectory).collect(Collectors.toList())).hasSize(1);
try ( Stream<Path> stream = Files.list(this.tmpDirectory) ) {
assertThat(stream.collect(Collectors.toList())).hasSize(1);
}
}
}