Report coverage

[blueyed]

TODO:

• requires Test with Neovim also #864
• rename flags to py27/py36

Includes/rebased on #864 for now.

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@b33392c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #863   +/-   ##
=========================================
  Coverage          ?   22.99%           
=========================================
  Files             ?        2           
  Lines             ?      574           
  Branches          ?      112           
=========================================
  Hits              ?      132           
  Misses            ?      433           
  Partials          ?        9

Flag	Coverage Δ
#py27	21.6% <ø> (?)
#py36	20.9% <ø> (?)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b33392c...04f6222. Read the comment docs.

[blueyed]

For some reason py2 is not working: https://travis-ci.org/davidhalter/jedi-vim/jobs/409612945

:checkhealth reports it to be ok, but during tests has('python') appears to be false.

[blueyed]

Found it:

provider/pythonx: Could not load Python 2:^@ /home/travis/virtualenv/python2.7.14/bin/python2 is Python Coverage.py warning: --include is ignored because --source is set (include-ignored)^@ 2.7 and cannot provide Python 2.

A Coverage.py warning apparently causes pytest to not detect py2 correctly.. :/

[blueyed]

20.9% coverage.. \o/

Needs some cleanup, but should be good then already (let's get the branches/PRs it depends on though in anyway).

Next step would be covimerage to track coverage for the Vimscript part.

[blueyed]

Acutally > 25% (flags were broken).

Something looks a bit odd, e.g. https://codecov.io/gh/davidhalter/jedi-vim/src/ce0d16a4dfafdcd92bae67475dcd0d2b5b6fabae/pythonx/jedi_vim.py#L166 (is there an exception/error happening?).

[blueyed]

Should be good, but I want to cleanup/review it myself once more.

[blueyed]

I like it! ;)

Should be squash-merged (I can do it).

[davidhalter]

I'll just put you in charge for this. :) I like that we have coverage and don't really care how it's exactly done!

[blueyed]

Yeah! \o/