Skip to content

Feature/method covers overrides class covers nothing#3115

Merged
sebastianbergmann merged 3 commits into
sebastianbergmann:masterfrom
Danack:feature/method_covers_overrides_class_covers_nothing
May 7, 2018
Merged

Feature/method covers overrides class covers nothing#3115
sebastianbergmann merged 3 commits into
sebastianbergmann:masterfrom
Danack:feature/method_covers_overrides_class_covers_nothing

Conversation

@Danack

@Danack Danack commented May 7, 2018

Copy link
Copy Markdown
Contributor

Implements sebastianbergmann/php-code-coverage#611 aka allows a 'method covers' annotation to override the 'class coversNothing' annotation.

Reasonably chance this might need to wait until PHPUnit 7.2.0

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #3115 into master will increase coverage by <.01%.
The diff coverage is 90%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3115      +/-   ##
============================================
+ Coverage        81%   81.01%   +<.01%     
- Complexity     2914     2917       +3     
============================================
  Files           109      109              
  Lines          7630     7639       +9     
============================================
+ Hits           6181     6189       +8     
- Misses         1449     1450       +1
Impacted Files Coverage Δ Complexity Δ
src/Util/Test.php 94.44% <90%> (-0.11%) 215 <4> (+3)

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 d757cfa...7aca9db. Read the comment docs.

@sebastianbergmann sebastianbergmann merged commit 5c13945 into sebastianbergmann:master May 7, 2018
@keradus

keradus commented Jun 1, 2018

Copy link
Copy Markdown
Contributor

Really glad to see this, thanks @Danack, nice work here !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants