about summary refs log tree commit diff
path: root/test/test_download.py
diff options
context:
space:
mode:
authorPhilipp Hagemeister <phihag@phihag.de>2014-03-23 15:52:21 +0100
committerPhilipp Hagemeister <phihag@phihag.de>2014-03-23 15:52:21 +0100
commit257cfebfe6833a41b8ec2c3882b8666c15e454a1 (patch)
tree32c5e5fe6a5f0fe7345cff925bbe7c773713993d /test/test_download.py
parent6eefe53329ad22a2b2c7be123b8a9b758a6ea205 (diff)
downloadyoutube-dl-257cfebfe6833a41b8ec2c3882b8666c15e454a1.tar.gz
youtube-dl-257cfebfe6833a41b8ec2c3882b8666c15e454a1.tar.xz
youtube-dl-257cfebfe6833a41b8ec2c3882b8666c15e454a1.zip
[test] Move expect_info_dict out of test_download
Diffstat (limited to 'test/test_download.py')
-rw-r--r--test/test_download.py28
1 files changed, 5 insertions, 23 deletions
diff --git a/test/test_download.py b/test/test_download.py
index 815f5bb09..f4e5d120e 100644
--- a/test/test_download.py
+++ b/test/test_download.py
@@ -9,16 +9,16 @@ sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 from test.helper import (
     get_params,
     gettestcases,
-    try_rm,
+    expect_info_dict,
     md5,
-    report_warning
+    try_rm,
+    report_warning,
 )
 
 
 import hashlib
 import io
 import json
-import re
 import socket
 
 import youtube_dl.YoutubeDL
@@ -135,26 +135,8 @@ def generator(test_case):
                     self.assertEqual(md5_for_file, tc['md5'])
                 with io.open(info_json_fn, encoding='utf-8') as infof:
                     info_dict = json.load(infof)
-                for (info_field, expected) in tc.get('info_dict', {}).items():
-                    if isinstance(expected, compat_str) and expected.startswith('re:'):
-                        got = info_dict.get(info_field)
-                        match_str = expected[len('re:'):]
-                        match_rex = re.compile(match_str)
-
-                        self.assertTrue(
-                            isinstance(got, compat_str) and match_rex.match(got),
-                            u'field %s (value: %r) should match %r' % (info_field, got, match_str))
-                    elif isinstance(expected, type):
-                        got = info_dict.get(info_field)
-                        self.assertTrue(isinstance(got, expected),
-                            u'Expected type %r, but got value %r of type %r' % (expected, got, type(got)))
-                    else:
-                        if isinstance(expected, compat_str) and expected.startswith('md5:'):
-                            got = 'md5:' + md5(info_dict.get(info_field))
-                        else:
-                            got = info_dict.get(info_field)
-                        self.assertEqual(expected, got,
-                            u'invalid value for field %s, expected %r, got %r' % (info_field, expected, got))
+
+                expect_info_dict(self, tc.get('info_dict', {}), info_dict)
 
                 # Check for the presence of mandatory fields
                 for key in ('id', 'url', 'title', 'ext'):