Move error checking that should apply to all updates from cron build test to update assert

Created on 2 August 2023, over 1 year ago

Problem/Motivation

in \Drupal\Tests\automatic_updates\Build\CoreUpdateTest::testCron we have asserts checking there were no error or warning logged. in πŸ“Œ For web server dependent unattended updates run the entire life cycle in a separate process that will not be affected by hosting time limits Fixed we test cron 3 different ways so I made a helper function that includes this assert. But really this should already be in \Drupal\Tests\automatic_updates\Build\CoreUpdateTest::assertUpdateSuccessful

Steps to reproduce

Proposed resolution

Move the dblog checks to \Drupal\Tests\automatic_updates\Build\CoreUpdateTest::assertUpdateSuccessful for all that should apply to all updates

Major because it blocks πŸ“Œ For web server dependent unattended updates run the entire life cycle in a separate process that will not be affected by hosting time limits Fixed

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Fixed

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States tedbow Ithaca, NY, USA

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

Comments & Activities

Production build 0.71.5 2024