feat: simplify AssignRoleWizardPage tests by removing scope handling and updating navigation path

Author: bra-i-am

src/authz-module/wizard/AssignRoleWizard.test.tsx

@@ -100,7 +100,7 @@ const permissionsFor = ({ library = false, course = false } = {}) => [
 ];
 
 const renderWizard = (props = {}) => renderWrapper(
-  <AssignRoleWizard onClose={jest.fn()} scope="lib:123" {...props} />,
+  <AssignRoleWizard onClose={jest.fn()} {...props} />,
 );
 
 describe('AssignRoleWizard — Step 1', () => {
@@ -128,27 +128,6 @@ describe('AssignRoleWizard — Step 1', () => {
     expect(screen.getByTestId('users-input')).toHaveValue('alice');
   });
 
-  it('shows only library roles when user only has manage_library_team permission', () => {
-    mockUseValidateUserPermissions.mockReturnValue({ data: permissionsFor({ library: true }) });
-    renderWizard();
-    expect(screen.getByTestId('role-option-library_admin')).toBeInTheDocument();
-    expect(screen.queryByTestId('role-option-course_admin')).not.toBeInTheDocument();
-  });
-
-  it('shows only course roles when user only has manage_course_team permission', () => {
-    mockUseValidateUserPermissions.mockReturnValue({ data: permissionsFor({ course: true }) });
-    renderWizard();
-    expect(screen.getByTestId('role-option-course_admin')).toBeInTheDocument();
-    expect(screen.queryByTestId('role-option-library_admin')).not.toBeInTheDocument();
-  });
-
-  it('shows both course and library roles when user has both permissions', () => {
-    mockUseValidateUserPermissions.mockReturnValue({ data: permissionsFor({ library: true, course: true }) });
-    renderWizard();
-    expect(screen.getByTestId('role-option-library_admin')).toBeInTheDocument();
-    expect(screen.getByTestId('role-option-course_admin')).toBeInTheDocument();
-  });
-
   it('selecting a different role replaces the previous selection', async () => {
     const user = userEvent.setup();
     mockUseValidateUserPermissions.mockReturnValue({ data: permissionsFor({ library: true, course: true }) });

src/authz-module/wizard/AssignRoleWizardPage.test.tsx

@@ -16,8 +16,8 @@ jest.mock('../data/hooks', () => ({
 
 jest.mock('./AssignRoleWizard', () => ({
   __esModule: true,
-  default: ({ scope, initialUsers, onClose }: { scope: string; initialUsers?: string; onClose: () => void }) => (
-    <div data-testid="assign-role-wizard" data-scope={scope} data-users={initialUsers}>
+  default: ({ initialUsers, onClose }: { initialUsers?: string; onClose: () => void }) => (
+    <div data-testid="assign-role-wizard" data-users={initialUsers}>
       <button type="button" data-testid="wizard-close" onClick={onClose}>Close</button>
       Assign Role Wizard
     </div>
@@ -54,18 +54,12 @@ describe('AssignRoleWizardPage', () => {
     jest.clearAllMocks();
   });
 
-  it('renders the wizard when scope and library are present', () => {
+  it('renders the wizard when library is present', () => {
     setupMocks();
     renderWrapper(<AssignRoleWizardPage />);
     expect(screen.getByTestId('assign-role-wizard')).toBeInTheDocument();
   });
 
-  it('passes scope to the wizard', () => {
-    setupMocks({ scope: 'lib:456' });
-    renderWrapper(<AssignRoleWizardPage />);
-    expect(screen.getByTestId('assign-role-wizard')).toHaveAttribute('data-scope', 'lib:456');
-  });
-
   it('passes initialUsers from search params to the wizard', () => {
     setupMocks({ users: 'alice,bob' });
     renderWrapper(<AssignRoleWizardPage />);
@@ -78,7 +72,7 @@ describe('AssignRoleWizardPage', () => {
     expect(screen.getByRole('heading', { name: 'Assign Role' })).toBeInTheDocument();
   });
 
-  it('navigates to team members path when wizard onClose is triggered', async () => {
+  it('navigates to home path when wizard onClose is triggered', async () => {
     setupMocks();
     const mockNavigate = jest.fn();
     const { useNavigate } = jest.requireMock('react-router-dom');
@@ -88,6 +82,6 @@ describe('AssignRoleWizardPage', () => {
     renderWrapper(<AssignRoleWizardPage />);
     await user.click(screen.getByTestId('wizard-close'));
 
-    expect(mockNavigate).toHaveBeenCalledWith('/authz/libraries/lib:123');
+    expect(mockNavigate).toHaveBeenCalledWith('/authz');
   });
 });