Ad
  • Custom User Avatar

    What are u talking about? Have you tested my code in console?

    > var arr = groupAnagrams(["rat", "tar", "star"])
    > arr
    [ [ 'rat', 'tar' ], [ 'star' ] ]
    > arr.length
    2

    I cant understand, how you got that output.

  • Default User Avatar

    Thanks for responding so quickly. Honestly, I think the test case shouldn't deal with case insensitivity, rather the solutinons we code should, however I appreciate you addressing my concern.

    By the way, I did really enjoy the challenge. Thanks!

  • Custom User Avatar

    You're solution is almost right, but for the first Light test it failed because it generates the given object :

    > groupAnagrams(["rat", "tar", "star"])
    [ [ 'rat', 'tar' ], 
      [ 'star' ], 
      undefined ]
    

    The undefined should not be part of the output for the test to succeed.

    Cheers

  • Custom User Avatar

    I've fixed the assertSimilarUnsorted to compare lowercase string versions. Thanks for noticing it !

  • Default User Avatar

    I feel like the tests cases have invalid failures since it's case senistive. As a result "Rats" is not an anagram of "rats" which it most certainly is.

  • Custom User Avatar

    Light and edge tests do not validate my correct solution (tested for different inputs in console).
    Here is my solution: http://pastebin.com/LXRnTSTd