Problem/Motivation

Follow-up of #3435282: Automated Drupal 11 compatibility fixes for user_not_role.

Drupal core made a backwards-incompatible test change that causes tests to fail in Drupal 11, but not Drupal 10. Browser tests are annoying to debug so rather than waste time this issue soley focuses on Drupal 11+ test refactor.

Steps to reproduce

Run pipeline.

Proposed resolution

Refactor the browser tests possibly splitting into multiple tests to focus on specific users rather than having one test with one provider.

Remaining tasks

-

User interface changes

No

API changes

No

Data model changes

No

Issue fork user_not_role-3455705

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

Comments

mradcliffe created an issue. See original summary.

mradcliffe opened merge request !6

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Active » Postponed

It looks like this isn't possible with GitLab CI until Drupal 11 is the current major... So we'll have failing tests for a while when Drupal 11 is released and this can be resolved.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Postponed » Needs work

Re-opening.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Needs work » Postponed

Re-postponing on gitlab ci.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Postponed » Needs work

Unpostponing, tests are running on D11 by default now on this branch.

mradcliffe changed the visibility of the branch 3455705-refactor-tests-for to hidden.

mradcliffe changed the visibility of the branch 3455705-refactor-tests-for to active.

mradcliffe opened merge request !7

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Needs work » Needs review

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Needs review » Fixed

Merging for both 2.0.x and 1.0.x.

Status: Fixed » Closed (fixed)

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