assignment-from-no-return considers the wrong method if ambiguous

[cschramm]

Steps to reproduce

class Test():
    def test(self):
        pass

    def test(self):
        return True

A = Test().test()

Current behavior

5:4: E0102: method already defined line 2 (function-redefined)
5:4: R0201: Method could be a function (no-self-use)
8:0: E1111: Assigning to function call which doesn't return (assignment-from-no-return)

Expected behavior

5:4: E0102: method already defined line 2 (function-redefined)
5:4: R0201: Method could be a function (no-self-use)

pylint --version output

pylint 2.0.1
astroid 2.0.1
Python 3.6.5 (default, May 11 2018, 13:30:17) 
[GCC 7.3.0]

pylint-dev/pylint@9a9de85 adds that check for methods and 2aa27e9 causes the first of the definitions to be the only one that gets returned from infer_attribute. ❎

For ambiguous local functions, infer_name returns the correct definition. ✔️
For ambiguous module functions, infer_attribute returns all definitions and safe_infer detects that ambiguity so that none of them get actually checked. ❎ (I'd consider that an issue as well but it only causes false negatives, so it's not a huge one).

[brycepg]

Thanks for the report

I think I'll make the first result of ClassDef.getattr be the last defined name in a classdef

[samertm]

FWIW, we've also hit this issue.