• Custom User Avatar
    • Fixed formatting in the description
    • Made description language-agnostic
    • Organized the test suite to have 100% coverage
    • Added edge case to sample and fixed tests
    • Imprved failed assertion messages
  • Custom User Avatar

    assertResult shouldn't be called as part of creating a TestCases, otherwise the error message in case of failure is

    Test Suite Aborted
    Exception encountered when invoking run on a nested suite - Unable to load a Suite class that was discovered in the runpath: TestCases

  • Custom User Avatar

    Thank you.

  • Custom User Avatar

    that's great that you're obsessed with solving the kata but NOBODY WANTS YOU YELLING AT THEM LIKE THIS BECAUSE THINGS ARE NOT AS YOU WISH THEY WERE. if you want to see tests on a kata such as this one you are going to have to use console.log() to print the inputs. just because your code is passing the basic tests guarantees nothing, and if a knowledgeable coder can actually see your code and tell you there's a problem, the right thing to do would be say, "Thank you for reviewing my code", and get back to work.

  • Custom User Avatar

    console.log...

  • Custom User Avatar

    As I said before, I am just learning javascript and I can not revise my code if I can not see where it goes wrong ! I WANT TO SEE THE RANDOM TEST so I can determin what is wrong with my code ! JUST saying that I have a bug and I have to revise it from the beginning, is not helping me at all ! The problem is that I am obsessed with this kata ! ALL THE OTHER KATAS I HAVE COMPLETED, show all the random tests, even if there where 200 tests. Sometimes there are specific exceptions!

  • Custom User Avatar

    Done, I have included random tests in translation. Thank you.

  • Custom User Avatar

    (wave)
    Please, be very attentive because your code has a serious bug!
    Revise it once again from the beginning...

  • Custom User Avatar

    How all the student having same amount is taken.I'm getting wrong output everytime.BTW I'm multiplying the first three arguments with their respective fives,tens and twenties.

  • Custom User Avatar

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

  • Custom User Avatar

    I think I remember experiencing this as a CW bug... anyway, fix approved.

  • Custom User Avatar

    I don't know how it happened, I don't remember to changes it specifically. I think may be a problem when I try to create a fork for first time, the view was a bit different that when I create a translation and when I changed from a python view to Scala view may be I'd lose something. I don't know. I've solved it and includes it again. Thank you!

  • Custom User Avatar

    Why did you remove the example in the description?

  • Custom User Avatar

    Thank you for the feedback! There was a bug in random tests. I've solved it in a fork and its waiting for the approval of the author or a moderator.

  • Custom User Avatar

    It solves a bug in a random tests.

  • Loading more items...