diff --git a/src/discussions/common/AuthorLabel.jsx b/src/discussions/common/AuthorLabel.jsx index 83f6629c..b6a54160 100644 --- a/src/discussions/common/AuthorLabel.jsx +++ b/src/discussions/common/AuthorLabel.jsx @@ -75,7 +75,7 @@ function AuthorLabel({ ); - return linkToProfile && author && learnersTabEnabled + return linkToProfile && author && learnersTabEnabled && author !== messages.anonymous ? ( {!inIframe &&
} -
+
{!inIframe && }
- + {label} diff --git a/src/discussions/messages.js b/src/discussions/messages.js index e78880af..5ab58a84 100644 --- a/src/discussions/messages.js +++ b/src/discussions/messages.js @@ -153,6 +153,11 @@ const messages = defineMessages({ defaultMessage: 'Load more posts', description: 'Text on button for loading more posts by a user', }, + anonymous: { + id: 'discussions.post.anonymous.author', + defaultMessage: 'anonymous', + description: 'Author name displayed when a post is anonymous', + }, }); export default messages; diff --git a/src/discussions/posts/post-filter-bar/PostFilterBar.jsx b/src/discussions/posts/post-filter-bar/PostFilterBar.jsx index e69b4acd..3ce4e52a 100644 --- a/src/discussions/posts/post-filter-bar/PostFilterBar.jsx +++ b/src/discussions/posts/post-filter-bar/PostFilterBar.jsx @@ -39,7 +39,7 @@ const ActionItem = ({ aria-checked={value === selected} > - + {label} diff --git a/src/discussions/posts/post/PostHeader.jsx b/src/discussions/posts/post/PostHeader.jsx index bc71d32b..1784f400 100644 --- a/src/discussions/posts/post/PostHeader.jsx +++ b/src/discussions/posts/post/PostHeader.jsx @@ -109,7 +109,7 @@ function PostHeader({ && {intl.formatMessage(messages.answered)}}
) - :

{post.title}

} + :

{post.title}

} { expect(screen.queryByTestId('learner-posts-link')).toBeInTheDocument(); } }); + + it.each([ + true, + false, + ])('is only clickable if user is not anonymous', async (isAnonymous) => { + renderComponent({ ...mockPost, author: isAnonymous ? null : 'test-user' }); + if (isAnonymous) { + expect(screen.queryByTestId('learner-posts-link')).not.toBeInTheDocument(); + } else { + expect(screen.queryByTestId('learner-posts-link')).toBeInTheDocument(); + } + }); });