Allow ArrayValidator to use other validators to validate array elements

Author: Josh Mostafa

lib/apipie/validator.rb

@@ -225,7 +225,13 @@ def validate_item(value)
 
       def has_valid_type?(value)
         if @items_type
-          value.kind_of?(@items_type)
+          item_validator = BaseValidator.find('', @items_type, nil, nil)
+
+          if item_validator
+            item_validator.valid?(value)
+          else
+            value.kind_of?(@items_type)
+          end
         else
           true
         end

spec/lib/validators/array_validator_spec.rb

@@ -24,18 +24,38 @@ module Apipie::Validator
     end
 
     context "with a constraint on items type" do
-      let(:validator) { ArrayValidator.new(param_desc, Array, :of => String) }
+      let(:validator) { ArrayValidator.new(param_desc, Array, :of => type) }
 
-      it "accepts array of specified type" do
-        expect(validator.validate(['string1', 'string2'])).to eq(true)
-      end
+      context "String" do
+        let(:type) { String }
 
-      it "accepts empty array" do
-        expect(validator.validate([])).to eq(true)
+        it "accepts array of specified type" do
+          expect(validator.validate(['string1', 'string2'])).to eq(true)
+        end
+
+        it "accepts empty array" do
+          expect(validator.validate([])).to eq(true)
+        end
+
+        it "does not accepts array with other types" do
+          expect(validator.validate(['string1', true])).to eq(false)
+        end
       end
 
-      it "does not accepts array with other types" do
-        expect(validator.validate(['string1', true])).to eq(false)
+      context ":number" do
+        let(:type) { :decimal }
+
+        it "accepts array of specified type" do
+          expect(validator.validate([12, '14'])).to eq(true)
+        end
+
+        it "accepts empty array" do
+          expect(validator.validate([])).to eq(true)
+        end
+
+        it "does not accepts array with other types" do
+          expect(validator.validate([12, 'potato'])).to eq(false)
+        end
       end
     end