Objective

  1. rebuildContainer() only exists to synchronize the container of a child site with the parent site/test runner.

  2. It is meaningless and wrong to call rebuildContainer() in a test that does not extend from WebTestBase.

Proposed solution

  1. Move TestBase::rebuildContainer() into WebTestBase::rebuildContainer().

Comments

sun’s picture

Status: Active » Needs review
StatusFileSize
new5.54 KB

Status: Needs review » Needs work

The last submitted patch, 1: test.rebuildcontainer.1.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
StatusFileSize
new7.47 KB
new1.94 KB

That's exactly the 2 test failures that I expected :-)

sun’s picture

sun’s picture

Title: Limit rebuildContainer() to web tests (WebTestBase) » rebuildContainer() is not limited to web tests (WebTestBase)
Category: Task » Bug report
Priority: Normal » Major

This is an architectural bug, as is the sister issue #2253067: settingsSet() is not limited to KernelTestBase

ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community

makes complete sense to me

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • Commit bd248aa on 8.x by catch:
    Issue #2252967 by sun: RebuildContainer() is not limited to web tests (...

Status: Fixed » Closed (fixed)

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