Add code completions to `rope_autoimport` plugin

[tkrabel-db]

Fixes #403

Testing

• [x] Unit test
• [x] Manual test

[tkrabel-db]

It's weird. Unrelated tests are failing ...

[krassowski]

Tests are reaching the timeout:

https://github.com/python-lsp/python-lsp-server/blob/d4fb2764ccce0737df87394dc83650fc91e91827/.github/workflows/test-linux.yml#L28

Indeed 7 minutes for pytest here seems suspicious, but there is a small chance it could be GitHub temporarily throttling for no good reason.

[tkrabel-db]

@krassowski I see why the test takes so long.

• In config.py, we expect plugins to have an "enabled" field under their name in order to determine if the plugin is enabled or disabled
• I remove the top level "enable" from rope_autoimport
• This makes rope start for every test case, leading to the slow down

In any case, I do think the config should look more like

def pylsp_settings() -> Dict[str, Dict[str, Dict[str, Any]]]:
    # Default rope_completion to disabled
    return {
        "plugins": {
            "rope_autoimport": {
                "enabled": False,  # global "kill switch"
                "memory": False,
                "completions": {
                    "enabled": True, 
                },
                "code_actions": {
                    "enabled": True,
                },
            }
        }
    }

I will address this tomorrow. Need some sleep now :)

[tkrabel-db]

Couldn't resist @ccordoba12 @krassowski . I think my PR is ready to review now :)

[ccordoba12]

@krassowski, would you like to give this one another pass? Or should I merge?