[PP-1] Merge test methods in FieldUIRouteTest for better performance

Created on 10 July 2024, 8 days ago
Updated 14 July 2024, 5 days 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

Postponed

Version

11.0 🔥

Component
PHPUnit 

Last updated about 2 hours ago

Created by

🇩🇪Germany FeyP

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

Comments & Activities

Production build 0.69.0 2024