As mentioned below, the specs are incomplete: what if toggle contains strings that is contained in do/undo too?
"Performance test" is not actually testing for performance: input size is too small.
namespace should be passed in as an argument, not as a global variable.
Needs actual random tests.
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"].
["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.
Still wondering how to get kata out of beta process...
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.
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
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?
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 :)
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.
no namespace defined for tests