solidusio / solidus_stripe

πŸ’³ Integrate Solidus with Stripe
https://stripe.com
BSD 3-Clause "New" or "Revised" License
36 stars 61 forks source link

Add a basic codecov config to show commit status #204

Closed elia closed 1 year ago

elia commented 1 year ago

Summary

Fixes #211

Use coverage to understand what major areas or edge cases should be tested.

Correctly translates paths from the dummy-app to the templates dir, so that sources are visible in the CodeCov UI.

ERB look off from time to time, but it's not clear if a line is lost in translation or it's a ruby+ERB bug.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

codecov[bot] commented 1 year ago

Codecov Report

Merging #204 (4b0077f) into master (d74d901) will decrease coverage by 1.26%. The diff coverage is 100.00%.

:exclamation: Current head 4b0077f differs from pull request most recent head bca8001. Consider uploading reports for the commit bca8001 to get more accurate results

@@            Coverage Diff             @@
##           master     #204      +/-   ##
==========================================
- Coverage   94.16%   92.91%   -1.26%     
==========================================
  Files          15      124     +109     
  Lines         360     1580    +1220     
==========================================
+ Hits          339     1468    +1129     
- Misses         21      112      +91     
Impacted Files Coverage Ξ”
Rakefile 100.00% <100.00%> (ΓΈ)
config/routes.rb 100.00% <100.00%> (ΓΈ)

... and 125 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more