Inline Method refactoring is allowed in methods of the descriptor protocol

[jonhnanthan]

Steps to reproduce the behavior:

1. Code before refactoring:

class Descriptor:
    def __get__(self, instance, owner):
        return 40

2. Apply the Inline Method refactoring to the method __get__.

3. Expected code after refactoring: the same as the original. It would be nice if Rope could emit a warning that applying the refactoring to this type of class is not allowed.

4. Rope produces the result below:

class Descriptor:
    pass

[lieryan]

Thanks for testing and describing these corner cases.

We may be able to create a new InlineDescriptor mechanism to implement something of this nature, but I don't know if this is something that can be done in a generic way given how flexible __get__ can get.

I don't think there's much we can really do here, other than, as you say, forbidding this refactoring, which should be pretty simple, we can just check that we don't allow inlining this dunder method.