Problem/Motivation

Since the arrival of #540008: Add a container parameter that can remove the special behavior of UID#1, it is possible to disable the super user access policy added by default to user 1.

We need to add tests to confirm that Dashboard behaves as expected in that scenario.
See https://www.drupal.org/node/2910500 for instructions about how to proceed.

New tests will be added once 10.3 core version is released.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork dashboard-3441321

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

plopesc created an issue. See original summary.

pooja_sharma made their first commit to this issue’s fork.

pooja_sharma opened merge request !16

pooja_sharma’s picture

pooja_sharma commented

Added the permission for createUser() & removed the unrelated code for all test.

Please review, moving NR.

pooja_sharma’s picture

pooja_sharma commented
Status: Active » Needs review
plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Title: [PP] Define specific tests for disabled super user access policy » Define specific tests for disabled super user access policy
Status: Needs review » Needs work

Thank you for your work on this one @pooja_sharma.

Checked the MR and might be that the issue summary is not explaining the needs properly.

We are not expecting to refactor the way tests work with user and roles, that could be discussed in a different issue, though.

What we need for this issue is to create a new test for scenarios where the Super user access policy is turned off and confirm that module is still working as expected. See change record.

pooja_sharma opened merge request !17

pooja_sharma’s picture

pooja_sharma commented
Status: Needs work » Needs review

Tried to add test coverage with super user policy disabled, tried to test the functionality of some piece of code of module , apart from it nothing seems to left , correct me if I 'm missing anything.

Please review, moving NR.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Needs work

At the very least needs reroll.