bleedingdeacons / reach / 35540059982
86%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 95
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 86.437% (+0.04%) from 86.402%
35540059982

push

github

web-flow 
feat!: move to PHP 8.4/8.5 and PHPUnit 13, tested to WP 7.1.1 (#119)

Part of the suite-wide PHP 8.5 / WordPress 7.1.1 upgrade.

- Requires PHP 8.1 -> 8.4, header and readme.txt. WordPress 7.1
  recommends 8.4/8.5, and PHPUnit 13 requires >= 8.4.
- PHPUnit ^10.5 -> ^13.0, with all 350 docblock annotations converted
  to attributes (285 @test, 9 @covers, 56 @dataProvider). PHPUnit 12
  removed doc-comment metadata outright. Converted with Rector; the
  counts match the originals exactly.
- bleedingdeacons/wp-mocks ^2.0 -> ^3.0, the release that raises that
  package's own floor to 8.4 and allows PHPUnit 13.
- CI runs a matrix of 8.4 and 8.5 rather than a single 8.1. The build
  archive, both badge assertions and the Coveralls upload are pinned to
  the 8.4 leg: they are identical on both, and running them on each has
  the two legs writing the same paths at once.
- PHPStan analyses a min/max range instead of one pinned version. A
  single pin proves one end only -- at the ceiling it misses an
  8.5-only construct that breaks on 8.4, at the floor it misses what
  8.5 deprecates.
- readme.txt Tested up to -> 7.1.1.

Verified with the same gate sequence ci.yml runs, on PHP 8.5.7:
composer validate, audit, php -l, PHPStan level 8, PHPUnit, PHPCS and
build:production. The test count is unchanged, which is the check that
matters -- a mis-mapped @covers moves coverage attribution without
failing anything.

Two PHP 8.5 / PHPUnit 13 fixes beyond the mechanical conversion:

- A private test helper named attempt() collided with PHPUnit 13's new
  final TestCase::attempt(): int, which is a fatal error at load time and
  took the whole suite down. Renamed callAttempt(), after what it returns.
- Dropped two ReflectionMethod/ReflectionProperty::setAccessible() calls.
  They have been no-ops since PHP 8.1 and PHP 8.5 deprecates them outright.
  The rest of the suite had already removed theirs and left a comment
  saying why; these two were the stragglers.

BREAKING CH... (continued)

6386 of 7388 relevant lines covered (86.44%)

15.29 hits per line

Jobs
ID Job ID Ran Files Coverage
1 35540059982.1 95
86.44
GitHub Action Run
Source Files on build 35540059982