Chris@1517: # Redmine - project management software Chris@1517: # Copyright (C) 2006-2014 Jean-Philippe Lang Chris@1517: # Chris@1517: # This program is free software; you can redistribute it and/or Chris@1517: # modify it under the terms of the GNU General Public License Chris@1517: # as published by the Free Software Foundation; either version 2 Chris@1517: # of the License, or (at your option) any later version. Chris@1517: # Chris@1517: # This program is distributed in the hope that it will be useful, Chris@1517: # but WITHOUT ANY WARRANTY; without even the implied warranty of Chris@1517: # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the Chris@1517: # GNU General Public License for more details. Chris@1517: # Chris@1517: # You should have received a copy of the GNU General Public License Chris@1517: # along with this program; if not, write to the Free Software Chris@1517: # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. Chris@1517: Chris@1517: require File.expand_path('../../../test_helper', __FILE__) Chris@1517: Chris@1517: class Redmine::ApiTest::IssuesTest < Redmine::ApiTest::Base Chris@1517: fixtures :projects, Chris@1517: :users, Chris@1517: :roles, Chris@1517: :members, Chris@1517: :member_roles, Chris@1517: :issues, Chris@1517: :issue_statuses, Chris@1517: :issue_relations, Chris@1517: :versions, Chris@1517: :trackers, Chris@1517: :projects_trackers, Chris@1517: :issue_categories, Chris@1517: :enabled_modules, Chris@1517: :enumerations, Chris@1517: :attachments, Chris@1517: :workflows, Chris@1517: :custom_fields, Chris@1517: :custom_values, Chris@1517: :custom_fields_projects, Chris@1517: :custom_fields_trackers, Chris@1517: :time_entries, Chris@1517: :journals, Chris@1517: :journal_details, Chris@1517: :queries, Chris@1517: :attachments Chris@1517: Chris@1517: def setup Chris@1517: Setting.rest_api_enabled = '1' Chris@1517: end Chris@1517: Chris@1517: context "/issues" do Chris@1517: # Use a private project to make sure auth is really working and not just Chris@1517: # only showing public issues. Chris@1517: should_allow_api_authentication(:get, "/projects/private-child/issues.xml") Chris@1517: Chris@1517: should "contain metadata" do Chris@1517: get '/issues.xml' Chris@1517: Chris@1517: assert_tag :tag => 'issues', Chris@1517: :attributes => { Chris@1517: :type => 'array', Chris@1517: :total_count => assigns(:issue_count), Chris@1517: :limit => 25, Chris@1517: :offset => 0 Chris@1517: } Chris@1517: end Chris@1517: Chris@1517: context "with offset and limit" do Chris@1517: should "use the params" do Chris@1517: get '/issues.xml?offset=2&limit=3' Chris@1517: Chris@1517: assert_equal 3, assigns(:limit) Chris@1517: assert_equal 2, assigns(:offset) Chris@1517: assert_tag :tag => 'issues', :children => {:count => 3, :only => {:tag => 'issue'}} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with nometa param" do Chris@1517: should "not contain metadata" do Chris@1517: get '/issues.xml?nometa=1' Chris@1517: Chris@1517: assert_tag :tag => 'issues', Chris@1517: :attributes => { Chris@1517: :type => 'array', Chris@1517: :total_count => nil, Chris@1517: :limit => nil, Chris@1517: :offset => nil Chris@1517: } Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with nometa header" do Chris@1517: should "not contain metadata" do Chris@1517: get '/issues.xml', {}, {'X-Redmine-Nometa' => '1'} Chris@1517: Chris@1517: assert_tag :tag => 'issues', Chris@1517: :attributes => { Chris@1517: :type => 'array', Chris@1517: :total_count => nil, Chris@1517: :limit => nil, Chris@1517: :offset => nil Chris@1517: } Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with relations" do Chris@1517: should "display relations" do Chris@1517: get '/issues.xml?include=relations' Chris@1517: Chris@1517: assert_response :success Chris@1517: assert_equal 'application/xml', @response.content_type Chris@1517: assert_tag 'relations', Chris@1517: :parent => {:tag => 'issue', :child => {:tag => 'id', :content => '3'}}, Chris@1517: :children => {:count => 1}, Chris@1517: :child => { Chris@1517: :tag => 'relation', Chris@1517: :attributes => {:id => '2', :issue_id => '2', :issue_to_id => '3', Chris@1517: :relation_type => 'relates'} Chris@1517: } Chris@1517: assert_tag 'relations', Chris@1517: :parent => {:tag => 'issue', :child => {:tag => 'id', :content => '1'}}, Chris@1517: :children => {:count => 0} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with invalid query params" do Chris@1517: should "return errors" do Chris@1517: get '/issues.xml', {:f => ['start_date'], :op => {:start_date => '='}} Chris@1517: Chris@1517: assert_response :unprocessable_entity Chris@1517: assert_equal 'application/xml', @response.content_type Chris@1517: assert_tag 'errors', :child => {:tag => 'error', :content => "Start date can't be blank"} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with custom field filter" do Chris@1517: should "show only issues with the custom field value" do Chris@1517: get '/issues.xml', Chris@1517: {:set_filter => 1, :f => ['cf_1'], :op => {:cf_1 => '='}, Chris@1517: :v => {:cf_1 => ['MySQL']}} Chris@1517: expected_ids = Issue.visible. Chris@1517: joins(:custom_values). Chris@1517: where(:custom_values => {:custom_field_id => 1, :value => 'MySQL'}).map(&:id) Chris@1517: assert_select 'issues > issue > id', :count => expected_ids.count do |ids| Chris@1517: ids.each { |id| assert expected_ids.delete(id.children.first.content.to_i) } Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with custom field filter (shorthand method)" do Chris@1517: should "show only issues with the custom field value" do Chris@1517: get '/issues.xml', { :cf_1 => 'MySQL' } Chris@1517: Chris@1517: expected_ids = Issue.visible. Chris@1517: joins(:custom_values). Chris@1517: where(:custom_values => {:custom_field_id => 1, :value => 'MySQL'}).map(&:id) Chris@1517: Chris@1517: assert_select 'issues > issue > id', :count => expected_ids.count do |ids| Chris@1517: ids.each { |id| assert expected_ids.delete(id.children.first.content.to_i) } Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: def test_index_should_allow_timestamp_filtering Chris@1517: Issue.delete_all Chris@1517: Issue.generate!(:subject => '1').update_column(:updated_on, Time.parse("2014-01-02T10:25:00Z")) Chris@1517: Issue.generate!(:subject => '2').update_column(:updated_on, Time.parse("2014-01-02T12:13:00Z")) Chris@1517: Chris@1517: get '/issues.xml', Chris@1517: {:set_filter => 1, :f => ['updated_on'], :op => {:updated_on => '<='}, Chris@1517: :v => {:updated_on => ['2014-01-02T12:00:00Z']}} Chris@1517: assert_select 'issues>issue', :count => 1 Chris@1517: assert_select 'issues>issue>subject', :text => '1' Chris@1517: Chris@1517: get '/issues.xml', Chris@1517: {:set_filter => 1, :f => ['updated_on'], :op => {:updated_on => '>='}, Chris@1517: :v => {:updated_on => ['2014-01-02T12:00:00Z']}} Chris@1517: assert_select 'issues>issue', :count => 1 Chris@1517: assert_select 'issues>issue>subject', :text => '2' Chris@1517: Chris@1517: get '/issues.xml', Chris@1517: {:set_filter => 1, :f => ['updated_on'], :op => {:updated_on => '>='}, Chris@1517: :v => {:updated_on => ['2014-01-02T08:00:00Z']}} Chris@1517: assert_select 'issues>issue', :count => 2 Chris@1517: end Chris@1517: Chris@1517: context "/index.json" do Chris@1517: should_allow_api_authentication(:get, "/projects/private-child/issues.json") Chris@1517: end Chris@1517: Chris@1517: context "/index.xml with filter" do Chris@1517: should "show only issues with the status_id" do Chris@1517: get '/issues.xml?status_id=5' Chris@1517: Chris@1517: expected_ids = Issue.visible.where(:status_id => 5).map(&:id) Chris@1517: Chris@1517: assert_select 'issues > issue > id', :count => expected_ids.count do |ids| Chris@1517: ids.each { |id| assert expected_ids.delete(id.children.first.content.to_i) } Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "/index.json with filter" do Chris@1517: should "show only issues with the status_id" do Chris@1517: get '/issues.json?status_id=5' Chris@1517: Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: status_ids_used = json['issues'].collect {|j| j['status']['id'] } Chris@1517: assert_equal 3, status_ids_used.length Chris@1517: assert status_ids_used.all? {|id| id == 5 } Chris@1517: end Chris@1517: Chris@1517: end Chris@1517: Chris@1517: # Issue 6 is on a private project Chris@1517: context "/issues/6.xml" do Chris@1517: should_allow_api_authentication(:get, "/issues/6.xml") Chris@1517: end Chris@1517: Chris@1517: context "/issues/6.json" do Chris@1517: should_allow_api_authentication(:get, "/issues/6.json") Chris@1517: end Chris@1517: Chris@1517: context "GET /issues/:id" do Chris@1517: context "with journals" do Chris@1517: context ".xml" do Chris@1517: should "display journals" do Chris@1517: get '/issues/1.xml?include=journals' Chris@1517: Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'journals', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :child => { Chris@1517: :tag => 'journal', Chris@1517: :attributes => { :id => '1'}, Chris@1517: :child => { Chris@1517: :tag => 'details', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :child => { Chris@1517: :tag => 'detail', Chris@1517: :attributes => { :name => 'status_id' }, Chris@1517: :child => { Chris@1517: :tag => 'old_value', Chris@1517: :content => '1', Chris@1517: :sibling => { Chris@1517: :tag => 'new_value', Chris@1517: :content => '2' Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with custom fields" do Chris@1517: context ".xml" do Chris@1517: should "display custom fields" do Chris@1517: get '/issues/3.xml' Chris@1517: Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'custom_fields', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :child => { Chris@1517: :tag => 'custom_field', Chris@1517: :attributes => { :id => '1'}, Chris@1517: :child => { Chris@1517: :tag => 'value', Chris@1517: :content => 'MySQL' Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: Chris@1517: assert_nothing_raised do Chris@1517: Hash.from_xml(response.body).to_xml Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with multi custom fields" do Chris@1517: setup do Chris@1517: field = CustomField.find(1) Chris@1517: field.update_attribute :multiple, true Chris@1517: issue = Issue.find(3) Chris@1517: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1517: issue.save! Chris@1517: end Chris@1517: Chris@1517: context ".xml" do Chris@1517: should "display custom fields" do Chris@1517: get '/issues/3.xml' Chris@1517: assert_response :success Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'custom_fields', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :child => { Chris@1517: :tag => 'custom_field', Chris@1517: :attributes => { :id => '1'}, Chris@1517: :child => { Chris@1517: :tag => 'value', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :children => { :count => 2 } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: Chris@1517: xml = Hash.from_xml(response.body) Chris@1517: custom_fields = xml['issue']['custom_fields'] Chris@1517: assert_kind_of Array, custom_fields Chris@1517: field = custom_fields.detect {|f| f['id'] == '1'} Chris@1517: assert_kind_of Hash, field Chris@1517: assert_equal ['MySQL', 'Oracle'], field['value'].sort Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context ".json" do Chris@1517: should "display custom fields" do Chris@1517: get '/issues/3.json' Chris@1517: assert_response :success Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: custom_fields = json['issue']['custom_fields'] Chris@1517: assert_kind_of Array, custom_fields Chris@1517: field = custom_fields.detect {|f| f['id'] == 1} Chris@1517: assert_kind_of Hash, field Chris@1517: assert_equal ['MySQL', 'Oracle'], field['value'].sort Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with empty value for multi custom field" do Chris@1517: setup do Chris@1517: field = CustomField.find(1) Chris@1517: field.update_attribute :multiple, true Chris@1517: issue = Issue.find(3) Chris@1517: issue.custom_field_values = {1 => ['']} Chris@1517: issue.save! Chris@1517: end Chris@1517: Chris@1517: context ".xml" do Chris@1517: should "display custom fields" do Chris@1517: get '/issues/3.xml' Chris@1517: assert_response :success Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'custom_fields', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :child => { Chris@1517: :tag => 'custom_field', Chris@1517: :attributes => { :id => '1'}, Chris@1517: :child => { Chris@1517: :tag => 'value', Chris@1517: :attributes => { :type => 'array' }, Chris@1517: :children => { :count => 0 } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: Chris@1517: xml = Hash.from_xml(response.body) Chris@1517: custom_fields = xml['issue']['custom_fields'] Chris@1517: assert_kind_of Array, custom_fields Chris@1517: field = custom_fields.detect {|f| f['id'] == '1'} Chris@1517: assert_kind_of Hash, field Chris@1517: assert_equal [], field['value'] Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context ".json" do Chris@1517: should "display custom fields" do Chris@1517: get '/issues/3.json' Chris@1517: assert_response :success Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: custom_fields = json['issue']['custom_fields'] Chris@1517: assert_kind_of Array, custom_fields Chris@1517: field = custom_fields.detect {|f| f['id'] == 1} Chris@1517: assert_kind_of Hash, field Chris@1517: assert_equal [], field['value'].sort Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with attachments" do Chris@1517: context ".xml" do Chris@1517: should "display attachments" do Chris@1517: get '/issues/3.xml?include=attachments' Chris@1517: Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'attachments', Chris@1517: :children => {:count => 5}, Chris@1517: :child => { Chris@1517: :tag => 'attachment', Chris@1517: :child => { Chris@1517: :tag => 'filename', Chris@1517: :content => 'source.rb', Chris@1517: :sibling => { Chris@1517: :tag => 'content_url', Chris@1517: :content => 'http://www.example.com/attachments/download/4/source.rb' Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "with subtasks" do Chris@1517: setup do Chris@1517: @c1 = Issue.create!( Chris@1517: :status_id => 1, :subject => "child c1", Chris@1517: :tracker_id => 1, :project_id => 1, :author_id => 1, Chris@1517: :parent_issue_id => 1 Chris@1517: ) Chris@1517: @c2 = Issue.create!( Chris@1517: :status_id => 1, :subject => "child c2", Chris@1517: :tracker_id => 1, :project_id => 1, :author_id => 1, Chris@1517: :parent_issue_id => 1 Chris@1517: ) Chris@1517: @c3 = Issue.create!( Chris@1517: :status_id => 1, :subject => "child c3", Chris@1517: :tracker_id => 1, :project_id => 1, :author_id => 1, Chris@1517: :parent_issue_id => @c1.id Chris@1517: ) Chris@1517: end Chris@1517: Chris@1517: context ".xml" do Chris@1517: should "display children" do Chris@1517: get '/issues/1.xml?include=children' Chris@1517: Chris@1517: assert_tag :tag => 'issue', Chris@1517: :child => { Chris@1517: :tag => 'children', Chris@1517: :children => {:count => 2}, Chris@1517: :child => { Chris@1517: :tag => 'issue', Chris@1517: :attributes => {:id => @c1.id.to_s}, Chris@1517: :child => { Chris@1517: :tag => 'subject', Chris@1517: :content => 'child c1', Chris@1517: :sibling => { Chris@1517: :tag => 'children', Chris@1517: :children => {:count => 1}, Chris@1517: :child => { Chris@1517: :tag => 'issue', Chris@1517: :attributes => {:id => @c3.id.to_s} Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: } Chris@1517: end Chris@1517: Chris@1517: context ".json" do Chris@1517: should "display children" do Chris@1517: get '/issues/1.json?include=children' Chris@1517: Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: assert_equal([ Chris@1517: { Chris@1517: 'id' => @c1.id, 'subject' => 'child c1', 'tracker' => {'id' => 1, 'name' => 'Bug'}, Chris@1517: 'children' => [{'id' => @c3.id, 'subject' => 'child c3', Chris@1517: 'tracker' => {'id' => 1, 'name' => 'Bug'} }] Chris@1517: }, Chris@1517: { 'id' => @c2.id, 'subject' => 'child c2', 'tracker' => {'id' => 1, 'name' => 'Bug'} } Chris@1517: ], Chris@1517: json['issue']['children']) Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: test "GET /issues/:id.xml?include=watchers should include watchers" do Chris@1517: Watcher.create!(:user_id => 3, :watchable => Issue.find(1)) Chris@1517: Chris@1517: get '/issues/1.xml?include=watchers', {}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal 'application/xml', response.content_type Chris@1517: assert_select 'issue' do Chris@1517: assert_select 'watchers', Issue.find(1).watchers.count Chris@1517: assert_select 'watchers' do Chris@1517: assert_select 'user[id=3]' Chris@1517: end Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "POST /issues.xml" do Chris@1517: should_allow_api_authentication( Chris@1517: :post, Chris@1517: '/issues.xml', Chris@1517: {:issue => {:project_id => 1, :subject => 'API test', :tracker_id => 2, :status_id => 3}}, Chris@1517: {:success_code => :created} Chris@1517: ) Chris@1517: should "create an issue with the attributes" do Chris@1517: assert_difference('Issue.count') do Chris@1517: post '/issues.xml', Chris@1517: {:issue => {:project_id => 1, :subject => 'API test', Chris@1517: :tracker_id => 2, :status_id => 3}}, credentials('jsmith') Chris@1517: end Chris@1517: issue = Issue.order('id DESC').first Chris@1517: assert_equal 1, issue.project_id Chris@1517: assert_equal 2, issue.tracker_id Chris@1517: assert_equal 3, issue.status_id Chris@1517: assert_equal 'API test', issue.subject Chris@1517: Chris@1517: assert_response :created Chris@1517: assert_equal 'application/xml', @response.content_type Chris@1517: assert_tag 'issue', :child => {:tag => 'id', :content => issue.id.to_s} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: test "POST /issues.xml with watcher_user_ids should create issue with watchers" do Chris@1517: assert_difference('Issue.count') do Chris@1517: post '/issues.xml', Chris@1517: {:issue => {:project_id => 1, :subject => 'Watchers', Chris@1517: :tracker_id => 2, :status_id => 3, :watcher_user_ids => [3, 1]}}, credentials('jsmith') Chris@1517: assert_response :created Chris@1517: end Chris@1517: issue = Issue.order('id desc').first Chris@1517: assert_equal 2, issue.watchers.size Chris@1517: assert_equal [1, 3], issue.watcher_user_ids.sort Chris@1517: end Chris@1517: Chris@1517: context "POST /issues.xml with failure" do Chris@1517: should "have an errors tag" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: post '/issues.xml', {:issue => {:project_id => 1}}, credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: assert_tag :errors, :child => {:tag => 'error', :content => "Subject can't be blank"} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "POST /issues.json" do Chris@1517: should_allow_api_authentication(:post, Chris@1517: '/issues.json', Chris@1517: {:issue => {:project_id => 1, :subject => 'API test', Chris@1517: :tracker_id => 2, :status_id => 3}}, Chris@1517: {:success_code => :created}) Chris@1517: Chris@1517: should "create an issue with the attributes" do Chris@1517: assert_difference('Issue.count') do Chris@1517: post '/issues.json', Chris@1517: {:issue => {:project_id => 1, :subject => 'API test', Chris@1517: :tracker_id => 2, :status_id => 3}}, Chris@1517: credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: issue = Issue.order('id DESC').first Chris@1517: assert_equal 1, issue.project_id Chris@1517: assert_equal 2, issue.tracker_id Chris@1517: assert_equal 3, issue.status_id Chris@1517: assert_equal 'API test', issue.subject Chris@1517: end Chris@1517: Chris@1517: end Chris@1517: Chris@1517: context "POST /issues.json with failure" do Chris@1517: should "have an errors element" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: post '/issues.json', {:issue => {:project_id => 1}}, credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: assert json['errors'].include?("Subject can't be blank") Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: # Issue 6 is on a private project Chris@1517: context "PUT /issues/6.xml" do Chris@1517: setup do Chris@1517: @parameters = {:issue => {:subject => 'API update', :notes => 'A new note'}} Chris@1517: end Chris@1517: Chris@1517: should_allow_api_authentication(:put, Chris@1517: '/issues/6.xml', Chris@1517: {:issue => {:subject => 'API update', :notes => 'A new note'}}, Chris@1517: {:success_code => :ok}) Chris@1517: Chris@1517: should "not create a new issue" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: should "create a new journal" do Chris@1517: assert_difference('Journal.count') do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: should "add the note to the journal" do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: Chris@1517: journal = Journal.last Chris@1517: assert_equal "A new note", journal.notes Chris@1517: end Chris@1517: Chris@1517: should "update the issue" do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: Chris@1517: issue = Issue.find(6) Chris@1517: assert_equal "API update", issue.subject Chris@1517: end Chris@1517: Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/3.xml with custom fields" do Chris@1517: setup do Chris@1517: @parameters = { Chris@1517: :issue => {:custom_fields => [{'id' => '1', 'value' => 'PostgreSQL' }, Chris@1517: {'id' => '2', 'value' => '150'}]} Chris@1517: } Chris@1517: end Chris@1517: Chris@1517: should "update custom fields" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: put '/issues/3.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: issue = Issue.find(3) Chris@1517: assert_equal '150', issue.custom_value_for(2).value Chris@1517: assert_equal 'PostgreSQL', issue.custom_value_for(1).value Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/3.xml with multi custom fields" do Chris@1517: setup do Chris@1517: field = CustomField.find(1) Chris@1517: field.update_attribute :multiple, true Chris@1517: @parameters = { Chris@1517: :issue => {:custom_fields => [{'id' => '1', 'value' => ['MySQL', 'PostgreSQL'] }, Chris@1517: {'id' => '2', 'value' => '150'}]} Chris@1517: } Chris@1517: end Chris@1517: Chris@1517: should "update custom fields" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: put '/issues/3.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: issue = Issue.find(3) Chris@1517: assert_equal '150', issue.custom_value_for(2).value Chris@1517: assert_equal ['MySQL', 'PostgreSQL'], issue.custom_field_value(1).sort Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/3.xml with project change" do Chris@1517: setup do Chris@1517: @parameters = {:issue => {:project_id => 2, :subject => 'Project changed'}} Chris@1517: end Chris@1517: Chris@1517: should "update project" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: put '/issues/3.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: Chris@1517: issue = Issue.find(3) Chris@1517: assert_equal 2, issue.project_id Chris@1517: assert_equal 'Project changed', issue.subject Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/6.xml with failed update" do Chris@1517: setup do Chris@1517: @parameters = {:issue => {:subject => ''}} Chris@1517: end Chris@1517: Chris@1517: should "not create a new issue" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: should "not create a new journal" do Chris@1517: assert_no_difference('Journal.count') do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: should "have an errors tag" do Chris@1517: put '/issues/6.xml', @parameters, credentials('jsmith') Chris@1517: Chris@1517: assert_tag :errors, :child => {:tag => 'error', :content => "Subject can't be blank"} Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/6.json" do Chris@1517: setup do Chris@1517: @parameters = {:issue => {:subject => 'API update', :notes => 'A new note'}} Chris@1517: end Chris@1517: Chris@1517: should_allow_api_authentication(:put, Chris@1517: '/issues/6.json', Chris@1517: {:issue => {:subject => 'API update', :notes => 'A new note'}}, Chris@1517: {:success_code => :ok}) Chris@1517: Chris@1517: should "update the issue" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: assert_difference('Journal.count') do Chris@1517: put '/issues/6.json', @parameters, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal '', response.body Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: issue = Issue.find(6) Chris@1517: assert_equal "API update", issue.subject Chris@1517: journal = Journal.last Chris@1517: assert_equal "A new note", journal.notes Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "PUT /issues/6.json with failed update" do Chris@1517: should "return errors" do Chris@1517: assert_no_difference('Issue.count') do Chris@1517: assert_no_difference('Journal.count') do Chris@1517: put '/issues/6.json', {:issue => {:subject => ''}}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :unprocessable_entity Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: json = ActiveSupport::JSON.decode(response.body) Chris@1517: assert json['errors'].include?("Subject can't be blank") Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "DELETE /issues/1.xml" do Chris@1517: should_allow_api_authentication(:delete, Chris@1517: '/issues/6.xml', Chris@1517: {}, Chris@1517: {:success_code => :ok}) Chris@1517: Chris@1517: should "delete the issue" do Chris@1517: assert_difference('Issue.count', -1) do Chris@1517: delete '/issues/6.xml', {}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal '', response.body Chris@1517: end Chris@1517: Chris@1517: assert_nil Issue.find_by_id(6) Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: context "DELETE /issues/1.json" do Chris@1517: should_allow_api_authentication(:delete, Chris@1517: '/issues/6.json', Chris@1517: {}, Chris@1517: {:success_code => :ok}) Chris@1517: Chris@1517: should "delete the issue" do Chris@1517: assert_difference('Issue.count', -1) do Chris@1517: delete '/issues/6.json', {}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal '', response.body Chris@1517: end Chris@1517: Chris@1517: assert_nil Issue.find_by_id(6) Chris@1517: end Chris@1517: end Chris@1517: Chris@1517: test "POST /issues/:id/watchers.xml should add watcher" do Chris@1517: assert_difference 'Watcher.count' do Chris@1517: post '/issues/1/watchers.xml', {:user_id => 3}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal '', response.body Chris@1517: end Chris@1517: watcher = Watcher.order('id desc').first Chris@1517: assert_equal Issue.find(1), watcher.watchable Chris@1517: assert_equal User.find(3), watcher.user Chris@1517: end Chris@1517: Chris@1517: test "DELETE /issues/:id/watchers/:user_id.xml should remove watcher" do Chris@1517: Watcher.create!(:user_id => 3, :watchable => Issue.find(1)) Chris@1517: Chris@1517: assert_difference 'Watcher.count', -1 do Chris@1517: delete '/issues/1/watchers/3.xml', {}, credentials('jsmith') Chris@1517: Chris@1517: assert_response :ok Chris@1517: assert_equal '', response.body Chris@1517: end Chris@1517: assert_equal false, Issue.find(1).watched_by?(User.find(3)) Chris@1517: end Chris@1517: Chris@1517: def test_create_issue_with_uploaded_file Chris@1517: set_tmp_attachments_directory Chris@1517: # upload the file Chris@1517: assert_difference 'Attachment.count' do Chris@1517: post '/uploads.xml', 'test_create_with_upload', Chris@1517: {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) Chris@1517: assert_response :created Chris@1517: end Chris@1517: xml = Hash.from_xml(response.body) Chris@1517: token = xml['upload']['token'] Chris@1517: attachment = Attachment.order('id DESC').first Chris@1517: Chris@1517: # create the issue with the upload's token Chris@1517: assert_difference 'Issue.count' do Chris@1517: post '/issues.xml', Chris@1517: {:issue => {:project_id => 1, :subject => 'Uploaded file', Chris@1517: :uploads => [{:token => token, :filename => 'test.txt', Chris@1517: :content_type => 'text/plain'}]}}, Chris@1517: credentials('jsmith') Chris@1517: assert_response :created Chris@1517: end Chris@1517: issue = Issue.order('id DESC').first Chris@1517: assert_equal 1, issue.attachments.count Chris@1517: assert_equal attachment, issue.attachments.first Chris@1517: Chris@1517: attachment.reload Chris@1517: assert_equal 'test.txt', attachment.filename Chris@1517: assert_equal 'text/plain', attachment.content_type Chris@1517: assert_equal 'test_create_with_upload'.size, attachment.filesize Chris@1517: assert_equal 2, attachment.author_id Chris@1517: Chris@1517: # get the issue with its attachments Chris@1517: get "/issues/#{issue.id}.xml", :include => 'attachments' Chris@1517: assert_response :success Chris@1517: xml = Hash.from_xml(response.body) Chris@1517: attachments = xml['issue']['attachments'] Chris@1517: assert_kind_of Array, attachments Chris@1517: assert_equal 1, attachments.size Chris@1517: url = attachments.first['content_url'] Chris@1517: assert_not_nil url Chris@1517: Chris@1517: # download the attachment Chris@1517: get url Chris@1517: assert_response :success Chris@1517: end Chris@1517: Chris@1517: def test_update_issue_with_uploaded_file Chris@1517: set_tmp_attachments_directory Chris@1517: # upload the file Chris@1517: assert_difference 'Attachment.count' do Chris@1517: post '/uploads.xml', 'test_upload_with_upload', Chris@1517: {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) Chris@1517: assert_response :created Chris@1517: end Chris@1517: xml = Hash.from_xml(response.body) Chris@1517: token = xml['upload']['token'] Chris@1517: attachment = Attachment.order('id DESC').first Chris@1517: Chris@1517: # update the issue with the upload's token Chris@1517: assert_difference 'Journal.count' do Chris@1517: put '/issues/1.xml', Chris@1517: {:issue => {:notes => 'Attachment added', Chris@1517: :uploads => [{:token => token, :filename => 'test.txt', Chris@1517: :content_type => 'text/plain'}]}}, Chris@1517: credentials('jsmith') Chris@1517: assert_response :ok Chris@1517: assert_equal '', @response.body Chris@1517: end Chris@1517: Chris@1517: issue = Issue.find(1) Chris@1517: assert_include attachment, issue.attachments Chris@1517: end Chris@1517: end