ARTESCA-13607 // filter out iam storage manager role

Refactor SelectAccountIAMRole component and utils/hooks.ts
scality · Oct 21, 2024 · 9a2ff16 · 9a2ff16
1 parent 23fe25f
commit 9a2ff16
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/src/react/ui-elements/SelectAccountIAMRole.tsx b/src/react/ui-elements/SelectAccountIAMRole.tsx
@@ -1,10 +1,4 @@
-import {
-  Form,
-  FormGroup,
-  FormSection,
-  Stack,
-  useToast,
-} from '@scality/core-ui';
+import { Form, FormGroup, FormSection, useToast } from '@scality/core-ui';
 import { Select } from '@scality/core-ui/dist/next';
 import { IAM } from 'aws-sdk';
 import { Bucket } from 'aws-sdk/clients/s3';
@@ -26,7 +20,7 @@ import {
 } from '../next-architecture/ui/AccessibleAccountsAdapterProvider';
 import { AccountsLocationsEndpointsAdapterProvider } from '../next-architecture/ui/AccountsLocationsEndpointsAdapterProvider';
 import { getListRolesQuery } from '../queries';
-import { SCALITY_INTERNAL_ROLES, regexArn } from '../utils/hooks';
+import { SCALITY_IAM_ROLES, regexArn } from '../utils/hooks';
 
 class NoOpMetricsAdapter implements IMetricsAdapter {
   async listBucketsLatestUsedCapacity(
@@ -234,7 +228,7 @@ const SelectAccountIAMRoleWithAccount = (
   const roles = props.filterOutInternalRoles
     ? allRolesExceptHiddenOnes.filter((role) => {
         return (
-          SCALITY_INTERNAL_ROLES.includes(role.RoleName) ||
+          SCALITY_IAM_ROLES.includes(role.RoleName) ||
           !role.Arn.includes('role/scality-internal')
         );
       })

diff --git a/src/react/utils/hooks.ts b/src/react/utils/hooks.ts
@@ -161,6 +161,11 @@ export const SCALITY_INTERNAL_ROLES = [
   DATA_CONSUMER_ROLE,
   DATA_ACCESSOR_ROLE,
 ];
+export const SCALITY_IAM_ROLES = [
+  STORAGE_ACCOUNT_OWNER_ROLE,
+  DATA_CONSUMER_ROLE,
+  DATA_ACCESSOR_ROLE,
+];
 
 const reduxBasedEventDispatcher = () => {
   const dispatch = useDispatch();