From 48e7e6594e37b6b95f2d7293617832a97b8ed3a8 Mon Sep 17 00:00:00 2001 From: Marek Takac Date: Thu, 31 Jul 2014 17:16:19 +0200 Subject: [PATCH] Added basic spec for updating WP --- spec/api/work_package_resource_spec.rb | 44 +++++++++++++++++++++++++- 1 file changed, 43 insertions(+), 1 deletion(-) diff --git a/spec/api/work_package_resource_spec.rb b/spec/api/work_package_resource_spec.rb index 8ae4c26ffe..ddcc3785f5 100644 --- a/spec/api/work_package_resource_spec.rb +++ b/spec/api/work_package_resource_spec.rb @@ -64,7 +64,7 @@ h4. things we like }} let(:project) { FactoryGirl.create(:project, :identifier => 'test_project', :is_public => false) } - let(:role) { FactoryGirl.create(:role, permissions: [:view_work_packages, :view_timelines]) } + let(:role) { FactoryGirl.create(:role, permissions: [:view_work_packages, :view_timelines, :edit_work_packages]) } let(:current_user) { FactoryGirl.create(:user, member_in_project: project, member_through_role: role) } let(:watcher) do FactoryGirl @@ -198,4 +198,46 @@ h4. things we like end end + + describe '#patch' do + let(:patch_path) { "/api/v3/work_packages/#{work_package.id}" } + before(:each) do + allow(User).to receive(:current).and_return current_user + patch patch_path, params + end + subject(:response) { last_response } + + context 'user with needed permissions' do + context 'valid update' do + let(:params) do + { + subject: 'Updated subject', + type: FactoryGirl.create(:type).name, + raw_description: 'Updated description', + status: FactoryGirl.create(:status).name, + priority: FactoryGirl.create(:priority).name, + startDate: Time.now - 1.week, + dueDate: Time.now + 2.weeks, + percentageDone: 50, + } + end + + it 'should respond with 200' do + expect(response.status).to eq(200) + end + + it 'should responde with updated work package' do + expect(subject.body).to be_json_eql('Updated subject'.to_json).at_path('subject') + end + end + + context 'invalid update' do + + end + end + + context 'user without permissions' do + + end + end end