Ad
  • Custom User Avatar

    Hello! I am a Codewars bot, and I prepared a review of your new kata!

    Click to see the review

    Commonly occurring issues

    • 🛑 No debugging info on failure: Submission tests use test.assert_equals without messages and the it titles (like "Fixed basic tests", "Edge-ish tests", and "Random tests") do not include the specific inputs being tested, so when a case fails you won’t see which elves and target caused the failure. Consider adding assertion messages such as test.assert_equals(time_to_finish(elves, target), expected, f"elves={elves}, target={target}"), or structuring tests in a loop where each it title includes the concrete input values. The same issue is present in the example tests, where failed assertions would not show the corresponding input values either.

    Please mind that I am not a very smart bot, and you should verify my remarks with any resources available for kata authors and translators: