Merge c61e62aa7a into 6365e92589

Removed the 'ext' key from the new test's info_dict, as the downloaded entity is a playlist, not a standalone item, and as such, does not have an extension. All tests pass.
Updated an existing test with additional information such that all tests (other than the newly-added one) now pass.
2024-11-25 16:51:26 +01:00 · 2024-11-16 17:57:18 +01:00 · 2024-11-13 17:57:15 -06:00 · 2024-11-13 17:02:40 -06:00 · 2024-11-13 16:54:58 -06:00 · 2024-11-13 16:53:36 -06:00
1 changed files with 19 additions and 1 deletions
--- a/yt_dlp/extractor/ctvnews.py
+++ b/yt_dlp/extractor/ctvnews.py
@ -8,7 +8,7 @@ class CTVNewsIE(InfoExtractor):
    _VALID_URL = r'https?://(?:.+?\.)?ctvnews\.ca/(?:video\?(?:clip|playlist|bin)Id=|.*?)(?P<id>[0-9.]+)'
    _TESTS = [{
        'url': 'http://www.ctvnews.ca/video?clipId=901995',
-        'md5': '9b8624ba66351a23e0b6e1391971f9af',
+        'md5': 'b608f466c7fa24b9666c6439d766ab7e',
        'info_dict': {
            'id': '901995',
            'ext': 'flv',
@ -16,6 +16,14 @@ class CTVNewsIE(InfoExtractor):
            'description': 'md5:958dd3b4f5bbbf0ed4d045c790d89285',
            'timestamp': 1467286284,
            'upload_date': '20160630',
+            'categories': [],
+            'season_number': 0,
+            'season': 'Season 0',
+            'tags': [],
+            'series': 'CTV News National | Archive | Stories 2',
+            'season_id': '57981',
+            'thumbnail': r're:https?://.*\.jpg$',
+            'duration': 764.631,
        },
    }, {
        'url': 'http://www.ctvnews.ca/video?playlistId=1.2966224',
@ -40,6 +48,13 @@ class CTVNewsIE(InfoExtractor):
    }, {
        'url': 'http://vancouverisland.ctvnews.ca/video?clipId=761241',
        'only_matching': True,
+    }, {
+        'url': 'https://www.ctvnews.ca/business/respondents-to-bank-of-canada-questionnaire-largely-oppose-creating-a-digital-loonie-1.6665797',
+        'info_dict':
+        {
+            'id': '1.6665797',
+        },
+        'playlist_mincount': 1,
    }]

    def _real_extract(self, url):
@ -67,4 +82,7 @@ class CTVNewsIE(InfoExtractor):
                if 'getAuthStates("' in webpage:
                    entries = [ninecninemedia_url_result(clip_id) for clip_id in
                               self._search_regex(r'getAuthStates\("([\d+,]+)"', webpage, 'clip ids').split(',')]
+                else:
+                    entries = [ninecninemedia_url_result(clip_id) for clip_id in orderedSet(
+                        re.findall(r'axisId&#34;:&#34;(\d+)', webpage))]
            return self.playlist_result(entries, page_id)
Author	SHA1	Message	Date
Jackson Humphrey	ad27b3c592	Merge `c61e62aa7a` into `6365e92589`	2024-11-16 17:57:18 +01:00
Jackson Humphrey	c61e62aa7a	Removed the 'ext' key from the new test's info_dict, as the downloaded entity is a playlist, not a standalone item, and as such, does not have an extension. All tests pass.	2024-11-13 17:57:15 -06:00
Jackson Humphrey	44ba62ffd5	Updated an existing test with additional information such that all tests (other than the newly-added one) now pass.	2024-11-13 17:02:40 -06:00
Jackson Humphrey	7cd5bc9e9f	Added a draft test for the URL reported in Issue #8689 . This test fails for reasons I don't understand; I believe the correct extension should be 'mp4', but the current code reports an 'ext' of None.	2024-11-13 16:54:58 -06:00
Jackson Humphrey	c0ff69795f	Applied the patch provided by dirkf on Issue #8689 .	2024-11-13 16:53:36 -06:00