Ad
  • Custom User Avatar

    Order of events in each output array is not defined.

  • Custom User Avatar

    As mentioned below, the specs are incomplete: what if toggle contains strings that is contained in do/undo too?

  • Custom User Avatar

    "Performance test" is not actually testing for performance: input size is too small.

  • Custom User Avatar

    namespace should be passed in as an argument, not as a global variable.

  • Custom User Avatar

    Needs actual random tests.

  • Default User Avatar

    What should happen if you have the following?

    manageEventsStrings("open toggle foo baz", "close bar baz", "toggle");
    manageEventsStrings("open foo baz", "close toggle bar baz", "toggle");
    

    I'd expect that you shouldn't have toggle in either doActions or undoActions, since it's found in toggleActions, so the result from both of those should be ["open.codeWars foo.codeWars", "close.codeWars bar.codeWars", "toggle.codeWars baz.codeWars"].

    That isn't really clear from the description, though: any verb that is found in both doActions and undoActions should be added to toggleActions, and then any verb found in toggleActions should be skipped in both doActions and undoActions if it occurs in either of those. You only covered the case where it's found in both doActions and undoActions, not when it occurred in only one of them but also in toggleActions.

  • Default User Avatar

    Still wondering how to get kata out of beta process...

  • Custom User Avatar

    Recent changes in the codewars platform broke the original version of this kata. I have updated the example test fixture so that the tools are available. In any case, submitting your solution always worked; it was only the "run tests" that was broken.

  • Default User Avatar

    Intresting kata, but unfortunatelly i get:

    ReferenceError: codewars is not defined<:LF:> <:LF:>
    throw ex; ^ ReferenceError: codewars is not defined at [eval]:10:144 at [eval]:20:3 at at [eval]:3:6 at Object. ([eval]-wrapper:6:22) at at evalScript (node.js:536:25) at startup (node.js:80:7) at node.js:906:3 
    

    when i try to run tests

  • Default User Avatar

    It's still showing that this kata has one minor issue in the profile. Can't figure out how can I "close" it. Could you please help me with advice?

  • Default User Avatar

    I'd suggest adding non-letter characters (i.e. numbers, punctuation marks rtc.) and mention about it in description. This should make kata a little bit more universal :)

  • Default User Avatar

    Sorry! Thats my first try. I thought this should come from "preloaded" part of kata for both final and example tests.
    Updated kata. Thank you.

  • Custom User Avatar

    no namespace defined for tests