[engadget] Relax _VALID_URL
authorRemita Amine <remitamine@gmail.com>
Thu, 4 Aug 2016 15:34:47 +0000 (16:34 +0100)
committerRemita Amine <remitamine@gmail.com>
Thu, 4 Aug 2016 15:34:47 +0000 (16:34 +0100)
youtube_dl/extractor/engadget.py

index e5e57d48518d3dd3999dad650d0c32406079ce33..a39e9010d4c732a8bbb9d36f124a99fa2992990a 100644 (file)
@@ -4,9 +4,10 @@ from .common import InfoExtractor
 
 
 class EngadgetIE(InfoExtractor):
-    _VALID_URL = r'https?://www.engadget.com/video/(?P<id>\d+)'
+    _VALID_URL = r'https?://www.engadget.com/video/(?P<id>[^/?#]+)'
 
-    _TEST = {
+    _TESTS = [{
+        # video with 5min ID
         'url': 'http://www.engadget.com/video/518153925/',
         'md5': 'c6820d4828a5064447a4d9fc73f312c9',
         'info_dict': {
@@ -15,8 +16,12 @@ class EngadgetIE(InfoExtractor):
             'title': 'Samsung Galaxy Tab Pro 8.4 Review',
         },
         'add_ie': ['FiveMin'],
-    }
+    }, {
+        # video with vidible ID
+        'url': 'https://www.engadget.com/video/57a28462134aa15a39f0421a/',
+        'only_matching': True,
+    }]
 
     def _real_extract(self, url):
         video_id = self._match_id(url)
-        return self.url_result('5min:%s' % video_id)
+        return self.url_result('aol-video:%s' % video_id)