Merge pull request #458 from ghanshyam-lele/newstr-hash

Bug fix #454: Implement __hash__() in newstr

Author: jmadler

src/future/types/newstr.py

@@ -105,6 +105,7 @@ def __repr__(self):
         """
         Without the u prefix
         """
+
         value = super(newstr, self).__repr__()
         # assert value[0] == u'u'
         return value[1:]
@@ -292,6 +293,13 @@ def __eq__(self, other):
         else:
             return False
 
+    def __hash__(self):
+        if (isinstance(self, unicode) or
+            isinstance(self, bytes) and not isnewbytes(self)):
+            return super(newstr, self).__hash__()
+        else:
+            raise NotImplementedError()
+
     def __ne__(self, other):
         if (isinstance(other, unicode) or
             isinstance(other, bytes) and not isnewbytes(other)):

tests/test_future/test_str.py

@@ -363,6 +363,10 @@ def test_eq(self):
             self.assertFalse(b'ABCD' == s)
         self.assertFalse(bytes(b'ABCD') == s)
 
+    def test_hash(self):
+        s = str('ABCD')
+        self.assertIsInstance(hash(s),int)
+
     def test_ne(self):
         s = str('ABCD')
         self.assertNotEqual('A', s)