From 30c3b755289a8a987892d002759d599929886362 Mon Sep 17 00:00:00 2001 From: Dave Long Date: Tue, 14 Feb 2023 09:48:23 +0000 Subject: [PATCH] Issue #3231184 by mglaman, smustgrave, longwave: ResourceTestBase::setUpAuthorization references wrong permission grant methods --- .../tests/src/Functional/ResourceTestBase.php | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/core/modules/jsonapi/tests/src/Functional/ResourceTestBase.php b/core/modules/jsonapi/tests/src/Functional/ResourceTestBase.php index 7a2b19a45f7..b30537eb07c 100644 --- a/core/modules/jsonapi/tests/src/Functional/ResourceTestBase.php +++ b/core/modules/jsonapi/tests/src/Functional/ResourceTestBase.php @@ -607,21 +607,14 @@ abstract class ResourceTestBase extends BrowserTestBase { /** * Sets up the necessary authorization. * - * In case of a test verifying publicly accessible REST resources: grant - * permissions to the anonymous user role. - * - * In case of a test verifying behavior when using a particular authentication - * provider: create a user with a particular set of permissions. - * * Because of the $method parameter, it's possible to first set up - * authentication for only GET, then add POST, et cetera. This then also + * authorization for only GET, then add POST, et cetera. This then also * allows for verifying a 403 in case of missing authorization. * * @param string $method - * The HTTP method for which to set up authentication. + * The HTTP method for which to set up authorization. * - * @see ::grantPermissionsToAnonymousRole() - * @see ::grantPermissionsToAuthenticatedRole() + * @see ::grantPermissionsToTestedRole() */ abstract protected function setUpAuthorization($method);