ssqs #614
ssqs #614
Conversation
There was a problem hiding this comment.
Great job on implementing the getHumanAge function and its tests! 🎉 The code meets the task requirements and constraints, and the tests are well-structured. However, please double-check the test case descriptions and expected results for ages 14 and 23 to ensure they align with the logic of your function. Keep up the great work! 🚀
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| test('should return [0, 0] for age 14 for both cat and dog', () => { | ||
| expect(getHumanAge(14, 14)) | ||
| .toEqual([0, 0]); |
There was a problem hiding this comment.
The test case description and expected result for age 14 returning [0, 0] might not be correct depending on the logic of getHumanAge. Verify if age 14 should indeed map to [0, 0] in human years.
| test('should return [1, 1] for age 23 for both cat and dog', () => { | ||
| expect(getHumanAge(23, 23)) | ||
| .toEqual([1, 1]); |
There was a problem hiding this comment.
The test case description and expected result for age 23 returning [1, 1] might not be correct depending on the logic of getHumanAge. Verify if age 23 should indeed map to [1, 1] in human years.
No description provided.