Ad
  • Custom User Avatar

    Re-raising Voile's issue closed by one very incompetent user. The task is a duplicate, and is solved not through bugfixing, but rewriting the code from scratch.

  • Default User Avatar

    I think he was trying to make sense out of testing.

  • Default User Avatar

    Fixed.

  • Custom User Avatar

    The example solution is O(n log n) where O(n) is perfectly possible, mutates its input and defines a spurious named constant.

    In short: WTF ?!?

    In medium: These are not Best Practices.

    In long: Bored now.

  • Custom User Avatar

    Duplicate.

    Also, these kind of bugfix is neither actual bugfixes, or meaningful at all.

  • Custom User Avatar

    Some sample tests are wrong.

    Test.assertNotEquals(pathologist(evidence), evidence[0] !== ['crossbow'], "this is not the murder weapon");
    Test.assertNotEquals(pathologist(evidence), evidence[0] !== ['crossbow'], "this is not the murder weapon")
    Test.assertNotSimilar(pathologist(evidence), evidence[0] !== ['crossbow'], "this is not the murder weapon")
    

    What are you testing there? The expected value there is true (a boolean) and note that you're comparing a string against an array, so returning anything that's not a booleans in the function, it'll pass.

  • Default User Avatar

    More tests needed.