Skip to content

Add pragma: no cover to avoid spurious cov changes#129

Merged
encukou merged 1 commit into
python-ldap:masterfrom
tiran:nocover
Dec 13, 2017
Merged

Add pragma: no cover to avoid spurious cov changes#129
encukou merged 1 commit into
python-ldap:masterfrom
tiran:nocover

Conversation

@tiran
Copy link
Copy Markdown
Member

@tiran tiran commented Dec 13, 2017

"pragma: no cover" was added to two helper functions in slapdtest to
avoid spurious changes in coverage whenever slapd has a hick up.

Closes: #127
Signed-off-by: Christian Heimes cheimes@redhat.com

"pragma: no cover" was added to two helper functions in slapdtest to
avoid spurious changes in coverage whenever slapd has a hick up.

Closes: python-ldap#127
Signed-off-by: Christian Heimes <cheimes@redhat.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 13, 2017

Codecov Report

Merging #129 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   68.87%   68.95%   +0.07%     
==========================================
  Files          49       49              
  Lines        4694     4683      -11     
  Branches      789      788       -1     
==========================================
- Hits         3233     3229       -4     
+ Misses       1111     1106       -5     
+ Partials      350      348       -2
Impacted Files Coverage Δ
Lib/slapdtest/_slapdtest.py 83.33% <100%> (+1.96%) ⬆️

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 8f2cded...7c4cee9. Read the comment docs.

@encukou encukou merged commit cf24a54 into python-ldap:master Dec 13, 2017
@tiran tiran deleted the nocover branch December 13, 2017 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants