Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

Enhancement: Collect code coverage #1824

Merged
merged 3 commits into from
Nov 28, 2019

Conversation

localheinz
Copy link
Contributor

@localheinz localheinz commented Oct 25, 2019

This PR

  • adds a whitelist filter configuration for collecting code coverage
  • adds a coverage target to Makefile for collecting code coverage
  • collects code coverage on Travis CI using https://codecov.io

@localheinz localheinz self-assigned this Oct 30, 2019
@localheinz localheinz changed the title Enhancement: Collect code coverage [WIP] Enhancement: Collect code coverage Nov 23, 2019
@localheinz localheinz changed the title [WIP] Enhancement: Collect code coverage Enhancement: Collect code coverage Nov 23, 2019
@localheinz localheinz added this to the 1.9.1 milestone Nov 23, 2019
@localheinz localheinz marked this pull request as ready for review November 23, 2019 11:42
- php: 7.1
- php: 7.2
- php: 7.3
env: WITH_COVERAGE=true
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably do not need to collect coverage on all builds - running on the latest version of PHP increases the likelihood that it's a fast run nonetheless.

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1824   +/-   ##
=========================================
  Coverage          ?   56.33%           
  Complexity        ?     2068           
=========================================
  Files             ?      306           
  Lines             ?     4851           
  Branches          ?        0           
=========================================
  Hits              ?     2733           
  Misses            ?     2118           
  Partials          ?        0

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 2c9b912...f4e0611. Read the comment docs.

@localheinz localheinz removed their assignment Nov 26, 2019
@pimjansen pimjansen merged commit 88bfb84 into fzaninotto:master Nov 28, 2019
@localheinz localheinz deleted the feature/coverage branch November 28, 2019 09:51
@localheinz
Copy link
Contributor Author

Thank you, @fzaninotto and @pimjansen!

@localheinz
Copy link
Contributor Author

Will add this to #1847 as well!

Partoo added a commit to Partoo/Faker that referenced this pull request Jun 21, 2020
* 'master' of github.com:Partoo/Faker: (294 commits)
  Fix (bug fzaninotto#1862): Do not clear entity manager for doctrine orm populator (fzaninotto#1995)
  Remove persian rude words. (fzaninotto#1879)
  Enhancement: Run builds on master using GitHub Actions
  Fix typo (fzaninotto#1927)
  Fix: Run build against PHP 7.4
  Fix: Aggregate badges in one place (fzaninotto#1866)
  Added changelog for the 1.9.1 release
  Fix: Reduce visibility of setUp() and tearDown() (fzaninotto#1821)
  Enhancement: Collect code coverage (fzaninotto#1824)
  Enhancement: Use all columns when running tests
  Add link to PHPStan extension to readme (fzaninotto#1834)
  Enhancement: Configure verbose output via phpunit.xml.dist
  Update master version
  Bump version to 1.9
  Curly braces for arrays is deprecated in PHP 7.4 (detected by PHPCompatibility)
  Updated indenting changelog
  Update changelog with 1.9.0 release
  Skipped lorumpixel test for the release where the service is pretty unstable
  Fix: Mark test classes as final
  Fix: Remove unnecessary class-level DocBlocks
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants