don't allow transfers of 0 hours and 0 minutes

Author: markets

app/assets/javascripts/give_time.js.coffee

@@ -5,8 +5,13 @@ jQuery.validator.addMethod "either-hours-minutes-informed", ((value, element) ->
 giveTimeReadyFn = () ->
   config =
     submitHandler: (form) ->
-      $(" #transfer_amount ").val($(" #transfer_hours ").val() * 3600 + $(" #transfer_minutes ").val() * 60)
-      form.submit()
+      amount = $("#transfer_hours").val() * 3600 + $("#transfer_minutes").val() * 60
+      $("#transfer_amount").val(amount)
+
+      if amount > 0
+        form.submit()
+      else
+        $(form).find('input:submit').after(' <label class="error">Time must be greater than 0</label>')
 
   $( "#new_transfer" ).validate(config)
 

app/controllers/transfers_controller.rb

@@ -12,7 +12,7 @@ def create
     if persister.save
       redirect_to redirect_target
     else
-      flash[:error] = transfer.errors.full_messages.to_sentence
+      redirect_to :back, alert: transfer.errors.full_messages.to_sentence
     end
   end
 

app/models/transfer.rb

@@ -17,6 +17,7 @@ class Transfer < ActiveRecord::Base
   belongs_to :operator, class_name: "User"
   has_many :movements
 
+  validates :amount, numericality: { greater_than: 0 }
   validate :different_source_and_destination
 
   after_create :make_movements
@@ -26,14 +27,6 @@ def make_movements
     movements.create(account: Account.find(destination_id), amount: amount.to_i)
   end
 
-  def movement_from
-    movements.detect {|m| m.amount < 0 }
-  end
-
-  def movement_to
-    movements.detect {|m| m.amount > 0 }
-  end
-
   def source_id
     source.respond_to?(:id) ? source.id : source
   end

app/views/transfers/new.html.erb

@@ -36,7 +36,7 @@
   </div>
 
   <div class="form-actions">
-    <%= f.button :submit, class: "btn btn-default", style: "margin-bottom: 20px;" %>
+    <%= f.button :submit, class: "btn btn-default" %>
   </div>
 <% end %>
 

spec/controllers/transfers_controller_spec.rb

@@ -207,5 +207,22 @@
         end
       end
     end
+
+    context 'with invalid params' do
+      let(:user) { member_giver.user }
+      let(:referer) { "/transfers/new?destination_account_id=#{member_taker.account.id}" }
+
+      before do
+        request.env["HTTP_REFERER"] = referer
+      end
+
+      it 'does not create any Transfer and redirects to :back if the amount is 0' do
+        expect {
+          post(:create, transfer: { amount: 0, destination: member_taker.account.id })
+        }.not_to change(Transfer, :count)
+
+        expect(response).to redirect_to(referer)
+      end
+    end
   end
 end