diff --git a/src/discussions/discussions-home/InformationBanner.test.jsx b/src/discussions/discussions-home/InformationBanner.test.jsx index 098ea0d0..b73935a3 100644 --- a/src/discussions/discussions-home/InformationBanner.test.jsx +++ b/src/discussions/discussions-home/InformationBanner.test.jsx @@ -118,8 +118,16 @@ describe('User is redirected according to url according to role', () => { .toHaveAttribute('href', process.env.TA_FEEDBACK_FORM); }); - test('admin/moderators are redirected to moderators feedback form', async () => { - store.dispatch(fetchConfigSuccess(getConfigData(true, ['Student', 'Moderator']))); + test('moderators/administrators are redirected to moderators feedback form', async () => { + store.dispatch(fetchConfigSuccess(getConfigData(true, ['Student', 'Moderator', 'Administrator']))); + renderComponent(true); + expect(screen.getByText(messages.shareFeedback.defaultMessage) + .closest('a')) + .toHaveAttribute('href', process.env.STAFF_FEEDBACK_FORM); + }); + + test('user with only isAdmin true are redirected to moderators feedback form', async () => { + store.dispatch(fetchConfigSuccess(getConfigData(true, ['Student']))); renderComponent(true); expect(screen.getByText(messages.shareFeedback.defaultMessage) .closest('a')) diff --git a/src/discussions/discussions-home/InformationsBanner.jsx b/src/discussions/discussions-home/InformationsBanner.jsx index 7550106d..1f9c3a1c 100644 --- a/src/discussions/discussions-home/InformationsBanner.jsx +++ b/src/discussions/discussions-home/InformationsBanner.jsx @@ -18,7 +18,7 @@ function InformationBanner({ const TAFeedbackLink = process.env.TA_FEEDBACK_FORM || ''; const staffFeedbackLink = process.env.STAFF_FEEDBACK_FORM || ''; const hideLearnMoreButton = ((userRoles.includes('Student') && userRoles.length === 1) || !userRoles.length) && !isAdmin; - const showStaffLink = !hideLearnMoreButton || userRoles.includes('Moderator') || userRoles.includes('Administrator') ; + const showStaffLink = !hideLearnMoreButton || userRoles.includes('Moderator') || userRoles.includes('Administrator'); return (