fix: avoid false positive when module-level names match class-level names

- Add scope comparision to avoid module-level constants to be incorrectly
classified as variables when a class-level attribute with the same name exists

Closes #10719

Author: joao-faria-dev

doc/whatsnew/fragments/10719.false_positive

@@ -0,0 +1,3 @@
+Fixed false positive for ``invalid-name`` where module-level constants were incorrectly classified as variables when a class-level attribute with the same name exists.
+
+Closes #10719

pylint/checkers/utils.py

@@ -1847,24 +1847,42 @@ def is_reassigned_before_current(node: nodes.NodeNG, varname: str) -> bool:
     """Check if the given variable name is reassigned in the same scope before the
     current node.
     """
-    return any(
-        a.name == varname and a.lineno < node.lineno
-        for a in node.scope().nodes_of_class(
-            (nodes.AssignName, nodes.ClassDef, nodes.FunctionDef)
-        )
-    )
+    node_scope = node.scope()
+    node_lineno = node.lineno
+    if node_lineno is None:
+        return False
+    for a in node_scope.nodes_of_class(
+        (nodes.AssignName, nodes.ClassDef, nodes.FunctionDef)
+    ):
+        if a.name == varname and a.lineno is not None and a.lineno < node_lineno:
+            if isinstance(a, (nodes.ClassDef, nodes.FunctionDef)):
+                if a.parent and a.parent.scope() == node_scope:
+                    return True
+            else:
+                if a.scope() == node_scope:
+                    return True
+    return False
 
 
 def is_reassigned_after_current(node: nodes.NodeNG, varname: str) -> bool:
     """Check if the given variable name is reassigned in the same scope after the
     current node.
     """
-    return any(
-        a.name == varname and a.lineno > node.lineno
-        for a in node.scope().nodes_of_class(
-            (nodes.AssignName, nodes.ClassDef, nodes.FunctionDef)
-        )
-    )
+    node_scope = node.scope()
+    node_lineno = node.lineno
+    if node_lineno is None:
+        return False
+    for a in node_scope.nodes_of_class(
+        (nodes.AssignName, nodes.ClassDef, nodes.FunctionDef)
+    ):
+        if a.name == varname and a.lineno is not None and a.lineno > node_lineno:
+            if isinstance(a, (nodes.ClassDef, nodes.FunctionDef)):
+                if a.parent and a.parent.scope() == node_scope:
+                    return True
+            else:
+                if a.scope() == node_scope:
+                    return True
+    return False
 
 
 def is_deleted_after_current(node: nodes.NodeNG, varname: str) -> bool:

tests/functional/c/const_class_attribute_same_name.py

@@ -0,0 +1,14 @@
+"""Test module-level constants with class attribute same name
+Regression test for #10719.
+"""
+# pylint: disable=missing-docstring, too-few-public-methods, redefined-builtin
+
+
+class Theme:
+    INPUT = ">>> "
+
+
+INPUT = Theme()
+input = Theme()
+OUTPUT = Theme()
+output = Theme()

tests/functional/n/no/no_dummy_redefined.py

@@ -1,4 +1,5 @@
 """Make sure warnings about redefinitions do not trigger for dummy variables."""
+# pylint: disable=invalid-name
 
 
 _, INTERESTING = 'a=b'.split('=')

tests/functional/n/no/no_dummy_redefined.txt

@@ -1 +1 @@
-redefined-outer-name:11:4:11:9:clobbering:Redefining name 'value' from outer scope (line 6):UNDEFINED
+redefined-outer-name:12:4:12:9:clobbering:Redefining name 'value' from outer scope (line 7):UNDEFINED