Problem/Motivation

Views tests install a module then immediately rebuild the container again, looks like it shouldn't be necessary, let's find out.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3613269

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

catch created an issue. See original summary.

catch opened merge request !16449

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Seems straight forward. Since it's a base class any concern for contrib/custom modules? Assuming not but wanted to ask.

catch’s picture

catch
he/him
Primary language English
commented

I think if this was going to fail one of the core tests would. It was probably an old workaround for using $this->container where it shouldn't be

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 1ffc3dce183 to main. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • longwave committed 1ffc3dce on main 
    test: #3613269 Avoid unnecessary container rebuild in views functional...