Problem/Motivation

The $modules property must be declared protected in tests. See https://www.drupal.org/node/2909426

Proposed resolution

Declare $modules as a protected property in tests.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork facets-3257445

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

idebr created an issue. See original summary.

idebr’s picture

Issue summary: View changes
Status: Active » Needs review
anagomes’s picture

Assigned: Unassigned » anagomes

anagomes’s picture

Assigned: anagomes » Unassigned

There were some tests from submodules that still had the $modules property as public, so I changed them to protected. And also opened a MR with the two commits (mine and @idebr's).

idebr’s picture

Status: Needs review » Reviewed & tested by the community

Thanks anagomes :)

Merge request is now complete.

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

mkalkbrenner’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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