Add error when attribute is present but value is nil and has no default

UPGRADING.md

@@ -1,5 +1,7 @@
 ## HEAD
 
+* No longer raises `Pavlov::ValidationError, "Missing arguments:` when an argument is nil but adds an error.
+
 # 0.1.8
 
 * Let custom validations use errors.add instead of raising exceptions

lib/pavlov/operation.rb

@@ -51,13 +51,15 @@ def raise_unauthorized(message = 'Unauthorized')
     end
 
     def check_validation
-      fail Pavlov::ValidationError, "Missing arguments: #{missing_arguments.inspect}" if missing_arguments.any?
+      check_missing_arguments
       validate
     end
 
-    def missing_arguments
-      attribute_set.select do |attribute|
-        !attribute.options.key?(:default) && send(attribute.name).nil?
+    def check_missing_arguments
+      attribute_set.each do |attribute|
+        if !attribute.options.key?(:default) && send(attribute.name).nil?
+          errors.add(attribute.name.to_s, "can't be blank")
+        end
       end
     end
 

spec/regression/case2_spec.rb

@@ -28,7 +28,7 @@ def list
       def members_to_remove
         list.zrange(0, -1).select do |id|
           activity = Activity[id]
-          ! valid(activity)
+          !valid(activity)
         end
       end
 

spec/regression/case4_spec.rb

@@ -15,7 +15,7 @@ class Interactors::ExampleModule::FollowUser
       arguments :user_name, :user_to_follow_user_name
 
       def authorized?
-        (!! pavlov_options[:current_user]) && (pavlov_options[:current_user].username == user_name)
+        (!!pavlov_options[:current_user]) && (pavlov_options[:current_user].username == user_name)
       end
 
       def user

spec/regression/case5_spec.rb

@@ -15,7 +15,7 @@ class Interactors::ExampleModule::Followers
       arguments :user_name, :skip, :take
 
       def authorized?
-        !! pavlov_options[:current_user]
+        !!pavlov_options[:current_user]
       end
 
       def validate