IDEA-CR-24258 check that parentheses belong to an argument list

python/src/com/jetbrains/python/documentation/PythonDocumentationProvider.java

@@ -627,7 +627,8 @@ public class PythonDocumentationProvider extends AbstractDocumentationProvider i
       if (PythonDialectsTokenSetProvider.INSTANCE.getKeywordTokens().contains(elementType)) {
         return contextElement;
       }
-      if (PyTokenTypes.LPAR == elementType || PyTokenTypes.RPAR == elementType) {
+      final PsiElement parent = contextElement.getParent();
+      if (parent instanceof PyArgumentList && (PyTokenTypes.LPAR == elementType || PyTokenTypes.RPAR == elementType)) {
         final PyCallExpression expression = PsiTreeUtil.getParentOfType(contextElement, PyCallExpression.class);
         if (expression != null) {
           final PyExpression callee = expression.getCallee();

python/testData/quickdoc/NotArgumentList.py

@@ -0,0 +1,9 @@
+# directly in function
+def foo():
+  "<the_doc>Doc of foo."
+  pass
+
+def bar(param):
+  pass
+
+bar((1, foo<the_ref>))

python/testSrc/com/jetbrains/python/PyQuickDocTest.java

@@ -300,6 +300,14 @@ public class PyQuickDocTest extends LightMarkedTestCase {
     checkByHTML(myProvider.generateDoc(element, originalElement));
   }
 
+  public void testNotArgumentList() {
+    Map<String, PsiElement> marks = loadTest();
+    final PsiElement originalElement = marks.get("<the_ref>");
+
+    final PsiElement element = myProvider.getCustomDocumentationElement(myFixture.getEditor(), myFile, originalElement);
+    assertNull(element);
+  }
+
   public void testDocstring() {
     Map<String, PsiElement> marks = loadTest();
     final PsiElement originalElement = marks.get("<the_ref>");