undid my change in pandas/core/ops/__init__.py and handle the case differently in pandas/tests/base/test_ops.py

Author: SaturnFromTitan

pandas/core/ops/__init__.py

@@ -265,7 +265,7 @@ def _get_opstr(op):
         rtruediv: "/",
         operator.floordiv: "//",
         rfloordiv: "//",
-        operator.mod: "%",
+        operator.mod: None,  # TODO: Why None for mod but '%' for rmod?
         rmod: "%",
         operator.pow: "**",
         rpow: "**",

pandas/tests/base/test_ops.py

@@ -1,5 +1,6 @@
 from datetime import datetime, timedelta
 from io import StringIO
+import operator
 import sys
 from typing import Any
 
@@ -86,9 +87,13 @@ def setup_method(self, method):
 
 @pytest.mark.parametrize("klass", [Series, DataFrame])
 def test_binary_ops_docs(klass, all_arithmetic_functions):
-    operator = all_arithmetic_functions
-    operator_name = _get_op_name(operator, special=False)
-    operator_symbol = _get_opstr(operator)
+    op = all_arithmetic_functions
+    operator_name = _get_op_name(op, special=False)
+    if op is operator.mod:
+        # _get_opstr returns 'None' for this operator
+        operator_symbol = "%"
+    else:
+        operator_symbol = _get_opstr(op)
     is_reverse = operator_name.startswith("r")
 
     operand1 = klass.__name__.lower()