bleedingdeacons / reach / 35528326095
86%
main: 86%

LAST BUILD BRANCH: test/pest-5
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%. First build
35528326095

Pull #119

github

crazeydave 
feat!: move to PHP 8.4/8.5 and PHPUnit 13, tested to WP 7.1.1

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 CHANGE: r... (continued)
Pull Request #119: feat!: move to PHP 8.4/8.5 and PHPUnit 13, tested to WP 7.1.1

6386 of 7388 relevant lines covered (86.44%)

15.29 hits per line

Jobs
ID Job ID Ran Files Coverage
1 35528326095.1 95
86.44
GitHub Action Run
Source Files on build 35528326095