Skip to content

Commit f6bd291

Browse files
dcoaarbrandes
authored andcommitted
fix: style issues and test
1 parent 9ef86c2 commit f6bd291

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

src/authz-module/index.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
font-weight: var(--pgn-typography-font-weight-base);
3030
}
3131

32-
svg {
32+
.pgn__icon {
3333
width: var(--pgn-size-icon-xs);
3434
height: var(--pgn-size-icon-xs);
3535
}

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import { fireEvent, screen } from '@testing-library/react';
1+
import { screen } from '@testing-library/react';
2+
import userEvent from '@testing-library/user-event';
23
import { renderWrapper } from '@src/setupTest';
34
import { initializeMockApp } from '@edx/frontend-platform/testing';
45
import { useLibrary } from '@src/authz-module/data/hooks';
@@ -96,11 +97,13 @@ describe('LibrariesTeamManager', () => {
9697
});
9798

9899
it('renders role cards when "Roles" tab is selected', async () => {
100+
const user = userEvent.setup();
101+
99102
renderWrapper(<LibrariesTeamManager />);
100103

101104
// Click on "Roles" tab
102105
const rolesTab = await screen.findByRole('tab', { name: /roles/i });
103-
fireEvent.click(rolesTab);
106+
await user.click(rolesTab);
104107

105108
const roleCards = await screen.findAllByTestId('role-card');
106109

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ const LibrariesTeamManager = () => {
5757
{!libraryRoles ? <Skeleton count={2} height={200} /> : null}
5858
{libraryRoles && libraryRoles.map(role => (
5959
<RoleCard
60-
key={`${role}-description`}
60+
key={`${role.role}-description`}
6161
title={role.name}
6262
userCounter={role.userCount}
6363
description={role.description}

0 commit comments

Comments
 (0)