Ad
  • Custom User Avatar

    Needs random tests

  • Custom User Avatar

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

  • Custom User Avatar

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

  • Custom User Avatar

    What about correcting your kata issue instead of solving a new one;-)? Isn't it the first task of an author?

  • Custom User Avatar

    Hi, can't see the new testcases, because cw doesn't update this code before i solved the kata (so my changed testcases work as before). What i can see is the error of your "submit testcases" as before. The following message appears, which is not an error produced by my code (returning an empty list displays same error, with hint of missing bracket):
    Test Results:
    [eval]:33
    Test.assertSimilar(caesarPyramid("b difflz buufnqu up njnjd dbftbs pof uipvtboe"), 1, 1000, "This is not one of Caesar's commissioned pyramids"]);;
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    SyntaxError: missing ) after argument list
    at Object.exports.runInThisContext (vm.js:53:16)
    at Object. ([eval]-wrapper:6:22)
    at
    at node.js:328:29
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

  • Default User Avatar

    Thank you for the feedback, I think I have resolved some of the issues you raised, except the no function name, I was unsure what you meant by this? PLease let me know what you think.

  • Custom User Avatar

    Normally i should mark different issues here;-)...
    Static tests are wrong - for example no function name, switched letters in one case "hundedr", generally wrong tests (use Test.assertSimilar instead of Test.assertEquals). Submit tests don't work too because of code errors...
    So nice idea of these combined and different tasks inside one kata, but at the moment bad implementation - sorry:-(...

  • Custom User Avatar

    I agree with that. A few random tests would be better to test if the solutions actually work

  • Default User Avatar

    Thank you for your feedback, I will include that in the description.

  • Default User Avatar

    Hi, you should specify some preconditions. Based on the tests:

    1. The list of cycles contains more than one unique integer bigger than 0.
    2. The result has to be floored.

    Regards,

    suic

  • Custom User Avatar

    Needs random testcases

  • Default User Avatar

    I run the code and all the test cases pass, however, upon submission I recieve this error message: 'Value is not what was expected', any suggestions?

  • Default User Avatar

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