Skip to content

Develop - #204

Open
Erasvx wants to merge 4 commits into
mate-academy:mainfrom
Erasvx:develop
Open

Erasvx wants to merge 4 commits into
mate-academy:mainfrom
Erasvx:develop

Conversation

@Erasvx

@Erasvx Erasvx commented Oct 24, 2026

Copy link
Copy Markdown

No description provided.

Comment thread src/arrayReverse.test.js Outdated
@@ -9,4 +9,4 @@

it(`should return an array`, () => {});

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ten test should return an array jest tu ciagle pusty, wiec zawsze przechodzi i nic nie sprawdza

Comment thread src/arrayReverse.test.js

it(`should return an array`, () => {});

it(`should return an empty string

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nazwa testu mowi o pustym stringu, a w srodku sprawdzasz ['Mate', 'Academy']. nazwa i kod musza sie zgadzac. albo zmien opis na cos w stylu should reverse words and keep their lengths, albo zrob tu faktycznie przypadek z [''] i oczekiwanie [''].

Comment thread src/arrayReverse.test.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

twoje 2 nowe testy sa dobre, ale wszystkie 3 przypadki to tablice z wieloma slowami. brakuje jednego slowa, np. ['Hell0'] -> ['0lleH'] z readme, oraz przypadku z pustym stringiem. bez nich zla implementacja moze dalej przechodzic.

@Erasvx
Erasvx requested a review from zibi-dev October 26, 2026 22:44

@zibi-dev zibi-dev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅

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.

2 participants