fix: add type for contextType and fix RolePermissions test typing

Author: dcoa

src/authz-module/roles-permissions/RolesPermissions.test.tsx

@@ -25,6 +25,7 @@ jest.mock('./course/constants', () => ({
   ],
   coursePermissions: [],
   courseResourceTypes: [],
+  courseRolesMetadata: [],
 }));
 
 jest.mock('./library/constants', () => ({
@@ -35,6 +36,7 @@ jest.mock('./library/constants', () => ({
   ],
   libraryPermissions: [],
   libraryResourceTypes: [],
+  libraryRolesMetadata: [],
 }));
 
 jest.mock('@openedx/paragon', () => ({

src/authz-module/roles-permissions/library/utils.test.ts

@@ -1,4 +1,5 @@
 import { createIntl } from '@edx/frontend-platform/i18n';
+import type { Role } from '@src/types';
 import { buildPermissionMatrixByResource } from './utils';
 
 const intl = createIntl({ locale: 'en', messages: {} });
@@ -14,15 +15,15 @@ const permissions = [
 const resources = [
   { key: 'library', label: 'Library', description: '' },
 ];
-const roles = [
+const roles: Role[] = [
   {
-    name: 'admin', permissions: ['create_library', 'edit_library'], userCount: 2, role: 'admin', description: '', contextType: '', scope: '',
+    name: 'admin', permissions: ['create_library', 'edit_library'], userCount: 2, role: 'admin', description: '', contextType: 'library', scope: '',
   },
   {
-    name: 'editor', permissions: ['edit_library'], userCount: 2, role: 'editor', description: '', contextType: '', scope: '',
+    name: 'editor', permissions: ['edit_library'], userCount: 2, role: 'editor', description: '', contextType: 'library', scope: '',
   },
   {
-    name: 'guest', permissions: [], userCount: 2, role: 'guest', description: '', contextType: '', scope: '',
+    name: 'guest', permissions: [], userCount: 2, role: 'guest', description: '', contextType: 'library', scope: '',
   },
 ];
 

src/types.ts

@@ -27,11 +27,13 @@ export interface LibraryMetadata {
   allowPublicRead: boolean;
 }
 
+export type ContextType = 'course' | 'library';
+
 export interface RoleMetadata {
   role: string;
   name: string;
   description: string;
-  contextType: string;
+  contextType: ContextType;
   disabled?: boolean;
 }
 // TODO: remove unnecessary fields when libraries gets removed