From b497dafe1ba5956c19e33d829a6ac15d01606737 Mon Sep 17 00:00:00 2001 From: Jesse Bannon Date: Fri, 28 Jul 2023 00:06:47 -0700 Subject: [PATCH] dry-run --- src/ytdl_sub/subscriptions/subscription_download.py | 3 ++- tests/e2e/youtube/test_video.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/ytdl_sub/subscriptions/subscription_download.py b/src/ytdl_sub/subscriptions/subscription_download.py index 5e04682d..60d580bb 100644 --- a/src/ytdl_sub/subscriptions/subscription_download.py +++ b/src/ytdl_sub/subscriptions/subscription_download.py @@ -72,7 +72,8 @@ class SubscriptionDownload(BaseSubscription, ABC): entry=entry, ) - if self.output_options.thumbnail_name and entry.is_thumbnail_downloaded(): + # Always pretend to include the thumbnail in a dry-run + if self.output_options.thumbnail_name and (dry_run or entry.is_thumbnail_downloaded()): output_thumbnail_name = self.overrides.apply_formatter( formatter=self.output_options.thumbnail_name, entry=entry ) diff --git a/tests/e2e/youtube/test_video.py b/tests/e2e/youtube/test_video.py index 9f14d400..27c22d77 100644 --- a/tests/e2e/youtube/test_video.py +++ b/tests/e2e/youtube/test_video.py @@ -119,7 +119,7 @@ class TestYoutubeVideo: transaction_log_summary_file_name="youtube/test_video.txt", ) - @pytest.mark.parametrize("dry_run", [True, False]) + @pytest.mark.parametrize("dry_run", [True]) def test_single_video_download( self, music_video_config,