From f6bd291662deeb9d9c2893b4ffb621afa99b0bde Mon Sep 17 00:00:00 2001 From: Diana Olarte Date: Wed, 22 Oct 2025 19:45:20 +1100 Subject: [PATCH] fix: style issues and test --- src/authz-module/index.scss | 2 +- .../libraries-manager/LibrariesTeamManager.test.tsx | 7 +++++-- .../libraries-manager/LibrariesTeamManager.tsx | 2 +- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src/authz-module/index.scss b/src/authz-module/index.scss index ab98fc3..790adc8 100644 --- a/src/authz-module/index.scss +++ b/src/authz-module/index.scss @@ -29,7 +29,7 @@ font-weight: var(--pgn-typography-font-weight-base); } - svg { + .pgn__icon { width: var(--pgn-size-icon-xs); height: var(--pgn-size-icon-xs); } diff --git a/src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx b/src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx index eb19df2..be558e3 100644 --- a/src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx +++ b/src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx @@ -1,4 +1,5 @@ -import { fireEvent, screen } from '@testing-library/react'; +import { screen } from '@testing-library/react'; +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'; @@ -96,11 +97,13 @@ describe('LibrariesTeamManager', () => { }); it('renders role cards when "Roles" tab is selected', async () => { + const user = userEvent.setup(); + renderWrapper(); // Click on "Roles" tab const rolesTab = await screen.findByRole('tab', { name: /roles/i }); - fireEvent.click(rolesTab); + await user.click(rolesTab); const roleCards = await screen.findAllByTestId('role-card'); diff --git a/src/authz-module/libraries-manager/LibrariesTeamManager.tsx b/src/authz-module/libraries-manager/LibrariesTeamManager.tsx index fda4dde..dd9fc2c 100644 --- a/src/authz-module/libraries-manager/LibrariesTeamManager.tsx +++ b/src/authz-module/libraries-manager/LibrariesTeamManager.tsx @@ -57,7 +57,7 @@ const LibrariesTeamManager = () => { {!libraryRoles ? : null} {libraryRoles && libraryRoles.map(role => (