danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[BACKEND] Make break message debug level and more clear (#496)

Browse source
This commit is contained in:
Jesse Bannon 2023-03-02 13:01:01 -08:00 committed by GitHub
parent 5d57eb14b8
commit 5d40d02eb3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/ytdl_sub/downloaders/downloader.py
View file

@ -369,14 +369,14 @@ class Downloader(DownloadArchiver, Generic[DownloaderOptionsT], ABC):
with self._listen_and_log_downloaded_info_json(log_prefix=log_prefix_on_info_json_dl): with self._listen_and_log_downloaded_info_json(log_prefix=log_prefix_on_info_json_dl):
_ = self.extract_info(ytdl_options_overrides=ytdl_options_overrides, **kwargs) _ = self.extract_info(ytdl_options_overrides=ytdl_options_overrides, **kwargs)
except RejectedVideoReached: except RejectedVideoReached:
download_logger.info( download_logger.debug(
"RejectedVideoReached, stopping additional downloads. " "RejectedVideoReached, stopping additional downloads "
"Disable by setting `ytdl_options.break_on_reject` to False." "(Can be disable by setting `ytdl_options.break_on_reject` to False)."
) )
except ExistingVideoReached: except ExistingVideoReached:
download_logger.info( download_logger.debug(
"ExistingVideoReached, stopping additional downloads. " "ExistingVideoReached, stopping additional downloads. "
"Disable by setting `ytdl_options.break_on_existing` to False." "(Can be disable by setting `ytdl_options.break_on_existing` to False)."
) )
except MaxDownloadsReached: except MaxDownloadsReached:
download_logger.info("MaxDownloadsReached, stopping additional downloads.") download_logger.info("MaxDownloadsReached, stopping additional downloads.")

8
tests/e2e/plugins/test_date_range.py
View file

@ -70,7 +70,7 @@ class TestDateRange:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="ExistingVideoReached, stopping additional downloads", expected_message="ExistingVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
transaction_log = recent_channel_subscription.download() transaction_log = recent_channel_subscription.download()
assert_transaction_log_matches( assert_transaction_log_matches(
@ -141,7 +141,7 @@ class TestDateRange:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="RejectedVideoReached, stopping additional downloads", expected_message="RejectedVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
transaction_log = recent_channel_subscription.download(dry_run=False) transaction_log = recent_channel_subscription.download(dry_run=False)
@ -161,7 +161,7 @@ class TestDateRange:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="ExistingVideoReached, stopping additional downloads", expected_message="ExistingVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
transaction_log = rolling_recent_channel_subscription.download(dry_run=dry_run) transaction_log = rolling_recent_channel_subscription.download(dry_run=dry_run)
@ -188,7 +188,7 @@ class TestDateRange:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="ExistingVideoReached, stopping additional downloads", expected_message="ExistingVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
transaction_log = rolling_recent_channel_subscription.download() transaction_log = rolling_recent_channel_subscription.download()

4
tests/e2e/youtube/test_playlist.py
View file

@ -85,7 +85,7 @@ class TestPlaylist:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="ExistingVideoReached, stopping additional downloads", expected_message="ExistingVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
_ = playlist_subscription.download() _ = playlist_subscription.download()
@ -133,7 +133,7 @@ class TestPlaylist:
with assert_logs( with assert_logs(
logger=ytdl_sub.downloaders.downloader.download_logger, logger=ytdl_sub.downloaders.downloader.download_logger,
expected_message="ExistingVideoReached, stopping additional downloads", expected_message="ExistingVideoReached, stopping additional downloads",
log_level="info", log_level="debug",
): ):
_ = mock_run_from_cli(args=args)[0][1] _ = mock_run_from_cli(args=args)[0][1]