1
0
mirror of https://github.com/yt-dlp/yt-dlp.git synced 2025-07-02 19:38:32 +00:00

Fixed coding convention issues and test cases

This commit is contained in:
Rohan Wadhwa 2025-04-25 22:49:03 -04:00
parent b4a8df0eba
commit 7cc2e6c19e

View File

@ -1,6 +1,7 @@
from .common import ExtractorError, InfoExtractor from .common import ExtractorError, InfoExtractor
from ..utils import determine_ext, join_nonempty from ..utils import determine_ext, join_nonempty
from ..utils.traversal import traverse_obj
class PlayerFmIE(InfoExtractor): class PlayerFmIE(InfoExtractor):
@ -22,7 +23,7 @@ class PlayerFmIE(InfoExtractor):
'id': 'thursday-april-24-2025', 'id': 'thursday-april-24-2025',
'thumbnail': r're:^https://.*\.(jpg|png)', 'thumbnail': r're:^https://.*\.(jpg|png)',
'title': 'Thursday, April 24, 2025', 'title': 'Thursday, April 24, 2025',
'creators': ['Lester Holt', 'NBC News'], 'creators': ['Lester Holt, NBC News'],
'description': '<p>Trump urges Putin to \'STOP\' after deadly Russian strike on Ukraine; More than 90,000 people have paid their respects to Pope Francis; Miami shooting leaves three injured; and more on tonights broadcast.</p>', 'description': '<p>Trump urges Putin to \'STOP\' after deadly Russian strike on Ukraine; More than 90,000 people have paid their respects to Pope Francis; Miami shooting leaves three injured; and more on tonights broadcast.</p>',
'duration': 1250, 'duration': 1250,
'ext': 'mp3', 'ext': 'mp3',
@ -33,8 +34,8 @@ class PlayerFmIE(InfoExtractor):
'id': 'ep-109-its-kicking-off-how-have-the-rules-for-kickoff-changed-what-are-the-best-approaches-to-getting-the-game-underway-and-how-could-we-improve-on-the-present-system', 'id': 'ep-109-its-kicking-off-how-have-the-rules-for-kickoff-changed-what-are-the-best-approaches-to-getting-the-game-underway-and-how-could-we-improve-on-the-present-system',
'thumbnail': r're:^https://.*\.(jpg|png)', 'thumbnail': r're:^https://.*\.(jpg|png)',
'title': '#109 It\'s kicking off! How have the rules for kickoff changed, what are the best approaches to getting the game underway, and how could we improve on the present system?', 'title': '#109 It\'s kicking off! How have the rules for kickoff changed, what are the best approaches to getting the game underway, and how could we improve on the present system?',
'creators': ['Soccer Roundup'], 'creators': ['TSS'],
'description': '`<p>Ryan is joined by Joe and Taylor to answer the age old question (in your best Jerry Seinfeld impression), "What\'s the deal with kickoff?!" How does a game get underway, how have the rules regarding kickoff evolved since the mid-1800s, what are the different approaches to getting your tactics right from kickoff, how could we improve upon the current system, and much much more!</p><p>The Soccer 101 theme, and plenty of other excellent music, can be found right here: <a href="https://aerialist.bandcamp.com/">https://aerialist.bandcamp.com</a>.</p>`', 'description': '<p>Ryan is joined by Joe and Taylor to answer the age old question (in your best Jerry Seinfeld impression), "What\'s the deal with kickoff?!" How does a game get underway, how have the rules regarding kickoff evolved since the mid-1800s, what are the different approaches to getting your tactics right from kickoff, how could we improve upon the current system, and much much more!</p><p>The Soccer 101 theme, and plenty of other excellent music, can be found right here: <a href="https://aerialist.bandcamp.com/">https://aerialist.bandcamp.com</a>.</p>',
'duration': 1765, 'duration': 1765,
'ext': 'mp3', 'ext': 'mp3',
}, },
@ -45,7 +46,12 @@ def _real_extract(self, url):
video_id = self._match_id(url) video_id = self._match_id(url)
data = self._download_json(url + '.json', None) data = self._download_json(url + '.json', None)
thumbnail = data['image']['url'] if 'image' in data else data['series']['image']['url'] title = data.get('title')
description = data.get('description')
duration = data.get('duration')
thumbnail = traverse_obj(data, ('image', 'url'), ('series', 'image', 'url'))
creators = [traverse_obj(data, ('series', 'author'))]
video_url = join_nonempty('https', self._search_regex(r'redirect.mp3/(.*)', data['url'], 'redirect'), delim='://') video_url = join_nonempty('https', self._search_regex(r'redirect.mp3/(.*)', data['url'], 'redirect'), delim='://')
if not video_url: if not video_url:
raise ExtractorError('URL to podcast not found', expected=True) raise ExtractorError('URL to podcast not found', expected=True)
@ -56,9 +62,9 @@ def _real_extract(self, url):
return { return {
'id': video_id, 'id': video_id,
'thumbnail': thumbnail, 'thumbnail': thumbnail,
'title': data['title'], 'title': title,
'creators': data['series']['author'].split(', '), 'creators': creators,
'description': data['description'], 'description': description,
'duration': data['duration'], 'duration': duration,
'formats': formats, 'formats': formats,
} }