diff options
author | Kevin McPhillips <github@kevinmcphillips.ca> | 2017-01-16 13:49:44 -0500 |
---|---|---|
committer | Kevin McPhillips <github@kevinmcphillips.ca> | 2017-01-16 14:40:41 -0500 |
commit | f90a08c193d4ec8267f4409b7a670c2b53e0621d (patch) | |
tree | 1fd8388950d3bc6c52a802e7ebb0888bba2d0ec3 /activesupport | |
parent | c8e92f174d9190fe4cb38199bc2e59d62ace0003 (diff) | |
download | rails-f90a08c193d4ec8267f4409b7a670c2b53e0621d.tar.gz rails-f90a08c193d4ec8267f4409b7a670c2b53e0621d.tar.bz2 rails-f90a08c193d4ec8267f4409b7a670c2b53e0621d.zip |
Raise ArgumentError if attempting to transliterate anything that is not a string
Diffstat (limited to 'activesupport')
-rw-r--r-- | activesupport/lib/active_support/inflector/transliterate.rb | 2 | ||||
-rw-r--r-- | activesupport/test/transliterate_test.rb | 18 |
2 files changed, 20 insertions, 0 deletions
diff --git a/activesupport/lib/active_support/inflector/transliterate.rb b/activesupport/lib/active_support/inflector/transliterate.rb index 3e78986e8e..de6dd2720b 100644 --- a/activesupport/lib/active_support/inflector/transliterate.rb +++ b/activesupport/lib/active_support/inflector/transliterate.rb @@ -57,6 +57,8 @@ module ActiveSupport # transliterate('Jürgen') # # => "Juergen" def transliterate(string, replacement = "?".freeze) + raise ArgumentError, "Can only transliterate strings. Received #{string.class.name}" unless string.is_a?(String) + I18n.transliterate(ActiveSupport::Multibyte::Unicode.normalize( ActiveSupport::Multibyte::Unicode.tidy_bytes(string), :c), replacement: replacement) diff --git a/activesupport/test/transliterate_test.rb b/activesupport/test/transliterate_test.rb index 040ddd25fc..466b69bcef 100644 --- a/activesupport/test/transliterate_test.rb +++ b/activesupport/test/transliterate_test.rb @@ -31,4 +31,22 @@ class TransliterateTest < ActiveSupport::TestCase def test_transliterate_should_allow_a_custom_replacement_char assert_equal "a*b", ActiveSupport::Inflector.transliterate("a索b", "*") end + + def test_transliterate_handles_empty_string + assert_equal "", ActiveSupport::Inflector.transliterate("") + end + + def test_transliterate_handles_nil + exception = assert_raises ArgumentError do + ActiveSupport::Inflector.transliterate(nil) + end + assert_equal "Can only transliterate strings. Received NilClass", exception.message + end + + def test_transliterate_handles_unknown_object + exception = assert_raises ArgumentError do + ActiveSupport::Inflector.transliterate(Object.new) + end + assert_equal "Can only transliterate strings. Received Object", exception.message + end end |