BUG: Fix bug when using df.query with "str.contains()"

doc/source/whatsnew/v0.25.0.rst

@@ -322,7 +322,7 @@ Sparse
 Other
 ^^^^^
 
--
+- Bug in :class:`BaseExprVisitor` for using named kwargs in :func:`eval` expressions.
 -
 -
 

pandas/core/computation/expr.py

@@ -632,11 +632,8 @@ def visit_Call_35(self, node, side=None, **kwargs):
                 if not isinstance(key, ast.keyword):
                     raise ValueError("keyword error in function call "
                                      "'{func}'".format(func=node.func.id))
-
                 if key.arg:
-                    # TODO: bug?
-                    kwargs.append(ast.keyword(
-                        keyword.arg, self.visit(keyword.value)))  # noqa
+                    kwargs[key.arg] = self.visit(key.value)()
 
             return self.const_type(res(*new_args, **kwargs), self.env)
 

pandas/tests/computation/test_eval.py

@@ -1903,3 +1903,15 @@ def test_validate_bool_args(self):
         for value in invalid_values:
             with pytest.raises(ValueError):
                 pd.eval("2+2", inplace=value)
+
+
+def test_query_str_contains():
+    df = pd.DataFrame([["I", "XYZ"], ["IJ", None]], columns=['A', 'B'])
+
+    expected = df[df["A"].str.contains("J")]
+    result = df.query("A.str.contains('J')", engine="python")
+    tm.assert_frame_equal(result, expected)
+
+    expected = df[df["B"].str.contains("Z", na=False)]
+    result = df.query("B.str.contains('Z', na=False)", engine="python")
+    tm.assert_frame_equal(result, expected)