2012-10-09 04:14:17 -04:00
|
|
|
# == Schema Information
|
|
|
|
#
|
|
|
|
# Table name: milestones
|
|
|
|
#
|
2012-11-19 13:24:05 -05:00
|
|
|
# id :integer not null, primary key
|
|
|
|
# title :string(255) not null
|
|
|
|
# project_id :integer not null
|
2012-10-09 04:14:17 -04:00
|
|
|
# description :text
|
|
|
|
# due_date :date
|
2014-04-09 08:05:03 -04:00
|
|
|
# created_at :datetime
|
|
|
|
# updated_at :datetime
|
2013-03-15 09:16:02 -04:00
|
|
|
# state :string(255)
|
2013-08-21 05:34:02 -04:00
|
|
|
# iid :integer
|
2012-10-09 04:14:17 -04:00
|
|
|
#
|
|
|
|
|
2012-04-08 17:28:58 -04:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2015-12-09 04:50:51 -05:00
|
|
|
describe Milestone, models: true do
|
2012-04-08 18:01:42 -04:00
|
|
|
describe "Associations" do
|
2015-02-12 13:17:35 -05:00
|
|
|
it { is_expected.to belong_to(:project) }
|
|
|
|
it { is_expected.to have_many(:issues) }
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
describe "Validation" do
|
2015-05-21 17:49:06 -04:00
|
|
|
before do
|
|
|
|
allow(subject).to receive(:set_iid).and_return(false)
|
|
|
|
end
|
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
it { is_expected.to validate_presence_of(:title) }
|
|
|
|
it { is_expected.to validate_presence_of(:project) }
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
|
|
|
|
2012-11-05 22:31:55 -05:00
|
|
|
let(:milestone) { create(:milestone) }
|
|
|
|
let(:issue) { create(:issue) }
|
2012-04-08 18:01:42 -04:00
|
|
|
|
2016-02-02 04:44:30 -05:00
|
|
|
describe "unique milestone title per project" do
|
|
|
|
it "shouldn't accept the same title in a project twice" do
|
|
|
|
new_milestone = Milestone.new(project: milestone.project, title: milestone.title)
|
|
|
|
expect(new_milestone).not_to be_valid
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should accept the same title in another project" do
|
|
|
|
project = build(:project)
|
|
|
|
new_milestone = Milestone.new(project: project, title: milestone.title)
|
|
|
|
|
|
|
|
expect(new_milestone).to be_valid
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2012-08-25 13:54:38 -04:00
|
|
|
describe "#percent_complete" do
|
|
|
|
it "should not count open issues" do
|
2012-04-08 18:01:42 -04:00
|
|
|
milestone.issues << issue
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.percent_complete).to eq(0)
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
|
|
|
|
2012-08-25 13:54:38 -04:00
|
|
|
it "should count closed issues" do
|
2013-02-18 04:10:09 -05:00
|
|
|
issue.close
|
2012-08-25 13:54:38 -04:00
|
|
|
milestone.issues << issue
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.percent_complete).to eq(100)
|
2012-08-25 13:54:38 -04:00
|
|
|
end
|
2012-04-08 18:01:42 -04:00
|
|
|
|
2012-08-25 13:54:38 -04:00
|
|
|
it "should recover from dividing by zero" do
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.issues).to receive(:count).and_return(0)
|
2015-05-26 14:44:04 -04:00
|
|
|
expect(milestone.percent_complete).to eq(0)
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2012-08-25 13:54:38 -04:00
|
|
|
describe "#expires_at" do
|
|
|
|
it "should be nil when due_date is unset" do
|
|
|
|
milestone.update_attributes(due_date: nil)
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.expires_at).to be_nil
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
|
|
|
|
2012-08-25 13:54:38 -04:00
|
|
|
it "should not be nil when due_date is set" do
|
|
|
|
milestone.update_attributes(due_date: Date.tomorrow)
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.expires_at).to be_present
|
2012-08-25 13:54:38 -04:00
|
|
|
end
|
2012-04-08 18:01:42 -04:00
|
|
|
end
|
2013-01-03 12:11:14 -05:00
|
|
|
|
|
|
|
describe :expired? do
|
|
|
|
context "expired" do
|
|
|
|
before do
|
2015-05-21 17:49:06 -04:00
|
|
|
allow(milestone).to receive(:due_date).and_return(Date.today.prev_year)
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
it { expect(milestone.expired?).to be_truthy }
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
context "not expired" do
|
|
|
|
before do
|
2015-05-21 17:49:06 -04:00
|
|
|
allow(milestone).to receive(:due_date).and_return(Date.today.next_year)
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
it { expect(milestone.expired?).to be_falsey }
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :percent_complete do
|
|
|
|
before do
|
2015-05-21 17:49:06 -04:00
|
|
|
allow(milestone).to receive_messages(
|
2013-01-03 12:11:14 -05:00
|
|
|
closed_items_count: 3,
|
|
|
|
total_items_count: 4
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
it { expect(milestone.percent_complete).to eq(75) }
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
describe :items_count do
|
|
|
|
before do
|
|
|
|
milestone.issues << create(:issue)
|
2013-02-18 04:10:09 -05:00
|
|
|
milestone.issues << create(:closed_issue)
|
2013-01-03 12:11:14 -05:00
|
|
|
milestone.merge_requests << create(:merge_request)
|
|
|
|
end
|
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
it { expect(milestone.closed_items_count).to eq(1) }
|
|
|
|
it { expect(milestone.open_items_count).to eq(2) }
|
|
|
|
it { expect(milestone.total_items_count).to eq(3) }
|
|
|
|
it { expect(milestone.is_empty?).to be_falsey }
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
describe :can_be_closed? do
|
2015-02-12 13:17:35 -05:00
|
|
|
it { expect(milestone.can_be_closed?).to be_truthy }
|
2013-01-03 12:11:14 -05:00
|
|
|
end
|
2013-02-18 04:38:29 -05:00
|
|
|
|
|
|
|
describe :is_empty? do
|
2013-02-18 08:52:39 -05:00
|
|
|
before do
|
2015-10-03 02:48:54 -04:00
|
|
|
create :closed_issue, milestone: milestone
|
|
|
|
create :merge_request, milestone: milestone
|
2013-02-18 08:52:39 -05:00
|
|
|
end
|
2013-02-18 04:38:29 -05:00
|
|
|
|
2013-02-18 08:52:39 -05:00
|
|
|
it 'Should return total count of issues and merge requests assigned to milestone' do
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.total_items_count).to eq 2
|
2013-02-18 04:38:29 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :can_be_closed? do
|
2013-02-18 08:52:39 -05:00
|
|
|
before do
|
2013-02-18 04:38:29 -05:00
|
|
|
milestone = create :milestone
|
2013-02-18 08:52:39 -05:00
|
|
|
create :closed_issue, milestone: milestone
|
|
|
|
|
2015-10-03 02:48:54 -04:00
|
|
|
create :issue
|
2013-02-18 08:52:39 -05:00
|
|
|
end
|
2013-02-18 04:38:29 -05:00
|
|
|
|
2013-07-29 06:46:00 -04:00
|
|
|
it 'should be true if milestone active and all nested issues closed' do
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.can_be_closed?).to be_truthy
|
2013-02-18 04:38:29 -05:00
|
|
|
end
|
|
|
|
|
2013-07-29 06:46:00 -04:00
|
|
|
it 'should be false if milestone active and not all nested issues closed' do
|
2013-02-18 08:52:39 -05:00
|
|
|
issue.milestone = milestone
|
2013-03-20 17:46:30 -04:00
|
|
|
issue.save
|
2013-02-18 04:38:29 -05:00
|
|
|
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(milestone.can_be_closed?).to be_falsey
|
2013-02-18 04:38:29 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-10-15 12:10:35 -04:00
|
|
|
describe '#sort_issues' do
|
|
|
|
let(:milestone) { create(:milestone) }
|
|
|
|
|
|
|
|
let(:issue1) { create(:issue, milestone: milestone, position: 1) }
|
|
|
|
let(:issue2) { create(:issue, milestone: milestone, position: 2) }
|
|
|
|
let(:issue3) { create(:issue, milestone: milestone, position: 3) }
|
|
|
|
let(:issue4) { create(:issue, position: 42) }
|
|
|
|
|
|
|
|
it 'sorts the given issues' do
|
|
|
|
milestone.sort_issues([issue3.id, issue2.id, issue1.id])
|
|
|
|
|
|
|
|
issue1.reload
|
|
|
|
issue2.reload
|
|
|
|
issue3.reload
|
|
|
|
|
|
|
|
expect(issue1.position).to eq(3)
|
|
|
|
expect(issue2.position).to eq(2)
|
|
|
|
expect(issue3.position).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'ignores issues not part of the milestone' do
|
|
|
|
milestone.sort_issues([issue3.id, issue2.id, issue1.id, issue4.id])
|
|
|
|
|
|
|
|
issue4.reload
|
|
|
|
|
|
|
|
expect(issue4.position).to eq(42)
|
|
|
|
end
|
|
|
|
end
|
2012-04-08 17:28:58 -04:00
|
|
|
end
|