Enable code_completion for robber.py

[hotay]

Issue

It's just a thing to consider but since we are using magic for robber.py, there's no code completion engine could understand this. So, we might need to consider this, if we can, this is a strong plus point to robber.py. Anyway, it would be a hard one, then, just consider.

[tklarryonline]

I agree that this should be considered much later since it's not that huge need. Despite PyCharm, I believe the code completion engine like Vim's YouCompleteMe could cope with this.

[hieueastagile]

@tklarryonline No, actually, if we still use magic in robber.py, YCM could not understand it too (we are using it so we can make sure :D). Anyway, I'm agree that it should be considered later.

[tklarryonline]

Oh really 😐. I thought YCM uses word-matching to suggest us the next possible use case...

[hieueastagile]

I can say that it's more intelligent than what pyCharm can do :P.

[hieueastagile]

Last update:

• I found that robber.py actuall works well with anaconda code-completion.
• And, regarding this issue that I raised in jedi repo: https://github.com/davidhalter/jedi/issues/825. We can set jedi.settings.auto_import_modules. And if it's possible for jedi then YCM should work since YCM is actually backed by jedi.
• I'm not sure how it should work with pyCharm.

So, let's do a quick summary for this issue:

• We need to try it out with jedi and write documentation for anacoda and jedi.
• We need to investigate how it should work with pyCharm, and yes, document it.

[catriuspham]

I cannot get Pycharm, Atom with Jedi, Sublime with Anacoda and Jedi to autocomplete robber.py. So I come up with an easier solution: we create a class named ExpectTemplate, then we inherit expect from it. The code will look like this:

class ExpectTemplate:
    @staticmethod
    def eq(expected):
        """
        :param expected:
        :return:
        """
        pass

class expect(ExpectTemplate):
    ...

This is what I get in Pycharm

The benefit of this method is that we can freely show the args/kwargs that the user need to provide. @hieueastagile What do you think?

[hieueastagile]

It seems hacky/workaround, huh?

[catriuspham]

Not really, it's like we predefine a class with an outline, then override it later.

[hieueastagile]

@catriuspham Honestly, I don't like it :D. ExpectTemplate might not be synced with what we register to expect, so it's considered a code smell.

[catriuspham]

@hieueastagile I gonna research more, though :stuck_out_tongue:

[hieueastagile]

wait to see your good news @catriuspham

[hieueastagile]

@catriuspham After a year, I found a way to do that :))): https://amir.rachum.com/blog/2016/10/05/python-dynamic-attributes/

[toantran-ea]

💯

[tklarryonline]

Why we didn't think of __dir__ in the first place :))))

[catriuspham]

Is this the real life? Is this just fantasy?

[hieueastagile]

:)))). Don't you mind to implement it @tklarryonline @catriuspham :))?