Fair play.

Thanks!

Thank you!

Thank you!

In the C# version there is an issue that can be solved in one of two ways: the class name can be renamed to Kata or the unit tests will need to call "SmallestIntegerFinder.FindSmallestInt(int []). Simple fix.

Very small suggestion. The solution method used by the random tests needs to be renamed to Solution per C# style guidelines.

Thanks for the suggestion. I will use that flag for similar issues going forward.

Very small issue. The solution method used by the random tests needs to be renamed to Solution per C# style guidelines.

There are some minor tweaks for the C# version:

• The method name needs to be "BasicOp" but I would even prefer "BasicOperation"
• Parameters should be "operation", "value1" and "value2" -- prefer the full names for modern IDEs as there is no reason to save chars

Great kata! This feels ready to move out of beta (at least in C#).

This solution is really clever. Learned something new today!

Thanks!
ssfcultra

This is very nice and concise.