Validations for reward model

app/controllers/pledges_controller.rb

@@ -1,5 +1,5 @@
 class PledgesController < ApplicationController
-  before_action :require_login
+  # before_action :not_authenticated
 
   def create
     @project = Project.find(params[:project_id])
@@ -15,5 +15,5 @@ def create
       render 'projects/show'
     end
   end
-  
+
 end

app/controllers/projects_controller.rb

@@ -1,5 +1,5 @@
 class ProjectsController < ApplicationController
-  before_action :require_login, only: [:new, :create]
+  before_action :not_authenticated, only: [:new, :create]
 
   def index
     @projects = Project.all

app/models/pledge.rb

@@ -1,7 +1,17 @@
 class Pledge < ApplicationRecord
   belongs_to :user
   belongs_to :project
+  #project.user_id != pledge.user_id
 
   validates :dollar_amount, presence: true
   validates :user, presence: true
+  validate :validate_backer_is_not_owner
+  validates :dollar_amount, :numericality => { :greater_than => 0 }
+
+  def validate_backer_is_not_owner
+    if project.user == user
+      errors.add(:user_id, 'Owner cant back project')
+    end
+  end
+
 end

app/models/project.rb

@@ -5,4 +5,5 @@ class Project < ActiveRecord::Base
   belongs_to :user # project owner
 
   validates :title, :description, :goal, :start_date, :end_date, presence: true
+  validates :user, presence: true
 end

app/models/reward.rb

@@ -1,3 +1,12 @@
 class Reward < ActiveRecord::Base
+
+  # Reward dollar_amount must be positive number
+  validates :dollar_amount, :numericality => { :greater_than => 0 }
+  validates :description, presence: true
+  validates :dollar_amount, presence: true
+
   belongs_to :project
+
+
+
 end

app/views/projects/_form.html.erb

@@ -1,3 +1,11 @@
+<% if @projects.errors.full_messages.any? %>
+  <% @projects.errors.full_messages.each do |msg| %>
+    <ul>
+      <li><%= msg %></li>
+    </ul>
+  <% end %>
+<% end %>
+
 <%= form_for(@project, html: {multipart: true}) do |f| %>
   <div>
     <%= f.label :title %>

test/models/pledge_test.rb

@@ -19,7 +19,7 @@ class PledgeTest < ActiveSupport::TestCase
     project = new_project
     project.user = owner
     project.save
-    pledge = Pledge.new(dollar_amount: 3.00, project: project)
+    pledge = Pledge.new( dollar_amount: 3.00, project: project)
     pledge.user = owner
     pledge.save
     assert pledge.invalid?, 'Owner should not be able to pledge towards own project'

test/models/reward_test.rb

@@ -36,6 +36,17 @@ class RewardTest < ActiveSupport::TestCase
     assert reward.new_record?, 'Reward should not save without a description'
   end
 
+  test 'Reward dollar_amount must be positive number' do
+    project = new_project
+    project.save
+    reward = Reward.create(
+      dollar_amount: -1,
+      description: 'Motherlover',
+      project: project
+    )
+    assert reward.invalid?
+  end
+
   def new_project
     Project.new(
       title:       'Cool new boardgame',