From 65b7b99805c3023d2a5122aa536e8eca37650989 Mon Sep 17 00:00:00 2001 From: Bimsara Bodaragama Date: Fri, 5 Jul 2024 11:16:54 +0530 Subject: [PATCH] Test: Add testPatchRoleWithGroupDisplayNameInsteadOfGroupIdThrowingErrors for SCIM error handling in V2 Roles operations --- .../common/impl/SCIMRoleManagerV2Test.java | 117 ++++++++++++++++++ .../src/test/resources/testng.xml | 1 + 2 files changed, 118 insertions(+) create mode 100644 components/org.wso2.carbon.identity.scim2.common/src/test/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2Test.java diff --git a/components/org.wso2.carbon.identity.scim2.common/src/test/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2Test.java b/components/org.wso2.carbon.identity.scim2.common/src/test/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2Test.java new file mode 100644 index 000000000..e1381443b --- /dev/null +++ b/components/org.wso2.carbon.identity.scim2.common/src/test/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2Test.java @@ -0,0 +1,117 @@ +/* + * Copyright (c) 2024, WSO2 LLC. (http://www.wso2.com). + * + * WSO2 LLC. licenses this file to you under the Apache License, + * Version 2.0 (the "License"); you may not use this file except + * in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ + +package org.wso2.carbon.identity.scim2.common.impl; + +import org.mockito.Mock; +import org.powermock.api.mockito.PowerMockito; +import org.powermock.core.classloader.annotations.PrepareForTest; +import org.powermock.modules.testng.PowerMockTestCase; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.BeforeMethod; +import org.testng.annotations.DataProvider; +import org.testng.annotations.Test; +import org.wso2.carbon.identity.core.util.IdentityUtil; +import org.wso2.carbon.identity.role.v2.mgt.core.RoleManagementService; +import org.wso2.carbon.identity.role.v2.mgt.core.exception.IdentityRoleManagementException; +import org.wso2.charon3.core.exceptions.BadRequestException; +import org.wso2.charon3.core.exceptions.CharonException; +import org.wso2.charon3.core.exceptions.ConflictException; +import org.wso2.charon3.core.exceptions.ForbiddenException; +import org.wso2.charon3.core.exceptions.NotFoundException; +import org.wso2.charon3.core.protocol.ResponseCodeConstants; +import org.wso2.charon3.core.schema.SCIMConstants; +import org.wso2.charon3.core.utils.codeutils.PatchOperation; + +import java.util.Collections; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +import static org.mockito.Mockito.when; +import static org.mockito.MockitoAnnotations.initMocks; +import static org.testng.AssertJUnit.assertEquals; + +/** + * Contains the unit test cases for SCIMRoleManagerV2. + */ +@PrepareForTest({SCIMRoleManagerV2.class, IdentityUtil.class}) +public class SCIMRoleManagerV2Test extends PowerMockTestCase { + + private static final String SAMPLE_TENANT_DOMAIN = "carbon.super"; + private static final String SAMPLE_VALID_ROLE_ID = "595f5508-f286-446a-86c4-5071e07b98fc"; + private static final String SAMPLE_GROUP_NAME = "testGroup"; + private static final String SAMPLE_VALID_ROLE_NAME = "admin"; + private static final int BAD_REQUEST = 400; + + @Mock + private RoleManagementService roleManagementService; + @Mock + private SCIMRoleManagerV2 scimRoleManagerV2; + + @BeforeClass + public void setUpClass() { + + initMocks(this); + } + + @BeforeMethod + public void setUpMethod() { + + scimRoleManagerV2 = PowerMockito.spy(new SCIMRoleManagerV2(roleManagementService, SAMPLE_TENANT_DOMAIN)); + PowerMockito.mockStatic(IdentityUtil.class); + } + + @DataProvider(name = "scimOperations") + public Object[][] provideScimOperations() { + + return new Object[][]{ + {SCIMConstants.OperationalConstants.ADD}, + {SCIMConstants.OperationalConstants.REMOVE}, + {SCIMConstants.OperationalConstants.REPLACE} + }; + } + + @Test(dataProvider = "scimOperations") + public void testPatchRoleWithGroupDisplayNameInsteadOfGroupIdThrowingErrors(String operation) + throws IdentityRoleManagementException, ForbiddenException, ConflictException, NotFoundException, + CharonException { + + Map> patchOperations = new HashMap<>(); + Map valueMap = new HashMap<>(); + valueMap.put(SCIMConstants.CommonSchemaConstants.DISPLAY, SAMPLE_GROUP_NAME); + valueMap.put(SCIMConstants.CommonSchemaConstants.VALUE, null); + + PatchOperation patchOperation = new PatchOperation(); + patchOperation.setOperation(operation); + patchOperation.setAttributeName(SCIMConstants.RoleSchemaConstants.GROUPS); + patchOperation.setValues(valueMap); + patchOperations.put(SCIMConstants.RoleSchemaConstants.GROUPS, Collections.singletonList(patchOperation)); + + when(roleManagementService.getRoleNameByRoleId(SAMPLE_VALID_ROLE_ID, SAMPLE_TENANT_DOMAIN)) + .thenReturn(SAMPLE_VALID_ROLE_NAME); + + try { + scimRoleManagerV2.patchRole(SAMPLE_VALID_ROLE_ID, patchOperations); + } catch (BadRequestException e) { + assertEquals(BAD_REQUEST, e.getStatus()); + assertEquals(ResponseCodeConstants.INVALID_VALUE, e.getScimType()); + assertEquals("Group id is required to update group of the role.", e.getDetail()); + } + } +} diff --git a/components/org.wso2.carbon.identity.scim2.common/src/test/resources/testng.xml b/components/org.wso2.carbon.identity.scim2.common/src/test/resources/testng.xml index 83c5f989d..e9b0cdcfd 100644 --- a/components/org.wso2.carbon.identity.scim2.common/src/test/resources/testng.xml +++ b/components/org.wso2.carbon.identity.scim2.common/src/test/resources/testng.xml @@ -34,6 +34,7 @@ +