Add is_class_var utils function

2 files changed, 12 insertions, 8 deletions

diff --git a/pylint/checkers/base.py b/pylint/checkers/base.py
index 33aa8a037..393a433fd 100644
--- a/pylint/checkers/base.py
+++ b/pylint/checkers/base.py
@@ -1988,16 +1988,10 @@ class NameChecker(_BasicChecker):
         elif isinstance(frame, astroid.ClassDef):
             if not list(frame.local_attr_ancestors(node.name)):
                 for ancestor in frame.ancestors():
-                    if (  # pylint: disable=too-many-boolean-expressions
+                    if (
                         ancestor.name == "Enum"
                         and ancestor.root().name == "enum"
-                        or isinstance(node.parent, astroid.AnnAssign)
-                        and (
-                            isinstance(node.parent.annotation, astroid.Subscript)
-                            and node.parent.annotation.value.name == "ClassVar"
-                            or isinstance(node.parent.annotation, astroid.Name)
-                            and node.parent.annotation.name == "ClassVar"
-                        )
+                        or utils.is_class_var(node)
                     ):
                         self._check_name("class_const", node.name, node)
                         break
diff --git a/pylint/checkers/utils.py b/pylint/checkers/utils.py
index 29f774a75..37abc3a87 100644
--- a/pylint/checkers/utils.py
+++ b/pylint/checkers/utils.py
@@ -1459,3 +1459,13 @@ def is_attribute_typed_annotation(
         ):
             return True
     return False
+
+
+def is_class_var(node: astroid.AssignName) -> bool:
+    """Test if node has `ClassVar` annotation."""
+    return isinstance(node.parent, astroid.AnnAssign) and (
+        isinstance(node.parent.annotation, astroid.Subscript)
+        and node.parent.annotation.value.name == "ClassVar"
+        or isinstance(node.parent.annotation, astroid.Name)
+        and node.parent.annotation.name == "ClassVar"
+    )