Solution #628
Solution #628
Conversation
There was a problem hiding this comment.
Great job on implementing the getHumanAge function and its tests! 🎉 The logic is sound, and the test coverage is comprehensive. While there's a minor inconsistency with handling string inputs, it doesn't impact the core functionality. Keep up the good work, and consider refining how string inputs are managed in the future. 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| }); | ||
|
|
||
| test("should correctly handle invalid types", () => { | ||
| expect(getHumanAge("15", 15)).toEqual([1, 1]); |
There was a problem hiding this comment.
The test case expect(getHumanAge("15", 15)).toEqual([1, 1]); assumes that a string input should be converted to a number. However, this behavior is not defined in the getHumanAge function, and typically, string inputs should result in an error or NaN. Consider updating the function to handle string inputs explicitly or adjust the test case to expect NaN.
Have a nice day!