Code lets pass user even if code is wrong because of inadequate number of cases

Challenge Confirm the Ending has an issue. User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.143 Safari/537.36. Please describe how to reproduce this issue, and include links to screenshots if possible.

My code:

function confirmEnding(str, target) {
  // "Never give up and good luck will find you."
  // -- Falcor
  var b=3;
  str = str.split('');
  target = target.split('');
  if(target.length ==1 && target[0] != str[str.length-1])
    return false;
   if(target.length ==1 && target[0] == str[str.length-1])
    return true;
  for(i=0;i<target.length;i++) {
  if (target[target.lenghth-i] == str[str.length-i])
    b++;
  }
  if(i==b )
    return true;
  else 
    return false;

}

confirmEnding("dename", "game");

[Bouncey]

Oh wow! @praveenoswal17 thanks for finding this bug, it seems so obvious when looking at it now.

But what an edge case to highlight it 👍 😄

First-timers only!

Add another test that should return true, but with a target.length > 4.

The code for this challenge lies here.

Following CONTRIBUTING.md will help you get set up with a local development environment, so you can test your changes.

If you have any issues whilst working on this, checkout the Contibutors Chat Room. Everyone here has been through the process of setting up and submitting their first PR to freeCodeCamp and can help with many issues you may come across.

Good luck and Happy Coding! 👍

[systimotic]

@praveenoswal17 I should clarify. Currently, checking the one length targets, returning true for the 4 length answers and returning false for everything else. Another passing solution would be to only check if the last two characters match. Adding a test that should return true with a target length larger than four fixes the first issue, but it doesn’t help the second issue. A test like I suggested in my previous comment should help with that.