Skip to content

refactor: remove phpunit CoversMethod - #29

Merged
didoda merged 1 commit into
bedita:masterfrom
didoda:refactor/5.x/phpunit-remove-coversmethod
Jun 9, 2026
Merged

refactor: remove phpunit CoversMethod#29
didoda merged 1 commit into
bedita:masterfrom
didoda:refactor/5.x/phpunit-remove-coversmethod

Conversation

@didoda

@didoda didoda commented Jun 9, 2026

Copy link
Copy Markdown
Member

This removes phpunit CoversMethod from tests files.
It is not "repeatable", so it originated phpstan errors.
CoversClass is enough.

@didoda didoda added this to the 5.x milestone Jun 9, 2026
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.6%. Comparing base (f4513c4) to head (67d6332).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             master     #29   +/-   ##
========================================
  Coverage      98.6%   98.6%           
  Complexity       69      69           
========================================
  Files             7       7           
  Lines           227     227           
========================================
  Hits            224     224           
  Misses            3       3           

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d5740a...67d6332. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@didoda
didoda merged commit 643d095 into bedita:master Jun 9, 2026
10 checks passed
@didoda
didoda deleted the refactor/5.x/phpunit-remove-coversmethod branch June 9, 2026 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant