fix: addressing pr comments

Author: jacobo-dominguez-wgu

src/authz-module/audit-user/index.test.tsx

@@ -5,7 +5,7 @@ import { MemoryRouter, Route, Routes } from 'react-router-dom';
 import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
 import { IntlProvider } from '@edx/frontend-platform/i18n';
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
-import { ToastManagerProvider } from '@src/authz-module/data/context/ToastManagerContext';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import AuditUserPage from './index';
 
 jest.mock('@edx/frontend-platform/auth', () => ({

src/authz-module/audit-user/index.tsx

@@ -23,14 +23,14 @@ import {
 import { useQuerySettings } from '@src/authz-module/hooks/useQuerySettings';
 import { useRevokeUserRoles, useUserAssignedRoles } from '@src/authz-module/data/hooks';
 import { Role } from 'types';
-import { useToastManager } from '@src/authz-module/data/context/ToastManagerContext';
+import { useToastManager } from '@src/components/ToastManager/ToastManagerContext';
 import messages from './messages';
 import ConfirmDeletionModal from '../components/ConfirmDeletionModal';
 
 const AuditUserPage = () => {
   const { formatMessage } = useIntl();
   const { username } = useParams();
-  const { authenticatedUser } = useContext(AppContext) as AppContextType;
+  const { authenticatedUser } = useContext(AppContext as React.Context<AppContextType>);
   const navigate = useNavigate();
   const {
     isLoading: isLoadingUser, data: user, isError: isErrorUser, error: errorUser,
@@ -148,7 +148,6 @@ const AuditUserPage = () => {
                 { Bold, Br },
               ),
             });
-            // authzQueryKeys.userRoles(username, querySettings),
             return;
           }
 

src/authz-module/authz-home/index.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 import { screen } from '@testing-library/react';
 import { useAllRoleAssignments, useOrgs, useScopes } from '@src/authz-module/data/hooks';
-import { ToastManagerProvider } from '@src/authz-module/data/context/ToastManagerContext';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import { renderWithAllProviders } from '@src/setupTest';
 import userEvent from '@testing-library/user-event';
 import AuthzHome from './index';

src/authz-module/components/messages.ts

@@ -134,7 +134,7 @@ const messages = defineMessages({
   },
   'authz.manage.removing.button': {
     id: 'authz.manage.removing.button',
-    defaultMessage: 'Removing...',
+    defaultMessage: 'Removing',
     description: 'AuthZ removing button title',
   },
   'authz.team.remove.user.modal.body.1': {

src/authz-module/data/hooks.ts

@@ -3,6 +3,7 @@ import {
 } from '@tanstack/react-query';
 import { appId } from '@src/constants';
 import { LibraryMetadata } from '@src/types';
+import { useQuerySettings } from '@src/authz-module/hooks/useQuerySettings';
 import {
   assignTeamMembersRole, AssignTeamMembersRoleRequest, getAllRoleAssignments,
   GetAllRoleAssignmentsResponse, getLibrary, getOrgs, GetOrgsResponse,
@@ -107,6 +108,7 @@ export const useAssignTeamMembersRole = () => {
  */
 export const useRevokeUserRoles = () => {
   const queryClient = useQueryClient();
+  const { querySettings: defaultQuerySettings } = useQuerySettings();
   return useMutation({
     mutationFn: async ({ data }: {
       data: RevokeUserRolesRequest
@@ -117,6 +119,7 @@ export const useRevokeUserRoles = () => {
       queryClient.invalidateQueries({
         queryKey: authzQueryKeys.userRoles(users, querySettings),
       });
+      queryClient.invalidateQueries({ queryKey: authzQueryKeys.allRoleAssignments(defaultQuerySettings) });
     },
   });
 };

src/authz-module/index.tsx

@@ -5,7 +5,7 @@ import { QueryErrorResetBoundary } from '@tanstack/react-query';
 import LoadingPage from '@src/components/LoadingPage';
 import LibrariesErrorFallback from '@src/authz-module/libraries-manager/ErrorPage';
 import { CustomErrors } from '@src/constants';
-import { ToastManagerProvider } from './data/context/ToastManagerContext';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import { LibrariesUserManager, LibrariesLayout, LibrariesTeamManager } from './libraries-manager';
 import AuthzHome from './authz-home';
 import AuditUserPage from './audit-user';

src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx

@@ -3,9 +3,9 @@ import userEvent from '@testing-library/user-event';
 import { renderWrapper } from '@src/setupTest';
 import { initializeMockApp } from '@edx/frontend-platform/testing';
 import { useLibrary } from '@src/authz-module/data/hooks';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import { useLibraryAuthZ } from './context';
 import LibrariesTeamManager from './LibrariesTeamManager';
-import { ToastManagerProvider } from '../data/context/ToastManagerContext';
 import { CONTENT_LIBRARY_PERMISSIONS } from '../constants';
 
 jest.mock('./context', () => {

src/authz-module/libraries-manager/LibrariesUserManager.test.tsx

@@ -2,10 +2,10 @@ import { useParams } from 'react-router-dom';
 import { screen, waitFor, within } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { renderWrapper } from '@src/setupTest';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import LibrariesUserManager from './LibrariesUserManager';
 import { useLibraryAuthZ } from './context';
 import { useLibrary, useTeamMembers, useRevokeUserRoles } from '../data/hooks';
-import { ToastManagerProvider } from '../data/context/ToastManagerContext';
 
 jest.mock('@edx/frontend-platform/logging', () => ({
   logError: jest.fn(),

src/authz-module/libraries-manager/LibrariesUserManager.tsx

@@ -4,7 +4,7 @@ import { useIntl } from '@edx/frontend-platform/i18n';
 import { Container, Skeleton } from '@openedx/paragon';
 import { ROUTES } from '@src/authz-module/constants';
 import { Role } from 'types';
-import { useToastManager } from '@src/authz-module/data/context/ToastManagerContext';
+import { useToastManager } from '@src/components/ToastManager/ToastManagerContext';
 import AuthZLayout from '../components/AuthZLayout';
 import { useLibraryAuthZ } from './context';
 import RoleCard from '../components/RoleCard';

src/authz-module/libraries-manager/components/AddNewTeamMemberModal/AddNewTeamMemberTrigger.test.tsx

@@ -3,7 +3,7 @@ import { screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { renderWrapper } from '@src/setupTest';
 import { useAssignTeamMembersRole } from '@src/authz-module/data/hooks';
-import { ToastManagerProvider } from '@src/authz-module/data/context/ToastManagerContext';
+import { ToastManagerProvider } from '@src/components/ToastManager/ToastManagerContext';
 import AddNewTeamMemberTrigger from './AddNewTeamMemberTrigger';
 
 jest.mock('@edx/frontend-platform/logging');