Fix false positive unused-private-member for accessing attributes…

… in a class using ``cls`` (#4965)

* Fix false positive ``unused-private-member`` for accessing attributes in
a class using ``cls``

Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>

ChangeLog

@@ -63,6 +63,10 @@ Release date: TBA
 
   Closes #4907
 
+* Fix false positive ``unused-private-member`` for accessing attributes in a class using ``cls``
+
+  Closes #4849
+
 * Fix false positive ``unused-private-member`` for private staticmethods accessed in classmethods.
 
   Closes #4849

doc/whatsnew/2.11.rst

@@ -70,6 +70,10 @@ Other Changes
 
   Closes #4907
 
+* Fix false positive ``unused-private-member`` for accessing attributes in a class using ``cls``
+
+  Closes #4849
+
 * Extended ``consider-using-in`` check to work for attribute access.
 
 * Setting ``min-similarity-lines`` to 0 now makes the similarty checker stop checking for duplicate code

pylint/checkers/classes.py

@@ -1003,11 +1003,15 @@ def _check_unused_private_attributes(self, node: nodes.ClassDef) -> None:
                 if attribute.attrname != assign_attr.attrname:
                     continue
 
-                if assign_attr.expr.name == "cls" and attribute.expr.name in [
-                    "cls",
-                    "self",
-                ]:
-                    # If assigned to cls.attrib, can be accessed by cls/self
+                if (
+                    assign_attr.expr.name
+                    in [
+                        "cls",
+                        node.name,
+                    ]
+                    and attribute.expr.name in ["cls", "self", node.name]
+                ):
+                    # If assigned to cls or class name, can be accessed by cls/self/class name
                     break
 
                 if (

tests/functional/u/unused/unused_private_member.py

@@ -242,6 +242,25 @@ def __init__(self):
             FalsePositive4681.__instance = False  # This should be fine
             FalsePositive4681.__should_cause_error = False  # [unused-private-member]
 
+# Accessing attributes of the class using `cls` should not result in a false positive
+# as long as it is used within the class
+class FalsePositive4681b:
+    __instance = None
+
+    @classmethod  # Use class method here
+    def instance(cls):
+        if cls.__instance is None:
+            cls()
+        return cls.__instance
+
+    def __init__(self):
+        try:
+            FalsePositive4681b.__instance = 42  # This should be fine
+        except Exception:  # pylint: disable=broad-except
+            print("Error")
+            FalsePositive4681b.__instance = False  # This should be fine
+
+
 # https://github.com/PyCQA/pylint/issues/4849
 # Accessing private static methods from classmethods via `cls` should not result in a
 # false positive

tests/functional/u/unused/unused_private_member.txt

@@ -14,6 +14,6 @@ unused-private-member:212:8:Crash4755Context.__init__:Unused private member `Cra
 unused-private-member:229:4:FalsePositive4681:Unused private member `FalsePositive4681.__should_cause_error`:HIGH
 unused-private-member:239:12:FalsePositive4681.__init__:Unused private member `FalsePositive4681.__should_cause_error`:HIGH
 unused-private-member:243:12:FalsePositive4681.__init__:Unused private member `FalsePositive4681.__should_cause_error`:HIGH
-unused-private-member:255:4:FalsePositive4849.__unused_private_method:Unused private member `FalsePositive4849.__unused_private_method()`:HIGH
-unused-private-member:272:4:Pony.__init_defaults:Unused private member `Pony.__init_defaults(self)`:HIGH
-unused-private-member:277:4:Pony.__get_fur_color:Unused private member `Pony.__get_fur_color(self)`:HIGH
+unused-private-member:274:4:FalsePositive4849.__unused_private_method:Unused private member `FalsePositive4849.__unused_private_method()`:HIGH
+unused-private-member:291:4:Pony.__init_defaults:Unused private member `Pony.__init_defaults(self)`:HIGH
+unused-private-member:296:4:Pony.__get_fur_color:Unused private member `Pony.__get_fur_color(self)`:HIGH