Skip to content

Commit e738f79

Browse files
committed
fix: update paragon version to revert Toast test changes due to extra alert role
1 parent 23b5aed commit e738f79

File tree

3 files changed

+19
-9
lines changed

3 files changed

+19
-9
lines changed

package-lock.json

Lines changed: 14 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@
6262
"@openedx-plugins/course-app-xpert_unit_summary": "file:plugins/course-apps/xpert_unit_summary",
6363
"@openedx/frontend-build": "^14.3.3",
6464
"@openedx/frontend-plugin-framework": "^1.7.0",
65-
"@openedx/paragon": "^23.4.5",
65+
"@openedx/paragon": "^23.5.0",
6666
"@redux-devtools/extension": "^3.3.0",
6767
"@reduxjs/toolkit": "1.9.7",
6868
"@tanstack/react-query": "4.36.1",

src/generic/processing-notification/ProcessingNotification.test.jsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,16 @@ describe('<ProcessingNotification />', () => {
2222
render(<ProcessingNotification {...props} close={() => {}} />);
2323
await screen.findByText(props.title);
2424
const undo = await screen.findByText('Undo');
25-
const alert = await screen.findAllByRole('alert', { hidden: true });
26-
expect(alert[1].classList.contains('processing-notification-hide-close-button')).toBeFalsy();
25+
const alert = await screen.findByRole('alert', { hidden: true });
26+
expect(alert.classList.contains('processing-notification-hide-close-button')).toBeFalsy();
2727
await userEvent.click(undo);
2828
expect(mockUndo).toHaveBeenCalled();
2929
});
3030

3131
it('add hide-close-button class if no close action is passed', async () => {
3232
render(<ProcessingNotification {...props} />);
3333
await screen.findByText(props.title);
34-
const alert = await screen.findAllByRole('alert', { hidden: true });
35-
expect(alert[1].classList.contains('processing-notification-hide-close-button')).toBeTruthy();
34+
const alert = await screen.findByRole('alert', { hidden: true });
35+
expect(alert.classList.contains('processing-notification-hide-close-button')).toBeTruthy();
3636
});
3737
});

0 commit comments

Comments
 (0)