##// END OF EJS Templates
Merged r10494 from trunk....
Jean-Philippe Lang -
r10313:fe617391084e
parent child
Show More
@@ -1,438 +1,438
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2012 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 class IssuesController < ApplicationController
19 19 menu_item :new_issue, :only => [:new, :create]
20 20 default_search_scope :issues
21 21
22 22 before_filter :find_issue, :only => [:show, :edit, :update]
23 23 before_filter :find_issues, :only => [:bulk_edit, :bulk_update, :destroy]
24 24 before_filter :find_project, :only => [:new, :create]
25 25 before_filter :authorize, :except => [:index]
26 26 before_filter :find_optional_project, :only => [:index]
27 27 before_filter :check_for_default_issue_status, :only => [:new, :create]
28 28 before_filter :build_new_issue_from_params, :only => [:new, :create]
29 29 accept_rss_auth :index, :show
30 30 accept_api_auth :index, :show, :create, :update, :destroy
31 31
32 32 rescue_from Query::StatementInvalid, :with => :query_statement_invalid
33 33
34 34 helper :journals
35 35 helper :projects
36 36 include ProjectsHelper
37 37 helper :custom_fields
38 38 include CustomFieldsHelper
39 39 helper :issue_relations
40 40 include IssueRelationsHelper
41 41 helper :watchers
42 42 include WatchersHelper
43 43 helper :attachments
44 44 include AttachmentsHelper
45 45 helper :queries
46 46 include QueriesHelper
47 47 helper :repositories
48 48 include RepositoriesHelper
49 49 helper :sort
50 50 include SortHelper
51 51 include IssuesHelper
52 52 helper :timelog
53 53 include Redmine::Export::PDF
54 54
55 55 def index
56 56 retrieve_query
57 57 sort_init(@query.sort_criteria.empty? ? [['id', 'desc']] : @query.sort_criteria)
58 58 sort_update(@query.sortable_columns)
59 59
60 60 if @query.valid?
61 61 case params[:format]
62 62 when 'csv', 'pdf'
63 63 @limit = Setting.issues_export_limit.to_i
64 64 when 'atom'
65 65 @limit = Setting.feeds_limit.to_i
66 66 when 'xml', 'json'
67 67 @offset, @limit = api_offset_and_limit
68 68 else
69 69 @limit = per_page_option
70 70 end
71 71
72 72 @issue_count = @query.issue_count
73 73 @issue_pages = Paginator.new self, @issue_count, @limit, params['page']
74 74 @offset ||= @issue_pages.current.offset
75 75 @issues = @query.issues(:include => [:assigned_to, :tracker, :priority, :category, :fixed_version],
76 76 :order => sort_clause,
77 77 :offset => @offset,
78 78 :limit => @limit)
79 79 @issue_count_by_group = @query.issue_count_by_group
80 80
81 81 respond_to do |format|
82 82 format.html { render :template => 'issues/index', :layout => !request.xhr? }
83 83 format.api {
84 84 Issue.load_relations(@issues) if include_in_api_response?('relations')
85 85 }
86 86 format.atom { render_feed(@issues, :title => "#{@project || Setting.app_title}: #{l(:label_issue_plural)}") }
87 87 format.csv { send_data(issues_to_csv(@issues, @project, @query, params), :type => 'text/csv; header=present', :filename => 'export.csv') }
88 88 format.pdf { send_data(issues_to_pdf(@issues, @project, @query), :type => 'application/pdf', :filename => 'export.pdf') }
89 89 end
90 90 else
91 91 respond_to do |format|
92 92 format.html { render(:template => 'issues/index', :layout => !request.xhr?) }
93 93 format.any(:atom, :csv, :pdf) { render(:nothing => true) }
94 94 format.api { render_validation_errors(@query) }
95 95 end
96 96 end
97 97 rescue ActiveRecord::RecordNotFound
98 98 render_404
99 99 end
100 100
101 101 def show
102 102 @journals = @issue.journals.find(:all, :include => [:user, :details], :order => "#{Journal.table_name}.created_on ASC")
103 103 @journals.each_with_index {|j,i| j.indice = i+1}
104 104 @journals.reverse! if User.current.wants_comments_in_reverse_order?
105 105
106 106 @changesets = @issue.changesets.visible.all
107 107 @changesets.reverse! if User.current.wants_comments_in_reverse_order?
108 108
109 109 @relations = @issue.relations.select {|r| r.other_issue(@issue) && r.other_issue(@issue).visible? }
110 110 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
111 111 @edit_allowed = User.current.allowed_to?(:edit_issues, @project)
112 112 @priorities = IssuePriority.active
113 113 @time_entry = TimeEntry.new(:issue => @issue, :project => @issue.project)
114 114 respond_to do |format|
115 115 format.html {
116 116 retrieve_previous_and_next_issue_ids
117 117 render :template => 'issues/show'
118 118 }
119 119 format.api
120 120 format.atom { render :template => 'journals/index', :layout => false, :content_type => 'application/atom+xml' }
121 121 format.pdf { send_data(issue_to_pdf(@issue), :type => 'application/pdf', :filename => "#{@project.identifier}-#{@issue.id}.pdf") }
122 122 end
123 123 end
124 124
125 125 # Add a new issue
126 126 # The new issue will be created from an existing one if copy_from parameter is given
127 127 def new
128 128 respond_to do |format|
129 129 format.html { render :action => 'new', :layout => !request.xhr? }
130 130 format.js { render :partial => 'update_form' }
131 131 end
132 132 end
133 133
134 134 def create
135 135 call_hook(:controller_issues_new_before_save, { :params => params, :issue => @issue })
136 136 @issue.save_attachments(params[:attachments] || (params[:issue] && params[:issue][:uploads]))
137 137 if @issue.save
138 138 call_hook(:controller_issues_new_after_save, { :params => params, :issue => @issue})
139 139 respond_to do |format|
140 140 format.html {
141 141 render_attachment_warning_if_needed(@issue)
142 142 flash[:notice] = l(:notice_issue_successful_create, :id => view_context.link_to("##{@issue.id}", issue_path(@issue), :title => @issue.subject))
143 143 redirect_to(params[:continue] ? { :action => 'new', :project_id => @issue.project, :issue => {:tracker_id => @issue.tracker, :parent_issue_id => @issue.parent_issue_id}.reject {|k,v| v.nil?} } :
144 144 { :action => 'show', :id => @issue })
145 145 }
146 146 format.api { render :action => 'show', :status => :created, :location => issue_url(@issue) }
147 147 end
148 148 return
149 149 else
150 150 respond_to do |format|
151 151 format.html { render :action => 'new' }
152 152 format.api { render_validation_errors(@issue) }
153 153 end
154 154 end
155 155 end
156 156
157 157 def edit
158 158 return unless update_issue_from_params
159 159
160 160 respond_to do |format|
161 161 format.html { }
162 162 format.xml { }
163 163 end
164 164 end
165 165
166 166 def update
167 167 return unless update_issue_from_params
168 168 @issue.save_attachments(params[:attachments] || (params[:issue] && params[:issue][:uploads]))
169 169 saved = false
170 170 begin
171 171 saved = @issue.save_issue_with_child_records(params, @time_entry)
172 172 rescue ActiveRecord::StaleObjectError
173 173 @conflict = true
174 174 if params[:last_journal_id]
175 175 @conflict_journals = @issue.journals_after(params[:last_journal_id]).all
176 176 end
177 177 end
178 178
179 179 if saved
180 180 render_attachment_warning_if_needed(@issue)
181 181 flash[:notice] = l(:notice_successful_update) unless @issue.current_journal.new_record?
182 182
183 183 respond_to do |format|
184 184 format.html { redirect_back_or_default({:action => 'show', :id => @issue}) }
185 185 format.api { render_api_ok }
186 186 end
187 187 else
188 188 respond_to do |format|
189 189 format.html { render :action => 'edit' }
190 190 format.api { render_validation_errors(@issue) }
191 191 end
192 192 end
193 193 end
194 194
195 195 # Bulk edit/copy a set of issues
196 196 def bulk_edit
197 197 @issues.sort!
198 198 @copy = params[:copy].present?
199 199 @notes = params[:notes]
200 200
201 201 if User.current.allowed_to?(:move_issues, @projects)
202 202 @allowed_projects = Issue.allowed_target_projects_on_move
203 203 if params[:issue]
204 204 @target_project = @allowed_projects.detect {|p| p.id.to_s == params[:issue][:project_id].to_s}
205 205 if @target_project
206 206 target_projects = [@target_project]
207 207 end
208 208 end
209 209 end
210 210 target_projects ||= @projects
211 211
212 212 if @copy
213 213 @available_statuses = [IssueStatus.default]
214 214 else
215 215 @available_statuses = @issues.map(&:new_statuses_allowed_to).reduce(:&)
216 216 end
217 217 @custom_fields = target_projects.map{|p|p.all_issue_custom_fields}.reduce(:&)
218 218 @assignables = target_projects.map(&:assignable_users).reduce(:&)
219 219 @trackers = target_projects.map(&:trackers).reduce(:&)
220 220 @versions = target_projects.map {|p| p.shared_versions.open}.reduce(:&)
221 221 @categories = target_projects.map {|p| p.issue_categories}.reduce(:&)
222 222 if @copy
223 223 @attachments_present = @issues.detect {|i| i.attachments.any?}.present?
224 224 @subtasks_present = @issues.detect {|i| !i.leaf?}.present?
225 225 end
226 226
227 227 @safe_attributes = @issues.map(&:safe_attribute_names).reduce(:&)
228 228 render :layout => false if request.xhr?
229 229 end
230 230
231 231 def bulk_update
232 232 @issues.sort!
233 233 @copy = params[:copy].present?
234 234 attributes = parse_params_for_bulk_issue_attributes(params)
235 235
236 236 unsaved_issue_ids = []
237 237 moved_issues = []
238 238
239 239 if @copy && params[:copy_subtasks].present?
240 240 # Descendant issues will be copied with the parent task
241 241 # Don't copy them twice
242 242 @issues.reject! {|issue| @issues.detect {|other| issue.is_descendant_of?(other)}}
243 243 end
244 244
245 245 @issues.each do |issue|
246 246 issue.reload
247 247 if @copy
248 248 issue = issue.copy({},
249 249 :attachments => params[:copy_attachments].present?,
250 250 :subtasks => params[:copy_subtasks].present?
251 251 )
252 252 end
253 253 journal = issue.init_journal(User.current, params[:notes])
254 254 issue.safe_attributes = attributes
255 255 call_hook(:controller_issues_bulk_edit_before_save, { :params => params, :issue => issue })
256 256 if issue.save
257 257 moved_issues << issue
258 258 else
259 259 # Keep unsaved issue ids to display them in flash error
260 260 unsaved_issue_ids << issue.id
261 261 end
262 262 end
263 263 set_flash_from_bulk_issue_save(@issues, unsaved_issue_ids)
264 264
265 265 if params[:follow]
266 266 if @issues.size == 1 && moved_issues.size == 1
267 267 redirect_to :controller => 'issues', :action => 'show', :id => moved_issues.first
268 268 elsif moved_issues.map(&:project).uniq.size == 1
269 269 redirect_to :controller => 'issues', :action => 'index', :project_id => moved_issues.map(&:project).first
270 270 end
271 271 else
272 272 redirect_back_or_default({:controller => 'issues', :action => 'index', :project_id => @project})
273 273 end
274 274 end
275 275
276 276 def destroy
277 277 @hours = TimeEntry.sum(:hours, :conditions => ['issue_id IN (?)', @issues]).to_f
278 278 if @hours > 0
279 279 case params[:todo]
280 280 when 'destroy'
281 281 # nothing to do
282 282 when 'nullify'
283 283 TimeEntry.update_all('issue_id = NULL', ['issue_id IN (?)', @issues])
284 284 when 'reassign'
285 285 reassign_to = @project.issues.find_by_id(params[:reassign_to_id])
286 286 if reassign_to.nil?
287 287 flash.now[:error] = l(:error_issue_not_found_in_project)
288 288 return
289 289 else
290 290 TimeEntry.update_all("issue_id = #{reassign_to.id}", ['issue_id IN (?)', @issues])
291 291 end
292 292 else
293 293 # display the destroy form if it's a user request
294 294 return unless api_request?
295 295 end
296 296 end
297 297 @issues.each do |issue|
298 298 begin
299 299 issue.reload.destroy
300 300 rescue ::ActiveRecord::RecordNotFound # raised by #reload if issue no longer exists
301 301 # nothing to do, issue was already deleted (eg. by a parent)
302 302 end
303 303 end
304 304 respond_to do |format|
305 305 format.html { redirect_back_or_default(:action => 'index', :project_id => @project) }
306 306 format.api { render_api_ok }
307 307 end
308 308 end
309 309
310 310 private
311 311 def find_issue
312 312 # Issue.visible.find(...) can not be used to redirect user to the login form
313 313 # if the issue actually exists but requires authentication
314 314 @issue = Issue.find(params[:id], :include => [:project, :tracker, :status, :author, :priority, :category])
315 315 unless @issue.visible?
316 316 deny_access
317 317 return
318 318 end
319 319 @project = @issue.project
320 320 rescue ActiveRecord::RecordNotFound
321 321 render_404
322 322 end
323 323
324 324 def find_project
325 325 project_id = params[:project_id] || (params[:issue] && params[:issue][:project_id])
326 326 @project = Project.find(project_id)
327 327 rescue ActiveRecord::RecordNotFound
328 328 render_404
329 329 end
330 330
331 331 def retrieve_previous_and_next_issue_ids
332 332 retrieve_query_from_session
333 333 if @query
334 334 sort_init(@query.sort_criteria.empty? ? [['id', 'desc']] : @query.sort_criteria)
335 335 sort_update(@query.sortable_columns, 'issues_index_sort')
336 336 limit = 500
337 337 issue_ids = @query.issue_ids(:order => sort_clause, :limit => (limit + 1), :include => [:assigned_to, :tracker, :priority, :category, :fixed_version])
338 338 if (idx = issue_ids.index(@issue.id)) && idx < limit
339 339 if issue_ids.size < 500
340 340 @issue_position = idx + 1
341 341 @issue_count = issue_ids.size
342 342 end
343 343 @prev_issue_id = issue_ids[idx - 1] if idx > 0
344 344 @next_issue_id = issue_ids[idx + 1] if idx < (issue_ids.size - 1)
345 345 end
346 346 end
347 347 end
348 348
349 349 # Used by #edit and #update to set some common instance variables
350 350 # from the params
351 351 # TODO: Refactor, not everything in here is needed by #edit
352 352 def update_issue_from_params
353 353 @edit_allowed = User.current.allowed_to?(:edit_issues, @project)
354 354 @time_entry = TimeEntry.new(:issue => @issue, :project => @issue.project)
355 355 @time_entry.attributes = params[:time_entry]
356 356
357 357 @notes = params[:notes] || (params[:issue].present? ? params[:issue][:notes] : nil)
358 358 @issue.init_journal(User.current, @notes)
359 359
360 360 issue_attributes = params[:issue]
361 361 if issue_attributes && params[:conflict_resolution]
362 362 case params[:conflict_resolution]
363 363 when 'overwrite'
364 364 issue_attributes = issue_attributes.dup
365 365 issue_attributes.delete(:lock_version)
366 366 when 'add_notes'
367 367 issue_attributes = {}
368 368 when 'cancel'
369 369 redirect_to issue_path(@issue)
370 370 return false
371 371 end
372 372 end
373 373 @issue.safe_attributes = issue_attributes
374 374 @priorities = IssuePriority.active
375 375 @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
376 376 true
377 377 end
378 378
379 379 # TODO: Refactor, lots of extra code in here
380 380 # TODO: Changing tracker on an existing issue should not trigger this
381 381 def build_new_issue_from_params
382 382 if params[:id].blank?
383 383 @issue = Issue.new
384 384 if params[:copy_from]
385 385 begin
386 386 @copy_from = Issue.visible.find(params[:copy_from])
387 387 @copy_attachments = params[:copy_attachments].present? || request.get?
388 388 @copy_subtasks = params[:copy_subtasks].present? || request.get?
389 389 @issue.copy_from(@copy_from, :attachments => @copy_attachments, :subtasks => @copy_subtasks)
390 390 rescue ActiveRecord::RecordNotFound
391 391 render_404
392 392 return
393 393 end
394 394 end
395 395 @issue.project = @project
396 396 else
397 397 @issue = @project.issues.visible.find(params[:id])
398 398 end
399 399
400 400 @issue.project = @project
401 @issue.author = User.current
401 @issue.author ||= User.current
402 402 # Tracker must be set before custom field values
403 403 @issue.tracker ||= @project.trackers.find((params[:issue] && params[:issue][:tracker_id]) || params[:tracker_id] || :first)
404 404 if @issue.tracker.nil?
405 405 render_error l(:error_no_tracker_in_project)
406 406 return false
407 407 end
408 408 @issue.start_date ||= Date.today if Setting.default_issue_start_date_to_creation_date?
409 409 @issue.safe_attributes = params[:issue]
410 410
411 411 @priorities = IssuePriority.active
412 412 @allowed_statuses = @issue.new_statuses_allowed_to(User.current, true)
413 413 @available_watchers = (@issue.project.users.sort + @issue.watcher_users).uniq
414 414 end
415 415
416 416 def check_for_default_issue_status
417 417 if IssueStatus.default.nil?
418 418 render_error l(:error_no_default_issue_status)
419 419 return false
420 420 end
421 421 end
422 422
423 423 def parse_params_for_bulk_issue_attributes(params)
424 424 attributes = (params[:issue] || {}).reject {|k,v| v.blank?}
425 425 attributes.keys.each {|k| attributes[k] = '' if attributes[k] == 'none'}
426 426 if custom = attributes[:custom_field_values]
427 427 custom.reject! {|k,v| v.blank?}
428 428 custom.keys.each do |k|
429 429 if custom[k].is_a?(Array)
430 430 custom[k] << '' if custom[k].delete('__none__')
431 431 else
432 432 custom[k] = '' if custom[k] == '__none__'
433 433 end
434 434 end
435 435 end
436 436 attributes
437 437 end
438 438 end
@@ -1,3658 +1,3670
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2012 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 require File.expand_path('../../test_helper', __FILE__)
19 19 require 'issues_controller'
20 20
21 21 class IssuesControllerTest < ActionController::TestCase
22 22 fixtures :projects,
23 23 :users,
24 24 :roles,
25 25 :members,
26 26 :member_roles,
27 27 :issues,
28 28 :issue_statuses,
29 29 :versions,
30 30 :trackers,
31 31 :projects_trackers,
32 32 :issue_categories,
33 33 :enabled_modules,
34 34 :enumerations,
35 35 :attachments,
36 36 :workflows,
37 37 :custom_fields,
38 38 :custom_values,
39 39 :custom_fields_projects,
40 40 :custom_fields_trackers,
41 41 :time_entries,
42 42 :journals,
43 43 :journal_details,
44 44 :queries,
45 45 :repositories,
46 46 :changesets
47 47
48 48 include Redmine::I18n
49 49
50 50 def setup
51 51 @controller = IssuesController.new
52 52 @request = ActionController::TestRequest.new
53 53 @response = ActionController::TestResponse.new
54 54 User.current = nil
55 55 end
56 56
57 57 def test_index
58 58 with_settings :default_language => "en" do
59 59 get :index
60 60 assert_response :success
61 61 assert_template 'index'
62 62 assert_not_nil assigns(:issues)
63 63 assert_nil assigns(:project)
64 64 assert_tag :tag => 'a', :content => /Can&#x27;t print recipes/
65 65 assert_tag :tag => 'a', :content => /Subproject issue/
66 66 # private projects hidden
67 67 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
68 68 assert_no_tag :tag => 'a', :content => /Issue on project 2/
69 69 # project column
70 70 assert_tag :tag => 'th', :content => /Project/
71 71 end
72 72 end
73 73
74 74 def test_index_should_not_list_issues_when_module_disabled
75 75 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
76 76 get :index
77 77 assert_response :success
78 78 assert_template 'index'
79 79 assert_not_nil assigns(:issues)
80 80 assert_nil assigns(:project)
81 81 assert_no_tag :tag => 'a', :content => /Can&#x27;t print recipes/
82 82 assert_tag :tag => 'a', :content => /Subproject issue/
83 83 end
84 84
85 85 def test_index_should_list_visible_issues_only
86 86 get :index, :per_page => 100
87 87 assert_response :success
88 88 assert_not_nil assigns(:issues)
89 89 assert_nil assigns(:issues).detect {|issue| !issue.visible?}
90 90 end
91 91
92 92 def test_index_with_project
93 93 Setting.display_subprojects_issues = 0
94 94 get :index, :project_id => 1
95 95 assert_response :success
96 96 assert_template 'index'
97 97 assert_not_nil assigns(:issues)
98 98 assert_tag :tag => 'a', :content => /Can&#x27;t print recipes/
99 99 assert_no_tag :tag => 'a', :content => /Subproject issue/
100 100 end
101 101
102 102 def test_index_with_project_and_subprojects
103 103 Setting.display_subprojects_issues = 1
104 104 get :index, :project_id => 1
105 105 assert_response :success
106 106 assert_template 'index'
107 107 assert_not_nil assigns(:issues)
108 108 assert_tag :tag => 'a', :content => /Can&#x27;t print recipes/
109 109 assert_tag :tag => 'a', :content => /Subproject issue/
110 110 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
111 111 end
112 112
113 113 def test_index_with_project_and_subprojects_should_show_private_subprojects
114 114 @request.session[:user_id] = 2
115 115 Setting.display_subprojects_issues = 1
116 116 get :index, :project_id => 1
117 117 assert_response :success
118 118 assert_template 'index'
119 119 assert_not_nil assigns(:issues)
120 120 assert_tag :tag => 'a', :content => /Can&#x27;t print recipes/
121 121 assert_tag :tag => 'a', :content => /Subproject issue/
122 122 assert_tag :tag => 'a', :content => /Issue of a private subproject/
123 123 end
124 124
125 125 def test_index_with_project_and_default_filter
126 126 get :index, :project_id => 1, :set_filter => 1
127 127 assert_response :success
128 128 assert_template 'index'
129 129 assert_not_nil assigns(:issues)
130 130
131 131 query = assigns(:query)
132 132 assert_not_nil query
133 133 # default filter
134 134 assert_equal({'status_id' => {:operator => 'o', :values => ['']}}, query.filters)
135 135 end
136 136
137 137 def test_index_with_project_and_filter
138 138 get :index, :project_id => 1, :set_filter => 1,
139 139 :f => ['tracker_id'],
140 140 :op => {'tracker_id' => '='},
141 141 :v => {'tracker_id' => ['1']}
142 142 assert_response :success
143 143 assert_template 'index'
144 144 assert_not_nil assigns(:issues)
145 145
146 146 query = assigns(:query)
147 147 assert_not_nil query
148 148 assert_equal({'tracker_id' => {:operator => '=', :values => ['1']}}, query.filters)
149 149 end
150 150
151 151 def test_index_with_short_filters
152 152 to_test = {
153 153 'status_id' => {
154 154 'o' => { :op => 'o', :values => [''] },
155 155 'c' => { :op => 'c', :values => [''] },
156 156 '7' => { :op => '=', :values => ['7'] },
157 157 '7|3|4' => { :op => '=', :values => ['7', '3', '4'] },
158 158 '=7' => { :op => '=', :values => ['7'] },
159 159 '!3' => { :op => '!', :values => ['3'] },
160 160 '!7|3|4' => { :op => '!', :values => ['7', '3', '4'] }},
161 161 'subject' => {
162 162 'This is a subject' => { :op => '=', :values => ['This is a subject'] },
163 163 'o' => { :op => '=', :values => ['o'] },
164 164 '~This is part of a subject' => { :op => '~', :values => ['This is part of a subject'] },
165 165 '!~This is part of a subject' => { :op => '!~', :values => ['This is part of a subject'] }},
166 166 'tracker_id' => {
167 167 '3' => { :op => '=', :values => ['3'] },
168 168 '=3' => { :op => '=', :values => ['3'] }},
169 169 'start_date' => {
170 170 '2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
171 171 '=2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
172 172 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
173 173 '<=2011-10-12' => { :op => '<=', :values => ['2011-10-12'] },
174 174 '><2011-10-01|2011-10-30' => { :op => '><', :values => ['2011-10-01', '2011-10-30'] },
175 175 '<t+2' => { :op => '<t+', :values => ['2'] },
176 176 '>t+2' => { :op => '>t+', :values => ['2'] },
177 177 't+2' => { :op => 't+', :values => ['2'] },
178 178 't' => { :op => 't', :values => [''] },
179 179 'w' => { :op => 'w', :values => [''] },
180 180 '>t-2' => { :op => '>t-', :values => ['2'] },
181 181 '<t-2' => { :op => '<t-', :values => ['2'] },
182 182 't-2' => { :op => 't-', :values => ['2'] }},
183 183 'created_on' => {
184 184 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
185 185 '<t-2' => { :op => '<t-', :values => ['2'] },
186 186 '>t-2' => { :op => '>t-', :values => ['2'] },
187 187 't-2' => { :op => 't-', :values => ['2'] }},
188 188 'cf_1' => {
189 189 'c' => { :op => '=', :values => ['c'] },
190 190 '!c' => { :op => '!', :values => ['c'] },
191 191 '!*' => { :op => '!*', :values => [''] },
192 192 '*' => { :op => '*', :values => [''] }},
193 193 'estimated_hours' => {
194 194 '=13.4' => { :op => '=', :values => ['13.4'] },
195 195 '>=45' => { :op => '>=', :values => ['45'] },
196 196 '<=125' => { :op => '<=', :values => ['125'] },
197 197 '><10.5|20.5' => { :op => '><', :values => ['10.5', '20.5'] },
198 198 '!*' => { :op => '!*', :values => [''] },
199 199 '*' => { :op => '*', :values => [''] }}
200 200 }
201 201
202 202 default_filter = { 'status_id' => {:operator => 'o', :values => [''] }}
203 203
204 204 to_test.each do |field, expression_and_expected|
205 205 expression_and_expected.each do |filter_expression, expected|
206 206
207 207 get :index, :set_filter => 1, field => filter_expression
208 208
209 209 assert_response :success
210 210 assert_template 'index'
211 211 assert_not_nil assigns(:issues)
212 212
213 213 query = assigns(:query)
214 214 assert_not_nil query
215 215 assert query.has_filter?(field)
216 216 assert_equal(default_filter.merge({field => {:operator => expected[:op], :values => expected[:values]}}), query.filters)
217 217 end
218 218 end
219 219 end
220 220
221 221 def test_index_with_project_and_empty_filters
222 222 get :index, :project_id => 1, :set_filter => 1, :fields => ['']
223 223 assert_response :success
224 224 assert_template 'index'
225 225 assert_not_nil assigns(:issues)
226 226
227 227 query = assigns(:query)
228 228 assert_not_nil query
229 229 # no filter
230 230 assert_equal({}, query.filters)
231 231 end
232 232
233 233 def test_index_with_project_custom_field_filter
234 234 field = ProjectCustomField.create!(:name => 'Client', :is_filter => true, :field_format => 'string')
235 235 CustomValue.create!(:custom_field => field, :customized => Project.find(3), :value => 'Foo')
236 236 CustomValue.create!(:custom_field => field, :customized => Project.find(5), :value => 'Foo')
237 237 filter_name = "project.cf_#{field.id}"
238 238 @request.session[:user_id] = 1
239 239
240 240 get :index, :set_filter => 1,
241 241 :f => [filter_name],
242 242 :op => {filter_name => '='},
243 243 :v => {filter_name => ['Foo']}
244 244 assert_response :success
245 245 assert_template 'index'
246 246 assert_equal [3, 5], assigns(:issues).map(&:project_id).uniq.sort
247 247 end
248 248
249 249 def test_index_with_query
250 250 get :index, :project_id => 1, :query_id => 5
251 251 assert_response :success
252 252 assert_template 'index'
253 253 assert_not_nil assigns(:issues)
254 254 assert_nil assigns(:issue_count_by_group)
255 255 end
256 256
257 257 def test_index_with_query_grouped_by_tracker
258 258 get :index, :project_id => 1, :query_id => 6
259 259 assert_response :success
260 260 assert_template 'index'
261 261 assert_not_nil assigns(:issues)
262 262 assert_not_nil assigns(:issue_count_by_group)
263 263 end
264 264
265 265 def test_index_with_query_grouped_by_list_custom_field
266 266 get :index, :project_id => 1, :query_id => 9
267 267 assert_response :success
268 268 assert_template 'index'
269 269 assert_not_nil assigns(:issues)
270 270 assert_not_nil assigns(:issue_count_by_group)
271 271 end
272 272
273 273 def test_index_with_query_grouped_by_user_custom_field
274 274 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
275 275 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
276 276 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
277 277 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
278 278 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
279 279
280 280 get :index, :project_id => 1, :set_filter => 1, :group_by => "cf_#{cf.id}"
281 281 assert_response :success
282 282
283 283 assert_select 'tr.group', 3
284 284 assert_select 'tr.group' do
285 285 assert_select 'a', :text => 'John Smith'
286 286 assert_select 'span.count', :text => '(1)'
287 287 end
288 288 assert_select 'tr.group' do
289 289 assert_select 'a', :text => 'Dave Lopper'
290 290 assert_select 'span.count', :text => '(2)'
291 291 end
292 292 end
293 293
294 294 def test_index_with_query_id_and_project_id_should_set_session_query
295 295 get :index, :project_id => 1, :query_id => 4
296 296 assert_response :success
297 297 assert_kind_of Hash, session[:query]
298 298 assert_equal 4, session[:query][:id]
299 299 assert_equal 1, session[:query][:project_id]
300 300 end
301 301
302 302 def test_index_with_invalid_query_id_should_respond_404
303 303 get :index, :project_id => 1, :query_id => 999
304 304 assert_response 404
305 305 end
306 306
307 307 def test_index_with_cross_project_query_in_session_should_show_project_issues
308 308 q = Query.create!(:name => "test", :user_id => 2, :is_public => false, :project => nil)
309 309 @request.session[:query] = {:id => q.id, :project_id => 1}
310 310
311 311 with_settings :display_subprojects_issues => '0' do
312 312 get :index, :project_id => 1
313 313 end
314 314 assert_response :success
315 315 assert_not_nil assigns(:query)
316 316 assert_equal q.id, assigns(:query).id
317 317 assert_equal 1, assigns(:query).project_id
318 318 assert_equal [1], assigns(:issues).map(&:project_id).uniq
319 319 end
320 320
321 321 def test_private_query_should_not_be_available_to_other_users
322 322 q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil)
323 323 @request.session[:user_id] = 3
324 324
325 325 get :index, :query_id => q.id
326 326 assert_response 403
327 327 end
328 328
329 329 def test_private_query_should_be_available_to_its_user
330 330 q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil)
331 331 @request.session[:user_id] = 2
332 332
333 333 get :index, :query_id => q.id
334 334 assert_response :success
335 335 end
336 336
337 337 def test_public_query_should_be_available_to_other_users
338 338 q = Query.create!(:name => "private", :user => User.find(2), :is_public => true, :project => nil)
339 339 @request.session[:user_id] = 3
340 340
341 341 get :index, :query_id => q.id
342 342 assert_response :success
343 343 end
344 344
345 345 def test_index_should_omit_page_param_in_export_links
346 346 get :index, :page => 2
347 347 assert_response :success
348 348 assert_select 'a.atom[href=/issues.atom]'
349 349 assert_select 'a.csv[href=/issues.csv]'
350 350 assert_select 'a.pdf[href=/issues.pdf]'
351 351 assert_select 'form#csv-export-form[action=/issues.csv]'
352 352 end
353 353
354 354 def test_index_csv
355 355 get :index, :format => 'csv'
356 356 assert_response :success
357 357 assert_not_nil assigns(:issues)
358 358 assert_equal 'text/csv; header=present', @response.content_type
359 359 assert @response.body.starts_with?("#,")
360 360 lines = @response.body.chomp.split("\n")
361 361 assert_equal assigns(:query).columns.size + 1, lines[0].split(',').size
362 362 end
363 363
364 364 def test_index_csv_with_project
365 365 get :index, :project_id => 1, :format => 'csv'
366 366 assert_response :success
367 367 assert_not_nil assigns(:issues)
368 368 assert_equal 'text/csv; header=present', @response.content_type
369 369 end
370 370
371 371 def test_index_csv_with_description
372 372 get :index, :format => 'csv', :description => '1'
373 373 assert_response :success
374 374 assert_not_nil assigns(:issues)
375 375 assert_equal 'text/csv; header=present', @response.content_type
376 376 assert @response.body.starts_with?("#,")
377 377 lines = @response.body.chomp.split("\n")
378 378 assert_equal assigns(:query).columns.size + 2, lines[0].split(',').size
379 379 end
380 380
381 381 def test_index_csv_with_spent_time_column
382 382 issue = Issue.create!(:project_id => 1, :tracker_id => 1, :subject => 'test_index_csv_with_spent_time_column', :author_id => 2)
383 383 TimeEntry.create!(:project => issue.project, :issue => issue, :hours => 7.33, :user => User.find(2), :spent_on => Date.today)
384 384
385 385 get :index, :format => 'csv', :set_filter => '1', :c => %w(subject spent_hours)
386 386 assert_response :success
387 387 assert_equal 'text/csv; header=present', @response.content_type
388 388 lines = @response.body.chomp.split("\n")
389 389 assert_include "#{issue.id},#{issue.subject},7.33", lines
390 390 end
391 391
392 392 def test_index_csv_with_all_columns
393 393 get :index, :format => 'csv', :columns => 'all'
394 394 assert_response :success
395 395 assert_not_nil assigns(:issues)
396 396 assert_equal 'text/csv; header=present', @response.content_type
397 397 assert @response.body.starts_with?("#,")
398 398 lines = @response.body.chomp.split("\n")
399 399 assert_equal assigns(:query).available_columns.size + 1, lines[0].split(',').size
400 400 end
401 401
402 402 def test_index_csv_with_multi_column_field
403 403 CustomField.find(1).update_attribute :multiple, true
404 404 issue = Issue.find(1)
405 405 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
406 406 issue.save!
407 407
408 408 get :index, :format => 'csv', :columns => 'all'
409 409 assert_response :success
410 410 lines = @response.body.chomp.split("\n")
411 411 assert lines.detect {|line| line.include?('"MySQL, Oracle"')}
412 412 end
413 413
414 414 def test_index_csv_big_5
415 415 with_settings :default_language => "zh-TW" do
416 416 str_utf8 = "\xe4\xb8\x80\xe6\x9c\x88"
417 417 str_big5 = "\xa4@\xa4\xeb"
418 418 if str_utf8.respond_to?(:force_encoding)
419 419 str_utf8.force_encoding('UTF-8')
420 420 str_big5.force_encoding('Big5')
421 421 end
422 422 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
423 423 :status_id => 1, :priority => IssuePriority.all.first,
424 424 :subject => str_utf8)
425 425 assert issue.save
426 426
427 427 get :index, :project_id => 1,
428 428 :f => ['subject'],
429 429 :op => '=', :values => [str_utf8],
430 430 :format => 'csv'
431 431 assert_equal 'text/csv; header=present', @response.content_type
432 432 lines = @response.body.chomp.split("\n")
433 433 s1 = "\xaa\xac\xbaA"
434 434 if str_utf8.respond_to?(:force_encoding)
435 435 s1.force_encoding('Big5')
436 436 end
437 437 assert lines[0].include?(s1)
438 438 assert lines[1].include?(str_big5)
439 439 end
440 440 end
441 441
442 442 def test_index_csv_cannot_convert_should_be_replaced_big_5
443 443 with_settings :default_language => "zh-TW" do
444 444 str_utf8 = "\xe4\xbb\xa5\xe5\x86\x85"
445 445 if str_utf8.respond_to?(:force_encoding)
446 446 str_utf8.force_encoding('UTF-8')
447 447 end
448 448 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
449 449 :status_id => 1, :priority => IssuePriority.all.first,
450 450 :subject => str_utf8)
451 451 assert issue.save
452 452
453 453 get :index, :project_id => 1,
454 454 :f => ['subject'],
455 455 :op => '=', :values => [str_utf8],
456 456 :c => ['status', 'subject'],
457 457 :format => 'csv',
458 458 :set_filter => 1
459 459 assert_equal 'text/csv; header=present', @response.content_type
460 460 lines = @response.body.chomp.split("\n")
461 461 s1 = "\xaa\xac\xbaA" # status
462 462 if str_utf8.respond_to?(:force_encoding)
463 463 s1.force_encoding('Big5')
464 464 end
465 465 assert lines[0].include?(s1)
466 466 s2 = lines[1].split(",")[2]
467 467 if s1.respond_to?(:force_encoding)
468 468 s3 = "\xa5H?" # subject
469 469 s3.force_encoding('Big5')
470 470 assert_equal s3, s2
471 471 elsif RUBY_PLATFORM == 'java'
472 472 assert_equal "??", s2
473 473 else
474 474 assert_equal "\xa5H???", s2
475 475 end
476 476 end
477 477 end
478 478
479 479 def test_index_csv_tw
480 480 with_settings :default_language => "zh-TW" do
481 481 str1 = "test_index_csv_tw"
482 482 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
483 483 :status_id => 1, :priority => IssuePriority.all.first,
484 484 :subject => str1, :estimated_hours => '1234.5')
485 485 assert issue.save
486 486 assert_equal 1234.5, issue.estimated_hours
487 487
488 488 get :index, :project_id => 1,
489 489 :f => ['subject'],
490 490 :op => '=', :values => [str1],
491 491 :c => ['estimated_hours', 'subject'],
492 492 :format => 'csv',
493 493 :set_filter => 1
494 494 assert_equal 'text/csv; header=present', @response.content_type
495 495 lines = @response.body.chomp.split("\n")
496 496 assert_equal "#{issue.id},1234.50,#{str1}", lines[1]
497 497
498 498 str_tw = "Traditional Chinese (\xe7\xb9\x81\xe9\xab\x94\xe4\xb8\xad\xe6\x96\x87)"
499 499 if str_tw.respond_to?(:force_encoding)
500 500 str_tw.force_encoding('UTF-8')
501 501 end
502 502 assert_equal str_tw, l(:general_lang_name)
503 503 assert_equal ',', l(:general_csv_separator)
504 504 assert_equal '.', l(:general_csv_decimal_separator)
505 505 end
506 506 end
507 507
508 508 def test_index_csv_fr
509 509 with_settings :default_language => "fr" do
510 510 str1 = "test_index_csv_fr"
511 511 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
512 512 :status_id => 1, :priority => IssuePriority.all.first,
513 513 :subject => str1, :estimated_hours => '1234.5')
514 514 assert issue.save
515 515 assert_equal 1234.5, issue.estimated_hours
516 516
517 517 get :index, :project_id => 1,
518 518 :f => ['subject'],
519 519 :op => '=', :values => [str1],
520 520 :c => ['estimated_hours', 'subject'],
521 521 :format => 'csv',
522 522 :set_filter => 1
523 523 assert_equal 'text/csv; header=present', @response.content_type
524 524 lines = @response.body.chomp.split("\n")
525 525 assert_equal "#{issue.id};1234,50;#{str1}", lines[1]
526 526
527 527 str_fr = "Fran\xc3\xa7ais"
528 528 if str_fr.respond_to?(:force_encoding)
529 529 str_fr.force_encoding('UTF-8')
530 530 end
531 531 assert_equal str_fr, l(:general_lang_name)
532 532 assert_equal ';', l(:general_csv_separator)
533 533 assert_equal ',', l(:general_csv_decimal_separator)
534 534 end
535 535 end
536 536
537 537 def test_index_pdf
538 538 ["en", "zh", "zh-TW", "ja", "ko"].each do |lang|
539 539 with_settings :default_language => lang do
540 540
541 541 get :index
542 542 assert_response :success
543 543 assert_template 'index'
544 544
545 545 if lang == "ja"
546 546 if RUBY_PLATFORM != 'java'
547 547 assert_equal "CP932", l(:general_pdf_encoding)
548 548 end
549 549 if RUBY_PLATFORM == 'java' && l(:general_pdf_encoding) == "CP932"
550 550 next
551 551 end
552 552 end
553 553
554 554 get :index, :format => 'pdf'
555 555 assert_response :success
556 556 assert_not_nil assigns(:issues)
557 557 assert_equal 'application/pdf', @response.content_type
558 558
559 559 get :index, :project_id => 1, :format => 'pdf'
560 560 assert_response :success
561 561 assert_not_nil assigns(:issues)
562 562 assert_equal 'application/pdf', @response.content_type
563 563
564 564 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
565 565 assert_response :success
566 566 assert_not_nil assigns(:issues)
567 567 assert_equal 'application/pdf', @response.content_type
568 568 end
569 569 end
570 570 end
571 571
572 572 def test_index_pdf_with_query_grouped_by_list_custom_field
573 573 get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
574 574 assert_response :success
575 575 assert_not_nil assigns(:issues)
576 576 assert_not_nil assigns(:issue_count_by_group)
577 577 assert_equal 'application/pdf', @response.content_type
578 578 end
579 579
580 580 def test_index_atom
581 581 get :index, :project_id => 'ecookbook', :format => 'atom'
582 582 assert_response :success
583 583 assert_template 'common/feed'
584 584
585 585 assert_tag :tag => 'link', :parent => {:tag => 'feed', :parent => nil },
586 586 :attributes => {:rel => 'self', :href => 'http://test.host/projects/ecookbook/issues.atom'}
587 587 assert_tag :tag => 'link', :parent => {:tag => 'feed', :parent => nil },
588 588 :attributes => {:rel => 'alternate', :href => 'http://test.host/projects/ecookbook/issues'}
589 589
590 590 assert_tag :tag => 'entry', :child => {
591 591 :tag => 'link',
592 592 :attributes => {:href => 'http://test.host/issues/1'}}
593 593 end
594 594
595 595 def test_index_sort
596 596 get :index, :sort => 'tracker,id:desc'
597 597 assert_response :success
598 598
599 599 sort_params = @request.session['issues_index_sort']
600 600 assert sort_params.is_a?(String)
601 601 assert_equal 'tracker,id:desc', sort_params
602 602
603 603 issues = assigns(:issues)
604 604 assert_not_nil issues
605 605 assert !issues.empty?
606 606 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
607 607 end
608 608
609 609 def test_index_sort_by_field_not_included_in_columns
610 610 Setting.issue_list_default_columns = %w(subject author)
611 611 get :index, :sort => 'tracker'
612 612 end
613 613
614 614 def test_index_sort_by_assigned_to
615 615 get :index, :sort => 'assigned_to'
616 616 assert_response :success
617 617 assignees = assigns(:issues).collect(&:assigned_to).compact
618 618 assert_equal assignees.sort, assignees
619 619 end
620 620
621 621 def test_index_sort_by_assigned_to_desc
622 622 get :index, :sort => 'assigned_to:desc'
623 623 assert_response :success
624 624 assignees = assigns(:issues).collect(&:assigned_to).compact
625 625 assert_equal assignees.sort.reverse, assignees
626 626 end
627 627
628 628 def test_index_group_by_assigned_to
629 629 get :index, :group_by => 'assigned_to', :sort => 'priority'
630 630 assert_response :success
631 631 end
632 632
633 633 def test_index_sort_by_author
634 634 get :index, :sort => 'author'
635 635 assert_response :success
636 636 authors = assigns(:issues).collect(&:author)
637 637 assert_equal authors.sort, authors
638 638 end
639 639
640 640 def test_index_sort_by_author_desc
641 641 get :index, :sort => 'author:desc'
642 642 assert_response :success
643 643 authors = assigns(:issues).collect(&:author)
644 644 assert_equal authors.sort.reverse, authors
645 645 end
646 646
647 647 def test_index_group_by_author
648 648 get :index, :group_by => 'author', :sort => 'priority'
649 649 assert_response :success
650 650 end
651 651
652 652 def test_index_sort_by_spent_hours
653 653 get :index, :sort => 'spent_hours:desc'
654 654 assert_response :success
655 655 hours = assigns(:issues).collect(&:spent_hours)
656 656 assert_equal hours.sort.reverse, hours
657 657 end
658 658
659 659 def test_index_sort_by_user_custom_field
660 660 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
661 661 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
662 662 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
663 663 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
664 664 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
665 665
666 666 get :index, :project_id => 1, :set_filter => 1, :sort => "cf_#{cf.id},id"
667 667 assert_response :success
668 668
669 669 assert_equal [2, 3, 1], assigns(:issues).select {|issue| issue.custom_field_value(cf).present?}.map(&:id)
670 670 end
671 671
672 672 def test_index_with_columns
673 673 columns = ['tracker', 'subject', 'assigned_to']
674 674 get :index, :set_filter => 1, :c => columns
675 675 assert_response :success
676 676
677 677 # query should use specified columns
678 678 query = assigns(:query)
679 679 assert_kind_of Query, query
680 680 assert_equal columns, query.column_names.map(&:to_s)
681 681
682 682 # columns should be stored in session
683 683 assert_kind_of Hash, session[:query]
684 684 assert_kind_of Array, session[:query][:column_names]
685 685 assert_equal columns, session[:query][:column_names].map(&:to_s)
686 686
687 687 # ensure only these columns are kept in the selected columns list
688 688 assert_tag :tag => 'select', :attributes => { :id => 'selected_columns' },
689 689 :children => { :count => 3 }
690 690 assert_no_tag :tag => 'option', :attributes => { :value => 'project' },
691 691 :parent => { :tag => 'select', :attributes => { :id => "selected_columns" } }
692 692 end
693 693
694 694 def test_index_without_project_should_implicitly_add_project_column_to_default_columns
695 695 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
696 696 get :index, :set_filter => 1
697 697
698 698 # query should use specified columns
699 699 query = assigns(:query)
700 700 assert_kind_of Query, query
701 701 assert_equal [:project, :tracker, :subject, :assigned_to], query.columns.map(&:name)
702 702 end
703 703
704 704 def test_index_without_project_and_explicit_default_columns_should_not_add_project_column
705 705 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
706 706 columns = ['tracker', 'subject', 'assigned_to']
707 707 get :index, :set_filter => 1, :c => columns
708 708
709 709 # query should use specified columns
710 710 query = assigns(:query)
711 711 assert_kind_of Query, query
712 712 assert_equal columns.map(&:to_sym), query.columns.map(&:name)
713 713 end
714 714
715 715 def test_index_with_custom_field_column
716 716 columns = %w(tracker subject cf_2)
717 717 get :index, :set_filter => 1, :c => columns
718 718 assert_response :success
719 719
720 720 # query should use specified columns
721 721 query = assigns(:query)
722 722 assert_kind_of Query, query
723 723 assert_equal columns, query.column_names.map(&:to_s)
724 724
725 725 assert_tag :td,
726 726 :attributes => {:class => 'cf_2 string'},
727 727 :ancestor => {:tag => 'table', :attributes => {:class => /issues/}}
728 728 end
729 729
730 730 def test_index_with_multi_custom_field_column
731 731 field = CustomField.find(1)
732 732 field.update_attribute :multiple, true
733 733 issue = Issue.find(1)
734 734 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
735 735 issue.save!
736 736
737 737 get :index, :set_filter => 1, :c => %w(tracker subject cf_1)
738 738 assert_response :success
739 739
740 740 assert_tag :td,
741 741 :attributes => {:class => /cf_1/},
742 742 :content => 'MySQL, Oracle'
743 743 end
744 744
745 745 def test_index_with_multi_user_custom_field_column
746 746 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
747 747 :tracker_ids => [1], :is_for_all => true)
748 748 issue = Issue.find(1)
749 749 issue.custom_field_values = {field.id => ['2', '3']}
750 750 issue.save!
751 751
752 752 get :index, :set_filter => 1, :c => ['tracker', 'subject', "cf_#{field.id}"]
753 753 assert_response :success
754 754
755 755 assert_tag :td,
756 756 :attributes => {:class => /cf_#{field.id}/},
757 757 :child => {:tag => 'a', :content => 'John Smith'}
758 758 end
759 759
760 760 def test_index_with_date_column
761 761 Issue.find(1).update_attribute :start_date, '1987-08-24'
762 762
763 763 with_settings :date_format => '%d/%m/%Y' do
764 764 get :index, :set_filter => 1, :c => %w(start_date)
765 765 assert_tag 'td', :attributes => {:class => /start_date/}, :content => '24/08/1987'
766 766 end
767 767 end
768 768
769 769 def test_index_with_done_ratio
770 770 Issue.find(1).update_attribute :done_ratio, 40
771 771
772 772 get :index, :set_filter => 1, :c => %w(done_ratio)
773 773 assert_tag 'td', :attributes => {:class => /done_ratio/},
774 774 :child => {:tag => 'table', :attributes => {:class => 'progress'},
775 775 :descendant => {:tag => 'td', :attributes => {:class => 'closed', :style => 'width: 40%;'}}
776 776 }
777 777 end
778 778
779 779 def test_index_with_spent_hours_column
780 780 get :index, :set_filter => 1, :c => %w(subject spent_hours)
781 781
782 782 assert_tag 'tr', :attributes => {:id => 'issue-3'},
783 783 :child => {
784 784 :tag => 'td', :attributes => {:class => /spent_hours/}, :content => '1.00'
785 785 }
786 786 end
787 787
788 788 def test_index_should_not_show_spent_hours_column_without_permission
789 789 Role.anonymous.remove_permission! :view_time_entries
790 790 get :index, :set_filter => 1, :c => %w(subject spent_hours)
791 791
792 792 assert_no_tag 'td', :attributes => {:class => /spent_hours/}
793 793 end
794 794
795 795 def test_index_with_fixed_version
796 796 get :index, :set_filter => 1, :c => %w(fixed_version)
797 797 assert_tag 'td', :attributes => {:class => /fixed_version/},
798 798 :child => {:tag => 'a', :content => '1.0', :attributes => {:href => '/versions/2'}}
799 799 end
800 800
801 801 def test_index_send_html_if_query_is_invalid
802 802 get :index, :f => ['start_date'], :op => {:start_date => '='}
803 803 assert_equal 'text/html', @response.content_type
804 804 assert_template 'index'
805 805 end
806 806
807 807 def test_index_send_nothing_if_query_is_invalid
808 808 get :index, :f => ['start_date'], :op => {:start_date => '='}, :format => 'csv'
809 809 assert_equal 'text/csv', @response.content_type
810 810 assert @response.body.blank?
811 811 end
812 812
813 813 def test_show_by_anonymous
814 814 get :show, :id => 1
815 815 assert_response :success
816 816 assert_template 'show'
817 817 assert_not_nil assigns(:issue)
818 818 assert_equal Issue.find(1), assigns(:issue)
819 819
820 820 # anonymous role is allowed to add a note
821 821 assert_tag :tag => 'form',
822 822 :descendant => { :tag => 'fieldset',
823 823 :child => { :tag => 'legend',
824 824 :content => /Notes/ } }
825 825 assert_tag :tag => 'title',
826 826 :content => "Bug #1: Can&#x27;t print recipes - eCookbook - Redmine"
827 827 end
828 828
829 829 def test_show_by_manager
830 830 @request.session[:user_id] = 2
831 831 get :show, :id => 1
832 832 assert_response :success
833 833
834 834 assert_tag :tag => 'a',
835 835 :content => /Quote/
836 836
837 837 assert_tag :tag => 'form',
838 838 :descendant => { :tag => 'fieldset',
839 839 :child => { :tag => 'legend',
840 840 :content => /Change properties/ } },
841 841 :descendant => { :tag => 'fieldset',
842 842 :child => { :tag => 'legend',
843 843 :content => /Log time/ } },
844 844 :descendant => { :tag => 'fieldset',
845 845 :child => { :tag => 'legend',
846 846 :content => /Notes/ } }
847 847 end
848 848
849 849 def test_show_should_display_update_form
850 850 @request.session[:user_id] = 2
851 851 get :show, :id => 1
852 852 assert_response :success
853 853
854 854 assert_tag 'form', :attributes => {:id => 'issue-form'}
855 855 assert_tag 'input', :attributes => {:name => 'issue[is_private]'}
856 856 assert_tag 'select', :attributes => {:name => 'issue[project_id]'}
857 857 assert_tag 'select', :attributes => {:name => 'issue[tracker_id]'}
858 858 assert_tag 'input', :attributes => {:name => 'issue[subject]'}
859 859 assert_tag 'textarea', :attributes => {:name => 'issue[description]'}
860 860 assert_tag 'select', :attributes => {:name => 'issue[status_id]'}
861 861 assert_tag 'select', :attributes => {:name => 'issue[priority_id]'}
862 862 assert_tag 'select', :attributes => {:name => 'issue[assigned_to_id]'}
863 863 assert_tag 'select', :attributes => {:name => 'issue[category_id]'}
864 864 assert_tag 'select', :attributes => {:name => 'issue[fixed_version_id]'}
865 865 assert_tag 'input', :attributes => {:name => 'issue[parent_issue_id]'}
866 866 assert_tag 'input', :attributes => {:name => 'issue[start_date]'}
867 867 assert_tag 'input', :attributes => {:name => 'issue[due_date]'}
868 868 assert_tag 'select', :attributes => {:name => 'issue[done_ratio]'}
869 869 assert_tag 'input', :attributes => { :name => 'issue[custom_field_values][2]' }
870 870 assert_no_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]'}
871 871 assert_tag 'textarea', :attributes => {:name => 'notes'}
872 872 end
873 873
874 874 def test_show_should_display_update_form_with_minimal_permissions
875 875 Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes]
876 876 WorkflowTransition.delete_all :role_id => 1
877 877
878 878 @request.session[:user_id] = 2
879 879 get :show, :id => 1
880 880 assert_response :success
881 881
882 882 assert_tag 'form', :attributes => {:id => 'issue-form'}
883 883 assert_no_tag 'input', :attributes => {:name => 'issue[is_private]'}
884 884 assert_no_tag 'select', :attributes => {:name => 'issue[project_id]'}
885 885 assert_no_tag 'select', :attributes => {:name => 'issue[tracker_id]'}
886 886 assert_no_tag 'input', :attributes => {:name => 'issue[subject]'}
887 887 assert_no_tag 'textarea', :attributes => {:name => 'issue[description]'}
888 888 assert_no_tag 'select', :attributes => {:name => 'issue[status_id]'}
889 889 assert_no_tag 'select', :attributes => {:name => 'issue[priority_id]'}
890 890 assert_no_tag 'select', :attributes => {:name => 'issue[assigned_to_id]'}
891 891 assert_no_tag 'select', :attributes => {:name => 'issue[category_id]'}
892 892 assert_no_tag 'select', :attributes => {:name => 'issue[fixed_version_id]'}
893 893 assert_no_tag 'input', :attributes => {:name => 'issue[parent_issue_id]'}
894 894 assert_no_tag 'input', :attributes => {:name => 'issue[start_date]'}
895 895 assert_no_tag 'input', :attributes => {:name => 'issue[due_date]'}
896 896 assert_no_tag 'select', :attributes => {:name => 'issue[done_ratio]'}
897 897 assert_no_tag 'input', :attributes => { :name => 'issue[custom_field_values][2]' }
898 898 assert_no_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]'}
899 899 assert_tag 'textarea', :attributes => {:name => 'notes'}
900 900 end
901 901
902 902 def test_show_should_display_update_form_with_workflow_permissions
903 903 Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes]
904 904
905 905 @request.session[:user_id] = 2
906 906 get :show, :id => 1
907 907 assert_response :success
908 908
909 909 assert_tag 'form', :attributes => {:id => 'issue-form'}
910 910 assert_no_tag 'input', :attributes => {:name => 'issue[is_private]'}
911 911 assert_no_tag 'select', :attributes => {:name => 'issue[project_id]'}
912 912 assert_no_tag 'select', :attributes => {:name => 'issue[tracker_id]'}
913 913 assert_no_tag 'input', :attributes => {:name => 'issue[subject]'}
914 914 assert_no_tag 'textarea', :attributes => {:name => 'issue[description]'}
915 915 assert_tag 'select', :attributes => {:name => 'issue[status_id]'}
916 916 assert_no_tag 'select', :attributes => {:name => 'issue[priority_id]'}
917 917 assert_tag 'select', :attributes => {:name => 'issue[assigned_to_id]'}
918 918 assert_no_tag 'select', :attributes => {:name => 'issue[category_id]'}
919 919 assert_tag 'select', :attributes => {:name => 'issue[fixed_version_id]'}
920 920 assert_no_tag 'input', :attributes => {:name => 'issue[parent_issue_id]'}
921 921 assert_no_tag 'input', :attributes => {:name => 'issue[start_date]'}
922 922 assert_no_tag 'input', :attributes => {:name => 'issue[due_date]'}
923 923 assert_tag 'select', :attributes => {:name => 'issue[done_ratio]'}
924 924 assert_no_tag 'input', :attributes => { :name => 'issue[custom_field_values][2]' }
925 925 assert_no_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]'}
926 926 assert_tag 'textarea', :attributes => {:name => 'notes'}
927 927 end
928 928
929 929 def test_show_should_not_display_update_form_without_permissions
930 930 Role.find(1).update_attribute :permissions, [:view_issues]
931 931
932 932 @request.session[:user_id] = 2
933 933 get :show, :id => 1
934 934 assert_response :success
935 935
936 936 assert_no_tag 'form', :attributes => {:id => 'issue-form'}
937 937 end
938 938
939 939 def test_update_form_should_not_display_inactive_enumerations
940 940 @request.session[:user_id] = 2
941 941 get :show, :id => 1
942 942 assert_response :success
943 943
944 944 assert ! IssuePriority.find(15).active?
945 945 assert_no_tag :option, :attributes => {:value => '15'},
946 946 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
947 947 end
948 948
949 949 def test_update_form_should_allow_attachment_upload
950 950 @request.session[:user_id] = 2
951 951 get :show, :id => 1
952 952
953 953 assert_tag :tag => 'form',
954 954 :attributes => {:id => 'issue-form', :method => 'post', :enctype => 'multipart/form-data'},
955 955 :descendant => {
956 956 :tag => 'input',
957 957 :attributes => {:type => 'file', :name => 'attachments[1][file]'}
958 958 }
959 959 end
960 960
961 961 def test_show_should_deny_anonymous_access_without_permission
962 962 Role.anonymous.remove_permission!(:view_issues)
963 963 get :show, :id => 1
964 964 assert_response :redirect
965 965 end
966 966
967 967 def test_show_should_deny_anonymous_access_to_private_issue
968 968 Issue.update_all(["is_private = ?", true], "id = 1")
969 969 get :show, :id => 1
970 970 assert_response :redirect
971 971 end
972 972
973 973 def test_show_should_deny_non_member_access_without_permission
974 974 Role.non_member.remove_permission!(:view_issues)
975 975 @request.session[:user_id] = 9
976 976 get :show, :id => 1
977 977 assert_response 403
978 978 end
979 979
980 980 def test_show_should_deny_non_member_access_to_private_issue
981 981 Issue.update_all(["is_private = ?", true], "id = 1")
982 982 @request.session[:user_id] = 9
983 983 get :show, :id => 1
984 984 assert_response 403
985 985 end
986 986
987 987 def test_show_should_deny_member_access_without_permission
988 988 Role.find(1).remove_permission!(:view_issues)
989 989 @request.session[:user_id] = 2
990 990 get :show, :id => 1
991 991 assert_response 403
992 992 end
993 993
994 994 def test_show_should_deny_member_access_to_private_issue_without_permission
995 995 Issue.update_all(["is_private = ?", true], "id = 1")
996 996 @request.session[:user_id] = 3
997 997 get :show, :id => 1
998 998 assert_response 403
999 999 end
1000 1000
1001 1001 def test_show_should_allow_author_access_to_private_issue
1002 1002 Issue.update_all(["is_private = ?, author_id = 3", true], "id = 1")
1003 1003 @request.session[:user_id] = 3
1004 1004 get :show, :id => 1
1005 1005 assert_response :success
1006 1006 end
1007 1007
1008 1008 def test_show_should_allow_assignee_access_to_private_issue
1009 1009 Issue.update_all(["is_private = ?, assigned_to_id = 3", true], "id = 1")
1010 1010 @request.session[:user_id] = 3
1011 1011 get :show, :id => 1
1012 1012 assert_response :success
1013 1013 end
1014 1014
1015 1015 def test_show_should_allow_member_access_to_private_issue_with_permission
1016 1016 Issue.update_all(["is_private = ?", true], "id = 1")
1017 1017 User.find(3).roles_for_project(Project.find(1)).first.update_attribute :issues_visibility, 'all'
1018 1018 @request.session[:user_id] = 3
1019 1019 get :show, :id => 1
1020 1020 assert_response :success
1021 1021 end
1022 1022
1023 1023 def test_show_should_not_disclose_relations_to_invisible_issues
1024 1024 Setting.cross_project_issue_relations = '1'
1025 1025 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
1026 1026 # Relation to a private project issue
1027 1027 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
1028 1028
1029 1029 get :show, :id => 1
1030 1030 assert_response :success
1031 1031
1032 1032 assert_tag :div, :attributes => { :id => 'relations' },
1033 1033 :descendant => { :tag => 'a', :content => /#2$/ }
1034 1034 assert_no_tag :div, :attributes => { :id => 'relations' },
1035 1035 :descendant => { :tag => 'a', :content => /#4$/ }
1036 1036 end
1037 1037
1038 1038 def test_show_should_list_subtasks
1039 1039 Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue')
1040 1040
1041 1041 get :show, :id => 1
1042 1042 assert_response :success
1043 1043 assert_tag 'div', :attributes => {:id => 'issue_tree'},
1044 1044 :descendant => {:tag => 'td', :content => /Child Issue/, :attributes => {:class => /subject/}}
1045 1045 end
1046 1046
1047 1047 def test_show_should_list_parents
1048 1048 issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue')
1049 1049
1050 1050 get :show, :id => issue.id
1051 1051 assert_response :success
1052 1052 assert_tag 'div', :attributes => {:class => 'subject'},
1053 1053 :descendant => {:tag => 'h3', :content => 'Child Issue'}
1054 1054 assert_tag 'div', :attributes => {:class => 'subject'},
1055 1055 :descendant => {:tag => 'a', :attributes => {:href => '/issues/1'}}
1056 1056 end
1057 1057
1058 1058 def test_show_should_not_display_prev_next_links_without_query_in_session
1059 1059 get :show, :id => 1
1060 1060 assert_response :success
1061 1061 assert_nil assigns(:prev_issue_id)
1062 1062 assert_nil assigns(:next_issue_id)
1063 1063
1064 1064 assert_no_tag 'div', :attributes => {:class => /next-prev-links/}
1065 1065 end
1066 1066
1067 1067 def test_show_should_display_prev_next_links_with_query_in_session
1068 1068 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil}
1069 1069 @request.session['issues_index_sort'] = 'id'
1070 1070
1071 1071 with_settings :display_subprojects_issues => '0' do
1072 1072 get :show, :id => 3
1073 1073 end
1074 1074
1075 1075 assert_response :success
1076 1076 # Previous and next issues for all projects
1077 1077 assert_equal 2, assigns(:prev_issue_id)
1078 1078 assert_equal 5, assigns(:next_issue_id)
1079 1079
1080 1080 assert_tag 'div', :attributes => {:class => /next-prev-links/}
1081 1081 assert_tag 'a', :attributes => {:href => '/issues/2'}, :content => /Previous/
1082 1082 assert_tag 'a', :attributes => {:href => '/issues/5'}, :content => /Next/
1083 1083
1084 1084 count = Issue.open.visible.count
1085 1085 assert_tag 'span', :attributes => {:class => 'position'}, :content => "3 of #{count}"
1086 1086 end
1087 1087
1088 1088 def test_show_should_display_prev_next_links_with_saved_query_in_session
1089 1089 query = Query.create!(:name => 'test', :is_public => true, :user_id => 1,
1090 1090 :filters => {'status_id' => {:values => ['5'], :operator => '='}},
1091 1091 :sort_criteria => [['id', 'asc']])
1092 1092 @request.session[:query] = {:id => query.id, :project_id => nil}
1093 1093
1094 1094 get :show, :id => 11
1095 1095
1096 1096 assert_response :success
1097 1097 assert_equal query, assigns(:query)
1098 1098 # Previous and next issues for all projects
1099 1099 assert_equal 8, assigns(:prev_issue_id)
1100 1100 assert_equal 12, assigns(:next_issue_id)
1101 1101
1102 1102 assert_tag 'a', :attributes => {:href => '/issues/8'}, :content => /Previous/
1103 1103 assert_tag 'a', :attributes => {:href => '/issues/12'}, :content => /Next/
1104 1104 end
1105 1105
1106 1106 def test_show_should_display_prev_next_links_with_query_and_sort_on_association
1107 1107 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil}
1108 1108
1109 1109 %w(project tracker status priority author assigned_to category fixed_version).each do |assoc_sort|
1110 1110 @request.session['issues_index_sort'] = assoc_sort
1111 1111
1112 1112 get :show, :id => 3
1113 1113 assert_response :success, "Wrong response status for #{assoc_sort} sort"
1114 1114
1115 1115 assert_tag 'div', :attributes => {:class => /next-prev-links/}, :content => /Previous/
1116 1116 assert_tag 'div', :attributes => {:class => /next-prev-links/}, :content => /Next/
1117 1117 end
1118 1118 end
1119 1119
1120 1120 def test_show_should_display_prev_next_links_with_project_query_in_session
1121 1121 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1}
1122 1122 @request.session['issues_index_sort'] = 'id'
1123 1123
1124 1124 with_settings :display_subprojects_issues => '0' do
1125 1125 get :show, :id => 3
1126 1126 end
1127 1127
1128 1128 assert_response :success
1129 1129 # Previous and next issues inside project
1130 1130 assert_equal 2, assigns(:prev_issue_id)
1131 1131 assert_equal 7, assigns(:next_issue_id)
1132 1132
1133 1133 assert_tag 'a', :attributes => {:href => '/issues/2'}, :content => /Previous/
1134 1134 assert_tag 'a', :attributes => {:href => '/issues/7'}, :content => /Next/
1135 1135 end
1136 1136
1137 1137 def test_show_should_not_display_prev_link_for_first_issue
1138 1138 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1}
1139 1139 @request.session['issues_index_sort'] = 'id'
1140 1140
1141 1141 with_settings :display_subprojects_issues => '0' do
1142 1142 get :show, :id => 1
1143 1143 end
1144 1144
1145 1145 assert_response :success
1146 1146 assert_nil assigns(:prev_issue_id)
1147 1147 assert_equal 2, assigns(:next_issue_id)
1148 1148
1149 1149 assert_no_tag 'a', :content => /Previous/
1150 1150 assert_tag 'a', :attributes => {:href => '/issues/2'}, :content => /Next/
1151 1151 end
1152 1152
1153 1153 def test_show_should_not_display_prev_next_links_for_issue_not_in_query_results
1154 1154 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'c'}}, :project_id => 1}
1155 1155 @request.session['issues_index_sort'] = 'id'
1156 1156
1157 1157 get :show, :id => 1
1158 1158
1159 1159 assert_response :success
1160 1160 assert_nil assigns(:prev_issue_id)
1161 1161 assert_nil assigns(:next_issue_id)
1162 1162
1163 1163 assert_no_tag 'a', :content => /Previous/
1164 1164 assert_no_tag 'a', :content => /Next/
1165 1165 end
1166 1166
1167 1167 def test_show_show_should_display_prev_next_links_with_query_sort_by_user_custom_field
1168 1168 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
1169 1169 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
1170 1170 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
1171 1171 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
1172 1172 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
1173 1173
1174 1174 query = Query.create!(:name => 'test', :is_public => true, :user_id => 1, :filters => {},
1175 1175 :sort_criteria => [["cf_#{cf.id}", 'asc'], ['id', 'asc']])
1176 1176 @request.session[:query] = {:id => query.id, :project_id => nil}
1177 1177
1178 1178 get :show, :id => 3
1179 1179 assert_response :success
1180 1180
1181 1181 assert_equal 2, assigns(:prev_issue_id)
1182 1182 assert_equal 1, assigns(:next_issue_id)
1183 1183 end
1184 1184
1185 1185 def test_show_should_display_link_to_the_assignee
1186 1186 get :show, :id => 2
1187 1187 assert_response :success
1188 1188 assert_select '.assigned-to' do
1189 1189 assert_select 'a[href=/users/3]'
1190 1190 end
1191 1191 end
1192 1192
1193 1193 def test_show_should_display_visible_changesets_from_other_projects
1194 1194 project = Project.find(2)
1195 1195 issue = project.issues.first
1196 1196 issue.changeset_ids = [102]
1197 1197 issue.save!
1198 1198 project.disable_module! :repository
1199 1199
1200 1200 @request.session[:user_id] = 2
1201 1201 get :show, :id => issue.id
1202 1202 assert_tag 'a', :attributes => {:href => "/projects/ecookbook/repository/revisions/3"}
1203 1203 end
1204 1204
1205 1205 def test_show_should_display_watchers
1206 1206 @request.session[:user_id] = 2
1207 1207 Issue.find(1).add_watcher User.find(2)
1208 1208
1209 1209 get :show, :id => 1
1210 1210 assert_select 'div#watchers ul' do
1211 1211 assert_select 'li' do
1212 1212 assert_select 'a[href=/users/2]'
1213 1213 assert_select 'a img[alt=Delete]'
1214 1214 end
1215 1215 end
1216 1216 end
1217 1217
1218 1218 def test_show_should_display_watchers_with_gravatars
1219 1219 @request.session[:user_id] = 2
1220 1220 Issue.find(1).add_watcher User.find(2)
1221 1221
1222 1222 with_settings :gravatar_enabled => '1' do
1223 1223 get :show, :id => 1
1224 1224 end
1225 1225
1226 1226 assert_select 'div#watchers ul' do
1227 1227 assert_select 'li' do
1228 1228 assert_select 'img.gravatar'
1229 1229 assert_select 'a[href=/users/2]'
1230 1230 assert_select 'a img[alt=Delete]'
1231 1231 end
1232 1232 end
1233 1233 end
1234 1234
1235 1235 def test_show_with_thumbnails_enabled_should_display_thumbnails
1236 1236 @request.session[:user_id] = 2
1237 1237
1238 1238 with_settings :thumbnails_enabled => '1' do
1239 1239 get :show, :id => 14
1240 1240 assert_response :success
1241 1241 end
1242 1242
1243 1243 assert_select 'div.thumbnails' do
1244 1244 assert_select 'a[href=/attachments/16/testfile.png]' do
1245 1245 assert_select 'img[src=/attachments/thumbnail/16]'
1246 1246 end
1247 1247 end
1248 1248 end
1249 1249
1250 1250 def test_show_with_thumbnails_disabled_should_not_display_thumbnails
1251 1251 @request.session[:user_id] = 2
1252 1252
1253 1253 with_settings :thumbnails_enabled => '0' do
1254 1254 get :show, :id => 14
1255 1255 assert_response :success
1256 1256 end
1257 1257
1258 1258 assert_select 'div.thumbnails', 0
1259 1259 end
1260 1260
1261 1261 def test_show_with_multi_custom_field
1262 1262 field = CustomField.find(1)
1263 1263 field.update_attribute :multiple, true
1264 1264 issue = Issue.find(1)
1265 1265 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
1266 1266 issue.save!
1267 1267
1268 1268 get :show, :id => 1
1269 1269 assert_response :success
1270 1270
1271 1271 assert_tag :td, :content => 'MySQL, Oracle'
1272 1272 end
1273 1273
1274 1274 def test_show_with_multi_user_custom_field
1275 1275 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
1276 1276 :tracker_ids => [1], :is_for_all => true)
1277 1277 issue = Issue.find(1)
1278 1278 issue.custom_field_values = {field.id => ['2', '3']}
1279 1279 issue.save!
1280 1280
1281 1281 get :show, :id => 1
1282 1282 assert_response :success
1283 1283
1284 1284 # TODO: should display links
1285 1285 assert_tag :td, :content => 'Dave Lopper, John Smith'
1286 1286 end
1287 1287
1288 1288 def test_show_atom
1289 1289 get :show, :id => 2, :format => 'atom'
1290 1290 assert_response :success
1291 1291 assert_template 'journals/index'
1292 1292 # Inline image
1293 1293 assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
1294 1294 end
1295 1295
1296 1296 def test_show_export_to_pdf
1297 1297 get :show, :id => 3, :format => 'pdf'
1298 1298 assert_response :success
1299 1299 assert_equal 'application/pdf', @response.content_type
1300 1300 assert @response.body.starts_with?('%PDF')
1301 1301 assert_not_nil assigns(:issue)
1302 1302 end
1303 1303
1304 1304 def test_show_export_to_pdf_with_ancestors
1305 1305 issue = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
1306 1306
1307 1307 get :show, :id => issue.id, :format => 'pdf'
1308 1308 assert_response :success
1309 1309 assert_equal 'application/pdf', @response.content_type
1310 1310 assert @response.body.starts_with?('%PDF')
1311 1311 end
1312 1312
1313 1313 def test_show_export_to_pdf_with_descendants
1314 1314 c1 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
1315 1315 c2 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
1316 1316 c3 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => c1.id)
1317 1317
1318 1318 get :show, :id => 1, :format => 'pdf'
1319 1319 assert_response :success
1320 1320 assert_equal 'application/pdf', @response.content_type
1321 1321 assert @response.body.starts_with?('%PDF')
1322 1322 end
1323 1323
1324 1324 def test_show_export_to_pdf_with_journals
1325 1325 get :show, :id => 1, :format => 'pdf'
1326 1326 assert_response :success
1327 1327 assert_equal 'application/pdf', @response.content_type
1328 1328 assert @response.body.starts_with?('%PDF')
1329 1329 end
1330 1330
1331 1331 def test_show_export_to_pdf_with_changesets
1332 1332 Issue.find(3).changesets = Changeset.find_all_by_id(100, 101, 102)
1333 1333
1334 1334 get :show, :id => 3, :format => 'pdf'
1335 1335 assert_response :success
1336 1336 assert_equal 'application/pdf', @response.content_type
1337 1337 assert @response.body.starts_with?('%PDF')
1338 1338 end
1339 1339
1340 1340 def test_get_new
1341 1341 @request.session[:user_id] = 2
1342 1342 get :new, :project_id => 1, :tracker_id => 1
1343 1343 assert_response :success
1344 1344 assert_template 'new'
1345 1345
1346 1346 assert_tag 'input', :attributes => {:name => 'issue[is_private]'}
1347 1347 assert_no_tag 'select', :attributes => {:name => 'issue[project_id]'}
1348 1348 assert_tag 'select', :attributes => {:name => 'issue[tracker_id]'}
1349 1349 assert_tag 'input', :attributes => {:name => 'issue[subject]'}
1350 1350 assert_tag 'textarea', :attributes => {:name => 'issue[description]'}
1351 1351 assert_tag 'select', :attributes => {:name => 'issue[status_id]'}
1352 1352 assert_tag 'select', :attributes => {:name => 'issue[priority_id]'}
1353 1353 assert_tag 'select', :attributes => {:name => 'issue[assigned_to_id]'}
1354 1354 assert_tag 'select', :attributes => {:name => 'issue[category_id]'}
1355 1355 assert_tag 'select', :attributes => {:name => 'issue[fixed_version_id]'}
1356 1356 assert_tag 'input', :attributes => {:name => 'issue[parent_issue_id]'}
1357 1357 assert_tag 'input', :attributes => {:name => 'issue[start_date]'}
1358 1358 assert_tag 'input', :attributes => {:name => 'issue[due_date]'}
1359 1359 assert_tag 'select', :attributes => {:name => 'issue[done_ratio]'}
1360 1360 assert_tag 'input', :attributes => { :name => 'issue[custom_field_values][2]', :value => 'Default string' }
1361 1361 assert_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]'}
1362 1362
1363 1363 # Be sure we don't display inactive IssuePriorities
1364 1364 assert ! IssuePriority.find(15).active?
1365 1365 assert_no_tag :option, :attributes => {:value => '15'},
1366 1366 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
1367 1367 end
1368 1368
1369 1369 def test_get_new_with_minimal_permissions
1370 1370 Role.find(1).update_attribute :permissions, [:add_issues]
1371 1371 WorkflowTransition.delete_all :role_id => 1
1372 1372
1373 1373 @request.session[:user_id] = 2
1374 1374 get :new, :project_id => 1, :tracker_id => 1
1375 1375 assert_response :success
1376 1376 assert_template 'new'
1377 1377
1378 1378 assert_no_tag 'input', :attributes => {:name => 'issue[is_private]'}
1379 1379 assert_no_tag 'select', :attributes => {:name => 'issue[project_id]'}
1380 1380 assert_tag 'select', :attributes => {:name => 'issue[tracker_id]'}
1381 1381 assert_tag 'input', :attributes => {:name => 'issue[subject]'}
1382 1382 assert_tag 'textarea', :attributes => {:name => 'issue[description]'}
1383 1383 assert_tag 'select', :attributes => {:name => 'issue[status_id]'}
1384 1384 assert_tag 'select', :attributes => {:name => 'issue[priority_id]'}
1385 1385 assert_tag 'select', :attributes => {:name => 'issue[assigned_to_id]'}
1386 1386 assert_tag 'select', :attributes => {:name => 'issue[category_id]'}
1387 1387 assert_tag 'select', :attributes => {:name => 'issue[fixed_version_id]'}
1388 1388 assert_no_tag 'input', :attributes => {:name => 'issue[parent_issue_id]'}
1389 1389 assert_tag 'input', :attributes => {:name => 'issue[start_date]'}
1390 1390 assert_tag 'input', :attributes => {:name => 'issue[due_date]'}
1391 1391 assert_tag 'select', :attributes => {:name => 'issue[done_ratio]'}
1392 1392 assert_tag 'input', :attributes => { :name => 'issue[custom_field_values][2]', :value => 'Default string' }
1393 1393 assert_no_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]'}
1394 1394 end
1395 1395
1396 1396 def test_get_new_with_list_custom_field
1397 1397 @request.session[:user_id] = 2
1398 1398 get :new, :project_id => 1, :tracker_id => 1
1399 1399 assert_response :success
1400 1400 assert_template 'new'
1401 1401
1402 1402 assert_tag 'select',
1403 1403 :attributes => {:name => 'issue[custom_field_values][1]', :class => 'list_cf'},
1404 1404 :children => {:count => 4},
1405 1405 :child => {:tag => 'option', :attributes => {:value => 'MySQL'}, :content => 'MySQL'}
1406 1406 end
1407 1407
1408 1408 def test_get_new_with_multi_custom_field
1409 1409 field = IssueCustomField.find(1)
1410 1410 field.update_attribute :multiple, true
1411 1411
1412 1412 @request.session[:user_id] = 2
1413 1413 get :new, :project_id => 1, :tracker_id => 1
1414 1414 assert_response :success
1415 1415 assert_template 'new'
1416 1416
1417 1417 assert_tag 'select',
1418 1418 :attributes => {:name => 'issue[custom_field_values][1][]', :multiple => 'multiple'},
1419 1419 :children => {:count => 3},
1420 1420 :child => {:tag => 'option', :attributes => {:value => 'MySQL'}, :content => 'MySQL'}
1421 1421 assert_tag 'input',
1422 1422 :attributes => {:name => 'issue[custom_field_values][1][]', :value => ''}
1423 1423 end
1424 1424
1425 1425 def test_get_new_with_multi_user_custom_field
1426 1426 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
1427 1427 :tracker_ids => [1], :is_for_all => true)
1428 1428
1429 1429 @request.session[:user_id] = 2
1430 1430 get :new, :project_id => 1, :tracker_id => 1
1431 1431 assert_response :success
1432 1432 assert_template 'new'
1433 1433
1434 1434 assert_tag 'select',
1435 1435 :attributes => {:name => "issue[custom_field_values][#{field.id}][]", :multiple => 'multiple'},
1436 1436 :children => {:count => Project.find(1).users.count},
1437 1437 :child => {:tag => 'option', :attributes => {:value => '2'}, :content => 'John Smith'}
1438 1438 assert_tag 'input',
1439 1439 :attributes => {:name => "issue[custom_field_values][#{field.id}][]", :value => ''}
1440 1440 end
1441 1441
1442 1442 def test_get_new_with_date_custom_field
1443 1443 field = IssueCustomField.create!(:name => 'Date', :field_format => 'date', :tracker_ids => [1], :is_for_all => true)
1444 1444
1445 1445 @request.session[:user_id] = 2
1446 1446 get :new, :project_id => 1, :tracker_id => 1
1447 1447 assert_response :success
1448 1448
1449 1449 assert_select 'input[name=?]', "issue[custom_field_values][#{field.id}]"
1450 1450 end
1451 1451
1452 1452 def test_get_new_with_text_custom_field
1453 1453 field = IssueCustomField.create!(:name => 'Text', :field_format => 'text', :tracker_ids => [1], :is_for_all => true)
1454 1454
1455 1455 @request.session[:user_id] = 2
1456 1456 get :new, :project_id => 1, :tracker_id => 1
1457 1457 assert_response :success
1458 1458
1459 1459 assert_select 'textarea[name=?]', "issue[custom_field_values][#{field.id}]"
1460 1460 end
1461 1461
1462 1462 def test_get_new_without_default_start_date_is_creation_date
1463 1463 Setting.default_issue_start_date_to_creation_date = 0
1464 1464
1465 1465 @request.session[:user_id] = 2
1466 1466 get :new, :project_id => 1, :tracker_id => 1
1467 1467 assert_response :success
1468 1468 assert_template 'new'
1469 1469
1470 1470 assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]',
1471 1471 :value => nil }
1472 1472 end
1473 1473
1474 1474 def test_get_new_with_default_start_date_is_creation_date
1475 1475 Setting.default_issue_start_date_to_creation_date = 1
1476 1476
1477 1477 @request.session[:user_id] = 2
1478 1478 get :new, :project_id => 1, :tracker_id => 1
1479 1479 assert_response :success
1480 1480 assert_template 'new'
1481 1481
1482 1482 assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]',
1483 1483 :value => Date.today.to_s }
1484 1484 end
1485 1485
1486 1486 def test_get_new_form_should_allow_attachment_upload
1487 1487 @request.session[:user_id] = 2
1488 1488 get :new, :project_id => 1, :tracker_id => 1
1489 1489
1490 1490 assert_select 'form[id=issue-form][method=post][enctype=multipart/form-data]' do
1491 1491 assert_select 'input[name=?][type=file]', 'attachments[1][file]'
1492 1492 assert_select 'input[name=?][maxlength=255]', 'attachments[1][description]'
1493 1493 end
1494 1494 end
1495 1495
1496 1496 def test_get_new_should_prefill_the_form_from_params
1497 1497 @request.session[:user_id] = 2
1498 1498 get :new, :project_id => 1,
1499 1499 :issue => {:tracker_id => 3, :description => 'Prefilled', :custom_field_values => {'2' => 'Custom field value'}}
1500 1500
1501 1501 issue = assigns(:issue)
1502 1502 assert_equal 3, issue.tracker_id
1503 1503 assert_equal 'Prefilled', issue.description
1504 1504 assert_equal 'Custom field value', issue.custom_field_value(2)
1505 1505
1506 1506 assert_tag 'select',
1507 1507 :attributes => {:name => 'issue[tracker_id]'},
1508 1508 :child => {:tag => 'option', :attributes => {:value => '3', :selected => 'selected'}}
1509 1509 assert_tag 'textarea',
1510 1510 :attributes => {:name => 'issue[description]'}, :content => "\nPrefilled"
1511 1511 assert_tag 'input',
1512 1512 :attributes => {:name => 'issue[custom_field_values][2]', :value => 'Custom field value'}
1513 1513 end
1514 1514
1515 1515 def test_get_new_should_mark_required_fields
1516 1516 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1517 1517 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1518 1518 WorkflowPermission.delete_all
1519 1519 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'required')
1520 1520 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required')
1521 1521 @request.session[:user_id] = 2
1522 1522
1523 1523 get :new, :project_id => 1
1524 1524 assert_response :success
1525 1525 assert_template 'new'
1526 1526
1527 1527 assert_select 'label[for=issue_start_date]' do
1528 1528 assert_select 'span[class=required]', 0
1529 1529 end
1530 1530 assert_select 'label[for=issue_due_date]' do
1531 1531 assert_select 'span[class=required]'
1532 1532 end
1533 1533 assert_select 'label[for=?]', "issue_custom_field_values_#{cf1.id}" do
1534 1534 assert_select 'span[class=required]', 0
1535 1535 end
1536 1536 assert_select 'label[for=?]', "issue_custom_field_values_#{cf2.id}" do
1537 1537 assert_select 'span[class=required]'
1538 1538 end
1539 1539 end
1540 1540
1541 1541 def test_get_new_should_not_display_readonly_fields
1542 1542 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1543 1543 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1544 1544 WorkflowPermission.delete_all
1545 1545 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'readonly')
1546 1546 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly')
1547 1547 @request.session[:user_id] = 2
1548 1548
1549 1549 get :new, :project_id => 1
1550 1550 assert_response :success
1551 1551 assert_template 'new'
1552 1552
1553 1553 assert_select 'input[name=?]', 'issue[start_date]'
1554 1554 assert_select 'input[name=?]', 'issue[due_date]', 0
1555 1555 assert_select 'input[name=?]', "issue[custom_field_values][#{cf1.id}]"
1556 1556 assert_select 'input[name=?]', "issue[custom_field_values][#{cf2.id}]", 0
1557 1557 end
1558 1558
1559 1559 def test_get_new_without_tracker_id
1560 1560 @request.session[:user_id] = 2
1561 1561 get :new, :project_id => 1
1562 1562 assert_response :success
1563 1563 assert_template 'new'
1564 1564
1565 1565 issue = assigns(:issue)
1566 1566 assert_not_nil issue
1567 1567 assert_equal Project.find(1).trackers.first, issue.tracker
1568 1568 end
1569 1569
1570 1570 def test_get_new_with_no_default_status_should_display_an_error
1571 1571 @request.session[:user_id] = 2
1572 1572 IssueStatus.delete_all
1573 1573
1574 1574 get :new, :project_id => 1
1575 1575 assert_response 500
1576 1576 assert_error_tag :content => /No default issue/
1577 1577 end
1578 1578
1579 1579 def test_get_new_with_no_tracker_should_display_an_error
1580 1580 @request.session[:user_id] = 2
1581 1581 Tracker.delete_all
1582 1582
1583 1583 get :new, :project_id => 1
1584 1584 assert_response 500
1585 1585 assert_error_tag :content => /No tracker/
1586 1586 end
1587 1587
1588 1588 def test_update_new_form
1589 1589 @request.session[:user_id] = 2
1590 1590 xhr :post, :new, :project_id => 1,
1591 1591 :issue => {:tracker_id => 2,
1592 1592 :subject => 'This is the test_new issue',
1593 1593 :description => 'This is the description',
1594 1594 :priority_id => 5}
1595 1595 assert_response :success
1596 1596 assert_template 'update_form'
1597 1597 assert_template 'form'
1598 1598 assert_equal 'text/javascript', response.content_type
1599 1599
1600 1600 issue = assigns(:issue)
1601 1601 assert_kind_of Issue, issue
1602 1602 assert_equal 1, issue.project_id
1603 1603 assert_equal 2, issue.tracker_id
1604 1604 assert_equal 'This is the test_new issue', issue.subject
1605 1605 end
1606 1606
1607 1607 def test_update_new_form_should_propose_transitions_based_on_initial_status
1608 1608 @request.session[:user_id] = 2
1609 1609 WorkflowTransition.delete_all
1610 1610 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 2)
1611 1611 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 5)
1612 1612 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 5, :new_status_id => 4)
1613 1613
1614 1614 xhr :post, :new, :project_id => 1,
1615 1615 :issue => {:tracker_id => 1,
1616 1616 :status_id => 5,
1617 1617 :subject => 'This is an issue'}
1618 1618
1619 1619 assert_equal 5, assigns(:issue).status_id
1620 1620 assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort
1621 1621 end
1622 1622
1623 1623 def test_post_create
1624 1624 @request.session[:user_id] = 2
1625 1625 assert_difference 'Issue.count' do
1626 1626 post :create, :project_id => 1,
1627 1627 :issue => {:tracker_id => 3,
1628 1628 :status_id => 2,
1629 1629 :subject => 'This is the test_new issue',
1630 1630 :description => 'This is the description',
1631 1631 :priority_id => 5,
1632 1632 :start_date => '2010-11-07',
1633 1633 :estimated_hours => '',
1634 1634 :custom_field_values => {'2' => 'Value for field 2'}}
1635 1635 end
1636 1636 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1637 1637
1638 1638 issue = Issue.find_by_subject('This is the test_new issue')
1639 1639 assert_not_nil issue
1640 1640 assert_equal 2, issue.author_id
1641 1641 assert_equal 3, issue.tracker_id
1642 1642 assert_equal 2, issue.status_id
1643 1643 assert_equal Date.parse('2010-11-07'), issue.start_date
1644 1644 assert_nil issue.estimated_hours
1645 1645 v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
1646 1646 assert_not_nil v
1647 1647 assert_equal 'Value for field 2', v.value
1648 1648 end
1649 1649
1650 1650 def test_post_new_with_group_assignment
1651 1651 group = Group.find(11)
1652 1652 project = Project.find(1)
1653 1653 project.members << Member.new(:principal => group, :roles => [Role.givable.first])
1654 1654
1655 1655 with_settings :issue_group_assignment => '1' do
1656 1656 @request.session[:user_id] = 2
1657 1657 assert_difference 'Issue.count' do
1658 1658 post :create, :project_id => project.id,
1659 1659 :issue => {:tracker_id => 3,
1660 1660 :status_id => 1,
1661 1661 :subject => 'This is the test_new_with_group_assignment issue',
1662 1662 :assigned_to_id => group.id}
1663 1663 end
1664 1664 end
1665 1665 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1666 1666
1667 1667 issue = Issue.find_by_subject('This is the test_new_with_group_assignment issue')
1668 1668 assert_not_nil issue
1669 1669 assert_equal group, issue.assigned_to
1670 1670 end
1671 1671
1672 1672 def test_post_create_without_start_date_and_default_start_date_is_not_creation_date
1673 1673 Setting.default_issue_start_date_to_creation_date = 0
1674 1674
1675 1675 @request.session[:user_id] = 2
1676 1676 assert_difference 'Issue.count' do
1677 1677 post :create, :project_id => 1,
1678 1678 :issue => {:tracker_id => 3,
1679 1679 :status_id => 2,
1680 1680 :subject => 'This is the test_new issue',
1681 1681 :description => 'This is the description',
1682 1682 :priority_id => 5,
1683 1683 :estimated_hours => '',
1684 1684 :custom_field_values => {'2' => 'Value for field 2'}}
1685 1685 end
1686 1686 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1687 1687
1688 1688 issue = Issue.find_by_subject('This is the test_new issue')
1689 1689 assert_not_nil issue
1690 1690 assert_nil issue.start_date
1691 1691 end
1692 1692
1693 1693 def test_post_create_without_start_date_and_default_start_date_is_creation_date
1694 1694 Setting.default_issue_start_date_to_creation_date = 1
1695 1695
1696 1696 @request.session[:user_id] = 2
1697 1697 assert_difference 'Issue.count' do
1698 1698 post :create, :project_id => 1,
1699 1699 :issue => {:tracker_id => 3,
1700 1700 :status_id => 2,
1701 1701 :subject => 'This is the test_new issue',
1702 1702 :description => 'This is the description',
1703 1703 :priority_id => 5,
1704 1704 :estimated_hours => '',
1705 1705 :custom_field_values => {'2' => 'Value for field 2'}}
1706 1706 end
1707 1707 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1708 1708
1709 1709 issue = Issue.find_by_subject('This is the test_new issue')
1710 1710 assert_not_nil issue
1711 1711 assert_equal Date.today, issue.start_date
1712 1712 end
1713 1713
1714 1714 def test_post_create_and_continue
1715 1715 @request.session[:user_id] = 2
1716 1716 assert_difference 'Issue.count' do
1717 1717 post :create, :project_id => 1,
1718 1718 :issue => {:tracker_id => 3, :subject => 'This is first issue', :priority_id => 5},
1719 1719 :continue => ''
1720 1720 end
1721 1721
1722 1722 issue = Issue.first(:order => 'id DESC')
1723 1723 assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook', :issue => {:tracker_id => 3}
1724 1724 assert_not_nil flash[:notice], "flash was not set"
1725 1725 assert_include %|<a href="/issues/#{issue.id}" title="This is first issue">##{issue.id}</a>|, flash[:notice], "issue link not found in the flash message"
1726 1726 end
1727 1727
1728 1728 def test_post_create_without_custom_fields_param
1729 1729 @request.session[:user_id] = 2
1730 1730 assert_difference 'Issue.count' do
1731 1731 post :create, :project_id => 1,
1732 1732 :issue => {:tracker_id => 1,
1733 1733 :subject => 'This is the test_new issue',
1734 1734 :description => 'This is the description',
1735 1735 :priority_id => 5}
1736 1736 end
1737 1737 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1738 1738 end
1739 1739
1740 1740 def test_post_create_with_multi_custom_field
1741 1741 field = IssueCustomField.find_by_name('Database')
1742 1742 field.update_attribute(:multiple, true)
1743 1743
1744 1744 @request.session[:user_id] = 2
1745 1745 assert_difference 'Issue.count' do
1746 1746 post :create, :project_id => 1,
1747 1747 :issue => {:tracker_id => 1,
1748 1748 :subject => 'This is the test_new issue',
1749 1749 :description => 'This is the description',
1750 1750 :priority_id => 5,
1751 1751 :custom_field_values => {'1' => ['', 'MySQL', 'Oracle']}}
1752 1752 end
1753 1753 assert_response 302
1754 1754 issue = Issue.first(:order => 'id DESC')
1755 1755 assert_equal ['MySQL', 'Oracle'], issue.custom_field_value(1).sort
1756 1756 end
1757 1757
1758 1758 def test_post_create_with_empty_multi_custom_field
1759 1759 field = IssueCustomField.find_by_name('Database')
1760 1760 field.update_attribute(:multiple, true)
1761 1761
1762 1762 @request.session[:user_id] = 2
1763 1763 assert_difference 'Issue.count' do
1764 1764 post :create, :project_id => 1,
1765 1765 :issue => {:tracker_id => 1,
1766 1766 :subject => 'This is the test_new issue',
1767 1767 :description => 'This is the description',
1768 1768 :priority_id => 5,
1769 1769 :custom_field_values => {'1' => ['']}}
1770 1770 end
1771 1771 assert_response 302
1772 1772 issue = Issue.first(:order => 'id DESC')
1773 1773 assert_equal [''], issue.custom_field_value(1).sort
1774 1774 end
1775 1775
1776 1776 def test_post_create_with_multi_user_custom_field
1777 1777 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
1778 1778 :tracker_ids => [1], :is_for_all => true)
1779 1779
1780 1780 @request.session[:user_id] = 2
1781 1781 assert_difference 'Issue.count' do
1782 1782 post :create, :project_id => 1,
1783 1783 :issue => {:tracker_id => 1,
1784 1784 :subject => 'This is the test_new issue',
1785 1785 :description => 'This is the description',
1786 1786 :priority_id => 5,
1787 1787 :custom_field_values => {field.id.to_s => ['', '2', '3']}}
1788 1788 end
1789 1789 assert_response 302
1790 1790 issue = Issue.first(:order => 'id DESC')
1791 1791 assert_equal ['2', '3'], issue.custom_field_value(field).sort
1792 1792 end
1793 1793
1794 1794 def test_post_create_with_required_custom_field_and_without_custom_fields_param
1795 1795 field = IssueCustomField.find_by_name('Database')
1796 1796 field.update_attribute(:is_required, true)
1797 1797
1798 1798 @request.session[:user_id] = 2
1799 1799 assert_no_difference 'Issue.count' do
1800 1800 post :create, :project_id => 1,
1801 1801 :issue => {:tracker_id => 1,
1802 1802 :subject => 'This is the test_new issue',
1803 1803 :description => 'This is the description',
1804 1804 :priority_id => 5}
1805 1805 end
1806 1806 assert_response :success
1807 1807 assert_template 'new'
1808 1808 issue = assigns(:issue)
1809 1809 assert_not_nil issue
1810 1810 assert_error_tag :content => /Database can&#x27;t be blank/
1811 1811 end
1812 1812
1813 1813 def test_create_should_validate_required_fields
1814 1814 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1815 1815 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1816 1816 WorkflowPermission.delete_all
1817 1817 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'required')
1818 1818 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required')
1819 1819 @request.session[:user_id] = 2
1820 1820
1821 1821 assert_no_difference 'Issue.count' do
1822 1822 post :create, :project_id => 1, :issue => {
1823 1823 :tracker_id => 2,
1824 1824 :status_id => 1,
1825 1825 :subject => 'Test',
1826 1826 :start_date => '',
1827 1827 :due_date => '',
1828 1828 :custom_field_values => {cf1.id.to_s => '', cf2.id.to_s => ''}
1829 1829 }
1830 1830 assert_response :success
1831 1831 assert_template 'new'
1832 1832 end
1833 1833
1834 1834 assert_error_tag :content => /Due date can&#x27;t be blank/i
1835 1835 assert_error_tag :content => /Bar can&#x27;t be blank/i
1836 1836 end
1837 1837
1838 1838 def test_create_should_ignore_readonly_fields
1839 1839 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1840 1840 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
1841 1841 WorkflowPermission.delete_all
1842 1842 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'readonly')
1843 1843 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly')
1844 1844 @request.session[:user_id] = 2
1845 1845
1846 1846 assert_difference 'Issue.count' do
1847 1847 post :create, :project_id => 1, :issue => {
1848 1848 :tracker_id => 2,
1849 1849 :status_id => 1,
1850 1850 :subject => 'Test',
1851 1851 :start_date => '2012-07-14',
1852 1852 :due_date => '2012-07-16',
1853 1853 :custom_field_values => {cf1.id.to_s => 'value1', cf2.id.to_s => 'value2'}
1854 1854 }
1855 1855 assert_response 302
1856 1856 end
1857 1857
1858 1858 issue = Issue.first(:order => 'id DESC')
1859 1859 assert_equal Date.parse('2012-07-14'), issue.start_date
1860 1860 assert_nil issue.due_date
1861 1861 assert_equal 'value1', issue.custom_field_value(cf1)
1862 1862 assert_nil issue.custom_field_value(cf2)
1863 1863 end
1864 1864
1865 1865 def test_post_create_with_watchers
1866 1866 @request.session[:user_id] = 2
1867 1867 ActionMailer::Base.deliveries.clear
1868 1868
1869 1869 assert_difference 'Watcher.count', 2 do
1870 1870 post :create, :project_id => 1,
1871 1871 :issue => {:tracker_id => 1,
1872 1872 :subject => 'This is a new issue with watchers',
1873 1873 :description => 'This is the description',
1874 1874 :priority_id => 5,
1875 1875 :watcher_user_ids => ['2', '3']}
1876 1876 end
1877 1877 issue = Issue.find_by_subject('This is a new issue with watchers')
1878 1878 assert_not_nil issue
1879 1879 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
1880 1880
1881 1881 # Watchers added
1882 1882 assert_equal [2, 3], issue.watcher_user_ids.sort
1883 1883 assert issue.watched_by?(User.find(3))
1884 1884 # Watchers notified
1885 1885 mail = ActionMailer::Base.deliveries.last
1886 1886 assert_not_nil mail
1887 1887 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
1888 1888 end
1889 1889
1890 1890 def test_post_create_subissue
1891 1891 @request.session[:user_id] = 2
1892 1892
1893 1893 assert_difference 'Issue.count' do
1894 1894 post :create, :project_id => 1,
1895 1895 :issue => {:tracker_id => 1,
1896 1896 :subject => 'This is a child issue',
1897 1897 :parent_issue_id => 2}
1898 1898 end
1899 1899 issue = Issue.find_by_subject('This is a child issue')
1900 1900 assert_not_nil issue
1901 1901 assert_equal Issue.find(2), issue.parent
1902 1902 end
1903 1903
1904 1904 def test_post_create_subissue_with_non_numeric_parent_id
1905 1905 @request.session[:user_id] = 2
1906 1906
1907 1907 assert_difference 'Issue.count' do
1908 1908 post :create, :project_id => 1,
1909 1909 :issue => {:tracker_id => 1,
1910 1910 :subject => 'This is a child issue',
1911 1911 :parent_issue_id => 'ABC'}
1912 1912 end
1913 1913 issue = Issue.find_by_subject('This is a child issue')
1914 1914 assert_not_nil issue
1915 1915 assert_nil issue.parent
1916 1916 end
1917 1917
1918 1918 def test_post_create_private
1919 1919 @request.session[:user_id] = 2
1920 1920
1921 1921 assert_difference 'Issue.count' do
1922 1922 post :create, :project_id => 1,
1923 1923 :issue => {:tracker_id => 1,
1924 1924 :subject => 'This is a private issue',
1925 1925 :is_private => '1'}
1926 1926 end
1927 1927 issue = Issue.first(:order => 'id DESC')
1928 1928 assert issue.is_private?
1929 1929 end
1930 1930
1931 1931 def test_post_create_private_with_set_own_issues_private_permission
1932 1932 role = Role.find(1)
1933 1933 role.remove_permission! :set_issues_private
1934 1934 role.add_permission! :set_own_issues_private
1935 1935
1936 1936 @request.session[:user_id] = 2
1937 1937
1938 1938 assert_difference 'Issue.count' do
1939 1939 post :create, :project_id => 1,
1940 1940 :issue => {:tracker_id => 1,
1941 1941 :subject => 'This is a private issue',
1942 1942 :is_private => '1'}
1943 1943 end
1944 1944 issue = Issue.first(:order => 'id DESC')
1945 1945 assert issue.is_private?
1946 1946 end
1947 1947
1948 1948 def test_post_create_should_send_a_notification
1949 1949 ActionMailer::Base.deliveries.clear
1950 1950 @request.session[:user_id] = 2
1951 1951 assert_difference 'Issue.count' do
1952 1952 post :create, :project_id => 1,
1953 1953 :issue => {:tracker_id => 3,
1954 1954 :subject => 'This is the test_new issue',
1955 1955 :description => 'This is the description',
1956 1956 :priority_id => 5,
1957 1957 :estimated_hours => '',
1958 1958 :custom_field_values => {'2' => 'Value for field 2'}}
1959 1959 end
1960 1960 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
1961 1961
1962 1962 assert_equal 1, ActionMailer::Base.deliveries.size
1963 1963 end
1964 1964
1965 1965 def test_post_create_should_preserve_fields_values_on_validation_failure
1966 1966 @request.session[:user_id] = 2
1967 1967 post :create, :project_id => 1,
1968 1968 :issue => {:tracker_id => 1,
1969 1969 # empty subject
1970 1970 :subject => '',
1971 1971 :description => 'This is a description',
1972 1972 :priority_id => 6,
1973 1973 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
1974 1974 assert_response :success
1975 1975 assert_template 'new'
1976 1976
1977 1977 assert_tag :textarea, :attributes => { :name => 'issue[description]' },
1978 1978 :content => "\nThis is a description"
1979 1979 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
1980 1980 :child => { :tag => 'option', :attributes => { :selected => 'selected',
1981 1981 :value => '6' },
1982 1982 :content => 'High' }
1983 1983 # Custom fields
1984 1984 assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
1985 1985 :child => { :tag => 'option', :attributes => { :selected => 'selected',
1986 1986 :value => 'Oracle' },
1987 1987 :content => 'Oracle' }
1988 1988 assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
1989 1989 :value => 'Value for field 2'}
1990 1990 end
1991 1991
1992 1992 def test_post_create_with_failure_should_preserve_watchers
1993 1993 assert !User.find(8).member_of?(Project.find(1))
1994 1994
1995 1995 @request.session[:user_id] = 2
1996 1996 post :create, :project_id => 1,
1997 1997 :issue => {:tracker_id => 1,
1998 1998 :watcher_user_ids => ['3', '8']}
1999 1999 assert_response :success
2000 2000 assert_template 'new'
2001 2001
2002 2002 assert_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]', :value => '2', :checked => nil}
2003 2003 assert_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]', :value => '3', :checked => 'checked'}
2004 2004 assert_tag 'input', :attributes => {:name => 'issue[watcher_user_ids][]', :value => '8', :checked => 'checked'}
2005 2005 end
2006 2006
2007 2007 def test_post_create_should_ignore_non_safe_attributes
2008 2008 @request.session[:user_id] = 2
2009 2009 assert_nothing_raised do
2010 2010 post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
2011 2011 end
2012 2012 end
2013 2013
2014 2014 def test_post_create_with_attachment
2015 2015 set_tmp_attachments_directory
2016 2016 @request.session[:user_id] = 2
2017 2017
2018 2018 assert_difference 'Issue.count' do
2019 2019 assert_difference 'Attachment.count' do
2020 2020 post :create, :project_id => 1,
2021 2021 :issue => { :tracker_id => '1', :subject => 'With attachment' },
2022 2022 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
2023 2023 end
2024 2024 end
2025 2025
2026 2026 issue = Issue.first(:order => 'id DESC')
2027 2027 attachment = Attachment.first(:order => 'id DESC')
2028 2028
2029 2029 assert_equal issue, attachment.container
2030 2030 assert_equal 2, attachment.author_id
2031 2031 assert_equal 'testfile.txt', attachment.filename
2032 2032 assert_equal 'text/plain', attachment.content_type
2033 2033 assert_equal 'test file', attachment.description
2034 2034 assert_equal 59, attachment.filesize
2035 2035 assert File.exists?(attachment.diskfile)
2036 2036 assert_equal 59, File.size(attachment.diskfile)
2037 2037 end
2038 2038
2039 2039 def test_post_create_with_failure_should_save_attachments
2040 2040 set_tmp_attachments_directory
2041 2041 @request.session[:user_id] = 2
2042 2042
2043 2043 assert_no_difference 'Issue.count' do
2044 2044 assert_difference 'Attachment.count' do
2045 2045 post :create, :project_id => 1,
2046 2046 :issue => { :tracker_id => '1', :subject => '' },
2047 2047 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
2048 2048 assert_response :success
2049 2049 assert_template 'new'
2050 2050 end
2051 2051 end
2052 2052
2053 2053 attachment = Attachment.first(:order => 'id DESC')
2054 2054 assert_equal 'testfile.txt', attachment.filename
2055 2055 assert File.exists?(attachment.diskfile)
2056 2056 assert_nil attachment.container
2057 2057
2058 2058 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
2059 2059 assert_tag 'span', :content => /testfile.txt/
2060 2060 end
2061 2061
2062 2062 def test_post_create_with_failure_should_keep_saved_attachments
2063 2063 set_tmp_attachments_directory
2064 2064 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
2065 2065 @request.session[:user_id] = 2
2066 2066
2067 2067 assert_no_difference 'Issue.count' do
2068 2068 assert_no_difference 'Attachment.count' do
2069 2069 post :create, :project_id => 1,
2070 2070 :issue => { :tracker_id => '1', :subject => '' },
2071 2071 :attachments => {'p0' => {'token' => attachment.token}}
2072 2072 assert_response :success
2073 2073 assert_template 'new'
2074 2074 end
2075 2075 end
2076 2076
2077 2077 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
2078 2078 assert_tag 'span', :content => /testfile.txt/
2079 2079 end
2080 2080
2081 2081 def test_post_create_should_attach_saved_attachments
2082 2082 set_tmp_attachments_directory
2083 2083 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
2084 2084 @request.session[:user_id] = 2
2085 2085
2086 2086 assert_difference 'Issue.count' do
2087 2087 assert_no_difference 'Attachment.count' do
2088 2088 post :create, :project_id => 1,
2089 2089 :issue => { :tracker_id => '1', :subject => 'Saved attachments' },
2090 2090 :attachments => {'p0' => {'token' => attachment.token}}
2091 2091 assert_response 302
2092 2092 end
2093 2093 end
2094 2094
2095 2095 issue = Issue.first(:order => 'id DESC')
2096 2096 assert_equal 1, issue.attachments.count
2097 2097
2098 2098 attachment.reload
2099 2099 assert_equal issue, attachment.container
2100 2100 end
2101 2101
2102 2102 context "without workflow privilege" do
2103 2103 setup do
2104 2104 WorkflowTransition.delete_all(["role_id = ?", Role.anonymous.id])
2105 2105 Role.anonymous.add_permission! :add_issues, :add_issue_notes
2106 2106 end
2107 2107
2108 2108 context "#new" do
2109 2109 should "propose default status only" do
2110 2110 get :new, :project_id => 1
2111 2111 assert_response :success
2112 2112 assert_template 'new'
2113 2113 assert_tag :tag => 'select',
2114 2114 :attributes => {:name => 'issue[status_id]'},
2115 2115 :children => {:count => 1},
2116 2116 :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}}
2117 2117 end
2118 2118
2119 2119 should "accept default status" do
2120 2120 assert_difference 'Issue.count' do
2121 2121 post :create, :project_id => 1,
2122 2122 :issue => {:tracker_id => 1,
2123 2123 :subject => 'This is an issue',
2124 2124 :status_id => 1}
2125 2125 end
2126 2126 issue = Issue.last(:order => 'id')
2127 2127 assert_equal IssueStatus.default, issue.status
2128 2128 end
2129 2129
2130 2130 should "ignore unauthorized status" do
2131 2131 assert_difference 'Issue.count' do
2132 2132 post :create, :project_id => 1,
2133 2133 :issue => {:tracker_id => 1,
2134 2134 :subject => 'This is an issue',
2135 2135 :status_id => 3}
2136 2136 end
2137 2137 issue = Issue.last(:order => 'id')
2138 2138 assert_equal IssueStatus.default, issue.status
2139 2139 end
2140 2140 end
2141 2141
2142 2142 context "#update" do
2143 2143 should "ignore status change" do
2144 2144 assert_difference 'Journal.count' do
2145 2145 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
2146 2146 end
2147 2147 assert_equal 1, Issue.find(1).status_id
2148 2148 end
2149 2149
2150 2150 should "ignore attributes changes" do
2151 2151 assert_difference 'Journal.count' do
2152 2152 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
2153 2153 end
2154 2154 issue = Issue.find(1)
2155 2155 assert_equal "Can't print recipes", issue.subject
2156 2156 assert_nil issue.assigned_to
2157 2157 end
2158 2158 end
2159 2159 end
2160 2160
2161 2161 context "with workflow privilege" do
2162 2162 setup do
2163 2163 WorkflowTransition.delete_all(["role_id = ?", Role.anonymous.id])
2164 2164 WorkflowTransition.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3)
2165 2165 WorkflowTransition.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4)
2166 2166 Role.anonymous.add_permission! :add_issues, :add_issue_notes
2167 2167 end
2168 2168
2169 2169 context "#update" do
2170 2170 should "accept authorized status" do
2171 2171 assert_difference 'Journal.count' do
2172 2172 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
2173 2173 end
2174 2174 assert_equal 3, Issue.find(1).status_id
2175 2175 end
2176 2176
2177 2177 should "ignore unauthorized status" do
2178 2178 assert_difference 'Journal.count' do
2179 2179 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
2180 2180 end
2181 2181 assert_equal 1, Issue.find(1).status_id
2182 2182 end
2183 2183
2184 2184 should "accept authorized attributes changes" do
2185 2185 assert_difference 'Journal.count' do
2186 2186 put :update, :id => 1, :notes => 'just trying', :issue => {:assigned_to_id => 2}
2187 2187 end
2188 2188 issue = Issue.find(1)
2189 2189 assert_equal 2, issue.assigned_to_id
2190 2190 end
2191 2191
2192 2192 should "ignore unauthorized attributes changes" do
2193 2193 assert_difference 'Journal.count' do
2194 2194 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed'}
2195 2195 end
2196 2196 issue = Issue.find(1)
2197 2197 assert_equal "Can't print recipes", issue.subject
2198 2198 end
2199 2199 end
2200 2200
2201 2201 context "and :edit_issues permission" do
2202 2202 setup do
2203 2203 Role.anonymous.add_permission! :add_issues, :edit_issues
2204 2204 end
2205 2205
2206 2206 should "accept authorized status" do
2207 2207 assert_difference 'Journal.count' do
2208 2208 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
2209 2209 end
2210 2210 assert_equal 3, Issue.find(1).status_id
2211 2211 end
2212 2212
2213 2213 should "ignore unauthorized status" do
2214 2214 assert_difference 'Journal.count' do
2215 2215 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
2216 2216 end
2217 2217 assert_equal 1, Issue.find(1).status_id
2218 2218 end
2219 2219
2220 2220 should "accept authorized attributes changes" do
2221 2221 assert_difference 'Journal.count' do
2222 2222 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
2223 2223 end
2224 2224 issue = Issue.find(1)
2225 2225 assert_equal "changed", issue.subject
2226 2226 assert_equal 2, issue.assigned_to_id
2227 2227 end
2228 2228 end
2229 2229 end
2230 2230
2231 2231 def test_new_as_copy
2232 2232 @request.session[:user_id] = 2
2233 2233 get :new, :project_id => 1, :copy_from => 1
2234 2234
2235 2235 assert_response :success
2236 2236 assert_template 'new'
2237 2237
2238 2238 assert_not_nil assigns(:issue)
2239 2239 orig = Issue.find(1)
2240 2240 assert_equal 1, assigns(:issue).project_id
2241 2241 assert_equal orig.subject, assigns(:issue).subject
2242 2242 assert assigns(:issue).copy?
2243 2243
2244 2244 assert_tag 'form', :attributes => {:id => 'issue-form', :action => '/projects/ecookbook/issues'}
2245 2245 assert_tag 'select', :attributes => {:name => 'issue[project_id]'}
2246 2246 assert_tag 'select', :attributes => {:name => 'issue[project_id]'},
2247 2247 :child => {:tag => 'option', :attributes => {:value => '1', :selected => 'selected'}, :content => 'eCookbook'}
2248 2248 assert_tag 'select', :attributes => {:name => 'issue[project_id]'},
2249 2249 :child => {:tag => 'option', :attributes => {:value => '2', :selected => nil}, :content => 'OnlineStore'}
2250 2250 assert_tag 'input', :attributes => {:name => 'copy_from', :value => '1'}
2251 2251 end
2252 2252
2253 2253 def test_new_as_copy_with_attachments_should_show_copy_attachments_checkbox
2254 2254 @request.session[:user_id] = 2
2255 2255 issue = Issue.find(3)
2256 2256 assert issue.attachments.count > 0
2257 2257 get :new, :project_id => 1, :copy_from => 3
2258 2258
2259 2259 assert_tag 'input', :attributes => {:name => 'copy_attachments', :type => 'checkbox', :checked => 'checked', :value => '1'}
2260 2260 end
2261 2261
2262 2262 def test_new_as_copy_without_attachments_should_not_show_copy_attachments_checkbox
2263 2263 @request.session[:user_id] = 2
2264 2264 issue = Issue.find(3)
2265 2265 issue.attachments.delete_all
2266 2266 get :new, :project_id => 1, :copy_from => 3
2267 2267
2268 2268 assert_no_tag 'input', :attributes => {:name => 'copy_attachments', :type => 'checkbox', :checked => 'checked', :value => '1'}
2269 2269 end
2270 2270
2271 2271 def test_new_as_copy_with_subtasks_should_show_copy_subtasks_checkbox
2272 2272 @request.session[:user_id] = 2
2273 2273 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
2274 2274 get :new, :project_id => 1, :copy_from => issue.id
2275 2275
2276 2276 assert_select 'input[type=checkbox][name=copy_subtasks][checked=checked][value=1]'
2277 2277 end
2278 2278
2279 2279 def test_new_as_copy_with_invalid_issue_should_respond_with_404
2280 2280 @request.session[:user_id] = 2
2281 2281 get :new, :project_id => 1, :copy_from => 99999
2282 2282 assert_response 404
2283 2283 end
2284 2284
2285 2285 def test_create_as_copy_on_different_project
2286 2286 @request.session[:user_id] = 2
2287 2287 assert_difference 'Issue.count' do
2288 2288 post :create, :project_id => 1, :copy_from => 1,
2289 2289 :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => 'Copy'}
2290 2290
2291 2291 assert_not_nil assigns(:issue)
2292 2292 assert assigns(:issue).copy?
2293 2293 end
2294 2294 issue = Issue.first(:order => 'id DESC')
2295 2295 assert_redirected_to "/issues/#{issue.id}"
2296 2296
2297 2297 assert_equal 2, issue.project_id
2298 2298 assert_equal 3, issue.tracker_id
2299 2299 assert_equal 'Copy', issue.subject
2300 2300 end
2301 2301
2302 2302 def test_create_as_copy_should_copy_attachments
2303 2303 @request.session[:user_id] = 2
2304 2304 issue = Issue.find(3)
2305 2305 count = issue.attachments.count
2306 2306 assert count > 0
2307 2307
2308 2308 assert_difference 'Issue.count' do
2309 2309 assert_difference 'Attachment.count', count do
2310 2310 assert_no_difference 'Journal.count' do
2311 2311 post :create, :project_id => 1, :copy_from => 3,
2312 2312 :issue => {:project_id => '1', :tracker_id => '3', :status_id => '1', :subject => 'Copy with attachments'},
2313 2313 :copy_attachments => '1'
2314 2314 end
2315 2315 end
2316 2316 end
2317 2317 copy = Issue.first(:order => 'id DESC')
2318 2318 assert_equal count, copy.attachments.count
2319 2319 assert_equal issue.attachments.map(&:filename).sort, copy.attachments.map(&:filename).sort
2320 2320 end
2321 2321
2322 2322 def test_create_as_copy_without_copy_attachments_option_should_not_copy_attachments
2323 2323 @request.session[:user_id] = 2
2324 2324 issue = Issue.find(3)
2325 2325 count = issue.attachments.count
2326 2326 assert count > 0
2327 2327
2328 2328 assert_difference 'Issue.count' do
2329 2329 assert_no_difference 'Attachment.count' do
2330 2330 assert_no_difference 'Journal.count' do
2331 2331 post :create, :project_id => 1, :copy_from => 3,
2332 2332 :issue => {:project_id => '1', :tracker_id => '3', :status_id => '1', :subject => 'Copy with attachments'}
2333 2333 end
2334 2334 end
2335 2335 end
2336 2336 copy = Issue.first(:order => 'id DESC')
2337 2337 assert_equal 0, copy.attachments.count
2338 2338 end
2339 2339
2340 2340 def test_create_as_copy_with_attachments_should_add_new_files
2341 2341 @request.session[:user_id] = 2
2342 2342 issue = Issue.find(3)
2343 2343 count = issue.attachments.count
2344 2344 assert count > 0
2345 2345
2346 2346 assert_difference 'Issue.count' do
2347 2347 assert_difference 'Attachment.count', count + 1 do
2348 2348 assert_no_difference 'Journal.count' do
2349 2349 post :create, :project_id => 1, :copy_from => 3,
2350 2350 :issue => {:project_id => '1', :tracker_id => '3', :status_id => '1', :subject => 'Copy with attachments'},
2351 2351 :copy_attachments => '1',
2352 2352 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
2353 2353 end
2354 2354 end
2355 2355 end
2356 2356 copy = Issue.first(:order => 'id DESC')
2357 2357 assert_equal count + 1, copy.attachments.count
2358 2358 end
2359 2359
2360 2360 def test_create_as_copy_should_copy_subtasks
2361 2361 @request.session[:user_id] = 2
2362 2362 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
2363 2363 count = issue.descendants.count
2364 2364
2365 2365 assert_difference 'Issue.count', count+1 do
2366 2366 assert_no_difference 'Journal.count' do
2367 2367 post :create, :project_id => 1, :copy_from => issue.id,
2368 2368 :issue => {:project_id => '1', :tracker_id => '3', :status_id => '1', :subject => 'Copy with subtasks'},
2369 2369 :copy_subtasks => '1'
2370 2370 end
2371 2371 end
2372 2372 copy = Issue.where(:parent_id => nil).first(:order => 'id DESC')
2373 2373 assert_equal count, copy.descendants.count
2374 2374 assert_equal issue.descendants.map(&:subject).sort, copy.descendants.map(&:subject).sort
2375 2375 end
2376 2376
2377 2377 def test_create_as_copy_without_copy_subtasks_option_should_not_copy_subtasks
2378 2378 @request.session[:user_id] = 2
2379 2379 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
2380 2380
2381 2381 assert_difference 'Issue.count', 1 do
2382 2382 assert_no_difference 'Journal.count' do
2383 2383 post :create, :project_id => 1, :copy_from => 3,
2384 2384 :issue => {:project_id => '1', :tracker_id => '3', :status_id => '1', :subject => 'Copy with subtasks'}
2385 2385 end
2386 2386 end
2387 2387 copy = Issue.where(:parent_id => nil).first(:order => 'id DESC')
2388 2388 assert_equal 0, copy.descendants.count
2389 2389 end
2390 2390
2391 2391 def test_create_as_copy_with_failure
2392 2392 @request.session[:user_id] = 2
2393 2393 post :create, :project_id => 1, :copy_from => 1,
2394 2394 :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => ''}
2395 2395
2396 2396 assert_response :success
2397 2397 assert_template 'new'
2398 2398
2399 2399 assert_not_nil assigns(:issue)
2400 2400 assert assigns(:issue).copy?
2401 2401
2402 2402 assert_tag 'form', :attributes => {:id => 'issue-form', :action => '/projects/ecookbook/issues'}
2403 2403 assert_tag 'select', :attributes => {:name => 'issue[project_id]'}
2404 2404 assert_tag 'select', :attributes => {:name => 'issue[project_id]'},
2405 2405 :child => {:tag => 'option', :attributes => {:value => '1', :selected => nil}, :content => 'eCookbook'}
2406 2406 assert_tag 'select', :attributes => {:name => 'issue[project_id]'},
2407 2407 :child => {:tag => 'option', :attributes => {:value => '2', :selected => 'selected'}, :content => 'OnlineStore'}
2408 2408 assert_tag 'input', :attributes => {:name => 'copy_from', :value => '1'}
2409 2409 end
2410 2410
2411 2411 def test_create_as_copy_on_project_without_permission_should_ignore_target_project
2412 2412 @request.session[:user_id] = 2
2413 2413 assert !User.find(2).member_of?(Project.find(4))
2414 2414
2415 2415 assert_difference 'Issue.count' do
2416 2416 post :create, :project_id => 1, :copy_from => 1,
2417 2417 :issue => {:project_id => '4', :tracker_id => '3', :status_id => '1', :subject => 'Copy'}
2418 2418 end
2419 2419 issue = Issue.first(:order => 'id DESC')
2420 2420 assert_equal 1, issue.project_id
2421 2421 end
2422 2422
2423 2423 def test_get_edit
2424 2424 @request.session[:user_id] = 2
2425 2425 get :edit, :id => 1
2426 2426 assert_response :success
2427 2427 assert_template 'edit'
2428 2428 assert_not_nil assigns(:issue)
2429 2429 assert_equal Issue.find(1), assigns(:issue)
2430 2430
2431 2431 # Be sure we don't display inactive IssuePriorities
2432 2432 assert ! IssuePriority.find(15).active?
2433 2433 assert_no_tag :option, :attributes => {:value => '15'},
2434 2434 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
2435 2435 end
2436 2436
2437 2437 def test_get_edit_should_display_the_time_entry_form_with_log_time_permission
2438 2438 @request.session[:user_id] = 2
2439 2439 Role.find_by_name('Manager').update_attribute :permissions, [:view_issues, :edit_issues, :log_time]
2440 2440
2441 2441 get :edit, :id => 1
2442 2442 assert_tag 'input', :attributes => {:name => 'time_entry[hours]'}
2443 2443 end
2444 2444
2445 2445 def test_get_edit_should_not_display_the_time_entry_form_without_log_time_permission
2446 2446 @request.session[:user_id] = 2
2447 2447 Role.find_by_name('Manager').remove_permission! :log_time
2448 2448
2449 2449 get :edit, :id => 1
2450 2450 assert_no_tag 'input', :attributes => {:name => 'time_entry[hours]'}
2451 2451 end
2452 2452
2453 2453 def test_get_edit_with_params
2454 2454 @request.session[:user_id] = 2
2455 2455 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 },
2456 2456 :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => TimeEntryActivity.first.id }
2457 2457 assert_response :success
2458 2458 assert_template 'edit'
2459 2459
2460 2460 issue = assigns(:issue)
2461 2461 assert_not_nil issue
2462 2462
2463 2463 assert_equal 5, issue.status_id
2464 2464 assert_tag :select, :attributes => { :name => 'issue[status_id]' },
2465 2465 :child => { :tag => 'option',
2466 2466 :content => 'Closed',
2467 2467 :attributes => { :selected => 'selected' } }
2468 2468
2469 2469 assert_equal 7, issue.priority_id
2470 2470 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
2471 2471 :child => { :tag => 'option',
2472 2472 :content => 'Urgent',
2473 2473 :attributes => { :selected => 'selected' } }
2474 2474
2475 2475 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => '2.5' }
2476 2476 assert_tag :select, :attributes => { :name => 'time_entry[activity_id]' },
2477 2477 :child => { :tag => 'option',
2478 2478 :attributes => { :selected => 'selected', :value => TimeEntryActivity.first.id } }
2479 2479 assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => 'test_get_edit_with_params' }
2480 2480 end
2481 2481
2482 2482 def test_get_edit_with_multi_custom_field
2483 2483 field = CustomField.find(1)
2484 2484 field.update_attribute :multiple, true
2485 2485 issue = Issue.find(1)
2486 2486 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
2487 2487 issue.save!
2488 2488
2489 2489 @request.session[:user_id] = 2
2490 2490 get :edit, :id => 1
2491 2491 assert_response :success
2492 2492 assert_template 'edit'
2493 2493
2494 2494 assert_tag 'select', :attributes => {:name => 'issue[custom_field_values][1][]', :multiple => 'multiple'}
2495 2495 assert_tag 'select', :attributes => {:name => 'issue[custom_field_values][1][]'},
2496 2496 :child => {:tag => 'option', :attributes => {:value => 'MySQL', :selected => 'selected'}}
2497 2497 assert_tag 'select', :attributes => {:name => 'issue[custom_field_values][1][]'},
2498 2498 :child => {:tag => 'option', :attributes => {:value => 'PostgreSQL', :selected => nil}}
2499 2499 assert_tag 'select', :attributes => {:name => 'issue[custom_field_values][1][]'},
2500 2500 :child => {:tag => 'option', :attributes => {:value => 'Oracle', :selected => 'selected'}}
2501 2501 end
2502 2502
2503 2503 def test_update_edit_form
2504 2504 @request.session[:user_id] = 2
2505 2505 xhr :put, :new, :project_id => 1,
2506 2506 :id => 1,
2507 2507 :issue => {:tracker_id => 2,
2508 2508 :subject => 'This is the test_new issue',
2509 2509 :description => 'This is the description',
2510 2510 :priority_id => 5}
2511 2511 assert_response :success
2512 2512 assert_equal 'text/javascript', response.content_type
2513 2513 assert_template 'update_form'
2514 2514 assert_template 'form'
2515 2515
2516 2516 issue = assigns(:issue)
2517 2517 assert_kind_of Issue, issue
2518 2518 assert_equal 1, issue.id
2519 2519 assert_equal 1, issue.project_id
2520 2520 assert_equal 2, issue.tracker_id
2521 2521 assert_equal 'This is the test_new issue', issue.subject
2522 2522 end
2523 2523
2524 def test_update_edit_form_should_keep_issue_author
2525 @request.session[:user_id] = 3
2526 xhr :put, :new, :project_id => 1, :id => 1, :issue => {:subject => 'Changed'}
2527 assert_response :success
2528 assert_equal 'text/javascript', response.content_type
2529
2530 issue = assigns(:issue)
2531 assert_equal User.find(2), issue.author
2532 assert_equal 2, issue.author_id
2533 assert_not_equal User.current, issue.author
2534 end
2535
2524 2536 def test_update_edit_form_should_propose_transitions_based_on_initial_status
2525 2537 @request.session[:user_id] = 2
2526 2538 WorkflowTransition.delete_all
2527 2539 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 1)
2528 2540 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 5)
2529 2541 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 5, :new_status_id => 4)
2530 2542
2531 2543 xhr :put, :new, :project_id => 1,
2532 2544 :id => 2,
2533 2545 :issue => {:tracker_id => 2,
2534 2546 :status_id => 5,
2535 2547 :subject => 'This is an issue'}
2536 2548
2537 2549 assert_equal 5, assigns(:issue).status_id
2538 2550 assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort
2539 2551 end
2540 2552
2541 2553 def test_update_edit_form_with_project_change
2542 2554 @request.session[:user_id] = 2
2543 2555 xhr :put, :new, :project_id => 1,
2544 2556 :id => 1,
2545 2557 :issue => {:project_id => 2,
2546 2558 :tracker_id => 2,
2547 2559 :subject => 'This is the test_new issue',
2548 2560 :description => 'This is the description',
2549 2561 :priority_id => 5}
2550 2562 assert_response :success
2551 2563 assert_template 'form'
2552 2564
2553 2565 issue = assigns(:issue)
2554 2566 assert_kind_of Issue, issue
2555 2567 assert_equal 1, issue.id
2556 2568 assert_equal 2, issue.project_id
2557 2569 assert_equal 2, issue.tracker_id
2558 2570 assert_equal 'This is the test_new issue', issue.subject
2559 2571 end
2560 2572
2561 2573 def test_put_update_without_custom_fields_param
2562 2574 @request.session[:user_id] = 2
2563 2575 ActionMailer::Base.deliveries.clear
2564 2576
2565 2577 issue = Issue.find(1)
2566 2578 assert_equal '125', issue.custom_value_for(2).value
2567 2579 old_subject = issue.subject
2568 2580 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
2569 2581
2570 2582 assert_difference('Journal.count') do
2571 2583 assert_difference('JournalDetail.count', 2) do
2572 2584 put :update, :id => 1, :issue => {:subject => new_subject,
2573 2585 :priority_id => '6',
2574 2586 :category_id => '1' # no change
2575 2587 }
2576 2588 end
2577 2589 end
2578 2590 assert_redirected_to :action => 'show', :id => '1'
2579 2591 issue.reload
2580 2592 assert_equal new_subject, issue.subject
2581 2593 # Make sure custom fields were not cleared
2582 2594 assert_equal '125', issue.custom_value_for(2).value
2583 2595
2584 2596 mail = ActionMailer::Base.deliveries.last
2585 2597 assert_not_nil mail
2586 2598 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
2587 2599 assert_mail_body_match "Subject changed from #{old_subject} to #{new_subject}", mail
2588 2600 end
2589 2601
2590 2602 def test_put_update_with_project_change
2591 2603 @request.session[:user_id] = 2
2592 2604 ActionMailer::Base.deliveries.clear
2593 2605
2594 2606 assert_difference('Journal.count') do
2595 2607 assert_difference('JournalDetail.count', 3) do
2596 2608 put :update, :id => 1, :issue => {:project_id => '2',
2597 2609 :tracker_id => '1', # no change
2598 2610 :priority_id => '6',
2599 2611 :category_id => '3'
2600 2612 }
2601 2613 end
2602 2614 end
2603 2615 assert_redirected_to :action => 'show', :id => '1'
2604 2616 issue = Issue.find(1)
2605 2617 assert_equal 2, issue.project_id
2606 2618 assert_equal 1, issue.tracker_id
2607 2619 assert_equal 6, issue.priority_id
2608 2620 assert_equal 3, issue.category_id
2609 2621
2610 2622 mail = ActionMailer::Base.deliveries.last
2611 2623 assert_not_nil mail
2612 2624 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
2613 2625 assert_mail_body_match "Project changed from eCookbook to OnlineStore", mail
2614 2626 end
2615 2627
2616 2628 def test_put_update_with_tracker_change
2617 2629 @request.session[:user_id] = 2
2618 2630 ActionMailer::Base.deliveries.clear
2619 2631
2620 2632 assert_difference('Journal.count') do
2621 2633 assert_difference('JournalDetail.count', 2) do
2622 2634 put :update, :id => 1, :issue => {:project_id => '1',
2623 2635 :tracker_id => '2',
2624 2636 :priority_id => '6'
2625 2637 }
2626 2638 end
2627 2639 end
2628 2640 assert_redirected_to :action => 'show', :id => '1'
2629 2641 issue = Issue.find(1)
2630 2642 assert_equal 1, issue.project_id
2631 2643 assert_equal 2, issue.tracker_id
2632 2644 assert_equal 6, issue.priority_id
2633 2645 assert_equal 1, issue.category_id
2634 2646
2635 2647 mail = ActionMailer::Base.deliveries.last
2636 2648 assert_not_nil mail
2637 2649 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
2638 2650 assert_mail_body_match "Tracker changed from Bug to Feature request", mail
2639 2651 end
2640 2652
2641 2653 def test_put_update_with_custom_field_change
2642 2654 @request.session[:user_id] = 2
2643 2655 issue = Issue.find(1)
2644 2656 assert_equal '125', issue.custom_value_for(2).value
2645 2657
2646 2658 assert_difference('Journal.count') do
2647 2659 assert_difference('JournalDetail.count', 3) do
2648 2660 put :update, :id => 1, :issue => {:subject => 'Custom field change',
2649 2661 :priority_id => '6',
2650 2662 :category_id => '1', # no change
2651 2663 :custom_field_values => { '2' => 'New custom value' }
2652 2664 }
2653 2665 end
2654 2666 end
2655 2667 assert_redirected_to :action => 'show', :id => '1'
2656 2668 issue.reload
2657 2669 assert_equal 'New custom value', issue.custom_value_for(2).value
2658 2670
2659 2671 mail = ActionMailer::Base.deliveries.last
2660 2672 assert_not_nil mail
2661 2673 assert_mail_body_match "Searchable field changed from 125 to New custom value", mail
2662 2674 end
2663 2675
2664 2676 def test_put_update_with_multi_custom_field_change
2665 2677 field = CustomField.find(1)
2666 2678 field.update_attribute :multiple, true
2667 2679 issue = Issue.find(1)
2668 2680 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
2669 2681 issue.save!
2670 2682
2671 2683 @request.session[:user_id] = 2
2672 2684 assert_difference('Journal.count') do
2673 2685 assert_difference('JournalDetail.count', 3) do
2674 2686 put :update, :id => 1,
2675 2687 :issue => {
2676 2688 :subject => 'Custom field change',
2677 2689 :custom_field_values => { '1' => ['', 'Oracle', 'PostgreSQL'] }
2678 2690 }
2679 2691 end
2680 2692 end
2681 2693 assert_redirected_to :action => 'show', :id => '1'
2682 2694 assert_equal ['Oracle', 'PostgreSQL'], Issue.find(1).custom_field_value(1).sort
2683 2695 end
2684 2696
2685 2697 def test_put_update_with_status_and_assignee_change
2686 2698 issue = Issue.find(1)
2687 2699 assert_equal 1, issue.status_id
2688 2700 @request.session[:user_id] = 2
2689 2701 assert_difference('TimeEntry.count', 0) do
2690 2702 put :update,
2691 2703 :id => 1,
2692 2704 :issue => { :status_id => 2, :assigned_to_id => 3 },
2693 2705 :notes => 'Assigned to dlopper',
2694 2706 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
2695 2707 end
2696 2708 assert_redirected_to :action => 'show', :id => '1'
2697 2709 issue.reload
2698 2710 assert_equal 2, issue.status_id
2699 2711 j = Journal.find(:first, :order => 'id DESC')
2700 2712 assert_equal 'Assigned to dlopper', j.notes
2701 2713 assert_equal 2, j.details.size
2702 2714
2703 2715 mail = ActionMailer::Base.deliveries.last
2704 2716 assert_mail_body_match "Status changed from New to Assigned", mail
2705 2717 # subject should contain the new status
2706 2718 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
2707 2719 end
2708 2720
2709 2721 def test_put_update_with_note_only
2710 2722 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
2711 2723 # anonymous user
2712 2724 put :update,
2713 2725 :id => 1,
2714 2726 :notes => notes
2715 2727 assert_redirected_to :action => 'show', :id => '1'
2716 2728 j = Journal.find(:first, :order => 'id DESC')
2717 2729 assert_equal notes, j.notes
2718 2730 assert_equal 0, j.details.size
2719 2731 assert_equal User.anonymous, j.user
2720 2732
2721 2733 mail = ActionMailer::Base.deliveries.last
2722 2734 assert_mail_body_match notes, mail
2723 2735 end
2724 2736
2725 2737 def test_put_update_with_note_and_spent_time
2726 2738 @request.session[:user_id] = 2
2727 2739 spent_hours_before = Issue.find(1).spent_hours
2728 2740 assert_difference('TimeEntry.count') do
2729 2741 put :update,
2730 2742 :id => 1,
2731 2743 :notes => '2.5 hours added',
2732 2744 :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
2733 2745 end
2734 2746 assert_redirected_to :action => 'show', :id => '1'
2735 2747
2736 2748 issue = Issue.find(1)
2737 2749
2738 2750 j = Journal.find(:first, :order => 'id DESC')
2739 2751 assert_equal '2.5 hours added', j.notes
2740 2752 assert_equal 0, j.details.size
2741 2753
2742 2754 t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
2743 2755 assert_not_nil t
2744 2756 assert_equal 2.5, t.hours
2745 2757 assert_equal spent_hours_before + 2.5, issue.spent_hours
2746 2758 end
2747 2759
2748 2760 def test_put_update_with_attachment_only
2749 2761 set_tmp_attachments_directory
2750 2762
2751 2763 # Delete all fixtured journals, a race condition can occur causing the wrong
2752 2764 # journal to get fetched in the next find.
2753 2765 Journal.delete_all
2754 2766
2755 2767 # anonymous user
2756 2768 assert_difference 'Attachment.count' do
2757 2769 put :update, :id => 1,
2758 2770 :notes => '',
2759 2771 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
2760 2772 end
2761 2773
2762 2774 assert_redirected_to :action => 'show', :id => '1'
2763 2775 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
2764 2776 assert j.notes.blank?
2765 2777 assert_equal 1, j.details.size
2766 2778 assert_equal 'testfile.txt', j.details.first.value
2767 2779 assert_equal User.anonymous, j.user
2768 2780
2769 2781 attachment = Attachment.first(:order => 'id DESC')
2770 2782 assert_equal Issue.find(1), attachment.container
2771 2783 assert_equal User.anonymous, attachment.author
2772 2784 assert_equal 'testfile.txt', attachment.filename
2773 2785 assert_equal 'text/plain', attachment.content_type
2774 2786 assert_equal 'test file', attachment.description
2775 2787 assert_equal 59, attachment.filesize
2776 2788 assert File.exists?(attachment.diskfile)
2777 2789 assert_equal 59, File.size(attachment.diskfile)
2778 2790
2779 2791 mail = ActionMailer::Base.deliveries.last
2780 2792 assert_mail_body_match 'testfile.txt', mail
2781 2793 end
2782 2794
2783 2795 def test_put_update_with_failure_should_save_attachments
2784 2796 set_tmp_attachments_directory
2785 2797 @request.session[:user_id] = 2
2786 2798
2787 2799 assert_no_difference 'Journal.count' do
2788 2800 assert_difference 'Attachment.count' do
2789 2801 put :update, :id => 1,
2790 2802 :issue => { :subject => '' },
2791 2803 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
2792 2804 assert_response :success
2793 2805 assert_template 'edit'
2794 2806 end
2795 2807 end
2796 2808
2797 2809 attachment = Attachment.first(:order => 'id DESC')
2798 2810 assert_equal 'testfile.txt', attachment.filename
2799 2811 assert File.exists?(attachment.diskfile)
2800 2812 assert_nil attachment.container
2801 2813
2802 2814 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
2803 2815 assert_tag 'span', :content => /testfile.txt/
2804 2816 end
2805 2817
2806 2818 def test_put_update_with_failure_should_keep_saved_attachments
2807 2819 set_tmp_attachments_directory
2808 2820 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
2809 2821 @request.session[:user_id] = 2
2810 2822
2811 2823 assert_no_difference 'Journal.count' do
2812 2824 assert_no_difference 'Attachment.count' do
2813 2825 put :update, :id => 1,
2814 2826 :issue => { :subject => '' },
2815 2827 :attachments => {'p0' => {'token' => attachment.token}}
2816 2828 assert_response :success
2817 2829 assert_template 'edit'
2818 2830 end
2819 2831 end
2820 2832
2821 2833 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
2822 2834 assert_tag 'span', :content => /testfile.txt/
2823 2835 end
2824 2836
2825 2837 def test_put_update_should_attach_saved_attachments
2826 2838 set_tmp_attachments_directory
2827 2839 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
2828 2840 @request.session[:user_id] = 2
2829 2841
2830 2842 assert_difference 'Journal.count' do
2831 2843 assert_difference 'JournalDetail.count' do
2832 2844 assert_no_difference 'Attachment.count' do
2833 2845 put :update, :id => 1,
2834 2846 :notes => 'Attachment added',
2835 2847 :attachments => {'p0' => {'token' => attachment.token}}
2836 2848 assert_redirected_to '/issues/1'
2837 2849 end
2838 2850 end
2839 2851 end
2840 2852
2841 2853 attachment.reload
2842 2854 assert_equal Issue.find(1), attachment.container
2843 2855
2844 2856 journal = Journal.first(:order => 'id DESC')
2845 2857 assert_equal 1, journal.details.size
2846 2858 assert_equal 'testfile.txt', journal.details.first.value
2847 2859 end
2848 2860
2849 2861 def test_put_update_with_attachment_that_fails_to_save
2850 2862 set_tmp_attachments_directory
2851 2863
2852 2864 # Delete all fixtured journals, a race condition can occur causing the wrong
2853 2865 # journal to get fetched in the next find.
2854 2866 Journal.delete_all
2855 2867
2856 2868 # Mock out the unsaved attachment
2857 2869 Attachment.any_instance.stubs(:create).returns(Attachment.new)
2858 2870
2859 2871 # anonymous user
2860 2872 put :update,
2861 2873 :id => 1,
2862 2874 :notes => '',
2863 2875 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
2864 2876 assert_redirected_to :action => 'show', :id => '1'
2865 2877 assert_equal '1 file(s) could not be saved.', flash[:warning]
2866 2878 end
2867 2879
2868 2880 def test_put_update_with_no_change
2869 2881 issue = Issue.find(1)
2870 2882 issue.journals.clear
2871 2883 ActionMailer::Base.deliveries.clear
2872 2884
2873 2885 put :update,
2874 2886 :id => 1,
2875 2887 :notes => ''
2876 2888 assert_redirected_to :action => 'show', :id => '1'
2877 2889
2878 2890 issue.reload
2879 2891 assert issue.journals.empty?
2880 2892 # No email should be sent
2881 2893 assert ActionMailer::Base.deliveries.empty?
2882 2894 end
2883 2895
2884 2896 def test_put_update_should_send_a_notification
2885 2897 @request.session[:user_id] = 2
2886 2898 ActionMailer::Base.deliveries.clear
2887 2899 issue = Issue.find(1)
2888 2900 old_subject = issue.subject
2889 2901 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
2890 2902
2891 2903 put :update, :id => 1, :issue => {:subject => new_subject,
2892 2904 :priority_id => '6',
2893 2905 :category_id => '1' # no change
2894 2906 }
2895 2907 assert_equal 1, ActionMailer::Base.deliveries.size
2896 2908 end
2897 2909
2898 2910 def test_put_update_with_invalid_spent_time_hours_only
2899 2911 @request.session[:user_id] = 2
2900 2912 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
2901 2913
2902 2914 assert_no_difference('Journal.count') do
2903 2915 put :update,
2904 2916 :id => 1,
2905 2917 :notes => notes,
2906 2918 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
2907 2919 end
2908 2920 assert_response :success
2909 2921 assert_template 'edit'
2910 2922
2911 2923 assert_error_tag :descendant => {:content => /Activity can&#x27;t be blank/}
2912 2924 assert_tag :textarea, :attributes => { :name => 'notes' }, :content => "\n"+notes
2913 2925 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
2914 2926 end
2915 2927
2916 2928 def test_put_update_with_invalid_spent_time_comments_only
2917 2929 @request.session[:user_id] = 2
2918 2930 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
2919 2931
2920 2932 assert_no_difference('Journal.count') do
2921 2933 put :update,
2922 2934 :id => 1,
2923 2935 :notes => notes,
2924 2936 :time_entry => {"comments"=>"this is my comment", "activity_id"=>"", "hours"=>""}
2925 2937 end
2926 2938 assert_response :success
2927 2939 assert_template 'edit'
2928 2940
2929 2941 assert_error_tag :descendant => {:content => /Activity can&#x27;t be blank/}
2930 2942 assert_error_tag :descendant => {:content => /Hours can&#x27;t be blank/}
2931 2943 assert_tag :textarea, :attributes => { :name => 'notes' }, :content => "\n"+notes
2932 2944 assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => "this is my comment" }
2933 2945 end
2934 2946
2935 2947 def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
2936 2948 issue = Issue.find(2)
2937 2949 @request.session[:user_id] = 2
2938 2950
2939 2951 put :update,
2940 2952 :id => issue.id,
2941 2953 :issue => {
2942 2954 :fixed_version_id => 4
2943 2955 }
2944 2956
2945 2957 assert_response :redirect
2946 2958 issue.reload
2947 2959 assert_equal 4, issue.fixed_version_id
2948 2960 assert_not_equal issue.project_id, issue.fixed_version.project_id
2949 2961 end
2950 2962
2951 2963 def test_put_update_should_redirect_back_using_the_back_url_parameter
2952 2964 issue = Issue.find(2)
2953 2965 @request.session[:user_id] = 2
2954 2966
2955 2967 put :update,
2956 2968 :id => issue.id,
2957 2969 :issue => {
2958 2970 :fixed_version_id => 4
2959 2971 },
2960 2972 :back_url => '/issues'
2961 2973
2962 2974 assert_response :redirect
2963 2975 assert_redirected_to '/issues'
2964 2976 end
2965 2977
2966 2978 def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
2967 2979 issue = Issue.find(2)
2968 2980 @request.session[:user_id] = 2
2969 2981
2970 2982 put :update,
2971 2983 :id => issue.id,
2972 2984 :issue => {
2973 2985 :fixed_version_id => 4
2974 2986 },
2975 2987 :back_url => 'http://google.com'
2976 2988
2977 2989 assert_response :redirect
2978 2990 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
2979 2991 end
2980 2992
2981 2993 def test_get_bulk_edit
2982 2994 @request.session[:user_id] = 2
2983 2995 get :bulk_edit, :ids => [1, 2]
2984 2996 assert_response :success
2985 2997 assert_template 'bulk_edit'
2986 2998
2987 2999 assert_tag :select, :attributes => {:name => 'issue[project_id]'}
2988 3000 assert_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
2989 3001
2990 3002 # Project specific custom field, date type
2991 3003 field = CustomField.find(9)
2992 3004 assert !field.is_for_all?
2993 3005 assert_equal 'date', field.field_format
2994 3006 assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
2995 3007
2996 3008 # System wide custom field
2997 3009 assert CustomField.find(1).is_for_all?
2998 3010 assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'}
2999 3011
3000 3012 # Be sure we don't display inactive IssuePriorities
3001 3013 assert ! IssuePriority.find(15).active?
3002 3014 assert_no_tag :option, :attributes => {:value => '15'},
3003 3015 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
3004 3016 end
3005 3017
3006 3018 def test_get_bulk_edit_on_different_projects
3007 3019 @request.session[:user_id] = 2
3008 3020 get :bulk_edit, :ids => [1, 2, 6]
3009 3021 assert_response :success
3010 3022 assert_template 'bulk_edit'
3011 3023
3012 3024 # Can not set issues from different projects as children of an issue
3013 3025 assert_no_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
3014 3026
3015 3027 # Project specific custom field, date type
3016 3028 field = CustomField.find(9)
3017 3029 assert !field.is_for_all?
3018 3030 assert !field.project_ids.include?(Issue.find(6).project_id)
3019 3031 assert_no_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
3020 3032 end
3021 3033
3022 3034 def test_get_bulk_edit_with_user_custom_field
3023 3035 field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true)
3024 3036
3025 3037 @request.session[:user_id] = 2
3026 3038 get :bulk_edit, :ids => [1, 2]
3027 3039 assert_response :success
3028 3040 assert_template 'bulk_edit'
3029 3041
3030 3042 assert_tag :select,
3031 3043 :attributes => {:name => "issue[custom_field_values][#{field.id}]", :class => 'user_cf'},
3032 3044 :children => {
3033 3045 :only => {:tag => 'option'},
3034 3046 :count => Project.find(1).users.count + 2 # "no change" + "none" options
3035 3047 }
3036 3048 end
3037 3049
3038 3050 def test_get_bulk_edit_with_version_custom_field
3039 3051 field = IssueCustomField.create!(:name => 'Affected version', :field_format => 'version', :is_for_all => true)
3040 3052
3041 3053 @request.session[:user_id] = 2
3042 3054 get :bulk_edit, :ids => [1, 2]
3043 3055 assert_response :success
3044 3056 assert_template 'bulk_edit'
3045 3057
3046 3058 assert_tag :select,
3047 3059 :attributes => {:name => "issue[custom_field_values][#{field.id}]"},
3048 3060 :children => {
3049 3061 :only => {:tag => 'option'},
3050 3062 :count => Project.find(1).shared_versions.count + 2 # "no change" + "none" options
3051 3063 }
3052 3064 end
3053 3065
3054 3066 def test_get_bulk_edit_with_multi_custom_field
3055 3067 field = CustomField.find(1)
3056 3068 field.update_attribute :multiple, true
3057 3069
3058 3070 @request.session[:user_id] = 2
3059 3071 get :bulk_edit, :ids => [1, 2]
3060 3072 assert_response :success
3061 3073 assert_template 'bulk_edit'
3062 3074
3063 3075 assert_tag :select,
3064 3076 :attributes => {:name => "issue[custom_field_values][1][]"},
3065 3077 :children => {
3066 3078 :only => {:tag => 'option'},
3067 3079 :count => field.possible_values.size + 1 # "none" options
3068 3080 }
3069 3081 end
3070 3082
3071 3083 def test_bulk_edit_should_only_propose_statuses_allowed_for_all_issues
3072 3084 WorkflowTransition.delete_all
3073 3085 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 1)
3074 3086 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3)
3075 3087 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4)
3076 3088 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 1)
3077 3089 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 3)
3078 3090 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 5)
3079 3091 @request.session[:user_id] = 2
3080 3092 get :bulk_edit, :ids => [1, 2]
3081 3093
3082 3094 assert_response :success
3083 3095 statuses = assigns(:available_statuses)
3084 3096 assert_not_nil statuses
3085 3097 assert_equal [1, 3], statuses.map(&:id).sort
3086 3098
3087 3099 assert_tag 'select', :attributes => {:name => 'issue[status_id]'},
3088 3100 :children => {:count => 3} # 2 statuses + "no change" option
3089 3101 end
3090 3102
3091 3103 def test_bulk_edit_should_propose_target_project_open_shared_versions
3092 3104 @request.session[:user_id] = 2
3093 3105 post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1}
3094 3106 assert_response :success
3095 3107 assert_template 'bulk_edit'
3096 3108 assert_equal Project.find(1).shared_versions.open.all.sort, assigns(:versions).sort
3097 3109 assert_tag 'select',
3098 3110 :attributes => {:name => 'issue[fixed_version_id]'},
3099 3111 :descendant => {:tag => 'option', :content => '2.0'}
3100 3112 end
3101 3113
3102 3114 def test_bulk_edit_should_propose_target_project_categories
3103 3115 @request.session[:user_id] = 2
3104 3116 post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1}
3105 3117 assert_response :success
3106 3118 assert_template 'bulk_edit'
3107 3119 assert_equal Project.find(1).issue_categories.sort, assigns(:categories).sort
3108 3120 assert_tag 'select',
3109 3121 :attributes => {:name => 'issue[category_id]'},
3110 3122 :descendant => {:tag => 'option', :content => 'Recipes'}
3111 3123 end
3112 3124
3113 3125 def test_bulk_update
3114 3126 @request.session[:user_id] = 2
3115 3127 # update issues priority
3116 3128 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
3117 3129 :issue => {:priority_id => 7,
3118 3130 :assigned_to_id => '',
3119 3131 :custom_field_values => {'2' => ''}}
3120 3132
3121 3133 assert_response 302
3122 3134 # check that the issues were updated
3123 3135 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
3124 3136
3125 3137 issue = Issue.find(1)
3126 3138 journal = issue.journals.find(:first, :order => 'created_on DESC')
3127 3139 assert_equal '125', issue.custom_value_for(2).value
3128 3140 assert_equal 'Bulk editing', journal.notes
3129 3141 assert_equal 1, journal.details.size
3130 3142 end
3131 3143
3132 3144 def test_bulk_update_with_group_assignee
3133 3145 group = Group.find(11)
3134 3146 project = Project.find(1)
3135 3147 project.members << Member.new(:principal => group, :roles => [Role.givable.first])
3136 3148
3137 3149 @request.session[:user_id] = 2
3138 3150 # update issues assignee
3139 3151 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
3140 3152 :issue => {:priority_id => '',
3141 3153 :assigned_to_id => group.id,
3142 3154 :custom_field_values => {'2' => ''}}
3143 3155
3144 3156 assert_response 302
3145 3157 assert_equal [group, group], Issue.find_all_by_id([1, 2]).collect {|i| i.assigned_to}
3146 3158 end
3147 3159
3148 3160 def test_bulk_update_on_different_projects
3149 3161 @request.session[:user_id] = 2
3150 3162 # update issues priority
3151 3163 post :bulk_update, :ids => [1, 2, 6], :notes => 'Bulk editing',
3152 3164 :issue => {:priority_id => 7,
3153 3165 :assigned_to_id => '',
3154 3166 :custom_field_values => {'2' => ''}}
3155 3167
3156 3168 assert_response 302
3157 3169 # check that the issues were updated
3158 3170 assert_equal [7, 7, 7], Issue.find([1,2,6]).map(&:priority_id)
3159 3171
3160 3172 issue = Issue.find(1)
3161 3173 journal = issue.journals.find(:first, :order => 'created_on DESC')
3162 3174 assert_equal '125', issue.custom_value_for(2).value
3163 3175 assert_equal 'Bulk editing', journal.notes
3164 3176 assert_equal 1, journal.details.size
3165 3177 end
3166 3178
3167 3179 def test_bulk_update_on_different_projects_without_rights
3168 3180 @request.session[:user_id] = 3
3169 3181 user = User.find(3)
3170 3182 action = { :controller => "issues", :action => "bulk_update" }
3171 3183 assert user.allowed_to?(action, Issue.find(1).project)
3172 3184 assert ! user.allowed_to?(action, Issue.find(6).project)
3173 3185 post :bulk_update, :ids => [1, 6], :notes => 'Bulk should fail',
3174 3186 :issue => {:priority_id => 7,
3175 3187 :assigned_to_id => '',
3176 3188 :custom_field_values => {'2' => ''}}
3177 3189 assert_response 403
3178 3190 assert_not_equal "Bulk should fail", Journal.last.notes
3179 3191 end
3180 3192
3181 3193 def test_bullk_update_should_send_a_notification
3182 3194 @request.session[:user_id] = 2
3183 3195 ActionMailer::Base.deliveries.clear
3184 3196 post(:bulk_update,
3185 3197 {
3186 3198 :ids => [1, 2],
3187 3199 :notes => 'Bulk editing',
3188 3200 :issue => {
3189 3201 :priority_id => 7,
3190 3202 :assigned_to_id => '',
3191 3203 :custom_field_values => {'2' => ''}
3192 3204 }
3193 3205 })
3194 3206
3195 3207 assert_response 302
3196 3208 assert_equal 2, ActionMailer::Base.deliveries.size
3197 3209 end
3198 3210
3199 3211 def test_bulk_update_project
3200 3212 @request.session[:user_id] = 2
3201 3213 post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'}
3202 3214 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
3203 3215 # Issues moved to project 2
3204 3216 assert_equal 2, Issue.find(1).project_id
3205 3217 assert_equal 2, Issue.find(2).project_id
3206 3218 # No tracker change
3207 3219 assert_equal 1, Issue.find(1).tracker_id
3208 3220 assert_equal 2, Issue.find(2).tracker_id
3209 3221 end
3210 3222
3211 3223 def test_bulk_update_project_on_single_issue_should_follow_when_needed
3212 3224 @request.session[:user_id] = 2
3213 3225 post :bulk_update, :id => 1, :issue => {:project_id => '2'}, :follow => '1'
3214 3226 assert_redirected_to '/issues/1'
3215 3227 end
3216 3228
3217 3229 def test_bulk_update_project_on_multiple_issues_should_follow_when_needed
3218 3230 @request.session[:user_id] = 2
3219 3231 post :bulk_update, :id => [1, 2], :issue => {:project_id => '2'}, :follow => '1'
3220 3232 assert_redirected_to '/projects/onlinestore/issues'
3221 3233 end
3222 3234
3223 3235 def test_bulk_update_tracker
3224 3236 @request.session[:user_id] = 2
3225 3237 post :bulk_update, :ids => [1, 2], :issue => {:tracker_id => '2'}
3226 3238 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
3227 3239 assert_equal 2, Issue.find(1).tracker_id
3228 3240 assert_equal 2, Issue.find(2).tracker_id
3229 3241 end
3230 3242
3231 3243 def test_bulk_update_status
3232 3244 @request.session[:user_id] = 2
3233 3245 # update issues priority
3234 3246 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status',
3235 3247 :issue => {:priority_id => '',
3236 3248 :assigned_to_id => '',
3237 3249 :status_id => '5'}
3238 3250
3239 3251 assert_response 302
3240 3252 issue = Issue.find(1)
3241 3253 assert issue.closed?
3242 3254 end
3243 3255
3244 3256 def test_bulk_update_priority
3245 3257 @request.session[:user_id] = 2
3246 3258 post :bulk_update, :ids => [1, 2], :issue => {:priority_id => 6}
3247 3259
3248 3260 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
3249 3261 assert_equal 6, Issue.find(1).priority_id
3250 3262 assert_equal 6, Issue.find(2).priority_id
3251 3263 end
3252 3264
3253 3265 def test_bulk_update_with_notes
3254 3266 @request.session[:user_id] = 2
3255 3267 post :bulk_update, :ids => [1, 2], :notes => 'Moving two issues'
3256 3268
3257 3269 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
3258 3270 assert_equal 'Moving two issues', Issue.find(1).journals.sort_by(&:id).last.notes
3259 3271 assert_equal 'Moving two issues', Issue.find(2).journals.sort_by(&:id).last.notes
3260 3272 end
3261 3273
3262 3274 def test_bulk_update_parent_id
3263 3275 @request.session[:user_id] = 2
3264 3276 post :bulk_update, :ids => [1, 3],
3265 3277 :notes => 'Bulk editing parent',
3266 3278 :issue => {:priority_id => '', :assigned_to_id => '', :status_id => '', :parent_issue_id => '2'}
3267 3279
3268 3280 assert_response 302
3269 3281 parent = Issue.find(2)
3270 3282 assert_equal parent.id, Issue.find(1).parent_id
3271 3283 assert_equal parent.id, Issue.find(3).parent_id
3272 3284 assert_equal [1, 3], parent.children.collect(&:id).sort
3273 3285 end
3274 3286
3275 3287 def test_bulk_update_custom_field
3276 3288 @request.session[:user_id] = 2
3277 3289 # update issues priority
3278 3290 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field',
3279 3291 :issue => {:priority_id => '',
3280 3292 :assigned_to_id => '',
3281 3293 :custom_field_values => {'2' => '777'}}
3282 3294
3283 3295 assert_response 302
3284 3296
3285 3297 issue = Issue.find(1)
3286 3298 journal = issue.journals.find(:first, :order => 'created_on DESC')
3287 3299 assert_equal '777', issue.custom_value_for(2).value
3288 3300 assert_equal 1, journal.details.size
3289 3301 assert_equal '125', journal.details.first.old_value
3290 3302 assert_equal '777', journal.details.first.value
3291 3303 end
3292 3304
3293 3305 def test_bulk_update_custom_field_to_blank
3294 3306 @request.session[:user_id] = 2
3295 3307 post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing custom field',
3296 3308 :issue => {:priority_id => '',
3297 3309 :assigned_to_id => '',
3298 3310 :custom_field_values => {'1' => '__none__'}}
3299 3311 assert_response 302
3300 3312 assert_equal '', Issue.find(1).custom_field_value(1)
3301 3313 assert_equal '', Issue.find(3).custom_field_value(1)
3302 3314 end
3303 3315
3304 3316 def test_bulk_update_multi_custom_field
3305 3317 field = CustomField.find(1)
3306 3318 field.update_attribute :multiple, true
3307 3319
3308 3320 @request.session[:user_id] = 2
3309 3321 post :bulk_update, :ids => [1, 2, 3], :notes => 'Bulk editing multi custom field',
3310 3322 :issue => {:priority_id => '',
3311 3323 :assigned_to_id => '',
3312 3324 :custom_field_values => {'1' => ['MySQL', 'Oracle']}}
3313 3325
3314 3326 assert_response 302
3315 3327
3316 3328 assert_equal ['MySQL', 'Oracle'], Issue.find(1).custom_field_value(1).sort
3317 3329 assert_equal ['MySQL', 'Oracle'], Issue.find(3).custom_field_value(1).sort
3318 3330 # the custom field is not associated with the issue tracker
3319 3331 assert_nil Issue.find(2).custom_field_value(1)
3320 3332 end
3321 3333
3322 3334 def test_bulk_update_multi_custom_field_to_blank
3323 3335 field = CustomField.find(1)
3324 3336 field.update_attribute :multiple, true
3325 3337
3326 3338 @request.session[:user_id] = 2
3327 3339 post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing multi custom field',
3328 3340 :issue => {:priority_id => '',
3329 3341 :assigned_to_id => '',
3330 3342 :custom_field_values => {'1' => ['__none__']}}
3331 3343 assert_response 302
3332 3344 assert_equal [''], Issue.find(1).custom_field_value(1)
3333 3345 assert_equal [''], Issue.find(3).custom_field_value(1)
3334 3346 end
3335 3347
3336 3348 def test_bulk_update_unassign
3337 3349 assert_not_nil Issue.find(2).assigned_to
3338 3350 @request.session[:user_id] = 2
3339 3351 # unassign issues
3340 3352 post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
3341 3353 assert_response 302
3342 3354 # check that the issues were updated
3343 3355 assert_nil Issue.find(2).assigned_to
3344 3356 end
3345 3357
3346 3358 def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject
3347 3359 @request.session[:user_id] = 2
3348 3360
3349 3361 post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4}
3350 3362
3351 3363 assert_response :redirect
3352 3364 issues = Issue.find([1,2])
3353 3365 issues.each do |issue|
3354 3366 assert_equal 4, issue.fixed_version_id
3355 3367 assert_not_equal issue.project_id, issue.fixed_version.project_id
3356 3368 end
3357 3369 end
3358 3370
3359 3371 def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter
3360 3372 @request.session[:user_id] = 2
3361 3373 post :bulk_update, :ids => [1,2], :back_url => '/issues'
3362 3374
3363 3375 assert_response :redirect
3364 3376 assert_redirected_to '/issues'
3365 3377 end
3366 3378
3367 3379 def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
3368 3380 @request.session[:user_id] = 2
3369 3381 post :bulk_update, :ids => [1,2], :back_url => 'http://google.com'
3370 3382
3371 3383 assert_response :redirect
3372 3384 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
3373 3385 end
3374 3386
3375 3387 def test_bulk_update_with_failure_should_set_flash
3376 3388 @request.session[:user_id] = 2
3377 3389 Issue.update_all("subject = ''", "id = 2") # Make it invalid
3378 3390 post :bulk_update, :ids => [1, 2], :issue => {:priority_id => 6}
3379 3391
3380 3392 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
3381 3393 assert_equal 'Failed to save 1 issue(s) on 2 selected: #2.', flash[:error]
3382 3394 end
3383 3395
3384 3396 def test_get_bulk_copy
3385 3397 @request.session[:user_id] = 2
3386 3398 get :bulk_edit, :ids => [1, 2, 3], :copy => '1'
3387 3399 assert_response :success
3388 3400 assert_template 'bulk_edit'
3389 3401
3390 3402 issues = assigns(:issues)
3391 3403 assert_not_nil issues
3392 3404 assert_equal [1, 2, 3], issues.map(&:id).sort
3393 3405
3394 3406 assert_select 'input[name=copy_attachments]'
3395 3407 end
3396 3408
3397 3409 def test_bulk_copy_to_another_project
3398 3410 @request.session[:user_id] = 2
3399 3411 assert_difference 'Issue.count', 2 do
3400 3412 assert_no_difference 'Project.find(1).issues.count' do
3401 3413 post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'}, :copy => '1'
3402 3414 end
3403 3415 end
3404 3416 assert_redirected_to '/projects/ecookbook/issues'
3405 3417
3406 3418 copies = Issue.all(:order => 'id DESC', :limit => issues.size)
3407 3419 copies.each do |copy|
3408 3420 assert_equal 2, copy.project_id
3409 3421 end
3410 3422 end
3411 3423
3412 3424 def test_bulk_copy_should_allow_not_changing_the_issue_attributes
3413 3425 @request.session[:user_id] = 2
3414 3426 issues = [
3415 3427 Issue.create!(:project_id => 1, :tracker_id => 1, :status_id => 1, :priority_id => 2, :subject => 'issue 1', :author_id => 1, :assigned_to_id => nil),
3416 3428 Issue.create!(:project_id => 2, :tracker_id => 3, :status_id => 2, :priority_id => 1, :subject => 'issue 2', :author_id => 2, :assigned_to_id => 3)
3417 3429 ]
3418 3430
3419 3431 assert_difference 'Issue.count', issues.size do
3420 3432 post :bulk_update, :ids => issues.map(&:id), :copy => '1',
3421 3433 :issue => {
3422 3434 :project_id => '', :tracker_id => '', :assigned_to_id => '',
3423 3435 :status_id => '', :start_date => '', :due_date => ''
3424 3436 }
3425 3437 end
3426 3438
3427 3439 copies = Issue.all(:order => 'id DESC', :limit => issues.size)
3428 3440 issues.each do |orig|
3429 3441 copy = copies.detect {|c| c.subject == orig.subject}
3430 3442 assert_not_nil copy
3431 3443 assert_equal orig.project_id, copy.project_id
3432 3444 assert_equal orig.tracker_id, copy.tracker_id
3433 3445 assert_equal orig.status_id, copy.status_id
3434 3446 assert_equal orig.assigned_to_id, copy.assigned_to_id
3435 3447 assert_equal orig.priority_id, copy.priority_id
3436 3448 end
3437 3449 end
3438 3450
3439 3451 def test_bulk_copy_should_allow_changing_the_issue_attributes
3440 3452 # Fixes random test failure with Mysql
3441 3453 # where Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2})
3442 3454 # doesn't return the expected results
3443 3455 Issue.delete_all("project_id=2")
3444 3456
3445 3457 @request.session[:user_id] = 2
3446 3458 assert_difference 'Issue.count', 2 do
3447 3459 assert_no_difference 'Project.find(1).issues.count' do
3448 3460 post :bulk_update, :ids => [1, 2], :copy => '1',
3449 3461 :issue => {
3450 3462 :project_id => '2', :tracker_id => '', :assigned_to_id => '4',
3451 3463 :status_id => '1', :start_date => '2009-12-01', :due_date => '2009-12-31'
3452 3464 }
3453 3465 end
3454 3466 end
3455 3467
3456 3468 copied_issues = Issue.all(:limit => 2, :order => 'id desc', :conditions => {:project_id => 2})
3457 3469 assert_equal 2, copied_issues.size
3458 3470 copied_issues.each do |issue|
3459 3471 assert_equal 2, issue.project_id, "Project is incorrect"
3460 3472 assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect"
3461 3473 assert_equal 1, issue.status_id, "Status is incorrect"
3462 3474 assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect"
3463 3475 assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect"
3464 3476 end
3465 3477 end
3466 3478
3467 3479 def test_bulk_copy_should_allow_adding_a_note
3468 3480 @request.session[:user_id] = 2
3469 3481 assert_difference 'Issue.count', 1 do
3470 3482 post :bulk_update, :ids => [1], :copy => '1',
3471 3483 :notes => 'Copying one issue',
3472 3484 :issue => {
3473 3485 :project_id => '', :tracker_id => '', :assigned_to_id => '4',
3474 3486 :status_id => '3', :start_date => '2009-12-01', :due_date => '2009-12-31'
3475 3487 }
3476 3488 end
3477 3489
3478 3490 issue = Issue.first(:order => 'id DESC')
3479 3491 assert_equal 1, issue.journals.size
3480 3492 journal = issue.journals.first
3481 3493 assert_equal 0, journal.details.size
3482 3494 assert_equal 'Copying one issue', journal.notes
3483 3495 end
3484 3496
3485 3497 def test_bulk_copy_should_allow_not_copying_the_attachments
3486 3498 attachment_count = Issue.find(3).attachments.size
3487 3499 assert attachment_count > 0
3488 3500 @request.session[:user_id] = 2
3489 3501
3490 3502 assert_difference 'Issue.count', 1 do
3491 3503 assert_no_difference 'Attachment.count' do
3492 3504 post :bulk_update, :ids => [3], :copy => '1',
3493 3505 :issue => {
3494 3506 :project_id => ''
3495 3507 }
3496 3508 end
3497 3509 end
3498 3510 end
3499 3511
3500 3512 def test_bulk_copy_should_allow_copying_the_attachments
3501 3513 attachment_count = Issue.find(3).attachments.size
3502 3514 assert attachment_count > 0
3503 3515 @request.session[:user_id] = 2
3504 3516
3505 3517 assert_difference 'Issue.count', 1 do
3506 3518 assert_difference 'Attachment.count', attachment_count do
3507 3519 post :bulk_update, :ids => [3], :copy => '1', :copy_attachments => '1',
3508 3520 :issue => {
3509 3521 :project_id => ''
3510 3522 }
3511 3523 end
3512 3524 end
3513 3525 end
3514 3526
3515 3527 def test_bulk_copy_should_allow_not_copying_the_subtasks
3516 3528 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
3517 3529 @request.session[:user_id] = 2
3518 3530
3519 3531 assert_difference 'Issue.count', 1 do
3520 3532 post :bulk_update, :ids => [issue.id], :copy => '1',
3521 3533 :issue => {
3522 3534 :project_id => ''
3523 3535 }
3524 3536 end
3525 3537 end
3526 3538
3527 3539 def test_bulk_copy_should_allow_copying_the_subtasks
3528 3540 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
3529 3541 count = issue.descendants.count
3530 3542 @request.session[:user_id] = 2
3531 3543
3532 3544 assert_difference 'Issue.count', count+1 do
3533 3545 post :bulk_update, :ids => [issue.id], :copy => '1', :copy_subtasks => '1',
3534 3546 :issue => {
3535 3547 :project_id => ''
3536 3548 }
3537 3549 end
3538 3550 copy = Issue.where(:parent_id => nil).order("id DESC").first
3539 3551 assert_equal count, copy.descendants.count
3540 3552 end
3541 3553
3542 3554 def test_bulk_copy_should_not_copy_selected_subtasks_twice
3543 3555 issue = Issue.generate_with_descendants!(Project.find(1), :subject => 'Parent')
3544 3556 count = issue.descendants.count
3545 3557 @request.session[:user_id] = 2
3546 3558
3547 3559 assert_difference 'Issue.count', count+1 do
3548 3560 post :bulk_update, :ids => issue.self_and_descendants.map(&:id), :copy => '1', :copy_subtasks => '1',
3549 3561 :issue => {
3550 3562 :project_id => ''
3551 3563 }
3552 3564 end
3553 3565 copy = Issue.where(:parent_id => nil).order("id DESC").first
3554 3566 assert_equal count, copy.descendants.count
3555 3567 end
3556 3568
3557 3569 def test_bulk_copy_to_another_project_should_follow_when_needed
3558 3570 @request.session[:user_id] = 2
3559 3571 post :bulk_update, :ids => [1], :copy => '1', :issue => {:project_id => 2}, :follow => '1'
3560 3572 issue = Issue.first(:order => 'id DESC')
3561 3573 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
3562 3574 end
3563 3575
3564 3576 def test_destroy_issue_with_no_time_entries
3565 3577 assert_nil TimeEntry.find_by_issue_id(2)
3566 3578 @request.session[:user_id] = 2
3567 3579
3568 3580 assert_difference 'Issue.count', -1 do
3569 3581 delete :destroy, :id => 2
3570 3582 end
3571 3583 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
3572 3584 assert_nil Issue.find_by_id(2)
3573 3585 end
3574 3586
3575 3587 def test_destroy_issues_with_time_entries
3576 3588 @request.session[:user_id] = 2
3577 3589
3578 3590 assert_no_difference 'Issue.count' do
3579 3591 delete :destroy, :ids => [1, 3]
3580 3592 end
3581 3593 assert_response :success
3582 3594 assert_template 'destroy'
3583 3595 assert_not_nil assigns(:hours)
3584 3596 assert Issue.find_by_id(1) && Issue.find_by_id(3)
3585 3597 assert_tag 'form',
3586 3598 :descendant => {:tag => 'input', :attributes => {:name => '_method', :value => 'delete'}}
3587 3599 end
3588 3600
3589 3601 def test_destroy_issues_and_destroy_time_entries
3590 3602 @request.session[:user_id] = 2
3591 3603
3592 3604 assert_difference 'Issue.count', -2 do
3593 3605 assert_difference 'TimeEntry.count', -3 do
3594 3606 delete :destroy, :ids => [1, 3], :todo => 'destroy'
3595 3607 end
3596 3608 end
3597 3609 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
3598 3610 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
3599 3611 assert_nil TimeEntry.find_by_id([1, 2])
3600 3612 end
3601 3613
3602 3614 def test_destroy_issues_and_assign_time_entries_to_project
3603 3615 @request.session[:user_id] = 2
3604 3616
3605 3617 assert_difference 'Issue.count', -2 do
3606 3618 assert_no_difference 'TimeEntry.count' do
3607 3619 delete :destroy, :ids => [1, 3], :todo => 'nullify'
3608 3620 end
3609 3621 end
3610 3622 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
3611 3623 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
3612 3624 assert_nil TimeEntry.find(1).issue_id
3613 3625 assert_nil TimeEntry.find(2).issue_id
3614 3626 end
3615 3627
3616 3628 def test_destroy_issues_and_reassign_time_entries_to_another_issue
3617 3629 @request.session[:user_id] = 2
3618 3630
3619 3631 assert_difference 'Issue.count', -2 do
3620 3632 assert_no_difference 'TimeEntry.count' do
3621 3633 delete :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
3622 3634 end
3623 3635 end
3624 3636 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
3625 3637 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
3626 3638 assert_equal 2, TimeEntry.find(1).issue_id
3627 3639 assert_equal 2, TimeEntry.find(2).issue_id
3628 3640 end
3629 3641
3630 3642 def test_destroy_issues_from_different_projects
3631 3643 @request.session[:user_id] = 2
3632 3644
3633 3645 assert_difference 'Issue.count', -3 do
3634 3646 delete :destroy, :ids => [1, 2, 6], :todo => 'destroy'
3635 3647 end
3636 3648 assert_redirected_to :controller => 'issues', :action => 'index'
3637 3649 assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6))
3638 3650 end
3639 3651
3640 3652 def test_destroy_parent_and_child_issues
3641 3653 parent = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Parent Issue')
3642 3654 child = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Child Issue', :parent_issue_id => parent.id)
3643 3655 assert child.is_descendant_of?(parent.reload)
3644 3656
3645 3657 @request.session[:user_id] = 2
3646 3658 assert_difference 'Issue.count', -2 do
3647 3659 delete :destroy, :ids => [parent.id, child.id], :todo => 'destroy'
3648 3660 end
3649 3661 assert_response 302
3650 3662 end
3651 3663
3652 3664 def test_default_search_scope
3653 3665 get :index
3654 3666 assert_tag :div, :attributes => {:id => 'quick-search'},
3655 3667 :child => {:tag => 'form',
3656 3668 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}
3657 3669 end
3658 3670 end
General Comments 0
You need to be logged in to leave comments. Login now