Skip to content

Commit 5d13622

Browse files
fix: make learner name capitalized (#800)
* fix: make learner name capitalized * test: fixed test cases
1 parent 9e7b7ae commit 5d13622

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/discussions/learners/LearnersView.test.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ describe('LearnersView', () => {
118118
renderComponent();
119119
});
120120

121-
expect(screen.queryByText('learner-1')).toBeInTheDocument();
121+
expect(screen.queryByText('Learner-1')).toBeInTheDocument();
122122
});
123123

124124
test('Most activity should be selected by default for the non-moderator role.', async () => {
@@ -176,7 +176,7 @@ describe('LearnersView', () => {
176176
const learners = await container.querySelectorAll('.discussion-post');
177177
const firstLearner = learners.item(0);
178178
const learnerAvatar = firstLearner.querySelector('[alt=learner-1]');
179-
const learnerTitle = within(firstLearner).queryByText('learner-1');
179+
const learnerTitle = within(firstLearner).queryByText('Learner-1');
180180
const stats = firstLearner.querySelectorAll('.icon-size');
181181

182182
expect(learners).toHaveLength(3);

src/discussions/learners/learner/LearnerCard.jsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import React, { useContext } from 'react';
22

3+
import capitalize from 'lodash/capitalize';
34
import { Link } from 'react-router-dom';
45

56
import { Routes } from '../../../data/constants';
@@ -39,7 +40,7 @@ const LearnerCard = ({ learner }) => {
3940
<div
4041
className="text-truncate font-weight-500 text-primary-500 font-style"
4142
>
42-
{username}
43+
{capitalize(username)}
4344
</div>
4445
</div>
4546
{threads !== null && (

0 commit comments

Comments
 (0)