From 900d8a00b1fcabca3070f6c122d2de57a8bc505e Mon Sep 17 00:00:00 2001 From: Bertocq Date: Fri, 15 Dec 2017 16:38:21 +0100 Subject: [PATCH] Make milestone's description a required attribute --- app/models/budget/investment/milestone.rb | 2 ++ spec/models/budget/investment/milestone_spec.rb | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/app/models/budget/investment/milestone.rb b/app/models/budget/investment/milestone.rb index 13724be21..98ea506e9 100644 --- a/app/models/budget/investment/milestone.rb +++ b/app/models/budget/investment/milestone.rb @@ -10,12 +10,14 @@ class Budget belongs_to :investment validates :title, presence: true + validates :description, presence: true validates :investment, presence: true validates :publication_date, presence: true def self.title_max_length 80 end + end end end diff --git a/spec/models/budget/investment/milestone_spec.rb b/spec/models/budget/investment/milestone_spec.rb index 32f040eaf..c66c9cd39 100644 --- a/spec/models/budget/investment/milestone_spec.rb +++ b/spec/models/budget/investment/milestone_spec.rb @@ -14,6 +14,11 @@ describe "Budget::Investment::Milestone" do expect(milestone).to_not be_valid end + it "Should not be valid without a description" do + milestone.description = nil + expect(milestone).to_not be_valid + end + it "Should not be valid without an investment" do milestone.investment_id = nil expect(milestone).to_not be_valid