X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/9a9b045372a6f48420a9a6dacfde52c34ab7abce..2f3c2cdb44def797f1b3107ae166c77e3eee26d2:/test/lib/i18n_test.rb diff --git a/test/lib/i18n_test.rb b/test/lib/i18n_test.rb index 09965e950..9c7d8e568 100644 --- a/test/lib/i18n_test.rb +++ b/test/lib/i18n_test.rb @@ -3,53 +3,55 @@ require "test_helper" class I18nTest < ActiveSupport::TestCase I18n.available_locales.each do |locale| define_method("test_#{locale.to_s.underscore}".to_sym) do - # plural_keys = plural_keys(locale) + without_i18n_exceptions do + # plural_keys = plural_keys(locale) - translation_keys.each do |key| - variables = [] + translation_keys.each do |key| + variables = [] - default_value = I18n.t(key, :locale => I18n.default_locale) + default_value = I18n.t(key, :locale => I18n.default_locale) - if default_value.is_a?(Hash) - variables.push("count") + if default_value.is_a?(Hash) + variables.push("count") - default_value.each do |_subkey, subvalue| - subvalue.scan(/%\{(\w+)\}/) do - variables.push($1) + default_value.each_value do |subvalue| + subvalue.scan(/%\{(\w+)\}/) do + variables.push(Regexp.last_match(1)) + end + end + else + default_value.scan(/%\{(\w+)\}/) do + variables.push(Regexp.last_match(1)) end end - else - default_value.scan(/%\{(\w+)\}/) do - variables.push($1) - end - end - if key =~ /^(active(model|record)\.)?errors\./ - variables.push("attribute") - end + variables.push("attribute") if key =~ /^(active(model|record)\.)?errors\./ - value = I18n.t(key, :locale => locale, :fallback => true) + value = I18n.t(key, :locale => locale, :fallback => true) - if value.is_a?(Hash) - value.each do |subkey, subvalue| - # assert plural_keys.include?(subkey), "#{key}.#{subkey} is not a valid plural key" + if value.is_a?(Hash) + value.each do |subkey, subvalue| + # assert plural_keys.include?(subkey), "#{key}.#{subkey} is not a valid plural key" - next if subvalue.nil? + next if subvalue.nil? - subvalue.scan(/%\{(\w+)\}/) do - assert variables.include?($1), "#{key}.#{subkey} uses unknown interpolation variable #{$1}" + subvalue.scan(/%\{(\w+)\}/) do + assert_includes variables, Regexp.last_match(1), "#{key}.#{subkey} uses unknown interpolation variable #{Regexp.last_match(1)}" + end end - end - else - assert value.is_a?(String), "#{key} is not a string" - value.scan(/%\{(\w+)\}/) do - assert variables.include?($1), "#{key} uses unknown interpolation variable #{$1}" + assert_includes value, :other, "#{key}.other plural key missing" + else + assert value.is_a?(String), "#{key} is not a string" + + value.scan(/%\{(\w+)\}/) do + assert_includes variables, Regexp.last_match(1), "#{key} uses unknown interpolation variable #{Regexp.last_match(1)}" + end end end - end - assert %w(ltr rtl).include?(I18n.t("html.dir", :locale => locale)), "html.dir must be ltr or rtl" + assert_includes %w[ltr rtl], I18n.t("html.dir", :locale => locale), "html.dir must be ltr or rtl" + end end end @@ -61,13 +63,14 @@ class I18nTest < ActiveSupport::TestCase I18n.t(scope || ".", :locale => I18n.default_locale).map do |key, value| scoped_key = scope ? "#{scope}.#{key}" : key - if value.is_a?(Hash) + case value + when Hash if value.keys - plural_keys == [] scoped_key else translation_keys(scoped_key) end - elsif value.is_a?(String) + when String scoped_key end end.flatten