Pig Latin Challenge - tests do not cover edge cases

@Greenheart I was reviewing old submissions and noticed that my implementation did not correctly return words that are all consonants (such as “my”) or where the first vowel appears at index > 2 (such as “phrase”), but did pass all the tests provided. Recommend adding these edge cases to the test suite (can get crazy like adding “schwartz” or something like that).

Offending solution (no judgement…I was learning JS at the time 😃 ):

function translatePigLatin(str) {
  var vowels = ['a', 'e', 'i', 'o', 'u'];
  for (var i = 0; i < vowels.length; i++) {
    if (str.charAt(0) == vowels[i]) {
      str+= "way";
      break;
    }
    else if (str.charAt(1) == vowels[i]) {
      str = str.slice(1) + str.charAt(0) + "ay";
      break;
    }
    else if (str.charAt(2) == vowels[i]) {
      str = str.slice(2) + str.substr(0, 2) + "ay";
      break;
    }
  }
  return str;
}

translatePigLatin("phrase");

[delbertlegg]

I’d be happy to fix it. I’ll work on it this evening…thanks!

[masters-george]

The instructions for the beta version of the challenge also need to tell the camper how to handle words that have no vowels (e.g. merely append “ay” at the end of the word).