Skip to content

Commit f00de05

Browse files
test: BrokenLinkTable tests updated
1 parent dc2d84a commit f00de05

2 files changed

Lines changed: 11 additions & 12 deletions

File tree

src/optimizer-page/scan-results/BrokenLinkTable.test.tsx

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import {
55
fireEvent,
66
waitFor,
77
} from '@testing-library/react';
8+
import { MemoryRouter } from 'react-router-dom';
89
import { IntlProvider } from '@edx/frontend-platform/i18n';
910
import { AppProvider } from '@edx/frontend-platform/react';
1011
import { initializeMockApp } from '@edx/frontend-platform';
@@ -86,9 +87,11 @@ const BrokenLinkTableWrapper: React.FC<BrokenLinkTableWrapperProps> = ({
8687

8788
const intlWrapper = (ui: React.ReactElement) =>
8889
render(
89-
<IntlProvider locale="en" messages={{}}>
90-
{ui}
91-
</IntlProvider>,
90+
<MemoryRouter>
91+
<IntlProvider locale="en" messages={{}}>
92+
{ui}
93+
</IntlProvider>
94+
</MemoryRouter>,
9295
);
9396

9497
describe('BrokenLinkTable', () => {
@@ -713,14 +716,11 @@ describe('BrokenLinkTable', () => {
713716

714717
const goToAnchor = screen.getByText('Test Block');
715718

716-
fireEvent.click(goToAnchor);
717-
718-
await waitFor(() => {
719-
expect(window.open).toHaveBeenCalledWith(
720-
'/course/course-v1:TestX+Test101+2024/container/unit-1#block-1',
721-
'_blank',
722-
);
723-
});
719+
expect(goToAnchor.closest('a')).toHaveAttribute(
720+
'href',
721+
'/course/course-v1:TestX+Test101+2024/container/unit-1#block-1',
722+
);
723+
expect(goToAnchor.closest('a')).toHaveAttribute('target', '_blank');
724724
});
725725

726726
it('BrokenLinkHref anchor opens the href URL', async () => {

src/optimizer-page/scan-results/BrokenLinkTable.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@ const BrokenLinkHref: FC<{ href: string; }> = ({ href }) => {
4343
};
4444

4545
const GoToBlock: FC<{ block: { url: string; displayName?: string; }; }> = ({ block }) => {
46-
4746
return (
4847
<div className="go-to-block-link-container">
4948
<Link to={block.url} className="broken-link" rel="noreferrer" target="_blank">

0 commit comments

Comments
 (0)