refactoring - 条件ステートメントの何が問題になっていますか? 母音を引き出そうとしている

Question

いくつかの if/elsif ステートメントを使用する場合、このコードを機能させることができますが、コードをきれいに保ちたいと考えています。このコードの何が問題なのか教えてください。["e"] を含む配列が表示されることを期待していましたが、[] を返します

def pull_vowels(str)

letters=str.split(//)
vowels=[]

0.upto(letters.count-1) {|idx|
  vowels<<letters[idx]  if letters[idx]==("a"||"e"||"i"||"o"||"u")
}

vowels
end

pull_vowels("test")