Valloric / ycmd / 1082

94%

push

travis-ci

Auto merge of #246 - vheon:feature-jedihttp, r=micbou

[READY] Use a http wrapper for jedi

This is the first step to support python completions for python version different from the one ycmd is currently running.

# What this change does

The jedi library is still used, but instead of calling it directly we now wrapped in a http+JSON server.

# Testing

~~The test have been updated to solicit the completer first with a `FileReadyToParse` event, which would start a JediHTTP server if one is not running (similarly to the `OmnisharpServer` tests). The tests for the `GoTo` subcommand family have been updated to test all three of the subcommands.~~

# Notes

Two subcommand have been added: `RestartServer` and `StopServer` but only the `RestartServer` is exposed through the user by the `DefinedSubcommand` method, this is because I think that the `StopServer` command is not useful for the user but is useful in the testing phase for shutting down the JediHTTP.

I have one question: would make sense to rename any reference from `jedi_completer` to `python_completer`? If it is I will make another PR.

@reinhrst If you could test this would be awesome :)

Fixes Valloric/YouCompleteMe#1827

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/246)
<!-- Reviewable:end -->

138 of 138 new or added lines in 1 file covered. (100.0%)

2381 of 2910 relevant lines covered (81.82%)

0.82 hits per line