Approved.

Updated ✅

Approved.

Fixed tests of final tests suite are missing mention of n in the title.

Minor issue: the class for final tests should probably be called Tests, not SampleTests.

Approved.

Approved.

• Initial solution doesn't compile. It should be either static def rooms or rooms = [:].
• Tests don't check that rooms is actually a Map. Additional test should be added.

• Please add the following edge cases:

  • "123."

  • "-12."

• Duplicate edge case of empty string in the edge cases' it block

• Random tests' order should be @Order(5)

• Assertion messages should be

  Incorrect answer for pin = "$pin"

Approved

Applied @Just4FunCoder's and hobsky's changesa

I did use a slightly modified version of your solution before, but now I use yours on the Random Tests.

Any update to this? If you want, you can check my solution on kata's discourse section to see how to avoid having to deal with modulo issues.

Approved latest fork

Approved