Suppress useless-super-delegation if return type changed (pylint-…

…dev#5822)

pylint/checkers/classes/class_checker.py

@@ -1263,6 +1263,13 @@ def _check_useless_super_delegation(self, function):
 
         if meth_node is not None:
 
+            if function.returns is not None and (
+                meth_node.returns is None
+                or meth_node.returns.as_string() != function.returns.as_string()
+            ):
+                # Override adds typing information to the return type
+                return
+
             def form_annotations(arguments):
                 annotations = chain(
                     (arguments.posonlyargs_annotations or []), arguments.annotations

tests/functional/u/useless/useless_super_delegation_py35.py

@@ -10,3 +10,37 @@ class UselessSuper(object):
 
     def useless(self, first, *, second=None, **kwargs): # [useless-super-delegation]
         return super().useless(first, second=second, **kwargs)
+
+# pylint: disable=wrong-import-position
+import random
+from typing import Any
+
+class ReturnTypeAny:
+    choices = ['a', 1, (2, 3)]
+
+    def draw(self) -> Any:
+        return random.choice(self.choices)
+
+class ReturnTypeNarrowed(ReturnTypeAny):
+    choices = [1, 2, 3]
+
+    def draw(self) -> int:
+        return super().draw()
+
+class NoReturnType:
+    choices = ['a', 1, (2, 3)]
+
+    def draw(self):
+        return random.choice(self.choices)
+
+class ReturnTypeSpecified(NoReturnType):
+    choices = ['a', 'b']
+
+    def draw(self) -> str:
+        return super().draw()
+
+class ReturnTypeSame(ReturnTypeAny):
+    choices = ['a', 'b']
+
+    def draw(self) -> Any: # [useless-super-delegation]
+        return super().draw()

tests/functional/u/useless/useless_super_delegation_py35.txt

@@ -1 +1,2 @@
 useless-super-delegation:11:4:11:15:UselessSuper.useless:Useless super delegation in method 'useless':UNDEFINED
+useless-super-delegation:45:4:45:12:ReturnTypeSame.draw:Useless super delegation in method 'draw':UNDEFINED