Fix most auto-correctable rubocop issues

[rails.git] / test / lib / i18n_test.rb

diff --git a/test/lib/i18n_test.rb b/test/lib/i18n_test.rb
index ea67e084eeab12223e3540468feccdac1b38145e..700966928a7c3f1eef5375d044417fec890d34b7 100644 (file)
--- a/test/lib/i18n_test.rb
+++ b/test/lib/i18n_test.rb
@@ -13,7 +13,7 @@ class I18nTest < ActiveSupport::TestCase
         if default_value.is_a?(Hash)
           variables.push("count")
 
-          default_value.each do |subkey,subvalue|
+          default_value.each do |_subkey, subvalue|
             subvalue.scan(/%\{(\w+)\}/) do
               variables.push($1)
             end
@@ -31,8 +31,8 @@ class I18nTest < ActiveSupport::TestCase
         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"
+          value.each do |subkey, subvalue|
+            #            assert plural_keys.include?(subkey), "#{key}.#{subkey} is not a valid plural key"
 
             unless subvalue.nil?
               subvalue.scan(/%\{(\w+)\}/) do
@@ -49,31 +49,33 @@ class I18nTest < ActiveSupport::TestCase
         end
       end
 
-      assert ["ltr", "rtl"].include?(I18n.t("html.dir", :locale => locale)), "html.dir must be ltr or rtl"
+      assert %w(ltr rtl).include?(I18n.t("html.dir", :locale => locale)), "html.dir must be ltr or rtl"
     end
   end
-private
+
+  private
+
   def translation_keys(scope = nil)
     plural_keys = plural_keys(I18n.default_locale)
 
-    I18n.t(scope || ".", :locale => I18n.default_locale).map do |key,value|
+    I18n.t(scope || ".", :locale => I18n.default_locale).map do |key, value|
       scoped_key = scope ? "#{scope}.#{key}" : key
 
-      if value.kind_of?(Hash)
+      if value.is_a?(Hash)
         if value.keys - plural_keys == []
           scoped_key
         else
           translation_keys(scoped_key)
         end
-      elsif value.kind_of?(String)
+      elsif value.is_a?(String)
         scoped_key
       end
     end.flatten
   end
 
   def plural_keys(locale)
-    I18n.t("i18n.plural.keys", :locale => locale, :raise => true) + [ :zero ]
+    I18n.t("i18n.plural.keys", :locale => locale, :raise => true) + [:zero]
   rescue I18n::MissingTranslationData
-    [ :zero, :one, :other ]
+    [:zero, :one, :other]
   end
 end