[ie/beatbump] Add new domain and update tests #8576
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
Site changed from beatbump.ml to beatbump.io, probably due to .ml returning to mali from Freenom.
Template
Before submitting a pull request make sure you have:
In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:
What is the purpose of your pull request?
Copilot Summary
🤖 Generated by Copilot at 73643b0
Summary
🌐🎵🛠️
Improved BeatBump extractor to handle multiple domains and provide better metadata. Updated
beatbump.py
with new classes, tests, and warnings.Walkthrough
_VALID_URL
regexes for bothBeatBumpVideoIE
andBeatBumpPlaylistIE
classes to support the.ml
and.io
domains and the different playlist types (link, link)uploader_url
field from the test case forBeatBumpVideoIE
, as it is not relevant for the extractor (link)channel_is_verified
andheatmap
fields to the test case forBeatBumpVideoIE
, as they are new features of the yt-dlp extractor (link).io
domain forBeatBumpVideoIE
, with theonly_matching
flag set to True (link)track
,uploader
,uploader_id
,modified_date
,uploader_url
,title
,tags
,channel_url
, andchannel_is_verified
fields for the test cases forBeatBumpPlaylistIE
, as they may differ from the YouTube values (link, link, link, link, link)expected_warnings
field to the test cases forBeatBumpPlaylistIE
, as the extractor may emit a warning message that YouTube Music is not directly supported (link, link, link).io
domain forBeatBumpPlaylistIE
, with theonly_matching
flag set to True (link)