The testAutomaticRedirects() fails in RedirectUITest in drupal 8.8 due to the updates of the path and path alias modules in core.

CommentFileSizeAuthor
#4 3100538-interdiff-2-4.txt1.61 KBarpad.rozsa
#4 3100538-redirectuitest_fails-4.patch1.58 KBarpad.rozsa
#2 3100538-redirectuitest_fails-2.patch1.25 KBarpad.rozsa

Comments

arpad.rozsa created an issue. See original summary.

arpad.rozsa’s picture

arpad.rozsa commented
Status: Active » Needs review
StatusFileSize
new 3100538-redirectuitest_fails-2.patch1.25 KB
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work

This will in turn fail on 8.7, which we still support.

As the change is pretty simple, lets add a check for if (version_compare(\Drupal::VERSION, '8.8', '>=')) { or so, so it works on both versions.

You could define a $field_suffix = '[0][value]'; variable that you conditionally set to empty or that and append it to those 3 fields.

arpad.rozsa’s picture

arpad.rozsa commented
Status: Needs work » Needs review
StatusFileSize
new 3100538-redirectuitest_fails-4.patch1.58 KB
new 3100538-interdiff-2-4.txt1.61 KB

Added the version check.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks.

  • Berdir committed 88dd7cd on 8.x-1.x authored by arpad.rozsa 
    Issue #3100538 by arpad.rozsa: RedirectUITest fails in D8.8

Status: Fixed » Closed (fixed)

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