Correct HSL and HSLA color formatting

lib/faker/color.rb

@@ -22,31 +22,17 @@ def rgb_color
         @rgb_colors
       end
 
-      def single_hsl_color
-        @single_hsla_color = Faker::Base::rand_in_range(0.0, 360.00).round(2)
-        @single_hsla_color
-      end
-
-      def alpha_channel
-        @alpha_channel = rand
-        @alpha_channel
-      end
-
       def hsl_color
-        @hsl_colors = []
-        3.times do
-          @hsl_colors.push single_hsl_color
-        end
-        @hsl_colors
+        hsl_values = []
+        hsl_values << (0..360).to_a.sample
+        2.times { hsl_values << rand.round(2) }
+        hsl_values
       end
 
       def hsla_color
-        @hsla_colors = []
-        3.times do
-          @hsla_colors.push single_hsl_color
-        end
-        @hsla_colors.push alpha_channel
-        @hsla_colors
+        hsla_values = hsl_color
+        hsla_values << rand.round(1)
+        hsla_values
       end
     end
   end

test/test_faker_color.rb

@@ -26,25 +26,24 @@ def test_rgb_color
     end
   end
 
-  def test_single_hsl_color
-    assert @tester.single_hsl_color.between?(0.0, 360.0)
-  end
-
   def test_hsl_color
     @result = @tester.hsl_color
     assert @result.length == 3
 
-    @result.each do |color|
-      assert color.between?(0.0, 360.0)
-    end
+    assert @result[0].between?(0, 360)
+    assert @result[0].is_a?(Fixnum)
+
+    assert @result[1].between?(0.0, 1.0)
+    assert @result[2].between?(0.0, 1.0)
   end
 
   def test_hsla_color
     @result = @tester.hsla_color
     assert @result.length == 4
 
-    @result.each do |color|
-      assert color.between?(0.0, 360.0) || color.between?(0.0, 1.0)
-    end
+    assert @result[0].between?(0, 360)
+    assert @result[1].between?(0.0, 1.0)
+    assert @result[2].between?(0.0, 1.0)
+    assert @result[3].between?(0.0, 1.0)
   end
 end