Problem/Motivation

\Drupal\Tests\UpdatePathTestTrait::runUpdates() doesn't offer the test coverage we think it does.

Steps to reproduce

See test-only patch.

Proposed resolution

Ensure drupal_load_updates(); is called prior to update_get_update_list();

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

N/a

CommentFileSizeAuthor
#2 3248879-2.patch1.5 KBalexpott
#2 3248879-2-test-only.patch981 bytesalexpott

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: 9.4.x-dev » 9.3.x-dev
StatusFileSize
new 3248879-2-test-only.patch981 bytes
new 3248879-2.patch1.5 KB
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review

The last submitted patch, 2: 3248879-2-test-only.patch, failed testing. View results

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

joachim’s picture

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

  • catch committed d77a860 on 10.0.x 
    Issue #3248879 by alexpott: UpdatePathTestTrait doesn't test what it...

  • catch committed c25e773 on 9.4.x 
    Issue #3248879 by alexpott: UpdatePathTestTrait doesn't test what it...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x and cherry-picked to 9.4.x, thanks!

Good job it's still passing... phew.

Status: Fixed » Closed (fixed)

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