Add rule priority to allow rules to validate data before rules that w…

…ill radically change data (#16)

lib/mini_defender/rule.rb

@@ -16,6 +16,13 @@ def self.available?
     true
   end
 
+  # Priority is used to sort rules in a rule set to allow rules to validate data
+  # before rules that will radically change data
+  # @return [Integer]
+  def priority
+    100
+  end
+
   # @param [MiniDefender::Validator] validator
   # @return [Boolean]
   def implicit?(validator)

lib/mini_defender/rules/array.rb

@@ -13,6 +13,10 @@ def self.make(args)
     new(args[0] || 'none')
   end
 
+  def priority
+    500
+  end
+
   def coerce(value)
     @data_mode == 'all' ? value : []
   end

lib/mini_defender/rules/hash.rb

@@ -19,6 +19,10 @@ def self.make(args)
     )
   end
 
+  def priority
+    500
+  end
+
   def coerce(value)
     @data_mode == 'all' ? value : {}
   end

lib/mini_defender/rules_factory.rb

@@ -11,16 +11,23 @@ def self.register(klass)
     end
 
     def init_set(rule_set)
-      rule_set = rule_set.split('|') if rule_set.is_a?(String)
-      raise ArgumentError, 'Rule set must be a string or an array' unless rule_set.is_a?(Array)
+      if rule_set.is_a?(String)
+        rule_set = rule_set.split('|')
+      end
+
+      unless rule_set.is_a?(Array)
+        raise ArgumentError, 'Rule set must be a string or an array'
+      end
 
-      rule_set.map do |rule|
+      rule_set = rule_set.map do |rule|
         unless rule.is_a?(String) || rule.is_a?(Rule)
           raise ArgumentError, 'Rule must be a string or an instance of MiniDefender::Rule'
         end
 
         rule.is_a?(String) ? init_rule(rule) : rule
       end
+
+      rule_set.sort_by! { |r| r.priority }
     end
 
     def init_rule(signature)

lib/mini_defender/validator.rb

@@ -62,7 +62,7 @@ def validate
 
           value_included &= !rule.excluded?(self)
 
-          if rule.passes?(k, value, self)
+          if rule.passes?(k, coerced, self)
             coerced = rule.coerce(coerced)
             force_coerce = rule.force_coerce?
           else

lib/mini_defender/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MiniDefender
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 end

test/rules/hash_test.rb

@@ -147,4 +147,25 @@ def test_hash_must_return_all_elements_when_all_mode_is_used_2
     assert_equal 'foo', validator.coerced['metadata']['1']
     assert_equal 'bar', validator.coerced['metadata']['2']
   end
+
+  def test_hash_should_be_delayed_to_all_size_rules_validation
+    validator = MiniDefender::Validator.new(
+      {
+        'metadata' => 'required|hash|size:2|max:2|min:2',
+        'metadata.foo' => 'required|string',
+        'metadata.zee' => 'required|string'
+      },
+      {
+        'metadata' => {
+          'foo' => 'bar',
+          'zee' => 'mee'
+        }
+      }
+    )
+
+    assert validator.passes?
+
+    assert_equal 2, validator.coerced['metadata'].length
+    assert_equal 'bar', validator.coerced['metadata']['foo']
+  end
 end