From 6974b7d03af8cb48554d7029d2d0d1f82bc98e2f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Wed, 28 Nov 2018 14:26:30 +0100 Subject: [PATCH] Remove redundant specs The same cases are tested in the previous `describe` block. --- spec/models/milestone_spec.rb | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/spec/models/milestone_spec.rb b/spec/models/milestone_spec.rb index 7a1817810..92cf96dd4 100644 --- a/spec/models/milestone_spec.rb +++ b/spec/models/milestone_spec.rb @@ -42,28 +42,6 @@ describe Milestone do end end - describe "#description_or_status_present?" do - let(:milestone) { build(:milestone) } - - it "is not valid when status is removed and there's no description" do - milestone.update(description: nil) - expect(milestone.update(status_id: nil)).to be false - end - - it "is not valid when description is removed and there's no status" do - milestone.update(status_id: nil) - expect(milestone.update(description: nil)).to be false - end - - it "is valid when description is removed and there is a status" do - expect(milestone.update(description: nil)).to be true - end - - it "is valid when status is removed and there is a description" do - expect(milestone.update(status_id: nil)).to be true - end - end - describe ".published" do it "uses the application's time zone date", :with_different_time_zone do published_in_local_time_zone = create(:milestone,