Contrib.social

Split up UpdaterFormTest to speed up test runs: from 13.5 to 10.5 minutes

Open on Drupal.org →
Created on 25 November 2022, about 2 years ago
Updated 20 January 2023, almost 2 years ago

Problem/Motivation

\Drupal\Tests\automatic_updates\Functional\UpdaterFormTest is >1,000 lines of code and has over a dozen test methods. It's the slowest test by far.

Because the way the concurrency works on drupalci, it will run multiple test classes concurrently but will not run multiple test methods within a single class concurrently, having 1 class with many test method methods is slower than the same exact test methods split over multiple test classes.

Proposed resolution

Split up \Drupal\Tests\automatic_updates\Functional\UpdaterFormTest into multiple test classes based on which tests are A) slower, B) semantically belong together.

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Assigned to yash.rode
  • Status changed to Needs work almost 2 years ago
  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Assigned to yash.rode
  • Status changed to Needs work almost 2 years ago
  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Assigned to yash.rode
  • Status changed to Needs work almost 2 years ago
  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Assigned to yash.rode
  • Status changed to Needs work almost 2 years ago
  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Assigned to yash.rode
  • Status changed to Needs work almost 2 years ago
  • Assigned to wim leers
  • Status changed to Needs review almost 2 years ago
  • Issue was unassigned.
  • Status changed to RTBC almost 2 years ago
  • Assigned to tedbow
  • Status changed to Fixed almost 2 years ago
  • Comment almost 2 years ago →
    System Message

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

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024