Merge test methods in FieldUIRouteTest for better performance

Created on 10 July 2024, 3 months ago
Updated 11 August 2024, 2 months ago

Problem/Motivation

Follow up for 📌 Fix Field UI tests that rely on UID1's super user behavior Active .

Drupal\Tests\field_ui\Functional\FieldUIRouteTest has two test methods. Each test method spins up it's own new Drupal instance, which causes some extra overhead. The testAdminRoute() performs a very basic test that could easily be done at the beginning of testFieldUIRoutes() before logging in with an account so that the test uses less time/resources.

Steps to reproduce

Not applicable.

Proposed resolution

Merge both test methods. Move the contents of testAdminRoute() to the beginning of testFieldUIRoutes() before logging in with an account.

Remaining tasks

  1. Create MR.
  2. Review.
  3. Commit.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

Not needed.

📌 Task
Status

Fixed

Version

10.3

Component
PHPUnit 

Last updated 1 day ago

Created by

🇩🇪Germany FeyP

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

Merge Requests

Comments & Activities

Production build 0.71.5 2024