1 # Redmine - project management software
2 # Copyright (C) 2006-2008 Jean-Philippe Lang
4 # This program is free software; you can redistribute it and/or
5 # modify it under the terms of the GNU General Public License
6 # as published by the Free Software Foundation; either version 2
7 # of the License, or (at your option) any later version.
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
14 # You should have received a copy of the GNU General Public License
15 # along with this program; if not, write to the Free Software
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18 require File.dirname(__FILE__) + '/../test_helper'
19 require 'issues_controller'
21 # Re-raise errors caught by the controller.
22 class IssuesController; def rescue_action(e) raise e end; end
24 class IssuesControllerTest < ActionController::TestCase
42 :custom_fields_projects,
43 :custom_fields_trackers,
50 @controller = IssuesController.new
51 @request = ActionController::TestRequest.new
52 @response = ActionController::TestResponse.new
56 def test_index_routing
58 {:method => :get, :path => '/issues'},
59 :controller => 'issues', :action => 'index'
64 Setting.default_language = 'en'
67 assert_response :success
68 assert_template 'index.rhtml'
69 assert_not_nil assigns(:issues)
70 assert_nil assigns(:project)
71 assert_tag :tag => 'a', :content => /Can't print recipes/
72 assert_tag :tag => 'a', :content => /Subproject issue/
73 # private projects hidden
74 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
75 assert_no_tag :tag => 'a', :content => /Issue on project 2/
77 assert_tag :tag => 'th', :content => /Project/
80 def test_index_should_not_list_issues_when_module_disabled
81 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
83 assert_response :success
84 assert_template 'index.rhtml'
85 assert_not_nil assigns(:issues)
86 assert_nil assigns(:project)
87 assert_no_tag :tag => 'a', :content => /Can't print recipes/
88 assert_tag :tag => 'a', :content => /Subproject issue/
91 def test_index_with_project_routing
93 {:method => :get, :path => '/projects/23/issues'},
94 :controller => 'issues', :action => 'index', :project_id => '23'
98 def test_index_should_not_list_issues_when_module_disabled
99 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
101 assert_response :success
102 assert_template 'index.rhtml'
103 assert_not_nil assigns(:issues)
104 assert_nil assigns(:project)
105 assert_no_tag :tag => 'a', :content => /Can't print recipes/
106 assert_tag :tag => 'a', :content => /Subproject issue/
109 def test_index_with_project_routing
111 {:method => :get, :path => 'projects/23/issues'},
112 :controller => 'issues', :action => 'index', :project_id => '23'
116 def test_index_with_project
117 Setting.display_subprojects_issues = 0
118 get :index, :project_id => 1
119 assert_response :success
120 assert_template 'index.rhtml'
121 assert_not_nil assigns(:issues)
122 assert_tag :tag => 'a', :content => /Can't print recipes/
123 assert_no_tag :tag => 'a', :content => /Subproject issue/
126 def test_index_with_project_and_subprojects
127 Setting.display_subprojects_issues = 1
128 get :index, :project_id => 1
129 assert_response :success
130 assert_template 'index.rhtml'
131 assert_not_nil assigns(:issues)
132 assert_tag :tag => 'a', :content => /Can't print recipes/
133 assert_tag :tag => 'a', :content => /Subproject issue/
134 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
137 def test_index_with_project_and_subprojects_should_show_private_subprojects
138 @request.session[:user_id] = 2
139 Setting.display_subprojects_issues = 1
140 get :index, :project_id => 1
141 assert_response :success
142 assert_template 'index.rhtml'
143 assert_not_nil assigns(:issues)
144 assert_tag :tag => 'a', :content => /Can't print recipes/
145 assert_tag :tag => 'a', :content => /Subproject issue/
146 assert_tag :tag => 'a', :content => /Issue of a private subproject/
149 def test_index_with_project_routing_formatted
151 {:method => :get, :path => 'projects/23/issues.pdf'},
152 :controller => 'issues', :action => 'index', :project_id => '23', :format => 'pdf'
155 {:method => :get, :path => 'projects/23/issues.atom'},
156 :controller => 'issues', :action => 'index', :project_id => '23', :format => 'atom'
160 def test_index_with_project_and_filter
161 get :index, :project_id => 1, :set_filter => 1
162 assert_response :success
163 assert_template 'index.rhtml'
164 assert_not_nil assigns(:issues)
167 def test_index_with_query
168 get :index, :project_id => 1, :query_id => 5
169 assert_response :success
170 assert_template 'index.rhtml'
171 assert_not_nil assigns(:issues)
172 assert_nil assigns(:issue_count_by_group)
175 def test_index_with_query_grouped_by_tracker
176 get :index, :project_id => 1, :query_id => 6
177 assert_response :success
178 assert_template 'index.rhtml'
179 assert_not_nil assigns(:issues)
180 assert_not_nil assigns(:issue_count_by_group)
183 def test_index_with_query_grouped_by_list_custom_field
184 get :index, :project_id => 1, :query_id => 9
185 assert_response :success
186 assert_template 'index.rhtml'
187 assert_not_nil assigns(:issues)
188 assert_not_nil assigns(:issue_count_by_group)
191 def test_index_sort_by_field_not_included_in_columns
192 Setting.issue_list_default_columns = %w(subject author)
193 get :index, :sort => 'tracker'
196 def test_index_csv_with_project
197 Setting.default_language = 'en'
199 get :index, :format => 'csv'
200 assert_response :success
201 assert_not_nil assigns(:issues)
202 assert_equal 'text/csv', @response.content_type
203 assert @response.body.starts_with?("#,")
205 get :index, :project_id => 1, :format => 'csv'
206 assert_response :success
207 assert_not_nil assigns(:issues)
208 assert_equal 'text/csv', @response.content_type
211 def test_index_formatted
213 {:method => :get, :path => 'issues.pdf'},
214 :controller => 'issues', :action => 'index', :format => 'pdf'
217 {:method => :get, :path => 'issues.atom'},
218 :controller => 'issues', :action => 'index', :format => 'atom'
223 get :index, :format => 'pdf'
224 assert_response :success
225 assert_not_nil assigns(:issues)
226 assert_equal 'application/pdf', @response.content_type
228 get :index, :project_id => 1, :format => 'pdf'
229 assert_response :success
230 assert_not_nil assigns(:issues)
231 assert_equal 'application/pdf', @response.content_type
233 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
234 assert_response :success
235 assert_not_nil assigns(:issues)
236 assert_equal 'application/pdf', @response.content_type
240 get :index, :sort => 'tracker,id:desc'
241 assert_response :success
243 sort_params = @request.session['issues_index_sort']
244 assert sort_params.is_a?(String)
245 assert_equal 'tracker,id:desc', sort_params
247 issues = assigns(:issues)
248 assert_not_nil issues
249 assert !issues.empty?
250 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
253 def test_index_with_columns
254 columns = ['tracker', 'subject', 'assigned_to']
255 get :index, :set_filter => 1, :query => { 'column_names' => columns}
256 assert_response :success
258 # query should use specified columns
259 query = assigns(:query)
260 assert_kind_of Query, query
261 assert_equal columns, query.column_names.map(&:to_s)
263 # columns should be stored in session
264 assert_kind_of Hash, session[:query]
265 assert_kind_of Array, session[:query][:column_names]
266 assert_equal columns, session[:query][:column_names].map(&:to_s)
270 get :gantt, :project_id => 1
271 assert_response :success
272 assert_template 'gantt.rhtml'
273 assert_not_nil assigns(:gantt)
274 events = assigns(:gantt).events
275 assert_not_nil events
276 # Issue with start and due dates
278 assert_not_nil i.due_date
279 assert events.include?(Issue.find(1))
280 # Issue with without due date but targeted to a version with date
282 assert_nil i.due_date
283 assert events.include?(i)
286 def test_cross_project_gantt
288 assert_response :success
289 assert_template 'gantt.rhtml'
290 assert_not_nil assigns(:gantt)
291 events = assigns(:gantt).events
292 assert_not_nil events
295 def test_gantt_export_to_pdf
296 get :gantt, :project_id => 1, :format => 'pdf'
297 assert_response :success
298 assert_equal 'application/pdf', @response.content_type
299 assert @response.body.starts_with?('%PDF')
300 assert_not_nil assigns(:gantt)
303 def test_cross_project_gantt_export_to_pdf
304 get :gantt, :format => 'pdf'
305 assert_response :success
306 assert_equal 'application/pdf', @response.content_type
307 assert @response.body.starts_with?('%PDF')
308 assert_not_nil assigns(:gantt)
311 if Object.const_defined?(:Magick)
313 get :gantt, :project_id => 1, :format => 'png'
314 assert_response :success
315 assert_equal 'image/png', @response.content_type
318 puts "RMagick not installed. Skipping tests !!!"
322 get :calendar, :project_id => 1
323 assert_response :success
324 assert_template 'calendar'
325 assert_not_nil assigns(:calendar)
328 def test_cross_project_calendar
330 assert_response :success
331 assert_template 'calendar'
332 assert_not_nil assigns(:calendar)
336 get :changes, :project_id => 1
337 assert_response :success
338 assert_not_nil assigns(:journals)
339 assert_equal 'application/atom+xml', @response.content_type
342 def test_show_routing
344 {:method => :get, :path => '/issues/64'},
345 :controller => 'issues', :action => 'show', :id => '64'
349 def test_show_routing_formatted
351 {:method => :get, :path => '/issues/2332.pdf'},
352 :controller => 'issues', :action => 'show', :id => '2332', :format => 'pdf'
355 {:method => :get, :path => '/issues/23123.atom'},
356 :controller => 'issues', :action => 'show', :id => '23123', :format => 'atom'
360 def test_show_by_anonymous
362 assert_response :success
363 assert_template 'show.rhtml'
364 assert_not_nil assigns(:issue)
365 assert_equal Issue.find(1), assigns(:issue)
367 # anonymous role is allowed to add a note
368 assert_tag :tag => 'form',
369 :descendant => { :tag => 'fieldset',
370 :child => { :tag => 'legend',
371 :content => /Notes/ } }
374 def test_show_by_manager
375 @request.session[:user_id] = 2
377 assert_response :success
379 assert_tag :tag => 'form',
380 :descendant => { :tag => 'fieldset',
381 :child => { :tag => 'legend',
382 :content => /Change properties/ } },
383 :descendant => { :tag => 'fieldset',
384 :child => { :tag => 'legend',
385 :content => /Log time/ } },
386 :descendant => { :tag => 'fieldset',
387 :child => { :tag => 'legend',
388 :content => /Notes/ } }
391 def test_show_should_deny_anonymous_access_without_permission
392 Role.anonymous.remove_permission!(:view_issues)
394 assert_response :redirect
397 def test_show_should_deny_non_member_access_without_permission
398 Role.non_member.remove_permission!(:view_issues)
399 @request.session[:user_id] = 9
404 def test_show_should_deny_member_access_without_permission
405 Role.find(1).remove_permission!(:view_issues)
406 @request.session[:user_id] = 2
411 def test_show_should_not_disclose_relations_to_invisible_issues
412 Setting.cross_project_issue_relations = '1'
413 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
414 # Relation to a private project issue
415 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
418 assert_response :success
420 assert_tag :div, :attributes => { :id => 'relations' },
421 :descendant => { :tag => 'a', :content => /#2$/ }
422 assert_no_tag :div, :attributes => { :id => 'relations' },
423 :descendant => { :tag => 'a', :content => /#4$/ }
427 get :show, :id => 2, :format => 'atom'
428 assert_response :success
429 assert_template 'changes.rxml'
431 assert @response.body.include?("<img src=\"http://test.host/attachments/download/10\" alt=\"\" />"), "Body did not match. Body: #{@response.body}"
436 {:method => :get, :path => '/projects/1/issues/new'},
437 :controller => 'issues', :action => 'new', :project_id => '1'
440 {:controller => 'issues', :action => 'new', :project_id => '1'},
441 {:method => :post, :path => '/projects/1/issues'}
445 def test_show_export_to_pdf
446 get :show, :id => 3, :format => 'pdf'
447 assert_response :success
448 assert_equal 'application/pdf', @response.content_type
449 assert @response.body.starts_with?('%PDF')
450 assert_not_nil assigns(:issue)
454 @request.session[:user_id] = 2
455 get :new, :project_id => 1, :tracker_id => 1
456 assert_response :success
457 assert_template 'new'
459 assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]',
460 :value => 'Default string' }
463 def test_get_new_without_tracker_id
464 @request.session[:user_id] = 2
465 get :new, :project_id => 1
466 assert_response :success
467 assert_template 'new'
469 issue = assigns(:issue)
471 assert_equal Project.find(1).trackers.first, issue.tracker
474 def test_get_new_with_no_default_status_should_display_an_error
475 @request.session[:user_id] = 2
476 IssueStatus.delete_all
478 get :new, :project_id => 1
480 assert_not_nil flash[:error]
481 assert_tag :tag => 'div', :attributes => { :class => /error/ },
482 :content => /No default issue/
485 def test_get_new_with_no_tracker_should_display_an_error
486 @request.session[:user_id] = 2
489 get :new, :project_id => 1
491 assert_not_nil flash[:error]
492 assert_tag :tag => 'div', :attributes => { :class => /error/ },
493 :content => /No tracker/
496 def test_update_new_form
497 @request.session[:user_id] = 2
498 xhr :post, :update_form, :project_id => 1,
499 :issue => {:tracker_id => 2,
500 :subject => 'This is the test_new issue',
501 :description => 'This is the description',
503 assert_response :success
504 assert_template 'attributes'
506 issue = assigns(:issue)
507 assert_kind_of Issue, issue
508 assert_equal 1, issue.project_id
509 assert_equal 2, issue.tracker_id
510 assert_equal 'This is the test_new issue', issue.subject
514 @request.session[:user_id] = 2
515 assert_difference 'Issue.count' do
516 post :new, :project_id => 1,
517 :issue => {:tracker_id => 3,
518 :subject => 'This is the test_new issue',
519 :description => 'This is the description',
521 :estimated_hours => '',
522 :custom_field_values => {'2' => 'Value for field 2'}}
524 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
526 issue = Issue.find_by_subject('This is the test_new issue')
528 assert_equal 2, issue.author_id
529 assert_equal 3, issue.tracker_id
530 assert_nil issue.estimated_hours
531 v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
533 assert_equal 'Value for field 2', v.value
536 def test_post_new_and_continue
537 @request.session[:user_id] = 2
538 post :new, :project_id => 1,
539 :issue => {:tracker_id => 3,
540 :subject => 'This is first issue',
543 assert_redirected_to :controller => 'issues', :action => 'new', :tracker_id => 3
546 def test_post_new_without_custom_fields_param
547 @request.session[:user_id] = 2
548 assert_difference 'Issue.count' do
549 post :new, :project_id => 1,
550 :issue => {:tracker_id => 1,
551 :subject => 'This is the test_new issue',
552 :description => 'This is the description',
555 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
558 def test_post_new_with_required_custom_field_and_without_custom_fields_param
559 field = IssueCustomField.find_by_name('Database')
560 field.update_attribute(:is_required, true)
562 @request.session[:user_id] = 2
563 post :new, :project_id => 1,
564 :issue => {:tracker_id => 1,
565 :subject => 'This is the test_new issue',
566 :description => 'This is the description',
568 assert_response :success
569 assert_template 'new'
570 issue = assigns(:issue)
572 assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values)
575 def test_post_new_with_watchers
576 @request.session[:user_id] = 2
577 ActionMailer::Base.deliveries.clear
579 assert_difference 'Watcher.count', 2 do
580 post :new, :project_id => 1,
581 :issue => {:tracker_id => 1,
582 :subject => 'This is a new issue with watchers',
583 :description => 'This is the description',
585 :watcher_user_ids => ['2', '3']}
587 issue = Issue.find_by_subject('This is a new issue with watchers')
589 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
592 assert_equal [2, 3], issue.watcher_user_ids.sort
593 assert issue.watched_by?(User.find(3))
595 mail = ActionMailer::Base.deliveries.last
596 assert_kind_of TMail::Mail, mail
597 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
600 def test_post_new_should_send_a_notification
601 ActionMailer::Base.deliveries.clear
602 @request.session[:user_id] = 2
603 assert_difference 'Issue.count' do
604 post :new, :project_id => 1,
605 :issue => {:tracker_id => 3,
606 :subject => 'This is the test_new issue',
607 :description => 'This is the description',
609 :estimated_hours => '',
610 :custom_field_values => {'2' => 'Value for field 2'}}
612 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
614 assert_equal 1, ActionMailer::Base.deliveries.size
617 def test_post_should_preserve_fields_values_on_validation_failure
618 @request.session[:user_id] = 2
619 post :new, :project_id => 1,
620 :issue => {:tracker_id => 1,
623 :description => 'This is a description',
625 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
626 assert_response :success
627 assert_template 'new'
629 assert_tag :textarea, :attributes => { :name => 'issue[description]' },
630 :content => 'This is a description'
631 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
632 :child => { :tag => 'option', :attributes => { :selected => 'selected',
636 assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
637 :child => { :tag => 'option', :attributes => { :selected => 'selected',
638 :value => 'Oracle' },
639 :content => 'Oracle' }
640 assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
641 :value => 'Value for field 2'}
644 def test_copy_routing
646 {:method => :get, :path => '/projects/world_domination/issues/567/copy'},
647 :controller => 'issues', :action => 'new', :project_id => 'world_domination', :copy_from => '567'
652 @request.session[:user_id] = 2
653 get :new, :project_id => 1, :copy_from => 1
654 assert_template 'new'
655 assert_not_nil assigns(:issue)
657 assert_equal orig.subject, assigns(:issue).subject
660 def test_edit_routing
662 {:method => :get, :path => '/issues/1/edit'},
663 :controller => 'issues', :action => 'edit', :id => '1'
665 assert_recognizes( #TODO: use a PUT on the issue URI isntead, need to adjust form
666 {:controller => 'issues', :action => 'edit', :id => '1'},
667 {:method => :post, :path => '/issues/1/edit'}
672 @request.session[:user_id] = 2
674 assert_response :success
675 assert_template 'edit'
676 assert_not_nil assigns(:issue)
677 assert_equal Issue.find(1), assigns(:issue)
680 def test_get_edit_with_params
681 @request.session[:user_id] = 2
682 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 }
683 assert_response :success
684 assert_template 'edit'
686 issue = assigns(:issue)
689 assert_equal 5, issue.status_id
690 assert_tag :select, :attributes => { :name => 'issue[status_id]' },
691 :child => { :tag => 'option',
692 :content => 'Closed',
693 :attributes => { :selected => 'selected' } }
695 assert_equal 7, issue.priority_id
696 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
697 :child => { :tag => 'option',
698 :content => 'Urgent',
699 :attributes => { :selected => 'selected' } }
702 def test_update_edit_form
703 @request.session[:user_id] = 2
704 xhr :post, :update_form, :project_id => 1,
706 :issue => {:tracker_id => 2,
707 :subject => 'This is the test_new issue',
708 :description => 'This is the description',
710 assert_response :success
711 assert_template 'attributes'
713 issue = assigns(:issue)
714 assert_kind_of Issue, issue
715 assert_equal 1, issue.id
716 assert_equal 1, issue.project_id
717 assert_equal 2, issue.tracker_id
718 assert_equal 'This is the test_new issue', issue.subject
721 def test_reply_routing
723 {:method => :post, :path => '/issues/1/quoted'},
724 :controller => 'issues', :action => 'reply', :id => '1'
728 def test_reply_to_issue
729 @request.session[:user_id] = 2
731 assert_response :success
732 assert_select_rjs :show, "update"
735 def test_reply_to_note
736 @request.session[:user_id] = 2
737 get :reply, :id => 1, :journal_id => 2
738 assert_response :success
739 assert_select_rjs :show, "update"
742 def test_post_edit_without_custom_fields_param
743 @request.session[:user_id] = 2
744 ActionMailer::Base.deliveries.clear
746 issue = Issue.find(1)
747 assert_equal '125', issue.custom_value_for(2).value
748 old_subject = issue.subject
749 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
751 assert_difference('Journal.count') do
752 assert_difference('JournalDetail.count', 2) do
753 post :edit, :id => 1, :issue => {:subject => new_subject,
755 :category_id => '1' # no change
759 assert_redirected_to :action => 'show', :id => '1'
761 assert_equal new_subject, issue.subject
762 # Make sure custom fields were not cleared
763 assert_equal '125', issue.custom_value_for(2).value
765 mail = ActionMailer::Base.deliveries.last
766 assert_kind_of TMail::Mail, mail
767 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
768 assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}")
771 def test_post_edit_with_custom_field_change
772 @request.session[:user_id] = 2
773 issue = Issue.find(1)
774 assert_equal '125', issue.custom_value_for(2).value
776 assert_difference('Journal.count') do
777 assert_difference('JournalDetail.count', 3) do
778 post :edit, :id => 1, :issue => {:subject => 'Custom field change',
780 :category_id => '1', # no change
781 :custom_field_values => { '2' => 'New custom value' }
785 assert_redirected_to :action => 'show', :id => '1'
787 assert_equal 'New custom value', issue.custom_value_for(2).value
789 mail = ActionMailer::Base.deliveries.last
790 assert_kind_of TMail::Mail, mail
791 assert mail.body.include?("Searchable field changed from 125 to New custom value")
794 def test_post_edit_with_status_and_assignee_change
795 issue = Issue.find(1)
796 assert_equal 1, issue.status_id
797 @request.session[:user_id] = 2
798 assert_difference('TimeEntry.count', 0) do
801 :issue => { :status_id => 2, :assigned_to_id => 3 },
802 :notes => 'Assigned to dlopper',
803 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
805 assert_redirected_to :action => 'show', :id => '1'
807 assert_equal 2, issue.status_id
808 j = issue.journals.find(:first, :order => 'id DESC')
809 assert_equal 'Assigned to dlopper', j.notes
810 assert_equal 2, j.details.size
812 mail = ActionMailer::Base.deliveries.last
813 assert mail.body.include?("Status changed from New to Assigned")
814 # subject should contain the new status
815 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
818 def test_post_edit_with_note_only
819 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
824 assert_redirected_to :action => 'show', :id => '1'
825 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
826 assert_equal notes, j.notes
827 assert_equal 0, j.details.size
828 assert_equal User.anonymous, j.user
830 mail = ActionMailer::Base.deliveries.last
831 assert mail.body.include?(notes)
834 def test_post_edit_with_note_and_spent_time
835 @request.session[:user_id] = 2
836 spent_hours_before = Issue.find(1).spent_hours
837 assert_difference('TimeEntry.count') do
840 :notes => '2.5 hours added',
841 :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first }
843 assert_redirected_to :action => 'show', :id => '1'
845 issue = Issue.find(1)
847 j = issue.journals.find(:first, :order => 'id DESC')
848 assert_equal '2.5 hours added', j.notes
849 assert_equal 0, j.details.size
851 t = issue.time_entries.find(:first, :order => 'id DESC')
853 assert_equal 2.5, t.hours
854 assert_equal spent_hours_before + 2.5, issue.spent_hours
857 def test_post_edit_with_attachment_only
858 set_tmp_attachments_directory
860 # Delete all fixtured journals, a race condition can occur causing the wrong
861 # journal to get fetched in the next find.
868 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
869 assert_redirected_to :action => 'show', :id => '1'
870 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
871 assert j.notes.blank?
872 assert_equal 1, j.details.size
873 assert_equal 'testfile.txt', j.details.first.value
874 assert_equal User.anonymous, j.user
876 mail = ActionMailer::Base.deliveries.last
877 assert mail.body.include?('testfile.txt')
880 def test_post_edit_with_no_change
881 issue = Issue.find(1)
883 ActionMailer::Base.deliveries.clear
888 assert_redirected_to :action => 'show', :id => '1'
891 assert issue.journals.empty?
892 # No email should be sent
893 assert ActionMailer::Base.deliveries.empty?
896 def test_post_edit_should_send_a_notification
897 @request.session[:user_id] = 2
898 ActionMailer::Base.deliveries.clear
899 issue = Issue.find(1)
900 old_subject = issue.subject
901 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
903 post :edit, :id => 1, :issue => {:subject => new_subject,
905 :category_id => '1' # no change
907 assert_equal 1, ActionMailer::Base.deliveries.size
910 def test_post_edit_with_invalid_spent_time
911 @request.session[:user_id] = 2
912 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
914 assert_no_difference('Journal.count') do
918 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
920 assert_response :success
921 assert_template 'edit'
923 assert_tag :textarea, :attributes => { :name => 'notes' },
925 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
928 def test_post_edit_should_allow_fixed_version_to_be_set_to_a_subproject
929 issue = Issue.find(2)
930 @request.session[:user_id] = 2
935 :fixed_version_id => 4
938 assert_response :redirect
940 assert_equal 4, issue.fixed_version_id
941 assert_not_equal issue.project_id, issue.fixed_version.project_id
944 def test_get_bulk_edit
945 @request.session[:user_id] = 2
946 get :bulk_edit, :ids => [1, 2]
947 assert_response :success
948 assert_template 'bulk_edit'
952 @request.session[:user_id] = 2
953 # update issues priority
954 post :bulk_edit, :ids => [1, 2], :priority_id => 7,
955 :assigned_to_id => '',
956 :custom_field_values => {'2' => ''},
957 :notes => 'Bulk editing'
959 # check that the issues were updated
960 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
962 issue = Issue.find(1)
963 journal = issue.journals.find(:first, :order => 'created_on DESC')
964 assert_equal '125', issue.custom_value_for(2).value
965 assert_equal 'Bulk editing', journal.notes
966 assert_equal 1, journal.details.size
969 def test_bullk_edit_should_send_a_notification
970 @request.session[:user_id] = 2
971 ActionMailer::Base.deliveries.clear
976 :assigned_to_id => '',
977 :custom_field_values => {'2' => ''},
978 :notes => 'Bulk editing'
982 assert_equal 2, ActionMailer::Base.deliveries.size
985 def test_bulk_edit_status
986 @request.session[:user_id] = 2
987 # update issues priority
988 post :bulk_edit, :ids => [1, 2], :priority_id => '',
989 :assigned_to_id => '',
991 :notes => 'Bulk editing status'
993 issue = Issue.find(1)
997 def test_bulk_edit_custom_field
998 @request.session[:user_id] = 2
999 # update issues priority
1000 post :bulk_edit, :ids => [1, 2], :priority_id => '',
1001 :assigned_to_id => '',
1002 :custom_field_values => {'2' => '777'},
1003 :notes => 'Bulk editing custom field'
1006 issue = Issue.find(1)
1007 journal = issue.journals.find(:first, :order => 'created_on DESC')
1008 assert_equal '777', issue.custom_value_for(2).value
1009 assert_equal 1, journal.details.size
1010 assert_equal '125', journal.details.first.old_value
1011 assert_equal '777', journal.details.first.value
1014 def test_bulk_unassign
1015 assert_not_nil Issue.find(2).assigned_to
1016 @request.session[:user_id] = 2
1018 post :bulk_edit, :ids => [1, 2], :notes => 'Bulk unassigning', :assigned_to_id => 'none'
1020 # check that the issues were updated
1021 assert_nil Issue.find(2).assigned_to
1024 def test_post_bulk_edit_should_allow_fixed_version_to_be_set_to_a_subproject
1025 @request.session[:user_id] = 2
1029 :fixed_version_id => 4
1031 assert_response :redirect
1032 issues = Issue.find([1,2])
1033 issues.each do |issue|
1034 assert_equal 4, issue.fixed_version_id
1035 assert_not_equal issue.project_id, issue.fixed_version.project_id
1039 def test_move_routing
1041 {:method => :get, :path => '/issues/1/move'},
1042 :controller => 'issues', :action => 'move', :id => '1'
1045 {:controller => 'issues', :action => 'move', :id => '1'},
1046 {:method => :post, :path => '/issues/1/move'}
1050 def test_move_one_issue_to_another_project
1051 @request.session[:user_id] = 2
1052 post :move, :id => 1, :new_project_id => 2, :tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
1053 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1054 assert_equal 2, Issue.find(1).project_id
1057 def test_move_one_issue_to_another_project_should_follow_when_needed
1058 @request.session[:user_id] = 2
1059 post :move, :id => 1, :new_project_id => 2, :follow => '1'
1060 assert_redirected_to '/issues/1'
1063 def test_bulk_move_to_another_project
1064 @request.session[:user_id] = 2
1065 post :move, :ids => [1, 2], :new_project_id => 2
1066 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1067 # Issues moved to project 2
1068 assert_equal 2, Issue.find(1).project_id
1069 assert_equal 2, Issue.find(2).project_id
1071 assert_equal 1, Issue.find(1).tracker_id
1072 assert_equal 2, Issue.find(2).tracker_id
1075 def test_bulk_move_to_another_tracker
1076 @request.session[:user_id] = 2
1077 post :move, :ids => [1, 2], :new_tracker_id => 2
1078 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1079 assert_equal 2, Issue.find(1).tracker_id
1080 assert_equal 2, Issue.find(2).tracker_id
1083 def test_bulk_copy_to_another_project
1084 @request.session[:user_id] = 2
1085 assert_difference 'Issue.count', 2 do
1086 assert_no_difference 'Project.find(1).issues.count' do
1087 post :move, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}
1090 assert_redirected_to 'projects/ecookbook/issues'
1093 context "#move via bulk copy" do
1094 should "allow not changing the issue's attributes" do
1095 @request.session[:user_id] = 2
1096 issue_before_move = Issue.find(1)
1097 assert_difference 'Issue.count', 1 do
1098 assert_no_difference 'Project.find(1).issues.count' do
1099 post :move, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => '', :status_id => '', :start_date => '', :due_date => ''
1102 issue_after_move = Issue.first(:order => 'id desc', :conditions => {:project_id => 2})
1103 assert_equal issue_before_move.tracker_id, issue_after_move.tracker_id
1104 assert_equal issue_before_move.status_id, issue_after_move.status_id
1105 assert_equal issue_before_move.assigned_to_id, issue_after_move.assigned_to_id
1108 should "allow changing the issue's attributes" do
1109 @request.session[:user_id] = 2
1110 assert_difference 'Issue.count', 2 do
1111 assert_no_difference 'Project.find(1).issues.count' do
1112 post :move, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}, :new_tracker_id => '', :assigned_to_id => 4, :status_id => 3, :start_date => '2009-12-01', :due_date => '2009-12-31'
1116 copied_issues = Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2})
1117 assert_equal 2, copied_issues.size
1118 copied_issues.each do |issue|
1119 assert_equal 2, issue.project_id, "Project is incorrect"
1120 assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect"
1121 assert_equal 3, issue.status_id, "Status is incorrect"
1122 assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect"
1123 assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect"
1128 def test_copy_to_another_project_should_follow_when_needed
1129 @request.session[:user_id] = 2
1130 post :move, :ids => [1], :new_project_id => 2, :copy_options => {:copy => '1'}, :follow => '1'
1131 issue = Issue.first(:order => 'id DESC')
1132 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
1135 def test_context_menu_one_issue
1136 @request.session[:user_id] = 2
1137 get :context_menu, :ids => [1]
1138 assert_response :success
1139 assert_template 'context_menu'
1140 assert_tag :tag => 'a', :content => 'Edit',
1141 :attributes => { :href => '/issues/1/edit',
1142 :class => 'icon-edit' }
1143 assert_tag :tag => 'a', :content => 'Closed',
1144 :attributes => { :href => '/issues/1/edit?issue%5Bstatus_id%5D=5',
1146 assert_tag :tag => 'a', :content => 'Immediate',
1147 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&priority_id=8',
1150 assert_tag :tag => 'a', :content => '2.0',
1151 :attributes => { :href => '/issues/bulk_edit?fixed_version_id=3&ids%5B%5D=1',
1153 assert_tag :tag => 'a', :content => 'eCookbook Subproject 1 - 2.0',
1154 :attributes => { :href => '/issues/bulk_edit?fixed_version_id=4&ids%5B%5D=1',
1157 assert_tag :tag => 'a', :content => 'Dave Lopper',
1158 :attributes => { :href => '/issues/bulk_edit?assigned_to_id=3&ids%5B%5D=1',
1160 assert_tag :tag => 'a', :content => 'Duplicate',
1161 :attributes => { :href => '/projects/ecookbook/issues/1/copy',
1162 :class => 'icon-duplicate' }
1163 assert_tag :tag => 'a', :content => 'Copy',
1164 :attributes => { :href => '/issues/move?copy_options%5Bcopy%5D=t&ids%5B%5D=1',
1165 :class => 'icon-copy' }
1166 assert_tag :tag => 'a', :content => 'Move',
1167 :attributes => { :href => '/issues/move?ids%5B%5D=1',
1168 :class => 'icon-move' }
1169 assert_tag :tag => 'a', :content => 'Delete',
1170 :attributes => { :href => '/issues/destroy?ids%5B%5D=1',
1171 :class => 'icon-del' }
1174 def test_context_menu_one_issue_by_anonymous
1175 get :context_menu, :ids => [1]
1176 assert_response :success
1177 assert_template 'context_menu'
1178 assert_tag :tag => 'a', :content => 'Delete',
1179 :attributes => { :href => '#',
1180 :class => 'icon-del disabled' }
1183 def test_context_menu_multiple_issues_of_same_project
1184 @request.session[:user_id] = 2
1185 get :context_menu, :ids => [1, 2]
1186 assert_response :success
1187 assert_template 'context_menu'
1188 assert_tag :tag => 'a', :content => 'Edit',
1189 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&ids%5B%5D=2',
1190 :class => 'icon-edit' }
1191 assert_tag :tag => 'a', :content => 'Immediate',
1192 :attributes => { :href => '/issues/bulk_edit?ids%5B%5D=1&ids%5B%5D=2&priority_id=8',
1194 assert_tag :tag => 'a', :content => 'Dave Lopper',
1195 :attributes => { :href => '/issues/bulk_edit?assigned_to_id=3&ids%5B%5D=1&ids%5B%5D=2',
1197 assert_tag :tag => 'a', :content => 'Copy',
1198 :attributes => { :href => '/issues/move?copy_options%5Bcopy%5D=t&ids%5B%5D=1&ids%5B%5D=2',
1199 :class => 'icon-copy' }
1200 assert_tag :tag => 'a', :content => 'Move',
1201 :attributes => { :href => '/issues/move?ids%5B%5D=1&ids%5B%5D=2',
1202 :class => 'icon-move' }
1203 assert_tag :tag => 'a', :content => 'Delete',
1204 :attributes => { :href => '/issues/destroy?ids%5B%5D=1&ids%5B%5D=2',
1205 :class => 'icon-del' }
1208 def test_context_menu_multiple_issues_of_different_project
1209 @request.session[:user_id] = 2
1210 get :context_menu, :ids => [1, 2, 4]
1211 assert_response :success
1212 assert_template 'context_menu'
1213 assert_tag :tag => 'a', :content => 'Delete',
1214 :attributes => { :href => '#',
1215 :class => 'icon-del disabled' }
1218 def test_destroy_routing
1219 assert_recognizes( #TODO: use DELETE on issue URI (need to change forms)
1220 {:controller => 'issues', :action => 'destroy', :id => '1'},
1221 {:method => :post, :path => '/issues/1/destroy'}
1225 def test_destroy_issue_with_no_time_entries
1226 assert_nil TimeEntry.find_by_issue_id(2)
1227 @request.session[:user_id] = 2
1228 post :destroy, :id => 2
1229 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1230 assert_nil Issue.find_by_id(2)
1233 def test_destroy_issues_with_time_entries
1234 @request.session[:user_id] = 2
1235 post :destroy, :ids => [1, 3]
1236 assert_response :success
1237 assert_template 'destroy'
1238 assert_not_nil assigns(:hours)
1239 assert Issue.find_by_id(1) && Issue.find_by_id(3)
1242 def test_destroy_issues_and_destroy_time_entries
1243 @request.session[:user_id] = 2
1244 post :destroy, :ids => [1, 3], :todo => 'destroy'
1245 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1246 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1247 assert_nil TimeEntry.find_by_id([1, 2])
1250 def test_destroy_issues_and_assign_time_entries_to_project
1251 @request.session[:user_id] = 2
1252 post :destroy, :ids => [1, 3], :todo => 'nullify'
1253 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1254 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1255 assert_nil TimeEntry.find(1).issue_id
1256 assert_nil TimeEntry.find(2).issue_id
1259 def test_destroy_issues_and_reassign_time_entries_to_another_issue
1260 @request.session[:user_id] = 2
1261 post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
1262 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
1263 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
1264 assert_equal 2, TimeEntry.find(1).issue_id
1265 assert_equal 2, TimeEntry.find(2).issue_id
1268 def test_default_search_scope
1270 assert_tag :div, :attributes => {:id => 'quick-search'},
1271 :child => {:tag => 'form',
1272 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}