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

Issue fork drupal-3476440

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

catch created an issue. See original summary.

bramr’s picture

bramr commented

Did your workshop at Drupalcon #Barcelona2024, i should have some time in the upcoming week to pick this up.

bramr opened merge request !9827

bramr’s picture

bramr commented
Status: Active » Needs review

Think it's ready for a review.

I agree with the solution to move clearCaches into the trait as proposed by issue: #3476433.

But this is a 5 min change and also depends on what gets merged first. Happy to adapt the code either way.

smustgrave’s picture

smustgrave commented
Status: Needs review » Postponed

#3476433: Add performance tests for admin/content is RTBC so lets postpone this one just temporarily

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.