#-- copyright # OpenProject is a project management system. # Copyright (C) 2012-2018 the OpenProject Foundation (OPF) # # This program is free software; you can redistribute it and/or # modify it under the terms of the GNU General Public License version 3. # # OpenProject is a fork of ChiliProject, which is a fork of Redmine. The copyright follows: # Copyright (C) 2006-2017 Jean-Philippe Lang # Copyright (C) 2010-2013 the ChiliProject Team # # This program is free software; you can redistribute it and/or # modify it under the terms of the GNU General Public License # as published by the Free Software Foundation; either version 2 # of the License, or (at your option) any later version. # # This program is distributed in the hope that it will be useful, # but WITHOUT ANY WARRANTY; without even the implied warranty of # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # GNU General Public License for more details. # # You should have received a copy of the GNU General Public License # along with this program; if not, write to the Free Software # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. # # See docs/COPYRIGHT.rdoc for more details. #++ require 'spec_helper' describe WorkPackage, type: :model do let(:stub_work_package) { FactoryGirl.build_stubbed(:work_package) } let(:stub_version) { FactoryGirl.build_stubbed(:version) } let(:stub_project) { FactoryGirl.build_stubbed(:project) } let(:user) { FactoryGirl.create(:user) } let(:type) { FactoryGirl.create(:type_standard) } let(:project) { FactoryGirl.create(:project, types: [type]) } let(:status) { FactoryGirl.create(:status) } let(:priority) { FactoryGirl.create(:priority) } let(:work_package) do WorkPackage.new.tap do |w| w.attributes = { project_id: project.id, type_id: type.id, author_id: user.id, status_id: status.id, priority: priority, subject: 'test_create', description: 'WorkPackage#create', estimated_hours: '1:30' } end end describe '.new' do context 'type' do let(:type2) { FactoryGirl.create(:type) } let(:project) { FactoryGirl.create(:project, types: [type, type2]) } before do project # loads types as well end context 'no project chosen' do it 'has no type set if no project was chosen' do expect(WorkPackage.new.type) .to be_nil end end context 'project chosen' do it 'has the provided type if one is provided' do expect(WorkPackage.new(project: project, type: type2).type) .to eql type2 end end end end describe 'create' do describe '#save' do subject { work_package.save } it { is_expected.to be_truthy } end describe '#estimated_hours' do before do work_package.save! work_package.reload end subject { work_package.estimated_hours } it { is_expected.to eq(1.5) } end describe 'minimal' do let(:work_package_minimal) { WorkPackage.new.tap do |w| w.attributes = { project_id: project.id, type_id: type.id, author_id: user.id, status_id: status.id, priority: priority, subject: 'test_create' } end } context 'save' do subject { work_package_minimal.save } it { is_expected.to be_truthy } end context 'description' do before do work_package_minimal.save! work_package_minimal.reload end subject { work_package_minimal.description } it { is_expected.to be_nil } end end describe '#assigned_to' do context 'group_assignment' do let(:group) { FactoryGirl.create(:group) } before do allow(Setting).to receive(:work_package_group_assignment).and_return(true) end subject { FactoryGirl.create(:work_package, assigned_to: group).assigned_to } it { is_expected.to eq(group) } end end end describe '#category' do let(:user_2) { FactoryGirl.create(:user, member_in_project: project) } let(:category) do FactoryGirl.create(:category, project: project, assigned_to: user_2) end before do work_package.attributes = { category_id: category.id } work_package.save! end subject { work_package.assigned_to } it { is_expected.to eq(category.assigned_to) } end describe '#assignable_assignees' do let(:value) { double('value') } before do allow(stub_work_package.project).to receive(:possible_assignees).and_return(value) end subject { stub_work_package.assignable_assignees } it 'calls project#possible_assignees and returns the value' do is_expected.to eql(value) end end describe '#assignable_responsibles' do let(:value) { double('value') } before do allow(stub_work_package.project).to receive(:possible_responsibles).and_return(value) end subject { stub_work_package.assignable_responsibles } it 'calls project#possible_responsibles and returns the value' do is_expected.to eql(value) end end describe 'responsible' do let(:group) { FactoryGirl.create(:group) } before { work_package.project.add_member! group, FactoryGirl.create(:role) } shared_context 'assign group as responsible' do before { work_package.responsible = group } end subject { work_package.valid? } context 'with assignable groups' do before { allow(Setting).to receive(:work_package_group_assignment?).and_return(true) } include_context 'assign group as responsible' it { is_expected.to be_truthy } end end describe '#assignable_versions' do let(:stub_version2) { FactoryGirl.build_stubbed(:version) } def stub_shared_versions(v = nil) versions = v ? [v] : [] allow(stub_work_package.project).to receive(:assignable_versions).and_return(versions) end it "should return all the project's shared versions" do stub_shared_versions(stub_version) expect(stub_work_package.assignable_versions).to eq([stub_version]) end it 'should return the former fixed_version if the version changed' do stub_shared_versions stub_work_package.fixed_version = stub_version2 allow(stub_work_package).to receive(:fixed_version_id_changed?).and_return true allow(stub_work_package).to receive(:fixed_version_id_was).and_return(stub_version.id) allow(Version).to receive(:find_by).with(id: stub_version.id).and_return(stub_version) expect(stub_work_package.assignable_versions).to eq([stub_version]) end it 'should return the current fixed_version if the version did not change' do stub_shared_versions stub_work_package.fixed_version = stub_version allow(stub_work_package).to receive(:fixed_version_id_changed?).and_return false expect(stub_work_package.assignable_versions).to eq([stub_version]) end end describe '#assignable_versions' do let!(:work_package) do wp = FactoryGirl.create(:work_package, project: project, fixed_version: version_current) # remove changes to fixed version factored into # assignable_versions calculation wp.reload wp end let!(:version_current) do FactoryGirl.create(:version, status: 'closed', project: project) end let!(:version_open) do FactoryGirl.create(:version, status: 'open', project: project) end let!(:version_locked) do FactoryGirl.create(:version, status: 'locked', project: project) end let!(:version_closed) do FactoryGirl.create(:version, status: 'closed', project: project) end let!(:version_other_project) do FactoryGirl.create(:version, status: 'open') end it 'returns all open versions of the project' do expect(work_package.assignable_versions) .to match_array [version_current, version_open] end end describe '#destroy' do let(:time_entry_1) { FactoryGirl.create(:time_entry, project: project, work_package: work_package) } let(:time_entry_2) { FactoryGirl.create(:time_entry, project: project, work_package: work_package) } before do time_entry_1 time_entry_2 work_package.destroy end context 'work package' do subject { WorkPackage.find_by(id: work_package.id) } it { is_expected.to be_nil } end context 'time entries' do subject { TimeEntry.find_by(work_package_id: work_package.id) } it { is_expected.to be_nil } end end describe '#done_ratio' do let(:status_new) { FactoryGirl.create(:status, name: 'New', is_default: true, is_closed: false, default_done_ratio: 50) } let(:status_assigned) { FactoryGirl.create(:status, name: 'Assigned', is_default: true, is_closed: false, default_done_ratio: 0) } let(:work_package_1) { FactoryGirl.create(:work_package, status: status_new) } let(:work_package_2) { FactoryGirl.create(:work_package, project: work_package_1.project, status: status_assigned, done_ratio: 30) } before { work_package_2 } describe '#value' do context 'work package field' do before { allow(Setting).to receive(:work_package_done_ratio).and_return 'field' } context 'work package 1' do subject { work_package_1.done_ratio } it { is_expected.to eq(0) } end context 'work package 2' do subject { work_package_2.done_ratio } it { is_expected.to eq(30) } end end context 'work package status' do before { allow(Setting).to receive(:work_package_done_ratio).and_return 'status' } context 'work package 1' do subject { work_package_1.done_ratio } it { is_expected.to eq(50) } end context 'work package 2' do subject { work_package_2.done_ratio } it { is_expected.to eq(0) } end end end describe '#update_done_ratio_from_status' do context 'work package field' do before do allow(Setting).to receive(:work_package_done_ratio).and_return 'field' work_package_1.update_done_ratio_from_status work_package_2.update_done_ratio_from_status end it 'does not update the done ratio' do expect(work_package_1.done_ratio).to eq(0) expect(work_package_2.done_ratio).to eq(30) end end context 'work package status' do before do allow(Setting).to receive(:work_package_done_ratio).and_return 'status' work_package_1.update_done_ratio_from_status work_package_2.update_done_ratio_from_status end it 'updates the done ratio' do expect(work_package_1.done_ratio).to eq(50) expect(work_package_2.done_ratio).to eq(0) end end end end describe '#group_by' do let(:type_2) { FactoryGirl.create(:type) } let(:priority_2) { FactoryGirl.create(:priority) } let(:project) { FactoryGirl.create(:project, types: [type, type_2]) } let(:version_1) { FactoryGirl.create(:version, project: project) } let(:version_2) { FactoryGirl.create(:version, project: project) } let(:category_1) { FactoryGirl.create(:category, project: project) } let(:category_2) { FactoryGirl.create(:category, project: project) } let(:user_2) { FactoryGirl.create(:user) } let(:work_package_1) { FactoryGirl.create(:work_package, author: user, assigned_to: user, responsible: user, project: project, type: type, priority: priority, fixed_version: version_1, category: category_1) } let(:work_package_2) { FactoryGirl.create(:work_package, author: user_2, assigned_to: user_2, responsible: user_2, project: project, type: type_2, priority: priority_2, fixed_version: version_2, category: category_2) } before do version_1 version_2 project.reload work_package_1 work_package_2 end shared_examples_for 'group by' do context 'size' do subject { groups.size } it { is_expected.to eq(2) } end context 'total' do subject { groups.inject(0) { |sum, group| sum + group['total'].to_i } } it { is_expected.to eq(2) } end end context 'by type' do let(:groups) { WorkPackage.by_type(project) } it_behaves_like 'group by' end context 'by version' do let(:groups) { WorkPackage.by_version(project) } it_behaves_like 'group by' end context 'by priority' do let(:groups) { WorkPackage.by_priority(project) } it_behaves_like 'group by' end context 'by category' do let(:groups) { WorkPackage.by_category(project) } it_behaves_like 'group by' end context 'by assigned to' do let(:groups) { WorkPackage.by_assigned_to(project) } it_behaves_like 'group by' end context 'by responsible' do let(:groups) { WorkPackage.by_responsible(project) } it_behaves_like 'group by' end context 'by author' do let(:groups) { WorkPackage.by_author(project) } it_behaves_like 'group by' end context 'by project' do let(:project_2) { FactoryGirl.create(:project, parent: project) } let(:work_package_3) { FactoryGirl.create(:work_package, project: project_2) } before { work_package_3 } let(:groups) { WorkPackage.by_author(project) } it_behaves_like 'group by' end end describe '#recently_updated' do let(:work_package_1) { FactoryGirl.create(:work_package) } let(:work_package_2) { FactoryGirl.create(:work_package) } before do work_package_1 work_package_2 without_timestamping do work_package_1.updated_at = 1.minute.ago work_package_1.save! end end context 'limit' do subject { WorkPackage.recently_updated.limit(1).first } it { is_expected.to eq(work_package_2) } end end describe '#on_active_project' do let(:project_archived) { FactoryGirl.create(:project, status: Project::STATUS_ARCHIVED) } let!(:work_package) { FactoryGirl.create(:work_package) } let(:work_package_in_archived_project) { FactoryGirl.create(:work_package, project: project_archived) } subject { WorkPackage.on_active_project.length } context 'one work package in active projects' do it { is_expected.to eq(1) } context 'and one work package in archived projects' do before { work_package_in_archived_project } it { is_expected.to eq(1) } end end end describe '#with_author' do let(:user) { FactoryGirl.create(:user) } let(:project_archived) { FactoryGirl.create(:project, status: Project::STATUS_ARCHIVED) } let!(:work_package) { FactoryGirl.create(:work_package, author: user) } let(:work_package_in_archived_project) { FactoryGirl.create(:work_package, project: project_archived, author: user) } subject { WorkPackage.with_author(user).length } context 'one work package in active projects' do it { is_expected.to eq(1) } context 'and one work package in archived projects' do before { work_package_in_archived_project } it { is_expected.to eq(2) } end end end describe '#recipients' do let(:project) { FactoryGirl.build_stubbed(:project) } let(:member) { FactoryGirl.build_stubbed(:user) } let(:author) { FactoryGirl.build_stubbed(:user) } let(:assignee) { FactoryGirl.build_stubbed(:user) } let(:responsible) { FactoryGirl.build_stubbed(:user) } let(:work_package) do FactoryGirl.build_stubbed(:work_package, author: author, assigned_to: assignee, responsible: responsible, project: project) end let(:project_notified_users) do [member] end let(:users_with_view_permission) do project_notified_users + [author, assignee, responsible] end before do allow(project) .to receive(:notified_users) .and_return(project_notified_users) allow(User) .to receive(:allowed) .and_return users_with_view_permission [author, assignee, responsible].each do |user| allow(user) .to receive(:notify_about?) .with(work_package) .and_return(true) end end it 'contains author, assignee, responsible and all from project#notified_users' do expect(work_package.recipients) .to match_array users_with_view_permission end context 'with users lacking the view permission' do let(:users_with_view_permission) do [] end it 'does not contain such users' do expect(work_package.recipients) .to be_empty end end context 'with author, assignee, responsible not interested' do before do [author, assignee, responsible].each do |user| allow(user) .to receive(:notify_about?) .with(work_package) .and_return(false) end end it 'does not contain such users' do expect(work_package.recipients) .to match_array project_notified_users end end context 'with author, assignee, responsible also being in project#notified_users' do let(:project_notified_users) do [member] + [author, assignee, responsible] end it 'contains the users but once' do expect(work_package.recipients) .to match_array project_notified_users end end context 'with a group' do let(:user1) { FactoryGirl.build_stubbed(:user) } let(:user2) { FactoryGirl.build_stubbed(:user) } let(:user3) { FactoryGirl.build_stubbed(:user) } let(:users_with_view_permission) do [user1, user3] end before do allow(user1) .to receive(:notify_about?) .with(work_package) .and_return(false) [user2, user3].each do |user| allow(user) .to receive(:notify_about?) .with(work_package) .and_return(true) end end context 'for assignee' do let(:assignee) do group = FactoryGirl.build_stubbed(:group) allow(group) .to receive(:users) .and_return([user1, user2, user3]) group end it 'returns those group members who want to be notified and who have the permission to see the work package' do expect(work_package.recipients) .to match_array [user3] end end context 'for responsible' do let(:responsible) do group = FactoryGirl.build_stubbed(:group) allow(group) .to receive(:users) .and_return([user1, user2, user3]) group end it 'returns those group members who want to be notified and who have the permission to see the work package' do expect(work_package.recipients) .to match_array [user3] end end end end describe '#add_time_entry' do it 'should return a new time entry' do expect(stub_work_package.add_time_entry).to be_a TimeEntry end it 'should already have the project assigned' do stub_work_package.project = stub_project expect(stub_work_package.add_time_entry.project).to eq(stub_project) end it 'should already have the work_package assigned' do expect(stub_work_package.add_time_entry.work_package).to eq(stub_work_package) end it 'should return an usaved entry' do expect(stub_work_package.add_time_entry).to be_new_record end end describe '.allowed_target_project_on_move' do let(:project) { FactoryGirl.create(:project) } let(:role) { FactoryGirl.create(:role, permissions: [:move_work_packages]) } let(:user) { FactoryGirl.create(:user, member_in_project: project, member_through_role: role) } context 'when having the move_work_packages permission' do it 'returns the project' do expect(WorkPackage.allowed_target_projects_on_move(user)) .to match_array [project] end end context 'when lacking the move_work_packages permission' do let(:role) { FactoryGirl.create(:role, permissions: []) } it 'does not return the project' do expect(WorkPackage.allowed_target_projects_on_move(user)) .to be_empty end end end describe '.allowed_target_project_on_create' do let(:project) { FactoryGirl.create(:project) } let(:role) { FactoryGirl.create(:role, permissions: [:add_work_packages]) } let(:user) { FactoryGirl.create(:user, member_in_project: project, member_through_role: role) } context 'when having the add_work_packages permission' do it 'returns the project' do expect(WorkPackage.allowed_target_projects_on_create(user)) .to match_array [project] end end context 'when lacking the add_work_packages permission' do let(:role) { FactoryGirl.create(:role, permissions: []) } it 'does not return the project' do expect(WorkPackage.allowed_target_projects_on_create(user)) .to be_empty end end end describe '#duration' do let(:instance) { send(subclass) } describe "w/ today as start date w/ tomorrow as due date" do before do work_package.start_date = Date.today work_package.due_date = Date.today + 1.day end it 'should have a duration of two' do expect(work_package.duration).to eq(2) end end describe "w/ today as start date w/ today as due date" do before do work_package.start_date = Date.today work_package.due_date = Date.today end it 'should have a duration of one' do expect(work_package.duration).to eq(1) end end describe "w/ today as start date w/o a due date" do before do work_package.start_date = Date.today work_package.due_date = nil end it 'should have a duration of one' do expect(work_package.duration).to eq(1) end end describe "w/o a start date w today as due date" do before do work_package.start_date = nil work_package.due_date = Date.today end it 'should have a duration of one' do expect(work_package.duration).to eq(1) end end end describe 'custom fields' do let(:included_cf) { FactoryGirl.build(:work_package_custom_field) } let(:other_cf) { FactoryGirl.build(:work_package_custom_field) } before do included_cf.save other_cf.save project.work_package_custom_fields << included_cf type.custom_fields << included_cf end it 'says to respond to valid custom field accessors' do expect(work_package.respond_to?(included_cf.accessor_name)).to be_truthy end it 'really responds to valid custom field accessors' do expect(work_package.send(included_cf.accessor_name)).to eql(nil) end it 'says to not respond to foreign custom field accessors' do expect(work_package.respond_to?(other_cf.accessor_name)).to be_falsey end it 'does really not respond to foreign custom field accessors' do expect { work_package.send(other_cf.accessor_name) }.to raise_error(NoMethodError) end it 'should not duplicate error messages when invalid' do cf1 = FactoryGirl.create(:work_package_custom_field, is_required: true) cf2 = FactoryGirl.create(:work_package_custom_field, is_required: true) # create work_package with one required custom field work_package = FactoryGirl.create :work_package work_package.reload work_package.project.work_package_custom_fields << cf1 work_package.type.custom_fields << cf1 # set that custom field with a value, should be fine work_package.custom_field_values = { cf1.id => 'test' } work_package.save! work_package.reload # now give the work_package another required custom field, but don't assign a value work_package.project.work_package_custom_fields << cf2 work_package.type.custom_fields << cf2 work_package.custom_field_values # #custom_field_values needs to be touched # that should not be valid expect(work_package).not_to be_valid # assert that there is only one error expect(work_package.errors.size).to eq 1 expect(work_package.errors["custom_field_#{cf2.id}"].size).to eq 1 end end describe 'changed_since' do let!(:work_package) do Timecop.travel(5.hours.ago) do FactoryGirl.create(:work_package) end end describe 'null' do subject { WorkPackage.changed_since(nil) } it { expect(subject).to match_array([work_package]) } end describe 'now' do subject { WorkPackage.changed_since(DateTime.now) } it { expect(subject).to be_empty } end describe 'work package update' do subject { WorkPackage.changed_since(work_package.reload.updated_at) } it { expect(subject).to match_array([work_package]) } end end end