This test should really have been a part of the sample tests.

• random_device should be used as a seeder, and should not be used as a RNG
• helper functons could be non-public
• Custom assertion messages with inputs could be helpful

Merge conflict !

Fixed

Fixed

There is using namespace std; in C++ Preloaded section which should be removed. See: https://github.com/codewars/content-issues/wiki/List-of-Cpp-Kata-to-Update

There is using namespace std; in C++ Preloaded section which should be removed. See: https://github.com/codewars/content-issues/wiki/List-of-Cpp-Kata-to-Update

Oh, thank you ;)

I think, that in general, .at() is safer than [] in terms of exceptions throwing.

Is there a difference between using '.at' and '[]'?

Fixed in list of C++ katas to update

C++ test code has compilation warnings

@Unnamed Thanks for the feedback. I just made this fork which resolves the issues above. Hopefully you can approve it now :) https://www.codewars.com/kumite/5f790b13be51af001a580e53?sel=5f84d2e100f3180023b97f9f

Merge conflict. Fork and update the description. Also, the sample tests still use include.