[PP-2] Test Seven's changes to Media Library

Created on 29 October 2019, almost 5 years ago
Updated 10 February 2024, 7 months ago

Problem/Motivation

In #3082690: Mark Media Library as a stable core module β†’ and related issues, a lot of Media Library's front-end code was moved into the Seven theme. Initially, it was all tested by MediaLibrarySevenTheme, which extended MediaLibraryTest and duplicated all of its coverage with Seven as the active theme.

However, because MediaLibraryTest was an unreliable, nightmarish monstrosity of a test, it was decided to remove MediaLibrarySevenTest from #3082690: Mark Media Library as a stable core module β†’ and, once MediaLibraryTest had been refactored and stabilized, restore the test coverage for Seven's modifications.

Proposed resolution

Add a new MediaLibrarySevenTest which runs through all parts of the media library that Seven touches, and asserts that the changes are made and do not break anything.

Remaining tasks

Add the test once #3087227: [META] Split up and refactor MediaLibraryTest β†’ , and ideally πŸ› [random test failure] Random failures building media library form after uploading image (WidgetUploadTest) Closed: cannot reproduce are addressed.

User interface changes

None.

API changes

None; Drupal's tests are not APIs.

Data model changes

None.

Release notes snippet

None needed.

πŸ“Œ Task
Status

Closed: outdated

Version

11.0 πŸ”₯

Component
MediaΒ  β†’

Last updated less than a minute ago

Created by

πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

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.71.5 2024