From d718f50e34c43f44b668360d27559e905d848140 Mon Sep 17 00:00:00 2001 From: litetex <40789489+litetex@users.noreply.github.com> Date: Wed, 5 Mar 2025 17:29:33 +0100 Subject: [PATCH] Fix compilation + Cleanup --- .../schabi/newpipe/downloader/MockDownloader.java | 7 ++++++- .../newpipe/downloader/RecordingDownloader.java | 15 +++++++++------ 2 files changed, 15 insertions(+), 7 deletions(-) diff --git a/extractor/src/test/java/org/schabi/newpipe/downloader/MockDownloader.java b/extractor/src/test/java/org/schabi/newpipe/downloader/MockDownloader.java index 234685e30..a218fdd63 100644 --- a/extractor/src/test/java/org/schabi/newpipe/downloader/MockDownloader.java +++ b/extractor/src/test/java/org/schabi/newpipe/downloader/MockDownloader.java @@ -7,7 +7,9 @@ import org.schabi.newpipe.extractor.downloader.Request; import org.schabi.newpipe.extractor.downloader.Response; import java.io.IOException; +import java.io.UncheckedIOException; import java.nio.file.Files; +import java.nio.file.Path; import java.nio.file.Paths; import java.util.HashMap; import java.util.Map; @@ -27,10 +29,11 @@ class MockDownloader extends Downloader { public MockDownloader(@Nonnull final String path) { this.path = path; this.mocks = new HashMap<>(); + try (final var directoryStream = Files.newDirectoryStream(Paths.get(path), entry -> entry.getFileName().toString() .startsWith(RecordingDownloader.FILE_NAME_PREFIX))) { - for (final var entry : directoryStream) { + for (final Path entry : directoryStream) { try (final var reader = Files.newBufferedReader(entry)) { final var response = new GsonBuilder() .create() @@ -38,6 +41,8 @@ class MockDownloader extends Downloader { mocks.put(response.getRequest(), response.getResponse()); } } + } catch (final IOException ioe) { + throw new UncheckedIOException(ioe); } } diff --git a/extractor/src/test/java/org/schabi/newpipe/downloader/RecordingDownloader.java b/extractor/src/test/java/org/schabi/newpipe/downloader/RecordingDownloader.java index ecc9aecb5..1c6454882 100644 --- a/extractor/src/test/java/org/schabi/newpipe/downloader/RecordingDownloader.java +++ b/extractor/src/test/java/org/schabi/newpipe/downloader/RecordingDownloader.java @@ -10,6 +10,7 @@ import org.schabi.newpipe.extractor.exceptions.ReCaptchaException; import java.io.IOException; import java.io.UncheckedIOException; import java.nio.file.Files; +import java.nio.file.Path; import java.nio.file.Paths; import javax.annotation.Nonnull; @@ -40,7 +41,7 @@ class RecordingDownloader extends Downloader { "(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)"; private int index = 0; - private final String path; + private final Path path; /** * Creates the folder described by {@code stringPath} if it does not exist. @@ -48,20 +49,22 @@ class RecordingDownloader extends Downloader { * @param stringPath Path to the folder where the json files will be saved to. */ public RecordingDownloader(final String stringPath) { - this.path = stringPath; - final var path = Paths.get(stringPath); + this.path = Paths.get(stringPath); + if (Files.exists(path)) { try (final var directoryStream = Files.newDirectoryStream(path, entry -> entry.getFileName().toString() .startsWith(RecordingDownloader.FILE_NAME_PREFIX))) { - for (final var entry : directoryStream) { + for (final Path entry : directoryStream) { Files.delete(entry); } + } catch (final IOException ioe) { + throw new UncheckedIOException(ioe); } } else { try { Files.createDirectories(path); - } catch (IOException e) { + } catch (final IOException e) { throw new UncheckedIOException(e); } } @@ -80,7 +83,7 @@ class RecordingDownloader extends Downloader { response.latestUrl() ); - final var outputPath = Paths.get(path).resolve(FILE_NAME_PREFIX + index + ".json"); + final Path outputPath = path.resolve(FILE_NAME_PREFIX + index + ".json"); index++; try (final var writer = Files.newBufferedWriter(outputPath)) { new GsonBuilder()