Skip to content

Commit

Permalink
Raise exception if there are no videos in the playlists
Browse files Browse the repository at this point in the history
  • Loading branch information
dan-niles committed Oct 14, 2024
1 parent f9d4b1e commit 7cbb02f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Changed

- Raise exception if there are no videos in the playlists (#347)

## [3.2.0] - 2024-10-11

### Deprecated
Expand Down
4 changes: 4 additions & 0 deletions scraper/src/youtube2zim/scraper.py
Original file line number Diff line number Diff line change
Expand Up @@ -618,6 +618,10 @@ def extract_videos_list(self):

for playlist in empty_playlists:
self.playlists.remove(playlist)

# Raise exception if no videos found in playlists
if not self.playlists:
raise Exception("No videos found in playlists")

Check warning on line 624 in scraper/src/youtube2zim/scraper.py

View check run for this annotation

Codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L624

Added line #L624 was not covered by tests
self.videos_ids = [*all_videos.keys()] # unpacking so it's subscriptable

def download_video_files(self, max_concurrency):
Expand Down

0 comments on commit 7cbb02f

Please sign in to comment.