mirror of
https://github.com/yt-dlp/yt-dlp.git
synced 2024-12-14 23:32:33 +00:00
[qqmusic:playlist] Capture errors and update tests
This commit is contained in:
parent
e698e4e533
commit
2fffb1dcd0
1 changed files with 23 additions and 6 deletions
|
@ -11,6 +11,7 @@ from ..utils import (
|
||||||
strip_jsonp,
|
strip_jsonp,
|
||||||
unescapeHTML,
|
unescapeHTML,
|
||||||
clean_html,
|
clean_html,
|
||||||
|
ExtractorError,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
@ -315,7 +316,7 @@ class QQMusicPlaylistIE(QQPlaylistBaseIE):
|
||||||
IE_DESC = 'QQ音乐 - 歌单'
|
IE_DESC = 'QQ音乐 - 歌单'
|
||||||
_VALID_URL = r'http://y\.qq\.com/#type=taoge&id=(?P<id>[0-9]+)'
|
_VALID_URL = r'http://y\.qq\.com/#type=taoge&id=(?P<id>[0-9]+)'
|
||||||
|
|
||||||
_TEST = {
|
_TESTS = [{
|
||||||
'url': 'http://y.qq.com/#type=taoge&id=3462654915',
|
'url': 'http://y.qq.com/#type=taoge&id=3462654915',
|
||||||
'info_dict': {
|
'info_dict': {
|
||||||
'id': '3462654915',
|
'id': '3462654915',
|
||||||
|
@ -323,7 +324,16 @@ class QQMusicPlaylistIE(QQPlaylistBaseIE):
|
||||||
'description': 'md5:d2c9d758a96b9888cf4fe82f603121d4',
|
'description': 'md5:d2c9d758a96b9888cf4fe82f603121d4',
|
||||||
},
|
},
|
||||||
'playlist_count': 40,
|
'playlist_count': 40,
|
||||||
}
|
'skip': 'playlist gone',
|
||||||
|
}, {
|
||||||
|
'url': 'http://y.qq.com/#type=taoge&id=1374105607',
|
||||||
|
'info_dict': {
|
||||||
|
'id': '1374105607',
|
||||||
|
'title': '易入人心的华语民谣',
|
||||||
|
'description': '民谣的歌曲易于传唱、、歌词朗朗伤口、旋律简单温馨。属于那种才入耳孔。却上心头的感觉。没有太多的复杂情绪。简单而直接地表达乐者的情绪,就是这样的简单才易入人心。',
|
||||||
|
},
|
||||||
|
'playlist_count': 20,
|
||||||
|
}]
|
||||||
|
|
||||||
def _real_extract(self, url):
|
def _real_extract(self, url):
|
||||||
list_id = self._match_id(url)
|
list_id = self._match_id(url)
|
||||||
|
@ -331,14 +341,21 @@ class QQMusicPlaylistIE(QQPlaylistBaseIE):
|
||||||
list_json = self._download_json(
|
list_json = self._download_json(
|
||||||
'http://i.y.qq.com/qzone-music/fcg-bin/fcg_ucc_getcdinfo_byids_cp.fcg?type=1&json=1&utf8=1&onlysong=0&disstid=%s'
|
'http://i.y.qq.com/qzone-music/fcg-bin/fcg_ucc_getcdinfo_byids_cp.fcg?type=1&json=1&utf8=1&onlysong=0&disstid=%s'
|
||||||
% list_id, list_id, 'Download list page',
|
% list_id, list_id, 'Download list page',
|
||||||
transform_source=strip_jsonp)['cdlist'][0]
|
transform_source=strip_jsonp)
|
||||||
|
if not len(list_json.get('cdlist', [])):
|
||||||
|
if list_json.get('code'):
|
||||||
|
raise ExtractorError(
|
||||||
|
'QQ Music said: error %d in fetching playlist info' % list_json['code'],
|
||||||
|
expected=True)
|
||||||
|
raise ExtractorError('Unable to get playlist info')
|
||||||
|
|
||||||
|
cdlist = list_json['cdlist'][0]
|
||||||
entries = [
|
entries = [
|
||||||
self.url_result(
|
self.url_result(
|
||||||
'http://y.qq.com/#type=song&mid=' + song['songmid'], 'QQMusic', song['songmid']
|
'http://y.qq.com/#type=song&mid=' + song['songmid'], 'QQMusic', song['songmid']
|
||||||
) for song in list_json['songlist']
|
) for song in cdlist['songlist']
|
||||||
]
|
]
|
||||||
|
|
||||||
list_name = list_json.get('dissname')
|
list_name = cdlist.get('dissname')
|
||||||
list_description = clean_html(unescapeHTML(list_json.get('desc')))
|
list_description = clean_html(unescapeHTML(cdlist.get('desc')))
|
||||||
return self.playlist_result(entries, list_id, list_name, list_description)
|
return self.playlist_result(entries, list_id, list_name, list_description)
|
||||||
|
|
Loading…
Reference in a new issue