diff --git a/scrapy/trunk/scrapy/tests/test_utils_misc.py b/scrapy/trunk/scrapy/tests/test_utils_misc.py new file mode 100644 index 000000000..994b165d5 --- /dev/null +++ b/scrapy/trunk/scrapy/tests/test_utils_misc.py @@ -0,0 +1,14 @@ +import unittest + +from scrapy.utils.misc import hash_values +from scrapy.core.exceptions import UsageError + +class UtilsMiscTestCase(unittest.TestCase): + def test_hash_values(self): + self.assertEqual(hash_values('some', 'values', 'to', 'hash'), + 'f37f5dc65beaaea35af05e16e26d439fd150c576') + + self.assertRaises(UsageError, hash_values, 'some', None, 'value') + +if __name__ == "__main__": + unittest.main() diff --git a/scrapy/trunk/scrapy/utils/misc.py b/scrapy/trunk/scrapy/utils/misc.py index b7d582c34..d69a028e1 100644 --- a/scrapy/trunk/scrapy/utils/misc.py +++ b/scrapy/trunk/scrapy/utils/misc.py @@ -2,7 +2,7 @@ Auxiliary functions which doesn't fit anywhere else """ import re -import sha +import hashlib from twisted.internet import defer @@ -104,7 +104,7 @@ def hash_values(*values): >>> hash_values('some', 'values', 'to', 'hash') 'f37f5dc65beaaea35af05e16e26d439fd150c576' """ - hash = sha.new() + hash = hashlib.sha1() for value in values: if value is None: message = "hash_values was passed None at argument index %d. This is a bug in the calling code" \