ffaker 2.22.0 → 2.23.0

data/lib/ffaker.rb

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require_relative 'ffaker/version'
+require 'pathname'
+
 module FFaker
   require_relative 'ffaker/utils/array_utils'
   require_relative 'ffaker/utils/module_utils'
@@ -28,197 +31,63 @@ module FFaker
     letterify(numerify(masks))
   end
 
-  autoload :Address, 'ffaker/address'
-  autoload :AddressAU, 'ffaker/address_au'
-  autoload :AddressBR, 'ffaker/address_br'
-  autoload :AddressCA, 'ffaker/address_ca'
-  autoload :AddressCH, 'ffaker/address_ch'
-  autoload :AddressCHDE, 'ffaker/address_ch_de'
-  autoload :AddressCHFR, 'ffaker/address_ch_fr'
-  autoload :AddressCHIT, 'ffaker/address_ch_it'
-  autoload :AddressDA, 'ffaker/address_da'
-  autoload :AddressDE, 'ffaker/address_de'
-  autoload :AddressFI, 'ffaker/address_fi'
-  autoload :AddressFR, 'ffaker/address_fr'
-  autoload :AddressGR, 'ffaker/address_gr'
-  autoload :AddressID, 'ffaker/address_id'
-  autoload :AddressIN, 'ffaker/address_in'
-  autoload :AddressIT, 'ffaker/address_it'
-  autoload :AddressJA, 'ffaker/address_ja'
-  autoload :AddressKR, 'ffaker/address_kr'
-  autoload :AddressMX, 'ffaker/address_mx'
-  autoload :AddressNL, 'ffaker/address_nl'
-  autoload :AddressPL, 'ffaker/address_pl'
-  autoload :AddressRU, 'ffaker/address_ru'
-  autoload :AddressSE, 'ffaker/address_se'
-  autoload :AddressSN, 'ffaker/address_sn'
-  autoload :AddressUA, 'ffaker/address_ua'
-  autoload :AddressUK, 'ffaker/address_uk'
-  autoload :AddressUS, 'ffaker/address_us'
-  autoload :Airline, 'ffaker/airline'
-  autoload :Animal, 'ffaker/animal'
-  autoload :AnimalBR, 'ffaker/animal_br'
-  autoload :AnimalCN, 'ffaker/animal_cn'
-  autoload :AnimalES, 'ffaker/animal_es'
-  autoload :AnimalPL, 'ffaker/animal_pl'
-  autoload :AnimalUS, 'ffaker/animal_us'
-  autoload :Avatar, 'ffaker/avatar'
-  autoload :AWS, 'ffaker/aws'
-  autoload :BaconIpsum, 'ffaker/bacon_ipsum'
-  autoload :Bank, 'ffaker/bank'
-  autoload :Book, 'ffaker/book'
-  autoload :Boolean, 'ffaker/boolean'
-  autoload :CheesyLingo, 'ffaker/cheesy_lingo'
-  autoload :Code, 'ffaker/code'
-  autoload :Color, 'ffaker/color'
-  autoload :ColorPL, 'ffaker/color_pl'
-  autoload :ColorUA, 'ffaker/color_ua'
-  autoload :Company, 'ffaker/company'
-  autoload :CompanyCN, 'ffaker/company_cn'
-  autoload :CompanyFR, 'ffaker/company_fr'
-  autoload :CompanyIT, 'ffaker/company_it'
-  autoload :CompanyJA, 'ffaker/company_ja'
-  autoload :CompanySE, 'ffaker/company_se'
-  autoload :Conference, 'ffaker/conference'
-  autoload :CoursesFR, 'ffaker/courses_fr'
-  autoload :Currency, 'ffaker/currency'
-  autoload :Date, 'ffaker/date'
-  autoload :DizzleIpsum, 'ffaker/dizzle_ipsum'
-  autoload :Education, 'ffaker/education'
-  autoload :EducationCN, 'ffaker/education_cn'
-  autoload :Filesystem, 'ffaker/filesystem'
-  autoload :Food, 'ffaker/food'
-  autoload :FoodPL, 'ffaker/food_pl'
-  autoload :FreedomIpsum, 'ffaker/freedom_ipsum'
-  autoload :Game, 'ffaker/game'
-  autoload :Gender, 'ffaker/gender'
-  autoload :GenderBR, 'ffaker/gender_br'
-  autoload :GenderCN, 'ffaker/gender_cn'
-  autoload :GenderID, 'ffaker/gender_id'
-  autoload :GenderIT, 'ffaker/gender_it'
-  autoload :GenderJA, 'ffaker/gender_ja'
-  autoload :GenderJP, 'ffaker/gender_jp'
-  autoload :GenderKR, 'ffaker/gender_kr'
-  autoload :GenderPL, 'ffaker/gender_pl'
-  autoload :GenderRU, 'ffaker/gender_ru'
-  autoload :Geolocation, 'ffaker/geolocation'
-  autoload :Guid, 'ffaker/guid'
-  autoload :HealthcareIpsum, 'ffaker/healthcare_ipsum'
-  autoload :HealthcareRU, 'ffaker/healthcare_ru'
-  autoload :HipsterIpsum, 'ffaker/hipster_ipsum'
-  autoload :HTMLIpsum, 'ffaker/html_ipsum'
-  autoload :Identification, 'ffaker/identification'
-  autoload :IdentificationBR, 'ffaker/identification_br'
-  autoload :IdentificationES, 'ffaker/identification_es'
-  autoload :IdentificationESCL, 'ffaker/identification_es_cl'
-  autoload :IdentificationESCO, 'ffaker/identification_es_co'
-  autoload :IdentificationIN, 'ffaker/identification_in'
-  autoload :IdentificationIT, 'ffaker/identification_it'
-  autoload :IdentificationKr, 'ffaker/identification_kr'
-  autoload :IdentificationMX, 'ffaker/identification_mx'
-  autoload :IdentificationPL, 'ffaker/identification_pl'
-  autoload :IdentificationTW, 'ffaker/identification_tw'
-  autoload :IdentificationEC, 'ffaker/identification_ec'
-  autoload :Image, 'ffaker/image'
-  autoload :Internet, 'ffaker/internet'
-  autoload :InternetSE, 'ffaker/internet_se'
-  autoload :Job, 'ffaker/job'
-  autoload :JobBR, 'ffaker/job_br'
-  autoload :JobCN, 'ffaker/job_cn'
-  autoload :JobFR, 'ffaker/job_fr'
-  autoload :JobIT, 'ffaker/job_it'
-  autoload :JobJA, 'ffaker/job_ja'
-  autoload :JobKR, 'ffaker/job_kr'
-  autoload :JobVN, 'ffaker/job_vn'
-  autoload :JoJo, 'ffaker/jo_jo'
-  autoload :Locale, 'ffaker/locale'
-  autoload :Lorem, 'ffaker/lorem'
-  autoload :LoremAR, 'ffaker/lorem_ar'
-  autoload :LoremBR, 'ffaker/lorem_br'
-  autoload :LoremCN, 'ffaker/lorem_cn'
-  autoload :LoremFR, 'ffaker/lorem_fr'
-  autoload :LoremIE, 'ffaker/lorem_ie'
-  autoload :LoremIT, 'ffaker/lorem_it'
-  autoload :LoremJA, 'ffaker/lorem_ja'
-  autoload :LoremKR, 'ffaker/lorem_kr'
-  autoload :LoremPL, 'ffaker/lorem_pl'
-  autoload :LoremRU, 'ffaker/lorem_ru'
-  autoload :LoremUA, 'ffaker/lorem_ua'
-  autoload :Movie, 'ffaker/movie'
-  autoload :Music, 'ffaker/music'
-  autoload :Name, 'ffaker/name'
-  autoload :NameAR, 'ffaker/name_ar'
-  autoload :NameBR, 'ffaker/name_br'
-  autoload :NameCN, 'ffaker/name_cn'
-  autoload :NameCS, 'ffaker/name_cs'
-  autoload :NameDA, 'ffaker/name_da'
-  autoload :NameDE, 'ffaker/name_de'
-  autoload :NameES, 'ffaker/name_es'
-  autoload :NameFR, 'ffaker/name_fr'
-  autoload :NameGA, 'ffaker/name_ga'
-  autoload :NameGR, 'ffaker/name_gr'
-  autoload :NameID, 'ffaker/name_id'
-  autoload :NameIN, 'ffaker/name_in'
-  autoload :NameIT, 'ffaker/name_it'
-  autoload :NameJA, 'ffaker/name_ja'
-  autoload :NameKH, 'ffaker/name_kh'
-  autoload :NameKR, 'ffaker/name_kr'
-  autoload :NameMX, 'ffaker/name_mx'
-  autoload :NameNB, 'ffaker/name_nb'
-  autoload :NameNL, 'ffaker/name_nl'
-  autoload :NamePH, 'ffaker/name_ph'
-  autoload :NamePL, 'ffaker/name_pl'
-  autoload :NameRU, 'ffaker/name_ru'
-  autoload :NameSE, 'ffaker/name_se'
-  autoload :NameSN, 'ffaker/name_sn'
-  autoload :NameTH, 'ffaker/name_th'
-  autoload :NameTW, 'ffaker/name_tw'
-  autoload :NameTHEN, 'ffaker/name_th_en'
-  autoload :NameUA, 'ffaker/name_ua'
-  autoload :NameVN, 'ffaker/name_vn'
-  autoload :NatoAlphabet, 'ffaker/nato_alphabet'
-  autoload :Number, 'ffaker/number'
-  autoload :PhoneNumber, 'ffaker/phone_number'
-  autoload :PhoneNumberAU, 'ffaker/phone_number_au'
-  autoload :PhoneNumberBR, 'ffaker/phone_number_br'
-  autoload :PhoneNumberCH, 'ffaker/phone_number_ch'
-  autoload :PhoneNumberCU, 'ffaker/phone_number_cu'
-  autoload :PhoneNumberDA, 'ffaker/phone_number_da'
-  autoload :PhoneNumberDE, 'ffaker/phone_number_de'
-  autoload :PhoneNumberFR, 'ffaker/phone_number_fr'
-  autoload :PhoneNumberID, 'ffaker/phone_number_id'
-  autoload :PhoneNumberIT, 'ffaker/phone_number_it'
-  autoload :PhoneNumberJA, 'ffaker/phone_number_ja'
-  autoload :PhoneNumberKR, 'ffaker/phone_number_kr'
-  autoload :PhoneNumberMX, 'ffaker/phone_number_mx'
-  autoload :PhoneNumberNL, 'ffaker/phone_number_nl'
-  autoload :PhoneNumberPL, 'ffaker/phone_number_pl'
-  autoload :PhoneNumberSE, 'ffaker/phone_number_se'
-  autoload :PhoneNumberSG, 'ffaker/phone_number_sg'
-  autoload :PhoneNumberSN, 'ffaker/phone_number_sn'
-  autoload :PhoneNumberTW, 'ffaker/phone_number_tw'
-  autoload :PhoneNumberUA, 'ffaker/phone_number_ua'
-  autoload :PhoneNumberRU, 'ffaker/phone_number_ru'
-  autoload :Product, 'ffaker/product'
-  autoload :SemVer, 'ffaker/sem_ver'
-  autoload :Skill, 'ffaker/skill'
-  autoload :Sport, 'ffaker/sport'
-  autoload :SportPL, 'ffaker/sport_pl'
-  autoload :SportUS, 'ffaker/sport_us'
-  autoload :SportRU, 'ffaker/sport_ru'
-  autoload :SSN, 'ffaker/ssn'
-  autoload :SSNMX, 'ffaker/ssn_mx'
-  autoload :SSNSE, 'ffaker/ssn_se'
-  autoload :String, 'ffaker/string'
-  autoload :Time, 'ffaker/time'
-  autoload :Tweet, 'ffaker/tweet'
-  autoload :Unit, 'ffaker/unit'
-  autoload :UnitEnglish, 'ffaker/unit_english'
-  autoload :UnitMetric, 'ffaker/unit_metric'
-  autoload :VERSION, 'ffaker/version'
-  autoload :Vehicle, 'ffaker/vehicle'
-  autoload :Venue, 'ffaker/venue'
-  autoload :Youtube, 'ffaker/youtube'
+  autoload_acronyms = {
+    'ar' => 'AR',
+    'au' => 'AU',
+    'aws' => 'AWS',
+    'br' => 'BR',
+    'ca' => 'CA',
+    'ch' => 'CH',
+    'cl' => 'CL',
+    'cn' => 'CN',
+    'co' => 'CO',
+    'cs' => 'CS',
+    'cu' => 'CU',
+    'da' => 'DA',
+    'de' => 'DE',
+    'ec' => 'EC',
+    'en' => 'EN',
+    'es' => 'ES',
+    'fi' => 'FI',
+    'fr' => 'FR',
+    'ga' => 'GA',
+    'gr' => 'GR',
+    'html' => 'HTML',
+    'id' => 'ID',
+    'ie' => 'IE',
+    'in' => 'IN',
+    'it' => 'IT',
+    'ja' => 'JA',
+    'jp' => 'JP',
+    'kh' => 'KH',
+    'kr' => 'KR',
+    'mx' => 'MX',
+    'nb' => 'NB',
+    'nl' => 'NL',
+    'ph' => 'PH',
+    'pl' => 'PL',
+    'ru' => 'RU',
+    'se' => 'SE',
+    'sg' => 'SG',
+    'sn' => 'SN',
+    'ssn' => 'SSN',
+    'th' => 'TH',
+    'tw' => 'TW',
+    'ua' => 'UA',
+    'uk' => 'UK',
+    'us' => 'US',
+    'vn' => 'VN'
+  }
+
+  directory = "#{__dir__}/ffaker"
+  Dir["#{directory}/*.rb"].each do |file_name|
+    relative_file_path = Pathname.new(file_name).relative_path_from(directory).to_s.chomp('.rb')
+    ## Don't consider files in sub-directories
+    constant_name = relative_file_path.split('_').map do |part|
+      autoload_acronyms.fetch(part) { part.capitalize }
+    end.join
+    autoload constant_name, file_name
+  end
 
   # Random Number Generator (RNG) used with ModuleUtils#fetch, #shuffle, #rand
   # in order to provide deterministic repeatability.

data/scripts/reference.rb

@@ -11,8 +11,8 @@ ICONS = {
   warning: '❗'
 }.freeze
 
-UTILS_MODULES = %i[ArrayUtils ModuleUtils RandomUtils Random]
-UTILS_METHODS = %i[k underscore fetch_sample rand shuffle unique luhn_check]
+UTILS_MODULES = %i[ArrayUtils ModuleUtils RandomUtils Random Version].freeze
+UTILS_METHODS = %i[k underscore fetch_sample rand shuffle unique luhn_check].freeze
 
 # Get a list of sections
 def faker_modules
@@ -36,12 +36,14 @@ end
 # Catch deprecation warnings.
 # This `#warn` overrides Kernel#warn
 def warn(msg)
-  $warnings << msg if $warnings
+  return unless Kernel.instance_variable_get(:@ffaker_warnings)
+
+  Kernel.instance_variable_set(:@ffaker_warnings, Kernel.instance_variable_get(:@ffaker_warnings) << msg)
 end
 
 def catch_warnings
-  $warnings = []
-  [yield, $warnings]
+  Kernel.instance_variable_set(:@ffaker_warnings, [])
+  [yield, Kernel.instance_variable_get(:@ffaker_warnings)]
 end
 
 def escape(str)
@@ -70,11 +72,7 @@ sections = faker_modules.map do |mod|
         examples, warnings = catch_warnings do
           Array.new(3) { mod.unique.send meth }
         end
-        right = if warnings.any?
-                  "#{ICONS[:warning]} *#{warnings.first}*"
-                else
-                  (escape examples.join(', ')).to_s
-                end
+        right = warnings.any? ? "#{ICONS[:warning]} *#{warnings.first}*" : (escape examples.join(', ')).to_s
       rescue StandardError => e
         right = "#{ICONS[:error]} #{e.class}: #{e.message}"
       end

data/test/helper.rb

@@ -14,7 +14,7 @@ module DeterministicHelper
   # the internal Random Number Generator state and compared the results of
   # each execution to make sure they are the same.
   def assert_deterministic(options = {}, &block)
-    raise ArgumentError, 'Must pass a block' unless block_given?
+    raise ArgumentError, 'Must pass a block' unless block
 
     options = { message: 'Results are not repeatable' }.merge(options)
 

data/test/test_avatar.rb

@@ -18,40 +18,63 @@ class TestAvatar < Test::Unit::TestCase
                  @tester.image)
   end
 
+  def test_image_output_with_keyword_arguments
+    output = capture_output do
+      @tester.image(format: 'jpg')
+    end
+
+    assert_equal ['', ''], output
+  end
+
+  def test_image_with_slug_as_positional_argument
+    assert_equal("#{ROBOHASH}/foobar.png?size=300x300", @tester.image('foobar'))
+  end
+
+  def test_image_output_with_positional_arguments
+    output = capture_output do
+      @tester.image('foobar')
+    end
+
+    assert_equal(
+      ['', "Positional arguments for Avatar#image are deprecated. Please use keyword arguments.\n"],
+      output
+    )
+  end
+
   def test_avatar_with_param
-    assert_equal("#{ROBOHASH}/faker.png?size=300x300", @tester.image('faker'))
+    assert_equal("#{ROBOHASH}/faker.png?size=300x300", @tester.image(slug: 'faker'))
   end
 
   def test_avatar_with_correct_size
     assert_equal("#{ROBOHASH}/faker.png?size=150x320",
-                 @tester.image('faker', '150x320'))
+                 @tester.image(slug: 'faker', size: '150x320'))
   end
 
   def test_avatar_with_incorrect_size
     assert_raise ArgumentError do
-      @tester.image(nil, '150x320z')
+      @tester.image(size: '150x320z')
     end
   end
 
   def test_avatar_with_supported_format
     assert_equal("#{ROBOHASH}/faker.jpg?size=300x300",
-                 @tester.image('faker', '300x300', 'jpg'))
+                 @tester.image(slug: 'faker', size: '300x300', format: 'jpg'))
   end
 
   def test_avatar_with_incorrect_format
     assert_raise ArgumentError do
-      @tester.image(nil, '300x300', 'wrong_format')
+      @tester.image(size: '300x300', format: 'wrong_format')
     end
   end
 
   def test_avatar_with_correct_background
     assert_equal("#{ROBOHASH}/faker.png?size=300x300&bgset=bg1",
-                 @tester.image('faker', '300x300', 'png', '1'))
+                 @tester.image(slug: 'faker', size: '300x300', format: 'png', bgset: '1'))
   end
 
   def test_avatar_with_incorrect_background
     assert_raise ArgumentError do
-      @tester.image('faker', '300x300', 'png', 'not_a_number')
+      @tester.image(slug: 'faker', size: '300x300', format: 'png', bgset: 'not_a_number')
     end
   end
 end

data/test/test_bank_us.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require_relative 'helper'
+
+class TestBankUS < Test::Unit::TestCase
+  include DeterministicHelper
+
+  assert_methods_are_deterministic(
+    FFaker::BankUS,
+    :account_number, :routing_number
+  )
+
+  def setup
+    @tester = FFaker::BankUS
+  end
+
+  def test_account_number
+    assert_instance_of String, @tester.account_number
+    assert_match(/\A\d{8}\z/, @tester.account_number(min_digits: 8, max_digits: 8))
+    assert_match(/\A\d{12}\z/, @tester.account_number(min_digits: 12, max_digits: 12))
+  end
+
+  def test_routing_number
+    routing_number = @tester.routing_number
+    assert_match(/\A\d{9}\z/, routing_number)
+
+    checksum = (
+      (7 * (routing_number[0].to_i + routing_number[3].to_i + routing_number[6].to_i)) +
+        (3 * (routing_number[1].to_i + routing_number[4].to_i + routing_number[7].to_i)) +
+        (9 * (routing_number[2].to_i + routing_number[5].to_i + routing_number[8].to_i))
+    )
+
+    assert_equal(0, checksum % 10, 'The routing number\'s checksum is not a multiple of ten')
+  end
+end

data/test/test_book.rb

@@ -39,6 +39,35 @@ class TestBook < Test::Unit::TestCase
                  @tester.cover)
   end
 
+  def test_cover_with_format
+    assert_match(%r{\Ahttps://robohash\.org/.+\.jpg\?size=300x300\z},
+                 @tester.cover(format: 'jpg'))
+  end
+
+  def test_cover_output_with_keyword_arguments
+    output = capture_output do
+      @tester.cover(format: 'jpg')
+    end
+
+    assert_equal ['', ''], output
+  end
+
+  def test_cover_with_slug_as_positional_argument
+    assert_match(%r{\Ahttps://robohash\.org/foobar\.png\?size=300x300\z},
+                 @tester.cover('foobar'))
+  end
+
+  def test_cover_output_with_positional_arguments
+    output = capture_output do
+      @tester.cover('foobar')
+    end
+
+    assert_equal(
+      ['', "Positional arguments for Book#cover are deprecated. Please use keyword arguments.\n"],
+      output
+    )
+  end
+
   def test_orly_cover
     assert_match(%r{\Ahttps://orly-appstore\.herokuapp\.com/generate},
                  @tester.orly_cover)

data/test/test_crypto.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require_relative 'helper'
+
+class TestCrypto < Test::Unit::TestCase
+  include DeterministicHelper
+
+  assert_methods_are_deterministic(FFaker::Crypto, :sha256)
+
+  def test_sha256
+    assert FFaker::Crypto.sha256.is_a?(String)
+    assert FFaker::Crypto.sha256.length == 64
+    assert_match(/\A[a-z\d]{64}\z/, FFaker::Crypto.sha256)
+  end
+end

data/test/test_date.rb

@@ -9,7 +9,7 @@ class TestFakerDate < Test::Unit::TestCase
     @tester = FFaker::Date
   end
 
-  assert_methods_are_deterministic(FFaker::Date, :backward, :forward)
+  assert_methods_are_deterministic(FFaker::Date, :backward, :forward, :birthday)
 
   def test_between
     from = Date.new(2015, 1, 1)
@@ -34,4 +34,14 @@ class TestFakerDate < Test::Unit::TestCase
     assert_random_between(today + 1..today + 30) { @tester.forward(30) }
     assert_instance_of Date, @tester.forward
   end
+
+  def test_birthday
+    today = Date.today
+
+    assert_random_between(today.prev_year(65).next_day..today.prev_year(18)) { @tester.birthday }
+    assert_random_between(today.prev_year(43).next_day..today.prev_year(42)) do
+      @tester.birthday(min_age: 42, max_age: 42)
+    end
+    assert_instance_of Date, @tester.birthday
+  end
 end

data/test/test_filesystem.rb

@@ -30,7 +30,33 @@ class TestFakerFilesystem < Test::Unit::TestCase
   end
 
   def test_file_name
-    assert_match %r{\A(?:[a-z_-]+[\\/])+[a-z_-]+\.\w{2,4}\z},
-                 @tester.file_name
+    assert_match %r{\A(?:[a-z_-]+[\\/])+[a-z_-]+\.\w{2,4}\z}, @tester.file_name
+  end
+
+  def test_file_name_with_extension
+    assert_match %r{\A(?:[a-z_-]+[\\/])+[a-z_-]+\.rb\z}, @tester.file_name(ext: 'rb')
+  end
+
+  def test_file_name_output_with_keyword_arguments
+    output = capture_output do
+      @tester.file_name(ext: 'rb')
+    end
+
+    assert_equal ['', ''], output
+  end
+
+  def test_file_name_with_directory_as_positional_argument
+    assert_match %r{\Asome_directory/[a-z_-]+\.\w{2,4}\z}, @tester.file_name('some_directory')
+  end
+
+  def test_file_name_output_with_positional_arguments
+    output = capture_output do
+      @tester.file_name('some_directory')
+    end
+
+    assert_equal(
+      ['', "Positional arguments for Filesystem#file_name are deprecated. Please use keyword arguments.\n"],
+      output
+    )
   end
 end

data/test/test_healthcare_ru.rb

@@ -5,7 +5,7 @@ require_relative 'helper'
 class TestHealthcareRU < Test::Unit::TestCase
   include DeterministicHelper
 
-  SPECIALIZATION_REGEX = /\A[А-Яа-я\ ()\-]+\z/
+  SPECIALIZATION_REGEX = /\A[А-Яа-я\ ()-]+\z/
 
   assert_methods_are_deterministic(FFaker::HealthcareRU, :doctor_specialization)
 

data/test/test_identification.rb

@@ -15,7 +15,6 @@ class TestFakerIdentification < Test::Unit::TestCase
   end
 
   def test_drivers_license
-    ###-###-##-###-#
     drivers_license_regex = /\A[A-Z]\d{3}-\d{3}-\d{2}-\d{3}-\d{1}\z/
     assert_match(drivers_license_regex, @tester.drivers_license)
   end
@@ -29,7 +28,18 @@ class TestFakerIdentification < Test::Unit::TestCase
   end
 
   def test_ethnicity
-    ethnicity_regex = %r{\A(African American|Asian/Pacific Islander|Caucasian|Hispanic|Native American|Multiracial|Other|Prefer not to respond)\z}
+    ethnicity_regex = %r{
+      \A(
+        African\ American|
+        Asian/Pacific\ Islander|
+        Caucasian|
+        Hispanic|
+        Native\ American|
+        Multiracial|
+        Other|
+        Prefer\ not\ to\ respond
+      )\z
+    }x
     assert_match(ethnicity_regex, @tester.ethnicity)
   end
 end

data/test/test_identification_ec.rb

@@ -27,7 +27,7 @@ class TestIdentificationEC < Test::Unit::TestCase
       end
     end
     mod = sum % 10
-    mod = 10 - mod if mod > 0
+    mod = 10 - mod if mod.positive?
     assert(ci[9].to_i == mod)
   end
 end

data/test/test_identification_es_mx.rb

@@ -22,7 +22,9 @@ class TestIdentificationMX < Test::Unit::TestCase
 
   def test_curp
     or_curp_states_abbr = FFaker::IdentificationMX::ESTADOS_CURP.join('|')
-    re = /\A[a-z][aeioux][a-z]{2}\d{2}[0-1]\d[0-3]\d[hm](?:#{or_curp_states_abbr})[bcdfghjklmñpqrstvwxyz]{3}[0-9a-z]\d\z/ui
+    re = /
+      \A[a-z][aeioux][a-z]{2}\d{2}[0-1]\d[0-3]\d[hm](?:#{or_curp_states_abbr})[bcdfghjklmñpqrstvwxyz]{3}[0-9a-z]\d\z
+    /uix
     assert_match(re, FFaker::IdentificationMX.curp)
   end
 end

data/test/test_identification_it.rb

@@ -28,7 +28,18 @@ class TestFakerIdentificationIT < Test::Unit::TestCase
   end
 
   def test_ethnicity
-    ethnicity_regex = %r{\A(Afroamericano|Asiatico/isolano del Pacifico|Caucasico|Ispanico|Nativo americano|Multirazziale|Altro|Preferisco non rispondere)\z}
+    ethnicity_regex = %r{
+      \A(
+        Afroamericano|
+        Asiatico/isolano\ del\ Pacifico|
+        Caucasico|
+        Ispanico|
+        Nativo\ americano|
+        Multirazziale|
+        Altro|
+        Preferisco\ non\ rispondere
+      )\z
+    }x
     assert_match(ethnicity_regex, @tester.ethnicity)
   end
 end

data/test/test_identification_kr.rb

@@ -2,13 +2,13 @@
 
 require_relative 'helper'
 
-class TestFakerIdentificationKr < Test::Unit::TestCase
+class TestFakerIdentificationKR < Test::Unit::TestCase
   include DeterministicHelper
 
-  assert_methods_are_deterministic(FFaker::IdentificationKr, :rrn)
+  assert_methods_are_deterministic(FFaker::IdentificationKR, :rrn)
 
   def setup
-    @tester = FFaker::IdentificationKr
+    @tester = FFaker::IdentificationKR
   end
 
   def test_rrn