Problem/Motivation

The MonitoringUITest has a failing function testCoreRequirementsUI().

Proposed resolution

Fix the test.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 fix_test_fails_in-2854786-2.patch670 bytesGinovski
testfails.png15.61 KBGinovski
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ginovski created an issue. See original summary.

Ginovski’s picture

Assigned: Unassigned » Ginovski
Status: Active » Needs review
FileSize
670 bytes

After #665790: Redesign the status report page got committed, the run cron messages were changed.

+++ b/core/modules/system/system.install
@@ -535,14 +535,20 @@ function system_requirements($phase) {
-        '#prefix' => '<br />',
-        '#markup' => t('To run cron from outside the site, go to <a href=":url">@cron</a>', [':url' => $cron_url, '@cron' => $cron_url]),
+        '#prefix' => '<span class="cron-description__run-cron">',
+        '#suffix' => '</span>',
+        '#type' => 'link',
+        '#title' => t('Run cron'),
+        '#url' => Url::fromRoute('system.run_cron'),

and

-        '#markup' => t('You can <a href=":cron">run cron manually</a>.', [':cron' => \Drupal::url('system.run_cron')]),
+        '#type' => 'link',
+        '#prefix' => '(',
+        '#title' => t('more information'),
+        '#suffix' => ')',
+        '#url' => Url::fromRoute('system.cron_settings'),

Changing assertions accordingly.

  • Berdir committed 1d67d92 on 8.x-1.x authored by Ginovski
    Issue #2854786 by Ginovski: Fix test fails in MonitoringUITest
    
Berdir’s picture

Status: Needs review » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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