Problem/Motivation

Method getMockForAbstractClass() of class PHPUnit\Framework\TestCase is deprecated in PHPUnit 10.

Some uses were removed as part of #3217904: [meta] Support PHPUnit 10 in Drupal 11, but there's a relevant number of places where this is still used.

Proposed resolution

Replace in most remaining views plugins. One instance remains, but that needs a general entity stub and will require a separate issue.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#7 3480419-nr-bot.txt90 bytesneeds-review-queue-bot

Issue fork drupal-3480419

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

mondrake created an issue. See original summary.

mondrake’s picture

Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Seems like a good refactor.

Wonder eventually if it should be documented somewhere all these Stubs?

catch’s picture

Status: Reviewed & tested by the community » Needs work

Needs a rebase for the phpstan baseline.

mondrake’s picture

Status: Needs work » Reviewed & tested by the community

rebased

needs-review-queue-bot’s picture

Status: Reviewed & tested by the community » Needs work
StatusFileSize
new90 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

mondrake’s picture

Status: Needs work » Reviewed & tested by the community

rebased

  • quietone committed 11103e23 on 11.1.x
    Issue #3480419 by mondrake, smustgrave, catch: Method...

  • quietone committed 88413a56 on 11.x
    Issue #3480419 by mondrake, smustgrave, catch: Method...
quietone’s picture

Version: 11.x-dev » 11.1.x-dev
Status: Reviewed & tested by the community » Fixed

@mondrake, thanks for keeping this up to date.

Committed to 11.x and cherry-picked to 11.1.x

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.