Ad
  • Custom User Avatar

    have c# random tests been sabotaged?

    Assert.That(Kata.MakesTheSentence(new List { 'j', 'm', 'j', 'f', 'd', 'c', 'n', '!', 't', 'I', 'u', 'b', 'k', 'y', 'n', 'e', 'm', 'l', 'k', 'e', 'x', 'j', 'p', 't', 'b', 'q', 'z', 't', 'd', 't', 'q', 'y', 'g', 'u', 'f', 'p', 'e', 'y', 'l', 'x', 'p', 'h', 'v', 'v', 'k', 'z', 'f', 'x' }, "Iftpyucn mpvxf ebfhdnj mgujqlkz eydtxez qpbkyv jttxlk!"), Is.EqualTo(false));

    False? really? How is that?

  • Custom User Avatar

    In at least one of the tests, a digit is passed in as part of the list of characters. Possibly in the sentence as well. I ended up having to add a test to filter out digits in order to get my solution to be accepted. And the way the description is written, it sounds like an invariant that the characters and sentences will only contain the approved characters, or at least makes no mention of how we should handle it if unapproved characters like digits appear.

  • Custom User Avatar

    The description should mention explicitly that all char in the input array have to be used: this imply a possible easier solution than checking if there are enough chars to make the sentence (and only when submitting we notice that we have to also check if there's no char remaining, adding it rather to rethink an easier and shorter solution ^^)

  • Custom User Avatar

    Published translations for C, C++, and Java. Please review.

  • Custom User Avatar

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