Почему этот метод для множественных названий категорий не работает во всех случаях?

Question

PLURALIZATION_EXCEPTIONS = "hardware",'memory'
def pluralize_category_name(name)
  category = name.split(' and ')
  exceptions_to_exp = ""

  category.map! { |e| 
    if e.match(/^[A-Z]+$/)  and !e.match(/^[A-Z]+S$/)
      e = e.pluralize
    end
    (PLURALIZATION_EXCEPTIONS.include?(e.downcase)  || e.match(/^[A-Z]+S$/) ||
      e.match(/[memory|hardware]/) )? e : e.pluralize
  }.join(' and ')
end

Тест и ожидание должны быть следующими:

it "properly pluralizes hardware as hardware" do
   pluralize_category_name("hardware").should == "hardware"
end

it "properly pluralizes UPS as UPS" do
   pluralize_category_name("UPS").should == "UPS"
end

it "properly pluralizes PDA and Portable Hardware as PDAs and Portable Hardware" do
   pluralize_category_name("PDA and Portable Hardware").should == "PDAs and Portable Hardware"
end

it "properly pluralizes perfume and cologne as perfumes and colognes" do
   pluralize_category_name("perfume and cologne").should == "perfumes and colognes"
end

Последний тест не пройден: (

HELP!

Answer 1

Я думаю, что ваша проблема в том, что в вашем состоянии

(PLURALIZATION_EXCEPTIONS.include?(e.downcase)  || e.match(/^[A-Z]+S$/) ||
      e.match(/[memory|hardware]/) )? e : e.pluralize

"perfume" совпадений /[memory|hardware]/.

[memory|hardware] - это класс символов , который соответствует любому из m, e, m, o, r и т. Д.

Возможно, вы имели в виду e.match(/(memory|hardware)]/i) вместо этого? Этот альтернативный шаблон пройдет ваши тесты, но он не использует вашу константу PLURALIZATION_EXCEPTIONS, поэтому потребуется обновление, если вы добавите какие-либо другие исключения.