Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: sebastianbergmann/phpunit
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: Vassyli/phpunit
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 3 commits
  • 6 files changed
  • 2 contributors

Commits on Jan 14, 2026

  1. Fixes #6470: Mocking a class with a covariant property hook setter le… 

    …ads to an unexpected fatal error

Set hooks allow covariant setters to expand the scope of a parameter (eg, converting strings into Datetimes). PHPUnit uses wrongly the type for the property for the setter instead of the setters type. @see Issue #6470.
    @Vassyli
    Vassyli committed Jan 14, 2026
    Configuration menu
    Copy the full SHA
    041daf8 View commit details
    Browse the repository at this point in the history

Commits on Jan 15, 2026

  1. Removes the require_once statement in testExpectationCanBeConfiguredF… 

    …orCovariantSetHookForPropertyOfExtendableClass.
    @Vassyli
    Vassyli committed Jan 15, 2026
    Configuration menu
    Copy the full SHA
    b4e7ee3 View commit details
    Browse the repository at this point in the history

  2. Makes HookedProperty parameter setterType nullable. 

    Hooked properties do not necessarily have a setter, therefore having a setterType on hooked properties that is not nullable leads to conflicts.

The return type of setterType is not nullable. If the property has a setter, it must have a setter type, too. The setter type can be the same, but doesn't need to be.
    @Vassyli
    Vassyli committed Jan 15, 2026
    Configuration menu
    Copy the full SHA
    087ddc5 View commit details
    Browse the repository at this point in the history
Loading