Necessary before release.

Look at interdiffs in #2603046: Support anonymous users and especially #2469431: BigPipe for auth users: first send+render the cheap parts of the page, then the expensive parts to find which parts need particular testing attention.

Note that to test the BigPipe placeholders that are replaced in JS, we'll need to simulate that without JS, much like we simulate the AJAX system in Drupal core's WebTestBase.

Unless of course we can start using BrowserTestBase soonish…

Test coverage added

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Issue summary: View changes

The child issue #2671958: SessionExistsCacheContext test coverage was added and fixed, and adds test coverage for SessionExistsCacheContext.

Wim Leers’s picture

Category: Task » Plan
Wim Leers’s picture

Issue summary: View changes
Wim Leers’s picture

Issue summary: View changes
Wim Leers’s picture

Issue summary: View changes
Wim Leers’s picture

Issue summary: View changes
Wim Leers’s picture

The child issues linked to explicitly from the IS represent the total set of test coverage that is needed. Once all children are fixed, I will close this meta issue.

Wim Leers’s picture

All of the test coverage issues/patches have landed. With the exception of #2675670: BigPipe response delivery test coverage with JS enabled & disabled (BigPipe + HtmlResponseBigPipeSubscriber). I've posted a WIP patch to that issue, and will continue & finish that last one tomorrow. Expect a new core patch at #2469431: BigPipe for auth users: first send+render the cheap parts of the page, then the expensive parts tomorrow!

Wim Leers’s picture

Status: Active » Fixed

All done!

Status: Fixed » Closed (fixed)

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