solution#572
Conversation
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your isIsogram implementation is correct and passes all existing tests. The core logic properly handles isogram detection including case-insensitivity and detecting repeating letters regardless of their position in the word. However, you'll need to add one missing test case. The task description explicitly lists isIsogram('Oops') === false as a reference example that should be tested. This verifies the function detects repeating letters like the double 'o' even when they aren't adjacent. Add a test for this to fully meet the requirements.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
|
|
||
| it('should return false for repeating letters in different cases', () => { | ||
| expect(isIsogram('Adam')).toBe(false); | ||
| }); |
There was a problem hiding this comment.
Missing test case: the task description explicitly shows isIsogram('Oops') === false as an example that should be tested. This verifies the function detects repeating letters like the double 'o' in 'Oops'.
2 participants
No description provided.