Ad
  • Default User Avatar

    This was a great Kata, and my first Beta since being able to provide feedback, so you get a 6th-Kyu Ready from me ^_^

  • Default User Avatar

    Right, sorry. I was in C#.

    I didn't realise that making an instructor that looked like:

    public Cube() {}

    Would still initialise an int to be 0 anyway. I thought that if you didn't assign it a default value, then the Get wouldn't work. Clearly I was wrong.

    IGNORE MY SUGGESTION, FOR IT IS FOLLY!

  • Default User Avatar

    Cannot currently complete in C#.

    The variable 'test' has been assigned but its value is never used.

  • Default User Avatar

    The only thing that could be better is that, while there is a test to determine whether or not there is a constructor that takes zero arguments, that test does not then check that the side actually gets set to 0 by default.

    You can produce an empty constructor that does nothing and still pass the test. I would suggest that, for that one particular test, after saying:

    Cube c = new Cube();

    Adding:

    Assert.AreEqual(0, c.GetSide());

    And then we would be forced to make a constructor that defaults to 0, as per the instructions in the description.

  • Default User Avatar

    You're literally comparing count to count + 1 of the same list, AFTER the value is added.

    It's the same as saying Assert.AreEqual(0, 1);

    It's NEVER going to be equal. If you empty the list completely it says:

    Expected: 0
    But was: 1

    In C#, this Kata is currently impossible.

  • Default User Avatar

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

  • Default User Avatar

    I'm getting the exact same issue. I don't understand why the author keeps putting the Random class in their test classes for these Kata, because it can't be used under normal circumstances. If it needs to be random, we can simply define a new Random variable in our own class.

  • Default User Avatar

    Upon running a test:

    "The variable 'testflagiO' is assigned but its value is never used".

  • Default User Avatar

    In C#:
    Expected: "0:0:0"
    But was: "00:00:00"

    Even though the string in the test clearly states that "00:00:00" is the correct answer. This needs to be fixed.

  • Default User Avatar

    Random tests just shouldn't be allowed for some Kata.
    There's no honour in getting lucky with RNG.