From a9a3feb383f307fbf3f36a27a794717d7c06602f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Paulo=20Diovani=20Gon=C3=A7alves?= <paulo@diovani.com>
Date: Thu, 21 Feb 2019 15:19:54 -0300
Subject: [PATCH] Chore/improve pt-BR specs (#1548)

* Improve pt-br address tests

* Fix and improve pt-br color methods

* Improve tests for email and license plate

* Use .match() method in tests
---
 test/test_pt_br_locale.rb | 41 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 40 insertions(+), 1 deletion(-)

diff --git a/test/test_pt_br_locale.rb b/test/test_pt_br_locale.rb
index 889ec2a03f..2a4bad5a65 100644
--- a/test/test_pt_br_locale.rb
+++ b/test/test_pt_br_locale.rb
@@ -13,12 +13,23 @@ def teardown
 
   def test_pt_br_address_methods
     assert Faker::Address.city.is_a? String
+
     assert Faker::Address.state_abbr.is_a? String
+    assert Faker::Address.state_abbr.match(/^[A-Z]{2}$/)
+
     assert Faker::Address.country.is_a? String
+
     assert Faker::Address.building_number.is_a? String
+    assert Faker::Address.building_number.match(/^([0-9]+)|(s\/n)$/)
+
     assert Faker::Address.street_suffix.is_a? String
+
     assert Faker::Address.secondary_address.is_a? String
+    assert Faker::Address.secondary_address.match(/^[a-zA-Z\.]+\s[0-9]+$/)
+
     assert Faker::Address.postcode.is_a? String
+    assert Faker::Address.postcode.match(/^[0-9]{5}\-[0-9]{3}$/)
+
     assert Faker::Address.state.is_a? String
     assert Faker::Address.city.is_a? String
     assert Faker::Address.street_name.is_a? String
@@ -26,7 +37,30 @@ def test_pt_br_address_methods
   end
 
   def test_pt_br_color_methods
-    assert Faker::Color.name.is_a? String
+    assert Faker::Color.hex_color.is_a? String
+    assert Faker::Color.hex_color.match(/^#([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})$/)
+
+    assert Faker::Color.color_name.is_a? String
+
+    assert Faker::Color.rgb_color.is_a? Array
+    Faker::Color.rgb_color.each do |value|
+      assert value >= 0 && value <= 255
+    end
+
+    assert Faker::Color.hsl_color.is_a? Array
+    hue = Faker::Color.hsl_color[0]
+    assert hue >= 0 && hue <= 360
+    Faker::Color.hsl_color[1..2].each do |value|
+      assert value >= 0 && value <= 1
+    end
+
+    assert Faker::Color.hsla_color.is_a? Array
+    assert Faker::Color.hsl_color.is_a? Array
+    hue = Faker::Color.hsl_color[0]
+    assert hue >= 0 && hue <= 360
+    Faker::Color.hsl_color[1..3].each do |value|
+      assert value >= 0 && value <= 1
+    end
   end
 
   def test_pt_br_company_methods
@@ -50,6 +84,8 @@ def test_pt_br_food_methods
 
   def test_pt_br_internet_methods
     assert Faker::Internet.free_email.is_a? String
+    assert Faker::Internet.free_email.match(/^[a-z0-9._\-]+@[a-z0-9]+.[a-z]+.([a-z]+)?$/i)
+
     assert Faker::Internet.domain_suffix.is_a? String
   end
 
@@ -93,7 +129,10 @@ def test_pt_br_university_methods
 
   def test_pt_br_vehicle_methods
     assert Faker::Vehicle.license_plate.is_a? String
+    assert Faker::Vehicle.license_plate.match(/^[A-Z]{3}\-[0-9]{4}/)
+
     assert Faker::Vehicle.license_plate('RJ').is_a? String
+    assert Faker::Vehicle.license_plate('RJ').match(/^[A-Z]{3}\-[0-9]{4}/)
   end
 
   def test_pt_br_gender_methods