Problem/Motivation

This test isn't a great candidate for a kernel test since it's an admin page and has lots of form submissions. It's usually the slowest functional test so let's try to speed it up.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3590722

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.

catch’s picture

catch
he/him
Primary language English
commented

Local before:

Time: :50.562, Memory: 14.00 MB

OK, but some tests were skipped!
Tests: 9, Assertions: 106, Skipped: 1.

Local after:

Time: 00:43.314, Memory: 14.00 MB

OK, but some tests were skipped!
Tests: 6, Assertions: 91, Skipped: 1.

catch opened merge request !15798

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review

On gitlab:

 109.340s Drupal\Tests\system\Functional\Module\DependencyTest                      5 passed, 1 skipped

Regularly see this taking about 180-200s although not possible to tell how much this is due to different runners.

https://git.drupalcode.org/project/drupal/-/jobs/9865558

catch’s picture

catch
he/him
Primary language English
commented
Issue tags: +Test suite performance
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Numbers in #2 look like a net improvement. And know if you've done similar things before combing the tests so I'll mark for you @catch.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 454adefc5b8 to main and dba2e4d4880 to 11.x. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • alexpott committed dba2e4d4 on 11.x 
    perf: #3590722 Speed up DependencyTest

By: catch
(cherry picked from...

  • alexpott committed 454adefc on main 
    perf: #3590722 Speed up DependencyTest

By: catch