Issue #3276645: Run readiness checks after stored results have been cleared in certain situations
2 unresolved threads
Merge request reports
Activity
- Resolved by Ted Bowman
added 20 commits
-
f15e6e67...29927706 - 17 commits from branch
project:8.x-2.x
- 348cdfe9 - Test changed
- 94021e46 - Added fix
- af9d129f - run Radinesscheck
Toggle commit list-
f15e6e67...29927706 - 17 commits from branch
added 4 commits
-
9ff0e3c8...5474065a - 2 commits from branch
project:8.x-2.x
- 72cdb464 - Merge branch '8.x-2.x' into 3276645-run-readiness-checks
- 71eb508b - Revert "revert to fix in PostApply"
-
9ff0e3c8...5474065a - 2 commits from branch
- Resolved by Ted Bowman
- Resolved by Ted Bowman
added 8 commits
Toggle commit list- Resolved by Ted Bowman
added 10 commits
-
ddc9ddf5 - 1 commit from branch
project:8.x-2.x
- 13d4ca64 - Test changed
- 1cfae838 - xAdded fix
- 309ad53b - run Radinesscheck
- 099fbc04 - Removed debug statement
- be488849 - use StatusCheck instead of ReadinessCheck
- 88eb3b9b - revert to fix in PostApply
- 0f270bed - Revert "revert to fix in PostApply"
- 8d91befe - Nitpicks
- 2b6683a0 - seperate test coverage
Toggle commit list-
ddc9ddf5 - 1 commit from branch
- Resolved by Ted Bowman
- Resolved by Ted Bowman
added 16 commits
-
30fef251...0bcd417a - 13 commits from branch
project:8.x-2.x
- b907bc77 - add comments to testStatusCheckerRunAfterUpdate
- 055467aa - Merge branch '8.x-2.x' into 3276645-run-readiness-checks
- ab270da6 - Remove unneeded change to test
Toggle commit list-
30fef251...0bcd417a - 13 commits from branch
- Resolved by Ted Bowman
- Resolved by Ted Bowman
added 1 commit
- ea8a270a - modified test to check with and without database updates
added 15 commits
-
81d6ccbd - 1 commit from branch
project:8.x-2.x
- 35b2a80a - Test changed
- fefa5475 - xAdded fix
- d92669d0 - run Radinesscheck
- ec6007d0 - Removed debug statement
- d330be20 - use StatusCheck instead of ReadinessCheck
- bbcd7062 - revert to fix in PostApply
- 39f7dd2b - Revert "revert to fix in PostApply"
- 21e358f7 - Nitpicks
- 842592c7 - seperate test coverage
- 927ba396 - changed test and onfinish
- 3146843e - rebase missed
- d337a015 - add comments to testStatusCheckerRunAfterUpdate
- ee4516db - Remove unneeded change to test
- 4e0505ac - modified test to check with and without database updates
Toggle commit list-
81d6ccbd - 1 commit from branch
- Resolved by Ted Bowman
added 18 commits
-
73fdc4a9...f37eed53 - 3 commits from branch
project:8.x-2.x
- 365e09c9 - Test changed
- 2ea9348c - xAdded fix
- 1ecc6c0b - run Radinesscheck
- cc55347a - Removed debug statement
- 3a84fd0d - use StatusCheck instead of ReadinessCheck
- af6ec80a - revert to fix in PostApply
- e1f76914 - Revert "revert to fix in PostApply"
- 80636b27 - Nitpicks
- 735ff148 - seperate test coverage
- 3ae7ec35 - changed test and onfinish
- 608b4edf - rebase missed
- ed835a78 - add comments to testStatusCheckerRunAfterUpdate
- 37cb44eb - Remove unneeded change to test
- 44809270 - modified test to check with and without database updates
- 69f8ef96 - moved StatusCheck run call before if else.
Toggle commit list-
73fdc4a9...f37eed53 - 3 commits from branch
- Resolved by Ted Bowman
- Resolved by Ted Bowman
- Resolved by Ted Bowman
- Resolved by Ted Bowman
- Resolved by Yash Rode
added 22 commits
-
df85e259...0d934d20 - 3 commits from branch
project:8.x-2.x
- 1ca179fc - Test changed
- 2b858ffc - xAdded fix
- f171c243 - run Radinesscheck
- f27bdd62 - Removed debug statement
- 0b39c8c3 - use StatusCheck instead of ReadinessCheck
- ca68ea75 - revert to fix in PostApply
- 528bae69 - Revert "revert to fix in PostApply"
- 4170fdab - Nitpicks
- a0838543 - seperate test coverage
- 5dab8d4d - changed test and onfinish
- 5902040f - rebase missed
- 7379eee6 - add comments to testStatusCheckerRunAfterUpdate
- ec163a0a - Remove unneeded change to test
- 15f10910 - modified test to check with and without database updates
- 0a84dd6b - moved StatusCheck run call before if else.
- 3603a7b4 - Added testStatusCheckerRunAfterUpdate
- 54934d62 - completed test in case of databse updates.
- 5ba4c0ae - removed debug
- c6e1c34e - nits except last one
Toggle commit list-
df85e259...0d934d20 - 3 commits from branch
749 $this->checkForUpdates(); 750 $page = $this->getSession()->getPage(); 751 // Navigate to the automatic updates form. 752 $this->drupalGet('/admin/modules/update'); 753 Stager::setFixturePath(__DIR__ . '/../../fixtures/drupal-9.8.1-installed'); 754 $page->pressButton('Update to 9.8.1'); 755 $this->checkForMetaRefresh(); 756 $this->assertUpdateStagedTimes(1); 757 $this->assertUpdateReady('9.8.1'); 758 // Set an error before completing the update. This error should be visible 759 // on admin pages after completing the update without having to explicitly 760 // run the status checks. 761 TestSubscriber1::setTestResult([ValidationResult::createError(['Error before continue.'])], StatusCheckEvent::class); 762 if ($has_database_updates) { 763 // Simulate a staged database update in the automatic_updates_test module. 764 // We must do this after the update has started, because the pending updates changed this line in version 23 of the diff
- Resolved by Ted Bowman
273 $assert_session->pageTextContains('Dynamic automatic_updates_update_1191934'); 274 $page->clickLink('Apply pending updates'); 275 $this->checkForMetaRefresh(); 276 $assert_session->pageTextContains('Updates were attempted.'); 277 } 278 else { 279 $page->pressButton('Continue'); 280 $this->checkForMetaRefresh(); 281 $assert_session->addressEquals('/admin/reports/updates'); 282 $assert_session->pageTextContainsOnce('Update complete!'); 283 } 284 // Status checks should display errors on admin page. 285 $this->drupalGet('/admin'); 286 // Confirm that the status checks were run and the new error is displayed. 287 $assert_session->pageTextContains('Error before continue.'); 288 $assert_session->pageTextContains(static::$errorsExplanation); changed this line in version 23 of the diff
added 2 commits
added 33 commits
-
1ba580ef...f066e2d7 - 11 commits from branch
project:8.x-2.x
- b3efdf39 - Test changed
- a653cd99 - xAdded fix
- 94155fa0 - run Radinesscheck
- 792ba903 - Removed debug statement
- 2ac433ea - use StatusCheck instead of ReadinessCheck
- 8a1da944 - revert to fix in PostApply
- 24c76e70 - Revert "revert to fix in PostApply"
- a94d8921 - Nitpicks
- d33ca42b - seperate test coverage
- 95d8c63f - changed test and onfinish
- e7aec912 - rebase missed
- 65549dda - add comments to testStatusCheckerRunAfterUpdate
- e12e60fd - Remove unneeded change to test
- 41816c4a - modified test to check with and without database updates
- 5d777d58 - moved StatusCheck run call before if else.
- 303b933d - Added testStatusCheckerRunAfterUpdate
- 6b87c4cb - completed test in case of databse updates.
- bf407442 - removed debug
- 249ac13c - nits except last one
- b6880f7a - typo
- 76f73f86 - comment line length
- e67413d8 - comment lenght and use statusMessageContains
Toggle commit list-
1ba580ef...f066e2d7 - 11 commits from branch
Please register or sign in to reply