X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/550c4a3a45814fde5c809334c85f1ebc47659a82..5fdddf2a8a98362ee646fb9cd22fadeaea185b46:/test/lib/i18n_test.rb diff --git a/test/lib/i18n_test.rb b/test/lib/i18n_test.rb index 9c7d8e568..a25e4887c 100644 --- a/test/lib/i18n_test.rb +++ b/test/lib/i18n_test.rb @@ -55,6 +55,32 @@ class I18nTest < ActiveSupport::TestCase end end + Rails.root.glob("config/locales/*.yml").each do |filename| + lang = File.basename(filename, ".yml") + define_method("test_#{lang}_for_raw_html".to_sym) do + yml = YAML.load_file(filename) + assert_nothing_raised do + check_values_for_raw_html(yml) + end + end + end + + def test_en_for_nil_values + en = YAML.load_file(Rails.root.join("config/locales/en.yml")) + assert_nothing_raised do + check_values_for_nil(en) + end + end + + # We should avoid using the key `zero:` in English, since that key + # is used for "numbers ending in zero" in other languages. + def test_en_for_zero_key + en = YAML.load_file(Rails.root.join("config/locales/en.yml")) + assert_nothing_raised do + check_keys_for_zero(en) + end + end + private def translation_keys(scope = nil) @@ -81,4 +107,35 @@ class I18nTest < ActiveSupport::TestCase rescue I18n::MissingTranslationData [:zero, :one, :other] end + + def check_values_for_raw_html(hash) + hash.each_pair do |k, v| + if v.is_a? Hash + check_values_for_raw_html(v) + else + next unless k.to_s.end_with?("_html") + raise "Avoid using raw html in '#{k}: #{v}'" if v.include? "<" + end + end + end + + def check_values_for_nil(hash) + hash.each_pair do |k, v| + if v.is_a? Hash + check_values_for_nil(v) + else + raise "Avoid nil values in '#{k}: nil'" if v.nil? + end + end + end + + def check_keys_for_zero(hash) + hash.each_pair do |k, v| + if v.is_a? Hash + check_keys_for_zero(v) + else + raise "Avoid using 'zero' key in '#{k}: #{v}'" if k.to_s == "zero" + end + end + end end