Ad
  • Custom User Avatar

    Looks tough but that is clearly not a production. Totally unreadable but there is some beauty in it.

  • Custom User Avatar

    The description is a total mess so is Item class. It is hard to write anything useful without changing it.

    Some products starts with sell_in above 50 or quality below 0. Nowhere in the description is it specified what should be done with it. Moreover Item constructor should be prepared for it and fix those numbers if neccessary or they should never apear!

    Random test are almost useless, without a way of repeating specific product you need to brute-force them in order to check every case. Special edge cases should be described or be taken into fixed tests!

    Starting code is a total mess it is better without it. Also without the access to Item class it is hard to make it close to domain model so I didn't even bother.

    I had high hopes for this kata but it wasn't anything good.

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution