ovv commented 6 years ago

The missing fixture was making pytest skip a big part of the tests

ovv commented 6 years ago

Thanks @jlaine for the catch.

I should really find a bit of time to improve the aiosip test tooling and add PyPI deploy from Travis.

codecov-io commented 6 years ago

Codecov Report

Merging #60 into master will increase coverage by 43.16%. The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #60       +/-   ##
===========================================
+ Coverage   27.56%   70.72%   +43.16%     
===========================================
  Files          15       15               
  Lines        1455     1455               
  Branches      260      260               
===========================================
+ Hits          401     1029      +628     
+ Misses       1041      315      -726     
- Partials       13      111       +98

Impacted Files	Coverage Δ
aiosip/pytest_plugin.py	`58.57% <0%> (+8.57%)`	:arrow_up:
aiosip/param.py	`90% <0%> (+15%)`	:arrow_up:
aiosip/contact.py	`75% <0%> (+17.85%)`	:arrow_up:
aiosip/uri.py	`68.42% <0%> (+29.82%)`	:arrow_up:
aiosip/application.py	`67.04% <0%> (+31.81%)`	:arrow_up:
aiosip/utils.py	`69.23% <0%> (+33.33%)`	:arrow_up:
aiosip/protocol.py	`62.37% <0%> (+41.58%)`	:arrow_up:
aiosip/dialplan.py	`90% <0%> (+42.5%)`	:arrow_up:
aiosip/transaction.py	`64.36% <0%> (+43.1%)`	:arrow_up:
aiosip/peers.py	`68.24% <0%> (+47.44%)`	:arrow_up:
... and 3 more

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 ba7c9f9...ce4e766. Read the comment docs.

Eyepea / aiosip

Add the missing loop fixture #60

Codecov Report