Problem/Motivation

In #3232761: Create local task links to the updater form we found that we could remove the call to checkForUpdates() in \Drupal\Tests\automatic_updates\Functional\UpdaterFormTest::testTableLooksCorrect. We no longer need this as our form refreshes the update.

Proposed resolution

Remove this call and others if necessary. To ensure our tests will fail if our form doesn't refreshes the update.

Issue fork automatic_updates-3233521

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kunal.sachdev created an issue. See original summary.

kunal.sachdev opened merge request !46

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
CreditAttribution: tedbow at Acquia commented
Status: Active » Reviewed & tested by the community

Looks good to me! thanks @kunal.sachdev

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
CreditAttribution: tedbow at Acquia commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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