Add performance test coverage for menu UI

Open on Drupal.org →

Created on 24 September 2024, 3 months ago

Problem/Motivation

The menu UI needs to deal with loading the menu tree, so we should add performance test coverage for it. Especially for the main admin menu which is huge.

This can probably be added to the standard profile or use the testing profile in a system test, since there are enough menu items from system module to test with.

Drupal\Tests\menu_ui\FunctionalJavascript\MenuPerformanceTest:testPerformance()~

Umami and standard performance tests can be used as examples.

OpenTelemetryNodePagePerformanceTest
OpenTelemetryFrontPagePerformanceTest
StandardPerformanceTest

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task

Status

Active

Version

11.0 🔥

Component

phpunit

Created by

🇬🇧United Kingdom catch

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!9827Add performance test coverage for menu UI
Open
🇳🇱Netherlands bramr
updated 3 months ago

Comments & Activities

Issue created by @catch
Comment 3 months ago →
🇳🇱Netherlands bramr
Did your workshop at Drupalcon #Barcelona2024, i should have some time in the upcoming week to pick this up.
Merge request !9827Issue #3476440:Add performance test coverage for menu UI → (Open) created by bramr
Comment 3 months ago →
🇳🇱Netherlands bramr
Think it's ready for a review.

I agree with the solution to move clearCaches into the trait as proposed by issue: #3476433 📌 Add performance tests for admin/content Active .

But this is a 5 min change and also depends on what gets merged first. Happy to adapt the code either way.
Comment 2 months ago →
🇺🇸United States smustgrave
📌 Add performance tests for admin/content Active is RTBC so lets postpone this one just temporarily

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024