SectionRenderTest contains coverage for multiple classes

Created on 6 February 2018, over 6 years ago
Updated 26 March 2024, 3 months ago

Problem/Motivation

The unit test SectionRenderTest originally tested the rendering of Sections, which was great.
But then the rendering of Sections was reduced to "loop through the array of components" and the actual logic moved to SectionComponent.
Now with #2937799: Allow greater flexibility within SectionComponent::toRenderArray() β†’ the actual logic is moving to an event subscriber.

The test uses Reflection to ensure the event subscriber is called by the component which is called by the section.

Proposed resolution

Split SectionRenderTest into 3 separate test classes, each testing a single class: Section, SectionComponent, and the new event subscriber.

Remaining tasks

Wait for #2937799: Allow greater flexibility within SectionComponent::toRenderArray() β†’ to go in.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component
Layout builderΒ  β†’

Last updated about 3 hours ago

Created by

πŸ‡ΊπŸ‡ΈUnited States tim.plunkett Philadelphia

Live updates comments and jobs are added and updated live.
  • Blocks-Layouts

    Blocks and Layouts Initiative. See the #2811175 Add layouts to Drupal issue.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024