diff options
author | Philipp Hagemeister <phihag@phihag.de> | 2013-06-23 21:33:11 +0200 |
---|---|---|
committer | Philipp Hagemeister <phihag@phihag.de> | 2013-06-23 21:33:11 +0200 |
commit | 51ce3a75c90059366aa4a87e84dc9cde13a4bc14 (patch) | |
tree | 9128340f309fab78f0a6151b898ec5aaed9c1c2b | |
parent | 335056663a6add065a1c9a6bb0c317957b0836a5 (diff) |
Improve error reporting for downloads
-rw-r--r-- | test/test_download.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/test/test_download.py b/test/test_download.py index 84b3204fe..6365b0340 100644 --- a/test/test_download.py +++ b/test/test_download.py @@ -125,11 +125,14 @@ def generator(test_case): self.assertEqual(md5_for_file, tc['md5']) with io.open(tc['file'] + '.info.json', encoding='utf-8') as infof: info_dict = json.load(infof) - for (info_field, value) in tc.get('info_dict', {}).items(): - if isinstance(value, compat_str) and value.startswith('md5:'): - self.assertEqual(value, 'md5:' + md5(info_dict.get(info_field))) + for (info_field, expected) in tc.get('info_dict', {}).items(): + if isinstance(expected, compat_str) and expected.startswith('md5:'): + self.assertEqual(expected, 'md5:' + md5(info_dict.get(info_field))) else: - self.assertEqual(value, info_dict.get(info_field), u'invalid value for field ' + info_field) + got = info_dict.get(info_field) + self.assertEqual( + expected, got, + u'invalid value for field %s, expected %r, got %r' % (info_field, expected, got)) # If checkable fields are missing from the test case, print the info_dict test_info_dict = dict((key, value if not isinstance(value, compat_str) or len(value) < 250 else 'md5:' + md5(value)) |