fix Range.__hash__

remove `other` from parameter list of `__hash__`
MagicStack · Nov 30, 2017 · 46f468c · 46f468c
1 parent 695b8f0
commit 46f468c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/asyncpg/types.py b/asyncpg/types.py
@@ -101,7 +101,7 @@ def __eq__(self, other):
             other._empty
         )
 
-    def __hash__(self, other):
+    def __hash__(self):
         return hash((
             self._lower,
             self._upper,

diff --git a/tests/test_codecs.py b/tests/test_codecs.py
@@ -817,6 +817,16 @@ async def test_range_types(self):
                 ValueError, 'expected 0, 1 or 2 elements'):
             await self.con.fetch("SELECT $1::int4range", (0, 2, 3))
 
+        cases = [(asyncpg.Range(0, 1), asyncpg.Range(0, 1), 1),
+                 (asyncpg.Range(0, 1), asyncpg.Range(0, 2), 2),
+                 (asyncpg.Range(empty=True), asyncpg.Range(0, 2), 2),
+                 (asyncpg.Range(empty=True), asyncpg.Range(empty=True), 1),
+                 (asyncpg.Range(0, 1, upper_inc=True), asyncpg.Range(0, 1), 2),
+                 ]
+        for obj_a, obj_b, count in cases:
+            dic = {obj_a: 1, obj_b: 2}
+            self.assertEqual(len(dic), count)
+
     async def test_extra_codec_alias(self):
         """Test encoding/decoding of a builtin non-pg_catalog codec."""
         await self.con.execute('''