Skip to content

Commit

Permalink
Merge pull request #283 from wearefine/rm-acronym-inflectors
Browse files Browse the repository at this point in the history
restore inflections initializer to default with no acronym inflection…
  • Loading branch information
jamesmk authored Jul 25, 2017
2 parents 4964f17 + af8ddd7 commit 0da3d48
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 13 deletions.
26 changes: 16 additions & 10 deletions config/initializers/inflections.rb
Original file line number Diff line number Diff line change
@@ -1,10 +1,16 @@
ActiveSupport::Inflector.inflections(:en) do |inflect|
inflect.acronym 'LOL'
inflect.acronym 'PDF'
inflect.acronym 'SEO'
inflect.acronym 'CEO'
inflect.acronym 'SKU'
inflect.acronym 'URL'
inflect.acronym 'AVA'
inflect.acronym 'CTA'
end
# Be sure to restart your server when you modify this file.

# Add new inflection rules using the following format. Inflections
# are locale specific, and you may define rules for as many different
# locales as you wish. All of these examples are active by default:
# ActiveSupport::Inflector.inflections(:en) do |inflect|
# inflect.plural /^(ox)$/i, '\1en'
# inflect.singular /^(ox)en/i, '\1'
# inflect.irregular 'person', 'people'
# inflect.uncountable %w( fish sheep )
# end

# These inflection rules are supported but not enabled by default:
# ActiveSupport::Inflector.inflections(:en) do |inflect|
# inflect.acronym 'RESTful'
# end
4 changes: 1 addition & 3 deletions spec/features/form_helpers/fae_input_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,7 @@
end
end

scenario 'should capitalize acronyms in fields, and display common helper texts' do
expect(page).to have_content('SEO Title')
expect(page).to have_content('SEO Description')
scenario 'should display common helper texts' do
expect(page).to have_content('Company Name | Keyword-driven description of the page section. Approx 65 characters.')
expect(page).to have_content('Displayed in search engine results. Under 150 characters.')
end
Expand Down

0 comments on commit 0da3d48

Please sign in to comment.