-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rewrite kernel tests to functional tests for failing testGetPermissions() #427
Comments
@ekes and @millnut I've just been testing the group content permissions in the browser and I'm getting some funny results. Fresh install
Now then...
Expected behaviour: the admin1 user on site1 should not be able to create blog posts. Observed behaviour: the admin1 user on site1 CAN create blog posts. |
@hc-konrad follow this issue for udates! |
@xtianjohnson following for updates. |
Manual testing core D10 + group + group_permissions. Granting group_permissions to a role any group seems to effect all the groups (at least that the user is a member of). So far so... there's certainly a thing going on between group + group_permissions at the moment that's not specific to localgov microsites. |
Thanks for digging into this more @ekes it looks like we have ended up on the conclusion with it being group_permissions, do you think it's worth adding that as a note to https://www.drupal.org/project/group_permissions/issues/3388880 |
https://www.drupal.org/project/group_permissions/issues/3395306 The test certainly confirms the cache clear issue, but also if I'm correct shows removal of permissions failing again. |
I'll be honest here. My temptation is now to remove group_permissions and use custom entity access to set if content can be created - it will then always be possible to be viewed if it has been created. |
See https://github.com/localgovdrupal/localgov_microsites_group/blob/2.x/tests/src/Kernel/GroupPermissionsHelperTest.php#L141
#396 (comment)
This is the test that is failing:
https://github.com/localgovdrupal/localgov_microsites_group/blob/2.x/tests/src/Kernel/GroupPermissionsHelperTest.php#L141
Can anyone re-write this test to test in a different way? @ekes was wondering if maybe a functional browser test to do the same thing might be appropriate?
@millnut might be able to look at this 😄
The text was updated successfully, but these errors were encountered: