bpo-37555: Replacing __eq__ with == to sidestep NotImplemented

ElizabethU · ElizabethU · commit f295eaca5bce · 2019-07-19T20:45:52.000-07:00
bool(NotImplemented) returns True, so it's necessary to use ==
instead of __eq__ in this comparison.
diff --git a/Lib/unittest/mock.py b/Lib/unittest/mock.py
@@ -348,7 +348,7 @@ def __eq__(self, other):
         self_list = list(self)
         other_list = list(other)
         # checking equality both directions is necessary for ANY to work
-        return self_list.__eq__(other_list) or other_list.__eq__(self_list)
+        return self_list == other_list or other_list == self_list
 
 
 def _check_and_set_parent(parent, value, name, new_name):
@@ -2411,8 +2411,8 @@ def __eq__(self, other):
         self_params = self_args, self_kwargs
         other_params = other_args, other_kwargs
         return (
-            self_params.__eq__(other_params)
-            or other_params.__eq__(self_params)
+            self_params == other_params
+            or other_params == self_params
         )
 
 
