Problem/Motivation

One of the very slowest functional tests.

Steps to reproduce

Proposed resolution

Consolidate two methods that do assertions that don't have anything to do with actually running the updates into one method - this loads the database one less time.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3413730

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 opened merge request !6102

catch’s picture

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

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

That works. Probably more we can do but the whole job is finishing faster with that small change.

catch’s picture

catch
he/him
Primary language English
commented
Issue summary: View changes
smustgrave’s picture

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

Single file don't see any reason this would cause a problem.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 3b059f784e to 11.x and c0bfe53b07 to 10.2.x. Thanks!

  • longwave committed c0bfe53b on 10.2.x 
    Issue #3413730 by catch: Speed up UpdatePathTestBaseTest

(cherry picked...

  • longwave committed 3b059f78 on 11.x 
    Issue #3413730 by catch: Speed up UpdatePathTestBaseTest

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.