Skip to content

Commit 216d6d2

Browse files
committed
#955 updated tests
1 parent 0b17344 commit 216d6d2

3 files changed

Lines changed: 9 additions & 4 deletions

File tree

src/frontend/src/components/ActionsMenu.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ const ActionsMenu: React.FC<ActionsMenuProps> = ({ buttons, title = 'Actions' })
7676
button.onClick();
7777
}}
7878
disabled={button.disabled}
79-
aria-disabled={button.disabled ? 'true' : undefined}
8079
>
8180
{button.icon && <ListItemIcon>{button.icon}</ListItemIcon>}
8281
{button.title}

src/frontend/src/tests/pages/ProjectDetailPage/ProjectViewContainer.test.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,14 @@ describe('Rendering Project View Container', () => {
5151
vi.spyOn(userHooks, 'useCurrentUser').mockReturnValue(exampleGuestUser);
5252

5353
fireEvent.click(screen.getByText('Actions'));
54-
expect(screen.getByText('Edit')).toHaveAttribute('aria-disabled', 'true');
55-
expect(screen.getByText('Request Change')).toHaveAttribute('aria-disabled', 'true');
54+
55+
const editMenuItem = screen.getByText('Edit').closest('li');
56+
expect(editMenuItem).not.toBeNull();
57+
expect(editMenuItem?.querySelector('svg')).toHaveAttribute('aria-hidden','true');
58+
59+
const reqChangeMenuItem = screen.getByText('Request Change');
60+
expect(reqChangeMenuItem).not.toBeNull();
61+
expect(reqChangeMenuItem?.querySelector('svg')).toHaveAttribute('aria-hidden','true');
5662
});
5763

5864
it('enables the buttons for admin users', async () => {

src/frontend/src/tests/pages/WorkPackageDetailPage/WorkPackagePage.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,6 @@ describe('work package container', () => {
123123

124124
fireEvent.click(screen.getByText('Actions'));
125125

126-
expect(screen.getByText('Edit')).toHaveAttribute('aria-disabled');
126+
expect(screen.queryByText('Edit')).toBeNull();
127127
});
128128
});

0 commit comments

Comments
 (0)