issues_test.rb
547 lines
| 17.2 KiB
| text/x-ruby
|
RubyLexer
|
r3196 | # Redmine - project management software | ||
|
r6175 | # Copyright (C) 2006-2011 Jean-Philippe Lang | ||
|
r3196 | # | ||
# 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. | ||||
|
r4395 | require File.expand_path('../../../test_helper', __FILE__) | ||
|
r3196 | |||
|
r4243 | class ApiTest::IssuesTest < ActionController::IntegrationTest | ||
|
r3196 | fixtures :projects, | ||
:users, | ||||
:roles, | ||||
:members, | ||||
:member_roles, | ||||
:issues, | ||||
:issue_statuses, | ||||
:versions, | ||||
:trackers, | ||||
:projects_trackers, | ||||
:issue_categories, | ||||
:enabled_modules, | ||||
:enumerations, | ||||
:attachments, | ||||
:workflows, | ||||
:custom_fields, | ||||
:custom_values, | ||||
:custom_fields_projects, | ||||
:custom_fields_trackers, | ||||
:time_entries, | ||||
:journals, | ||||
:journal_details, | ||||
|
r6175 | :queries, | ||
:attachments | ||||
|
r3196 | |||
def setup | ||||
Setting.rest_api_enabled = '1' | ||||
end | ||||
|
r3651 | |||
context "/index.xml" do | ||||
|
r4375 | # Use a private project to make sure auth is really working and not just | ||
# only showing public issues. | ||||
|
r4250 | should_allow_api_authentication(:get, "/projects/private-child/issues.xml") | ||
|
r4375 | |||
should "contain metadata" do | ||||
get '/issues.xml' | ||||
assert_tag :tag => 'issues', | ||||
:attributes => { | ||||
:type => 'array', | ||||
:total_count => assigns(:issue_count), | ||||
:limit => 25, | ||||
:offset => 0 | ||||
} | ||||
end | ||||
context "with offset and limit" do | ||||
should "use the params" do | ||||
get '/issues.xml?offset=2&limit=3' | ||||
assert_equal 3, assigns(:limit) | ||||
assert_equal 2, assigns(:offset) | ||||
assert_tag :tag => 'issues', :children => {:count => 3, :only => {:tag => 'issue'}} | ||||
end | ||||
end | ||||
context "with nometa param" do | ||||
should "not contain metadata" do | ||||
get '/issues.xml?nometa=1' | ||||
assert_tag :tag => 'issues', | ||||
:attributes => { | ||||
:type => 'array', | ||||
:total_count => nil, | ||||
:limit => nil, | ||||
:offset => nil | ||||
} | ||||
end | ||||
end | ||||
context "with nometa header" do | ||||
should "not contain metadata" do | ||||
get '/issues.xml', {}, {'X-Redmine-Nometa' => '1'} | ||||
assert_tag :tag => 'issues', | ||||
:attributes => { | ||||
:type => 'array', | ||||
:total_count => nil, | ||||
:limit => nil, | ||||
:offset => nil | ||||
} | ||||
end | ||||
end | ||||
|
r3196 | end | ||
|
r3652 | |||
context "/index.json" do | ||||
|
r4250 | should_allow_api_authentication(:get, "/projects/private-child/issues.json") | ||
|
r3652 | end | ||
|
r3651 | context "/index.xml with filter" do | ||
|
r4250 | should_allow_api_authentication(:get, "/projects/private-child/issues.xml?status_id=5") | ||
|
r3651 | should "show only issues with the status_id" do | ||
|
r4250 | get '/issues.xml?status_id=5' | ||
|
r3651 | assert_tag :tag => 'issues', | ||
:children => { :count => Issue.visible.count(:conditions => {:status_id => 5}), | ||||
:only => { :tag => 'issue' } } | ||||
end | ||||
|
r3196 | end | ||
|
r3651 | |||
|
r3652 | context "/index.json with filter" do | ||
|
r4250 | should_allow_api_authentication(:get, "/projects/private-child/issues.json?status_id=5") | ||
|
r3652 | |||
should "show only issues with the status_id" do | ||||
|
r4250 | get '/issues.json?status_id=5' | ||
|
r3652 | json = ActiveSupport::JSON.decode(response.body) | ||
|
r4344 | status_ids_used = json['issues'].collect {|j| j['status']['id'] } | ||
|
r3652 | assert_equal 3, status_ids_used.length | ||
assert status_ids_used.all? {|id| id == 5 } | ||||
end | ||||
end | ||||
|
r4250 | # Issue 6 is on a private project | ||
context "/issues/6.xml" do | ||||
should_allow_api_authentication(:get, "/issues/6.xml") | ||||
|
r3651 | end | ||
|
r4250 | context "/issues/6.json" do | ||
should_allow_api_authentication(:get, "/issues/6.json") | ||||
|
r3652 | end | ||
|
r4351 | |||
context "GET /issues/:id" do | ||||
|
r4369 | context "with journals" do | ||
context ".xml" do | ||||
should "display journals" do | ||||
|
r4372 | get '/issues/1.xml?include=journals' | ||
|
r4369 | |||
assert_tag :tag => 'issue', | ||||
:child => { | ||||
:tag => 'journals', | ||||
:attributes => { :type => 'array' }, | ||||
:child => { | ||||
:tag => 'journal', | ||||
:attributes => { :id => '1'}, | ||||
:child => { | ||||
:tag => 'details', | ||||
:attributes => { :type => 'array' }, | ||||
:child => { | ||||
:tag => 'detail', | ||||
:attributes => { :name => 'status_id' }, | ||||
:child => { | ||||
:tag => 'old_value', | ||||
:content => '1', | ||||
:sibling => { | ||||
:tag => 'new_value', | ||||
:content => '2' | ||||
} | ||||
} | ||||
} | ||||
} | ||||
} | ||||
} | ||||
end | ||||
end | ||||
end | ||||
|
r4366 | context "with custom fields" do | ||
context ".xml" do | ||||
should "display custom fields" do | ||||
get '/issues/3.xml' | ||||
assert_tag :tag => 'issue', | ||||
:child => { | ||||
:tag => 'custom_fields', | ||||
:attributes => { :type => 'array' }, | ||||
:child => { | ||||
:tag => 'custom_field', | ||||
:attributes => { :id => '1'}, | ||||
:child => { | ||||
:tag => 'value', | ||||
:content => 'MySQL' | ||||
} | ||||
} | ||||
} | ||||
assert_nothing_raised do | ||||
Hash.from_xml(response.body).to_xml | ||||
end | ||||
end | ||||
end | ||||
end | ||||
|
r6175 | context "with attachments" do | ||
context ".xml" do | ||||
should "display attachments" do | ||||
get '/issues/3.xml?include=attachments' | ||||
assert_tag :tag => 'issue', | ||||
:child => { | ||||
:tag => 'attachments', | ||||
:children => {:count => 5}, | ||||
:child => { | ||||
:tag => 'attachment', | ||||
:child => { | ||||
:tag => 'filename', | ||||
:content => 'source.rb', | ||||
:sibling => { | ||||
:tag => 'content_url', | ||||
:content => 'http://www.example.com/attachments/download/4/source.rb' | ||||
} | ||||
} | ||||
} | ||||
} | ||||
end | ||||
end | ||||
end | ||||
|
r4351 | context "with subtasks" do | ||
setup do | ||||
@c1 = Issue.generate!(:status_id => 1, :subject => "child c1", :tracker_id => 1, :project_id => 1, :parent_issue_id => 1) | ||||
@c2 = Issue.generate!(:status_id => 1, :subject => "child c2", :tracker_id => 1, :project_id => 1, :parent_issue_id => 1) | ||||
@c3 = Issue.generate!(:status_id => 1, :subject => "child c3", :tracker_id => 1, :project_id => 1, :parent_issue_id => @c1.id) | ||||
end | ||||
context ".xml" do | ||||
should "display children" do | ||||
|
r4372 | get '/issues/1.xml?include=children' | ||
|
r4351 | |||
assert_tag :tag => 'issue', | ||||
:child => { | ||||
:tag => 'children', | ||||
:children => {:count => 2}, | ||||
:child => { | ||||
:tag => 'issue', | ||||
:attributes => {:id => @c1.id.to_s}, | ||||
:child => { | ||||
:tag => 'subject', | ||||
:content => 'child c1', | ||||
:sibling => { | ||||
:tag => 'children', | ||||
:children => {:count => 1}, | ||||
:child => { | ||||
:tag => 'issue', | ||||
:attributes => {:id => @c3.id.to_s} | ||||
} | ||||
} | ||||
} | ||||
} | ||||
} | ||||
end | ||||
context ".json" do | ||||
should "display children" do | ||||
|
r4372 | get '/issues/1.json?include=children' | ||
|
r4351 | |||
json = ActiveSupport::JSON.decode(response.body) | ||||
assert_equal([ | ||||
{ | ||||
'id' => @c1.id, 'subject' => 'child c1', 'tracker' => {'id' => 1, 'name' => 'Bug'}, | ||||
'children' => [{ 'id' => @c3.id, 'subject' => 'child c3', 'tracker' => {'id' => 1, 'name' => 'Bug'} }] | ||||
}, | ||||
{ 'id' => @c2.id, 'subject' => 'child c2', 'tracker' => {'id' => 1, 'name' => 'Bug'} } | ||||
], | ||||
json['issue']['children']) | ||||
end | ||||
end | ||||
end | ||||
end | ||||
end | ||||
|
r3652 | |||
|
r3651 | context "POST /issues.xml" do | ||
|
r4251 | should_allow_api_authentication(:post, | ||
'/issues.xml', | ||||
{:issue => {:project_id => 1, :subject => 'API test', :tracker_id => 2, :status_id => 3}}, | ||||
{:success_code => :created}) | ||||
|
r3651 | |||
should "create an issue with the attributes" do | ||||
|
r4251 | assert_difference('Issue.count') do | ||
post '/issues.xml', {:issue => {:project_id => 1, :subject => 'API test', :tracker_id => 2, :status_id => 3}}, :authorization => credentials('jsmith') | ||||
|
r3651 | end | ||
|
r4251 | |||
issue = Issue.first(:order => 'id DESC') | ||||
assert_equal 1, issue.project_id | ||||
assert_equal 2, issue.tracker_id | ||||
assert_equal 3, issue.status_id | ||||
assert_equal 'API test', issue.subject | ||||
|
r4283 | |||
assert_response :created | ||||
assert_equal 'application/xml', @response.content_type | ||||
assert_tag 'issue', :child => {:tag => 'id', :content => issue.id.to_s} | ||||
|
r3196 | end | ||
end | ||||
|
r3651 | context "POST /issues.xml with failure" do | ||
|
r4251 | should_allow_api_authentication(:post, | ||
'/issues.xml', | ||||
{:issue => {:project_id => 1}}, | ||||
{:success_code => :unprocessable_entity}) | ||||
|
r3651 | |||
should "have an errors tag" do | ||||
|
r4251 | assert_no_difference('Issue.count') do | ||
post '/issues.xml', {:issue => {:project_id => 1}}, :authorization => credentials('jsmith') | ||||
end | ||||
|
r3651 | assert_tag :errors, :child => {:tag => 'error', :content => "Subject can't be blank"} | ||
end | ||||
|
r3196 | end | ||
|
r3652 | |||
context "POST /issues.json" do | ||||
|
r4251 | should_allow_api_authentication(:post, | ||
'/issues.json', | ||||
{:issue => {:project_id => 1, :subject => 'API test', :tracker_id => 2, :status_id => 3}}, | ||||
{:success_code => :created}) | ||||
|
r3652 | |||
should "create an issue with the attributes" do | ||||
|
r4251 | assert_difference('Issue.count') do | ||
post '/issues.json', {:issue => {:project_id => 1, :subject => 'API test', :tracker_id => 2, :status_id => 3}}, :authorization => credentials('jsmith') | ||||
|
r3652 | end | ||
|
r4251 | |||
issue = Issue.first(:order => 'id DESC') | ||||
assert_equal 1, issue.project_id | ||||
assert_equal 2, issue.tracker_id | ||||
assert_equal 3, issue.status_id | ||||
assert_equal 'API test', issue.subject | ||||
|
r3652 | end | ||
|
r4251 | |||
|
r3652 | end | ||
context "POST /issues.json with failure" do | ||||
|
r4251 | should_allow_api_authentication(:post, | ||
'/issues.json', | ||||
{:issue => {:project_id => 1}}, | ||||
{:success_code => :unprocessable_entity}) | ||||
|
r3652 | |||
should "have an errors element" do | ||||
|
r4251 | assert_no_difference('Issue.count') do | ||
post '/issues.json', {:issue => {:project_id => 1}}, :authorization => credentials('jsmith') | ||||
end | ||||
|
r3652 | json = ActiveSupport::JSON.decode(response.body) | ||
|
r4344 | assert json['errors'].include?(['subject', "can't be blank"]) | ||
|
r3652 | end | ||
end | ||||
|
r4252 | # Issue 6 is on a private project | ||
context "PUT /issues/6.xml" do | ||||
|
r3651 | setup do | ||
|
r4252 | @parameters = {:issue => {:subject => 'API update', :notes => 'A new note'}} | ||
@headers = { :authorization => credentials('jsmith') } | ||||
|
r3196 | end | ||
|
r3651 | |||
|
r4252 | should_allow_api_authentication(:put, | ||
'/issues/6.xml', | ||||
{:issue => {:subject => 'API update', :notes => 'A new note'}}, | ||||
{:success_code => :ok}) | ||||
|
r3651 | |||
should "not create a new issue" do | ||||
|
r4252 | assert_no_difference('Issue.count') do | ||
put '/issues/6.xml', @parameters, @headers | ||||
end | ||||
|
r3196 | end | ||
|
r3651 | |||
should "create a new journal" do | ||||
|
r4252 | assert_difference('Journal.count') do | ||
put '/issues/6.xml', @parameters, @headers | ||||
end | ||||
|
r3651 | end | ||
|
r3934 | should "add the note to the journal" do | ||
|
r4252 | put '/issues/6.xml', @parameters, @headers | ||
|
r3934 | journal = Journal.last | ||
assert_equal "A new note", journal.notes | ||||
end | ||||
|
r3651 | should "update the issue" do | ||
|
r4252 | put '/issues/6.xml', @parameters, @headers | ||
issue = Issue.find(6) | ||||
assert_equal "API update", issue.subject | ||||
|
r3196 | end | ||
|
r3651 | |||
|
r3196 | end | ||
|
r3651 | |||
|
r4367 | context "PUT /issues/3.xml with custom fields" do | ||
setup do | ||||
@parameters = {:issue => {:custom_fields => [{'id' => '1', 'value' => 'PostgreSQL' }, {'id' => '2', 'value' => '150'}]}} | ||||
@headers = { :authorization => credentials('jsmith') } | ||||
end | ||||
should "update custom fields" do | ||||
assert_no_difference('Issue.count') do | ||||
put '/issues/3.xml', @parameters, @headers | ||||
end | ||||
issue = Issue.find(3) | ||||
assert_equal '150', issue.custom_value_for(2).value | ||||
assert_equal 'PostgreSQL', issue.custom_value_for(1).value | ||||
end | ||||
end | ||||
|
r4252 | context "PUT /issues/6.xml with failed update" do | ||
|
r3651 | setup do | ||
|
r4252 | @parameters = {:issue => {:subject => ''}} | ||
@headers = { :authorization => credentials('jsmith') } | ||||
|
r3651 | end | ||
|
r4252 | |||
should_allow_api_authentication(:put, | ||||
'/issues/6.xml', | ||||
{:issue => {:subject => ''}}, # Missing subject should fail | ||||
{:success_code => :unprocessable_entity}) | ||||
|
r3651 | should "not create a new issue" do | ||
|
r4252 | assert_no_difference('Issue.count') do | ||
put '/issues/6.xml', @parameters, @headers | ||||
end | ||||
|
r3651 | end | ||
should "not create a new journal" do | ||||
|
r4252 | assert_no_difference('Journal.count') do | ||
put '/issues/6.xml', @parameters, @headers | ||||
end | ||||
|
r3651 | end | ||
should "have an errors tag" do | ||||
|
r4252 | put '/issues/6.xml', @parameters, @headers | ||
|
r3651 | assert_tag :errors, :child => {:tag => 'error', :content => "Subject can't be blank"} | ||
end | ||||
end | ||||
|
r4252 | context "PUT /issues/6.json" do | ||
|
r3652 | setup do | ||
|
r4252 | @parameters = {:issue => {:subject => 'API update', :notes => 'A new note'}} | ||
@headers = { :authorization => credentials('jsmith') } | ||||
|
r3652 | end | ||
|
r4252 | should_allow_api_authentication(:put, | ||
'/issues/6.json', | ||||
{:issue => {:subject => 'API update', :notes => 'A new note'}}, | ||||
{:success_code => :ok}) | ||||
|
r3652 | |||
should "not create a new issue" do | ||||
|
r4252 | assert_no_difference('Issue.count') do | ||
put '/issues/6.json', @parameters, @headers | ||||
end | ||||
|
r3652 | end | ||
should "create a new journal" do | ||||
|
r4252 | assert_difference('Journal.count') do | ||
put '/issues/6.json', @parameters, @headers | ||||
end | ||||
|
r3652 | end | ||
|
r3934 | should "add the note to the journal" do | ||
|
r4252 | put '/issues/6.json', @parameters, @headers | ||
|
r3934 | journal = Journal.last | ||
assert_equal "A new note", journal.notes | ||||
end | ||||
|
r3652 | should "update the issue" do | ||
|
r4252 | put '/issues/6.json', @parameters, @headers | ||
issue = Issue.find(6) | ||||
assert_equal "API update", issue.subject | ||||
|
r3652 | end | ||
|
r4252 | |||
|
r3652 | end | ||
|
r4252 | context "PUT /issues/6.json with failed update" do | ||
|
r3652 | setup do | ||
|
r4252 | @parameters = {:issue => {:subject => ''}} | ||
@headers = { :authorization => credentials('jsmith') } | ||||
|
r3652 | end | ||
|
r4252 | |||
should_allow_api_authentication(:put, | ||||
'/issues/6.json', | ||||
{:issue => {:subject => ''}}, # Missing subject should fail | ||||
{:success_code => :unprocessable_entity}) | ||||
|
r3652 | should "not create a new issue" do | ||
|
r4252 | assert_no_difference('Issue.count') do | ||
put '/issues/6.json', @parameters, @headers | ||||
end | ||||
|
r3652 | end | ||
should "not create a new journal" do | ||||
|
r4252 | assert_no_difference('Journal.count') do | ||
put '/issues/6.json', @parameters, @headers | ||||
end | ||||
|
r3652 | end | ||
should "have an errors attribute" do | ||||
|
r4252 | put '/issues/6.json', @parameters, @headers | ||
|
r3652 | json = ActiveSupport::JSON.decode(response.body) | ||
|
r4344 | assert json['errors'].include?(['subject', "can't be blank"]) | ||
|
r3652 | end | ||
end | ||||
|
r3651 | context "DELETE /issues/1.xml" do | ||
|
r4253 | should_allow_api_authentication(:delete, | ||
'/issues/6.xml', | ||||
{}, | ||||
{:success_code => :ok}) | ||||
|
r3651 | |||
should "delete the issue" do | ||||
|
r4253 | assert_difference('Issue.count',-1) do | ||
delete '/issues/6.xml', {}, :authorization => credentials('jsmith') | ||||
end | ||||
assert_nil Issue.find_by_id(6) | ||||
|
r3651 | end | ||
|
r3196 | end | ||
|
r3652 | |||
context "DELETE /issues/1.json" do | ||||
|
r4253 | should_allow_api_authentication(:delete, | ||
'/issues/6.json', | ||||
{}, | ||||
{:success_code => :ok}) | ||||
|
r3652 | |||
should "delete the issue" do | ||||
|
r4253 | assert_difference('Issue.count',-1) do | ||
delete '/issues/6.json', {}, :authorization => credentials('jsmith') | ||||
end | ||||
assert_nil Issue.find_by_id(6) | ||||
|
r3652 | end | ||
end | ||||
|
r3196 | def credentials(user, password=nil) | ||
ActionController::HttpAuthentication::Basic.encode_credentials(user, password || user) | ||||
end | ||||
end | ||||