Skip to content

task solution#169

Open
mesu96 wants to merge 4 commits intomate-academy:mainfrom
mesu96:develop
Open

task solution#169
mesu96 wants to merge 4 commits intomate-academy:mainfrom
mesu96:develop

Conversation

@mesu96
Copy link

@mesu96 mesu96 commented Oct 8, 2025

No description provided.

Copy link

@Zibi95 Zibi95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Read README.md carefuly. This task is about writing tests for a function

@mesu96 mesu96 closed this Oct 8, 2025
@mesu96 mesu96 reopened this Oct 8, 2025
@mesu96 mesu96 requested a review from Zibi95 October 8, 2025 07:33
Comment on lines 16 to 19
it(`should return an empty string
if original array consists of an empty string`, () => {
expect(arrayReverse(['Mate', 'Academy'])).toEqual(['ymed', 'acAetaM']);
});
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test name doesn’t match expectations

Comment on lines +25 to +32
it(`should reverse all strings and their order correctly`, () => {
expect(arrayReverse(['I', 'am', 'a', 'student!'])).toEqual([
'!',
'tn',
'e',
'dutsamaI',
]);
});
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should reverse all characters globally and split preserving original word lengths

I would call it like that

@mesu96 mesu96 requested a review from Zibi95 October 10, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants