Problem/Motivation

PHPUnit reports,

1x: The Drupal\Tests\field\Traits\EntityReferenceTestTrait is deprecated in drupal:10.2.0 and is removed from drupal:11.0.0. Instead, use \Drupal\Tests\field\Traits\EntityReferenceFieldCreationTrait. See https://www.drupal.org/node/3401941

Steps to reproduce

run phpunit list-groups

Proposed resolution

update the tests to use the new class.

CommentFileSizeAuthor
#3 issues-3415901-EntityReferenceTestTrait-deprecated-fixes.patch746 bytesviren18febs

Issue fork default_content-3415901

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:

About issue forks

Comments

BenStallings created an issue. See original summary.

viren18febS made their first commit to this issue’s fork.

viren18febs’s picture

viren18febs CreditAttribution: viren18febs at Concinnity Media Technologies for Concinnity Media Technologies commented
Status: Active » Needs review
StatusFileSize
new issues-3415901-EntityReferenceTestTrait-deprecated-fixes.patch746 bytes

Hi, I have fixed the issues and added a patch for this.
Thanks

viren18febS opened merge request !40

benstallings’s picture

benstallings CreditAttribution: benstallings at Share Good USA commented

The patch looks good, but the MR is not mergeable.

ankitv18 made their first commit to this issue’s fork.

ankitv18’s picture

ankitv18 CreditAttribution: ankitv18 as a volunteer and at Acquia commented
Version: 8.x-1.x-dev » 2.0.x-dev

Rebased the MR!40 with 2.0.x-dev ~~ now it is ready for review.

deepakkm made their first commit to this issue’s fork.

berdir’s picture

berdir
Primary language German
Location Switzerland
CreditAttribution: berdir at MD Systems GmbH commented
Status: Needs review » Closed (duplicate)

It makes more sense to do this as part of #3428127: Automated Drupal 11 compatibility fixes for default_content because it's needed for D11 compatibility and we can't test otherwise.

On the other hand, this breaks compatibility with version 10.1 and earlier, at least for running tests, so it's not something to do lightly.