Adjust old tests

Created on 19 April 2023, over 1 year ago
Updated 23 August 2023, over 1 year ago

Problem/Motivation

The current tests are old and should be adjusted to the new Drupal 9/10 Test standards.

Steps to reproduce

Proposed resolution

Adjust the tests accordingly.

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇩🇪Germany Grevil

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

Comments & Activities

  • Issue created by @Grevil
  • Open in Jenkins → Open on Drupal.org →
    Core: 10.1.x + Environment: PHP 8.2 & MySQL 8
    last update over 1 year ago
    3 pass
  • @grevil opened merge request.
  • Status changed to Needs review over 1 year ago
  • 🇩🇪Germany Grevil

    Alright! The tests should be fixed now.

    I almost went insane trying to figure out, why the "media/{id}" route didn't resolve even with the "standalone_url" setting enabled. Turns out, the routing needs to get rebuild after changing that setting inside tests.

    Furthermore, I needed to adjust your "media->label() present" check, as on default the label won't get displayed and I didn't want to tinker with the media view_mode settings.

    Please review!

  • Status changed to RTBC over 1 year ago
  • 🇩🇪Germany Anybody Porta Westfalica

    Nice work @Grevil! Tests pass again and changes make sense.

    Switching to Olivero is untypical and makes the tests slower and more memory-consuming. Are there any good alternatives in using a more lightweight theme which also allows selectors?

    Still I'll set this RTBC for the maintainer to decide, how to proceed. We could also keep it this way or change the theme later in a follow-up.

  • First commit to issue fork.
  • Status changed to Fixed over 1 year ago
  • 🇪🇸Spain marcoscano Barcelona, Spain

    Good work, thanks! And sorry for the delayed review.

  • 🇩🇪Germany Grevil

    Thanks! No worries, great, that they are in now! 😁

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

Production build 0.71.5 2024