Chris@0
|
1 # Redmine - project management software
|
Chris@0
|
2 # Copyright (C) 2006-2008 Jean-Philippe Lang
|
Chris@0
|
3 #
|
Chris@0
|
4 # This program is free software; you can redistribute it and/or
|
Chris@0
|
5 # modify it under the terms of the GNU General Public License
|
Chris@0
|
6 # as published by the Free Software Foundation; either version 2
|
Chris@0
|
7 # of the License, or (at your option) any later version.
|
Chris@0
|
8 #
|
Chris@0
|
9 # This program is distributed in the hope that it will be useful,
|
Chris@0
|
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
|
Chris@0
|
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
Chris@0
|
12 # GNU General Public License for more details.
|
Chris@0
|
13 #
|
Chris@0
|
14 # You should have received a copy of the GNU General Public License
|
Chris@0
|
15 # along with this program; if not, write to the Free Software
|
Chris@0
|
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
Chris@0
|
17
|
Chris@0
|
18 require File.dirname(__FILE__) + '/../test_helper'
|
Chris@0
|
19 require 'issues_controller'
|
Chris@0
|
20
|
Chris@0
|
21 # Re-raise errors caught by the controller.
|
Chris@0
|
22 class IssuesController; def rescue_action(e) raise e end; end
|
Chris@0
|
23
|
Chris@0
|
24 class IssuesControllerTest < ActionController::TestCase
|
Chris@0
|
25 fixtures :projects,
|
Chris@0
|
26 :users,
|
Chris@0
|
27 :roles,
|
Chris@0
|
28 :members,
|
Chris@0
|
29 :member_roles,
|
Chris@0
|
30 :issues,
|
Chris@0
|
31 :issue_statuses,
|
Chris@0
|
32 :versions,
|
Chris@0
|
33 :trackers,
|
Chris@0
|
34 :projects_trackers,
|
Chris@0
|
35 :issue_categories,
|
Chris@0
|
36 :enabled_modules,
|
Chris@0
|
37 :enumerations,
|
Chris@0
|
38 :attachments,
|
Chris@0
|
39 :workflows,
|
Chris@0
|
40 :custom_fields,
|
Chris@0
|
41 :custom_values,
|
Chris@0
|
42 :custom_fields_projects,
|
Chris@0
|
43 :custom_fields_trackers,
|
Chris@0
|
44 :time_entries,
|
Chris@0
|
45 :journals,
|
Chris@0
|
46 :journal_details,
|
Chris@0
|
47 :queries
|
Chris@0
|
48
|
Chris@0
|
49 def setup
|
Chris@0
|
50 @controller = IssuesController.new
|
Chris@0
|
51 @request = ActionController::TestRequest.new
|
Chris@0
|
52 @response = ActionController::TestResponse.new
|
Chris@0
|
53 User.current = nil
|
Chris@0
|
54 end
|
Chris@0
|
55
|
Chris@0
|
56 def test_index
|
Chris@0
|
57 Setting.default_language = 'en'
|
Chris@0
|
58
|
Chris@0
|
59 get :index
|
Chris@0
|
60 assert_response :success
|
Chris@0
|
61 assert_template 'index.rhtml'
|
Chris@0
|
62 assert_not_nil assigns(:issues)
|
Chris@0
|
63 assert_nil assigns(:project)
|
Chris@0
|
64 assert_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
65 assert_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
66 # private projects hidden
|
Chris@0
|
67 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
|
Chris@0
|
68 assert_no_tag :tag => 'a', :content => /Issue on project 2/
|
Chris@0
|
69 # project column
|
Chris@0
|
70 assert_tag :tag => 'th', :content => /Project/
|
Chris@0
|
71 end
|
Chris@0
|
72
|
Chris@0
|
73 def test_index_should_not_list_issues_when_module_disabled
|
Chris@0
|
74 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
|
Chris@0
|
75 get :index
|
Chris@0
|
76 assert_response :success
|
Chris@0
|
77 assert_template 'index.rhtml'
|
Chris@0
|
78 assert_not_nil assigns(:issues)
|
Chris@0
|
79 assert_nil assigns(:project)
|
Chris@0
|
80 assert_no_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
81 assert_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
82 end
|
Chris@0
|
83
|
Chris@0
|
84 def test_index_should_not_list_issues_when_module_disabled
|
Chris@0
|
85 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
|
Chris@0
|
86 get :index
|
Chris@0
|
87 assert_response :success
|
Chris@0
|
88 assert_template 'index.rhtml'
|
Chris@0
|
89 assert_not_nil assigns(:issues)
|
Chris@0
|
90 assert_nil assigns(:project)
|
Chris@0
|
91 assert_no_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
92 assert_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
93 end
|
Chris@0
|
94
|
Chris@0
|
95 def test_index_with_project
|
Chris@0
|
96 Setting.display_subprojects_issues = 0
|
Chris@0
|
97 get :index, :project_id => 1
|
Chris@0
|
98 assert_response :success
|
Chris@0
|
99 assert_template 'index.rhtml'
|
Chris@0
|
100 assert_not_nil assigns(:issues)
|
Chris@0
|
101 assert_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
102 assert_no_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
103 end
|
Chris@0
|
104
|
Chris@0
|
105 def test_index_with_project_and_subprojects
|
Chris@0
|
106 Setting.display_subprojects_issues = 1
|
Chris@0
|
107 get :index, :project_id => 1
|
Chris@0
|
108 assert_response :success
|
Chris@0
|
109 assert_template 'index.rhtml'
|
Chris@0
|
110 assert_not_nil assigns(:issues)
|
Chris@0
|
111 assert_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
112 assert_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
113 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
|
Chris@0
|
114 end
|
Chris@0
|
115
|
Chris@0
|
116 def test_index_with_project_and_subprojects_should_show_private_subprojects
|
Chris@0
|
117 @request.session[:user_id] = 2
|
Chris@0
|
118 Setting.display_subprojects_issues = 1
|
Chris@0
|
119 get :index, :project_id => 1
|
Chris@0
|
120 assert_response :success
|
Chris@0
|
121 assert_template 'index.rhtml'
|
Chris@0
|
122 assert_not_nil assigns(:issues)
|
Chris@0
|
123 assert_tag :tag => 'a', :content => /Can't print recipes/
|
Chris@0
|
124 assert_tag :tag => 'a', :content => /Subproject issue/
|
Chris@0
|
125 assert_tag :tag => 'a', :content => /Issue of a private subproject/
|
Chris@0
|
126 end
|
Chris@0
|
127
|
Chris@0
|
128 def test_index_with_project_and_filter
|
Chris@0
|
129 get :index, :project_id => 1, :set_filter => 1
|
Chris@0
|
130 assert_response :success
|
Chris@0
|
131 assert_template 'index.rhtml'
|
Chris@0
|
132 assert_not_nil assigns(:issues)
|
Chris@0
|
133 end
|
Chris@0
|
134
|
Chris@0
|
135 def test_index_with_query
|
Chris@0
|
136 get :index, :project_id => 1, :query_id => 5
|
Chris@0
|
137 assert_response :success
|
Chris@0
|
138 assert_template 'index.rhtml'
|
Chris@0
|
139 assert_not_nil assigns(:issues)
|
Chris@0
|
140 assert_nil assigns(:issue_count_by_group)
|
Chris@0
|
141 end
|
Chris@0
|
142
|
Chris@0
|
143 def test_index_with_query_grouped_by_tracker
|
Chris@0
|
144 get :index, :project_id => 1, :query_id => 6
|
Chris@0
|
145 assert_response :success
|
Chris@0
|
146 assert_template 'index.rhtml'
|
Chris@0
|
147 assert_not_nil assigns(:issues)
|
Chris@0
|
148 assert_not_nil assigns(:issue_count_by_group)
|
Chris@0
|
149 end
|
Chris@0
|
150
|
Chris@0
|
151 def test_index_with_query_grouped_by_list_custom_field
|
Chris@0
|
152 get :index, :project_id => 1, :query_id => 9
|
Chris@0
|
153 assert_response :success
|
Chris@0
|
154 assert_template 'index.rhtml'
|
Chris@0
|
155 assert_not_nil assigns(:issues)
|
Chris@0
|
156 assert_not_nil assigns(:issue_count_by_group)
|
Chris@0
|
157 end
|
Chris@0
|
158
|
Chris@0
|
159 def test_index_sort_by_field_not_included_in_columns
|
Chris@0
|
160 Setting.issue_list_default_columns = %w(subject author)
|
Chris@0
|
161 get :index, :sort => 'tracker'
|
Chris@0
|
162 end
|
Chris@0
|
163
|
Chris@0
|
164 def test_index_csv_with_project
|
Chris@0
|
165 Setting.default_language = 'en'
|
Chris@0
|
166
|
Chris@0
|
167 get :index, :format => 'csv'
|
Chris@0
|
168 assert_response :success
|
Chris@0
|
169 assert_not_nil assigns(:issues)
|
Chris@0
|
170 assert_equal 'text/csv', @response.content_type
|
Chris@0
|
171 assert @response.body.starts_with?("#,")
|
Chris@0
|
172
|
Chris@0
|
173 get :index, :project_id => 1, :format => 'csv'
|
Chris@0
|
174 assert_response :success
|
Chris@0
|
175 assert_not_nil assigns(:issues)
|
Chris@0
|
176 assert_equal 'text/csv', @response.content_type
|
Chris@0
|
177 end
|
Chris@0
|
178
|
Chris@0
|
179 def test_index_pdf
|
Chris@0
|
180 get :index, :format => 'pdf'
|
Chris@0
|
181 assert_response :success
|
Chris@0
|
182 assert_not_nil assigns(:issues)
|
Chris@0
|
183 assert_equal 'application/pdf', @response.content_type
|
Chris@0
|
184
|
Chris@0
|
185 get :index, :project_id => 1, :format => 'pdf'
|
Chris@0
|
186 assert_response :success
|
Chris@0
|
187 assert_not_nil assigns(:issues)
|
Chris@0
|
188 assert_equal 'application/pdf', @response.content_type
|
Chris@0
|
189
|
Chris@0
|
190 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
|
Chris@0
|
191 assert_response :success
|
Chris@0
|
192 assert_not_nil assigns(:issues)
|
Chris@0
|
193 assert_equal 'application/pdf', @response.content_type
|
Chris@0
|
194 end
|
Chris@0
|
195
|
Chris@0
|
196 def test_index_pdf_with_query_grouped_by_list_custom_field
|
Chris@0
|
197 get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
|
Chris@0
|
198 assert_response :success
|
Chris@0
|
199 assert_not_nil assigns(:issues)
|
Chris@0
|
200 assert_not_nil assigns(:issue_count_by_group)
|
Chris@0
|
201 assert_equal 'application/pdf', @response.content_type
|
Chris@0
|
202 end
|
Chris@0
|
203
|
Chris@0
|
204 def test_index_sort
|
Chris@0
|
205 get :index, :sort => 'tracker,id:desc'
|
Chris@0
|
206 assert_response :success
|
Chris@0
|
207
|
Chris@0
|
208 sort_params = @request.session['issues_index_sort']
|
Chris@0
|
209 assert sort_params.is_a?(String)
|
Chris@0
|
210 assert_equal 'tracker,id:desc', sort_params
|
Chris@0
|
211
|
Chris@0
|
212 issues = assigns(:issues)
|
Chris@0
|
213 assert_not_nil issues
|
Chris@0
|
214 assert !issues.empty?
|
Chris@0
|
215 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
|
Chris@0
|
216 end
|
Chris@0
|
217
|
Chris@0
|
218 def test_index_with_columns
|
Chris@0
|
219 columns = ['tracker', 'subject', 'assigned_to']
|
Chris@0
|
220 get :index, :set_filter => 1, :query => { 'column_names' => columns}
|
Chris@0
|
221 assert_response :success
|
Chris@0
|
222
|
Chris@0
|
223 # query should use specified columns
|
Chris@0
|
224 query = assigns(:query)
|
Chris@0
|
225 assert_kind_of Query, query
|
Chris@0
|
226 assert_equal columns, query.column_names.map(&:to_s)
|
Chris@0
|
227
|
Chris@0
|
228 # columns should be stored in session
|
Chris@0
|
229 assert_kind_of Hash, session[:query]
|
Chris@0
|
230 assert_kind_of Array, session[:query][:column_names]
|
Chris@0
|
231 assert_equal columns, session[:query][:column_names].map(&:to_s)
|
Chris@0
|
232 end
|
Chris@0
|
233
|
Chris@0
|
234 def test_show_by_anonymous
|
Chris@0
|
235 get :show, :id => 1
|
Chris@0
|
236 assert_response :success
|
Chris@0
|
237 assert_template 'show.rhtml'
|
Chris@0
|
238 assert_not_nil assigns(:issue)
|
Chris@0
|
239 assert_equal Issue.find(1), assigns(:issue)
|
Chris@0
|
240
|
Chris@0
|
241 # anonymous role is allowed to add a note
|
Chris@0
|
242 assert_tag :tag => 'form',
|
Chris@0
|
243 :descendant => { :tag => 'fieldset',
|
Chris@0
|
244 :child => { :tag => 'legend',
|
Chris@0
|
245 :content => /Notes/ } }
|
Chris@0
|
246 end
|
Chris@0
|
247
|
Chris@0
|
248 def test_show_by_manager
|
Chris@0
|
249 @request.session[:user_id] = 2
|
Chris@0
|
250 get :show, :id => 1
|
Chris@0
|
251 assert_response :success
|
Chris@0
|
252
|
Chris@0
|
253 assert_tag :tag => 'form',
|
Chris@0
|
254 :descendant => { :tag => 'fieldset',
|
Chris@0
|
255 :child => { :tag => 'legend',
|
Chris@0
|
256 :content => /Change properties/ } },
|
Chris@0
|
257 :descendant => { :tag => 'fieldset',
|
Chris@0
|
258 :child => { :tag => 'legend',
|
Chris@0
|
259 :content => /Log time/ } },
|
Chris@0
|
260 :descendant => { :tag => 'fieldset',
|
Chris@0
|
261 :child => { :tag => 'legend',
|
Chris@0
|
262 :content => /Notes/ } }
|
Chris@0
|
263 end
|
Chris@0
|
264
|
Chris@0
|
265 def test_show_should_deny_anonymous_access_without_permission
|
Chris@0
|
266 Role.anonymous.remove_permission!(:view_issues)
|
Chris@0
|
267 get :show, :id => 1
|
Chris@0
|
268 assert_response :redirect
|
Chris@0
|
269 end
|
Chris@0
|
270
|
Chris@0
|
271 def test_show_should_deny_non_member_access_without_permission
|
Chris@0
|
272 Role.non_member.remove_permission!(:view_issues)
|
Chris@0
|
273 @request.session[:user_id] = 9
|
Chris@0
|
274 get :show, :id => 1
|
Chris@0
|
275 assert_response 403
|
Chris@0
|
276 end
|
Chris@0
|
277
|
Chris@0
|
278 def test_show_should_deny_member_access_without_permission
|
Chris@0
|
279 Role.find(1).remove_permission!(:view_issues)
|
Chris@0
|
280 @request.session[:user_id] = 2
|
Chris@0
|
281 get :show, :id => 1
|
Chris@0
|
282 assert_response 403
|
Chris@0
|
283 end
|
Chris@0
|
284
|
Chris@0
|
285 def test_show_should_not_disclose_relations_to_invisible_issues
|
Chris@0
|
286 Setting.cross_project_issue_relations = '1'
|
Chris@0
|
287 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
|
Chris@0
|
288 # Relation to a private project issue
|
Chris@0
|
289 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
|
Chris@0
|
290
|
Chris@0
|
291 get :show, :id => 1
|
Chris@0
|
292 assert_response :success
|
Chris@0
|
293
|
Chris@0
|
294 assert_tag :div, :attributes => { :id => 'relations' },
|
Chris@0
|
295 :descendant => { :tag => 'a', :content => /#2$/ }
|
Chris@0
|
296 assert_no_tag :div, :attributes => { :id => 'relations' },
|
Chris@0
|
297 :descendant => { :tag => 'a', :content => /#4$/ }
|
Chris@0
|
298 end
|
Chris@0
|
299
|
Chris@0
|
300 def test_show_atom
|
Chris@0
|
301 get :show, :id => 2, :format => 'atom'
|
Chris@0
|
302 assert_response :success
|
Chris@14
|
303 assert_template 'journals/index.rxml'
|
Chris@0
|
304 # Inline image
|
Chris@0
|
305 assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
|
Chris@0
|
306 end
|
Chris@0
|
307
|
Chris@0
|
308 def test_show_export_to_pdf
|
Chris@0
|
309 get :show, :id => 3, :format => 'pdf'
|
Chris@0
|
310 assert_response :success
|
Chris@0
|
311 assert_equal 'application/pdf', @response.content_type
|
Chris@0
|
312 assert @response.body.starts_with?('%PDF')
|
Chris@0
|
313 assert_not_nil assigns(:issue)
|
Chris@0
|
314 end
|
Chris@0
|
315
|
Chris@0
|
316 def test_get_new
|
Chris@0
|
317 @request.session[:user_id] = 2
|
Chris@0
|
318 get :new, :project_id => 1, :tracker_id => 1
|
Chris@0
|
319 assert_response :success
|
Chris@0
|
320 assert_template 'new'
|
Chris@0
|
321
|
Chris@0
|
322 assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]',
|
Chris@0
|
323 :value => 'Default string' }
|
Chris@0
|
324 end
|
Chris@0
|
325
|
Chris@0
|
326 def test_get_new_without_tracker_id
|
Chris@0
|
327 @request.session[:user_id] = 2
|
Chris@0
|
328 get :new, :project_id => 1
|
Chris@0
|
329 assert_response :success
|
Chris@0
|
330 assert_template 'new'
|
Chris@0
|
331
|
Chris@0
|
332 issue = assigns(:issue)
|
Chris@0
|
333 assert_not_nil issue
|
Chris@0
|
334 assert_equal Project.find(1).trackers.first, issue.tracker
|
Chris@0
|
335 end
|
Chris@0
|
336
|
Chris@0
|
337 def test_get_new_with_no_default_status_should_display_an_error
|
Chris@0
|
338 @request.session[:user_id] = 2
|
Chris@0
|
339 IssueStatus.delete_all
|
Chris@0
|
340
|
Chris@0
|
341 get :new, :project_id => 1
|
Chris@0
|
342 assert_response 500
|
Chris@0
|
343 assert_not_nil flash[:error]
|
Chris@0
|
344 assert_tag :tag => 'div', :attributes => { :class => /error/ },
|
Chris@0
|
345 :content => /No default issue/
|
Chris@0
|
346 end
|
Chris@0
|
347
|
Chris@0
|
348 def test_get_new_with_no_tracker_should_display_an_error
|
Chris@0
|
349 @request.session[:user_id] = 2
|
Chris@0
|
350 Tracker.delete_all
|
Chris@0
|
351
|
Chris@0
|
352 get :new, :project_id => 1
|
Chris@0
|
353 assert_response 500
|
Chris@0
|
354 assert_not_nil flash[:error]
|
Chris@0
|
355 assert_tag :tag => 'div', :attributes => { :class => /error/ },
|
Chris@0
|
356 :content => /No tracker/
|
Chris@0
|
357 end
|
Chris@0
|
358
|
Chris@0
|
359 def test_update_new_form
|
Chris@0
|
360 @request.session[:user_id] = 2
|
Chris@14
|
361 xhr :post, :new, :project_id => 1,
|
Chris@0
|
362 :issue => {:tracker_id => 2,
|
Chris@0
|
363 :subject => 'This is the test_new issue',
|
Chris@0
|
364 :description => 'This is the description',
|
Chris@0
|
365 :priority_id => 5}
|
Chris@0
|
366 assert_response :success
|
Chris@0
|
367 assert_template 'attributes'
|
Chris@0
|
368
|
Chris@0
|
369 issue = assigns(:issue)
|
Chris@0
|
370 assert_kind_of Issue, issue
|
Chris@0
|
371 assert_equal 1, issue.project_id
|
Chris@0
|
372 assert_equal 2, issue.tracker_id
|
Chris@0
|
373 assert_equal 'This is the test_new issue', issue.subject
|
Chris@0
|
374 end
|
Chris@0
|
375
|
Chris@0
|
376 def test_post_create
|
Chris@0
|
377 @request.session[:user_id] = 2
|
Chris@0
|
378 assert_difference 'Issue.count' do
|
Chris@0
|
379 post :create, :project_id => 1,
|
Chris@0
|
380 :issue => {:tracker_id => 3,
|
Chris@0
|
381 :status_id => 2,
|
Chris@0
|
382 :subject => 'This is the test_new issue',
|
Chris@0
|
383 :description => 'This is the description',
|
Chris@0
|
384 :priority_id => 5,
|
Chris@0
|
385 :estimated_hours => '',
|
Chris@0
|
386 :custom_field_values => {'2' => 'Value for field 2'}}
|
Chris@0
|
387 end
|
Chris@0
|
388 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
|
Chris@0
|
389
|
Chris@0
|
390 issue = Issue.find_by_subject('This is the test_new issue')
|
Chris@0
|
391 assert_not_nil issue
|
Chris@0
|
392 assert_equal 2, issue.author_id
|
Chris@0
|
393 assert_equal 3, issue.tracker_id
|
Chris@0
|
394 assert_equal 2, issue.status_id
|
Chris@0
|
395 assert_nil issue.estimated_hours
|
Chris@0
|
396 v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
|
Chris@0
|
397 assert_not_nil v
|
Chris@0
|
398 assert_equal 'Value for field 2', v.value
|
Chris@0
|
399 end
|
Chris@0
|
400
|
Chris@0
|
401 def test_post_create_and_continue
|
Chris@0
|
402 @request.session[:user_id] = 2
|
Chris@0
|
403 post :create, :project_id => 1,
|
Chris@0
|
404 :issue => {:tracker_id => 3,
|
Chris@0
|
405 :subject => 'This is first issue',
|
Chris@0
|
406 :priority_id => 5},
|
Chris@0
|
407 :continue => ''
|
chris@22
|
408 assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook',
|
chris@22
|
409 :issue => {:tracker_id => 3}
|
Chris@0
|
410 end
|
Chris@0
|
411
|
Chris@0
|
412 def test_post_create_without_custom_fields_param
|
Chris@0
|
413 @request.session[:user_id] = 2
|
Chris@0
|
414 assert_difference 'Issue.count' do
|
Chris@0
|
415 post :create, :project_id => 1,
|
Chris@0
|
416 :issue => {:tracker_id => 1,
|
Chris@0
|
417 :subject => 'This is the test_new issue',
|
Chris@0
|
418 :description => 'This is the description',
|
Chris@0
|
419 :priority_id => 5}
|
Chris@0
|
420 end
|
Chris@0
|
421 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
|
Chris@0
|
422 end
|
Chris@0
|
423
|
Chris@0
|
424 def test_post_create_with_required_custom_field_and_without_custom_fields_param
|
Chris@0
|
425 field = IssueCustomField.find_by_name('Database')
|
Chris@0
|
426 field.update_attribute(:is_required, true)
|
Chris@0
|
427
|
Chris@0
|
428 @request.session[:user_id] = 2
|
Chris@0
|
429 post :create, :project_id => 1,
|
Chris@0
|
430 :issue => {:tracker_id => 1,
|
Chris@0
|
431 :subject => 'This is the test_new issue',
|
Chris@0
|
432 :description => 'This is the description',
|
Chris@0
|
433 :priority_id => 5}
|
Chris@0
|
434 assert_response :success
|
Chris@0
|
435 assert_template 'new'
|
Chris@0
|
436 issue = assigns(:issue)
|
Chris@0
|
437 assert_not_nil issue
|
Chris@0
|
438 assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values)
|
Chris@0
|
439 end
|
Chris@0
|
440
|
Chris@0
|
441 def test_post_create_with_watchers
|
Chris@0
|
442 @request.session[:user_id] = 2
|
Chris@0
|
443 ActionMailer::Base.deliveries.clear
|
Chris@0
|
444
|
Chris@0
|
445 assert_difference 'Watcher.count', 2 do
|
Chris@0
|
446 post :create, :project_id => 1,
|
Chris@0
|
447 :issue => {:tracker_id => 1,
|
Chris@0
|
448 :subject => 'This is a new issue with watchers',
|
Chris@0
|
449 :description => 'This is the description',
|
Chris@0
|
450 :priority_id => 5,
|
Chris@0
|
451 :watcher_user_ids => ['2', '3']}
|
Chris@0
|
452 end
|
Chris@0
|
453 issue = Issue.find_by_subject('This is a new issue with watchers')
|
Chris@0
|
454 assert_not_nil issue
|
Chris@0
|
455 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
|
Chris@0
|
456
|
Chris@0
|
457 # Watchers added
|
Chris@0
|
458 assert_equal [2, 3], issue.watcher_user_ids.sort
|
Chris@0
|
459 assert issue.watched_by?(User.find(3))
|
Chris@0
|
460 # Watchers notified
|
Chris@0
|
461 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
462 assert_kind_of TMail::Mail, mail
|
Chris@0
|
463 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
|
Chris@0
|
464 end
|
Chris@0
|
465
|
Chris@0
|
466 def test_post_create_subissue
|
Chris@0
|
467 @request.session[:user_id] = 2
|
Chris@0
|
468
|
Chris@0
|
469 assert_difference 'Issue.count' do
|
Chris@0
|
470 post :create, :project_id => 1,
|
Chris@0
|
471 :issue => {:tracker_id => 1,
|
Chris@0
|
472 :subject => 'This is a child issue',
|
Chris@0
|
473 :parent_issue_id => 2}
|
Chris@0
|
474 end
|
Chris@0
|
475 issue = Issue.find_by_subject('This is a child issue')
|
Chris@0
|
476 assert_not_nil issue
|
Chris@0
|
477 assert_equal Issue.find(2), issue.parent
|
Chris@0
|
478 end
|
Chris@0
|
479
|
Chris@0
|
480 def test_post_create_should_send_a_notification
|
Chris@0
|
481 ActionMailer::Base.deliveries.clear
|
Chris@0
|
482 @request.session[:user_id] = 2
|
Chris@0
|
483 assert_difference 'Issue.count' do
|
Chris@0
|
484 post :create, :project_id => 1,
|
Chris@0
|
485 :issue => {:tracker_id => 3,
|
Chris@0
|
486 :subject => 'This is the test_new issue',
|
Chris@0
|
487 :description => 'This is the description',
|
Chris@0
|
488 :priority_id => 5,
|
Chris@0
|
489 :estimated_hours => '',
|
Chris@0
|
490 :custom_field_values => {'2' => 'Value for field 2'}}
|
Chris@0
|
491 end
|
Chris@0
|
492 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
|
Chris@0
|
493
|
Chris@0
|
494 assert_equal 1, ActionMailer::Base.deliveries.size
|
Chris@0
|
495 end
|
Chris@0
|
496
|
Chris@0
|
497 def test_post_create_should_preserve_fields_values_on_validation_failure
|
Chris@0
|
498 @request.session[:user_id] = 2
|
Chris@0
|
499 post :create, :project_id => 1,
|
Chris@0
|
500 :issue => {:tracker_id => 1,
|
Chris@0
|
501 # empty subject
|
Chris@0
|
502 :subject => '',
|
Chris@0
|
503 :description => 'This is a description',
|
Chris@0
|
504 :priority_id => 6,
|
Chris@0
|
505 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
|
Chris@0
|
506 assert_response :success
|
Chris@0
|
507 assert_template 'new'
|
Chris@0
|
508
|
Chris@0
|
509 assert_tag :textarea, :attributes => { :name => 'issue[description]' },
|
Chris@0
|
510 :content => 'This is a description'
|
Chris@0
|
511 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
|
Chris@0
|
512 :child => { :tag => 'option', :attributes => { :selected => 'selected',
|
Chris@0
|
513 :value => '6' },
|
Chris@0
|
514 :content => 'High' }
|
Chris@0
|
515 # Custom fields
|
Chris@0
|
516 assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
|
Chris@0
|
517 :child => { :tag => 'option', :attributes => { :selected => 'selected',
|
Chris@0
|
518 :value => 'Oracle' },
|
Chris@0
|
519 :content => 'Oracle' }
|
Chris@0
|
520 assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
|
Chris@0
|
521 :value => 'Value for field 2'}
|
Chris@0
|
522 end
|
Chris@0
|
523
|
Chris@0
|
524 def test_post_create_should_ignore_non_safe_attributes
|
Chris@0
|
525 @request.session[:user_id] = 2
|
Chris@0
|
526 assert_nothing_raised do
|
Chris@0
|
527 post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
|
Chris@0
|
528 end
|
Chris@0
|
529 end
|
Chris@0
|
530
|
Chris@0
|
531 context "without workflow privilege" do
|
Chris@0
|
532 setup do
|
Chris@0
|
533 Workflow.delete_all(["role_id = ?", Role.anonymous.id])
|
Chris@0
|
534 Role.anonymous.add_permission! :add_issues
|
Chris@0
|
535 end
|
Chris@0
|
536
|
Chris@0
|
537 context "#new" do
|
Chris@0
|
538 should "propose default status only" do
|
Chris@0
|
539 get :new, :project_id => 1
|
Chris@0
|
540 assert_response :success
|
Chris@0
|
541 assert_template 'new'
|
Chris@0
|
542 assert_tag :tag => 'select',
|
Chris@0
|
543 :attributes => {:name => 'issue[status_id]'},
|
Chris@0
|
544 :children => {:count => 1},
|
Chris@0
|
545 :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}}
|
Chris@0
|
546 end
|
Chris@0
|
547
|
Chris@0
|
548 should "accept default status" do
|
Chris@0
|
549 assert_difference 'Issue.count' do
|
Chris@0
|
550 post :create, :project_id => 1,
|
Chris@0
|
551 :issue => {:tracker_id => 1,
|
Chris@0
|
552 :subject => 'This is an issue',
|
Chris@0
|
553 :status_id => 1}
|
Chris@0
|
554 end
|
Chris@0
|
555 issue = Issue.last(:order => 'id')
|
Chris@0
|
556 assert_equal IssueStatus.default, issue.status
|
Chris@0
|
557 end
|
Chris@0
|
558
|
Chris@0
|
559 should "ignore unauthorized status" do
|
Chris@0
|
560 assert_difference 'Issue.count' do
|
Chris@0
|
561 post :create, :project_id => 1,
|
Chris@0
|
562 :issue => {:tracker_id => 1,
|
Chris@0
|
563 :subject => 'This is an issue',
|
Chris@0
|
564 :status_id => 3}
|
Chris@0
|
565 end
|
Chris@0
|
566 issue = Issue.last(:order => 'id')
|
Chris@0
|
567 assert_equal IssueStatus.default, issue.status
|
Chris@0
|
568 end
|
Chris@0
|
569 end
|
Chris@0
|
570 end
|
Chris@0
|
571
|
Chris@0
|
572 def test_copy_issue
|
Chris@0
|
573 @request.session[:user_id] = 2
|
Chris@0
|
574 get :new, :project_id => 1, :copy_from => 1
|
Chris@0
|
575 assert_template 'new'
|
Chris@0
|
576 assert_not_nil assigns(:issue)
|
Chris@0
|
577 orig = Issue.find(1)
|
Chris@0
|
578 assert_equal orig.subject, assigns(:issue).subject
|
Chris@0
|
579 end
|
Chris@0
|
580
|
Chris@0
|
581 def test_get_edit
|
Chris@0
|
582 @request.session[:user_id] = 2
|
Chris@0
|
583 get :edit, :id => 1
|
Chris@0
|
584 assert_response :success
|
Chris@0
|
585 assert_template 'edit'
|
Chris@0
|
586 assert_not_nil assigns(:issue)
|
Chris@0
|
587 assert_equal Issue.find(1), assigns(:issue)
|
Chris@0
|
588 end
|
Chris@0
|
589
|
Chris@0
|
590 def test_get_edit_with_params
|
Chris@0
|
591 @request.session[:user_id] = 2
|
Chris@0
|
592 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 }
|
Chris@0
|
593 assert_response :success
|
Chris@0
|
594 assert_template 'edit'
|
Chris@0
|
595
|
Chris@0
|
596 issue = assigns(:issue)
|
Chris@0
|
597 assert_not_nil issue
|
Chris@0
|
598
|
Chris@0
|
599 assert_equal 5, issue.status_id
|
Chris@0
|
600 assert_tag :select, :attributes => { :name => 'issue[status_id]' },
|
Chris@0
|
601 :child => { :tag => 'option',
|
Chris@0
|
602 :content => 'Closed',
|
Chris@0
|
603 :attributes => { :selected => 'selected' } }
|
Chris@0
|
604
|
Chris@0
|
605 assert_equal 7, issue.priority_id
|
Chris@0
|
606 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
|
Chris@0
|
607 :child => { :tag => 'option',
|
Chris@0
|
608 :content => 'Urgent',
|
Chris@0
|
609 :attributes => { :selected => 'selected' } }
|
Chris@0
|
610 end
|
Chris@0
|
611
|
Chris@0
|
612 def test_update_edit_form
|
Chris@0
|
613 @request.session[:user_id] = 2
|
Chris@14
|
614 xhr :post, :new, :project_id => 1,
|
Chris@0
|
615 :id => 1,
|
Chris@0
|
616 :issue => {:tracker_id => 2,
|
Chris@0
|
617 :subject => 'This is the test_new issue',
|
Chris@0
|
618 :description => 'This is the description',
|
Chris@0
|
619 :priority_id => 5}
|
Chris@0
|
620 assert_response :success
|
Chris@0
|
621 assert_template 'attributes'
|
Chris@0
|
622
|
Chris@0
|
623 issue = assigns(:issue)
|
Chris@0
|
624 assert_kind_of Issue, issue
|
Chris@0
|
625 assert_equal 1, issue.id
|
Chris@0
|
626 assert_equal 1, issue.project_id
|
Chris@0
|
627 assert_equal 2, issue.tracker_id
|
Chris@0
|
628 assert_equal 'This is the test_new issue', issue.subject
|
Chris@0
|
629 end
|
Chris@0
|
630
|
Chris@0
|
631 def test_update_using_invalid_http_verbs
|
Chris@0
|
632 @request.session[:user_id] = 2
|
Chris@0
|
633 subject = 'Updated by an invalid http verb'
|
Chris@0
|
634
|
Chris@0
|
635 get :update, :id => 1, :issue => {:subject => subject}
|
Chris@0
|
636 assert_not_equal subject, Issue.find(1).subject
|
Chris@0
|
637
|
Chris@0
|
638 post :update, :id => 1, :issue => {:subject => subject}
|
Chris@0
|
639 assert_not_equal subject, Issue.find(1).subject
|
Chris@0
|
640
|
Chris@0
|
641 delete :update, :id => 1, :issue => {:subject => subject}
|
Chris@0
|
642 assert_not_equal subject, Issue.find(1).subject
|
Chris@0
|
643 end
|
Chris@0
|
644
|
Chris@0
|
645 def test_put_update_without_custom_fields_param
|
Chris@0
|
646 @request.session[:user_id] = 2
|
Chris@0
|
647 ActionMailer::Base.deliveries.clear
|
Chris@0
|
648
|
Chris@0
|
649 issue = Issue.find(1)
|
Chris@0
|
650 assert_equal '125', issue.custom_value_for(2).value
|
Chris@0
|
651 old_subject = issue.subject
|
Chris@0
|
652 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
|
Chris@0
|
653
|
Chris@0
|
654 assert_difference('Journal.count') do
|
Chris@0
|
655 assert_difference('JournalDetail.count', 2) do
|
Chris@0
|
656 put :update, :id => 1, :issue => {:subject => new_subject,
|
Chris@0
|
657 :priority_id => '6',
|
Chris@0
|
658 :category_id => '1' # no change
|
Chris@0
|
659 }
|
Chris@0
|
660 end
|
Chris@0
|
661 end
|
Chris@0
|
662 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
663 issue.reload
|
Chris@0
|
664 assert_equal new_subject, issue.subject
|
Chris@0
|
665 # Make sure custom fields were not cleared
|
Chris@0
|
666 assert_equal '125', issue.custom_value_for(2).value
|
Chris@0
|
667
|
Chris@0
|
668 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
669 assert_kind_of TMail::Mail, mail
|
Chris@0
|
670 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
|
Chris@0
|
671 assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}")
|
Chris@0
|
672 end
|
Chris@0
|
673
|
Chris@0
|
674 def test_put_update_with_custom_field_change
|
Chris@0
|
675 @request.session[:user_id] = 2
|
Chris@0
|
676 issue = Issue.find(1)
|
Chris@0
|
677 assert_equal '125', issue.custom_value_for(2).value
|
Chris@0
|
678
|
Chris@0
|
679 assert_difference('Journal.count') do
|
Chris@0
|
680 assert_difference('JournalDetail.count', 3) do
|
Chris@0
|
681 put :update, :id => 1, :issue => {:subject => 'Custom field change',
|
Chris@0
|
682 :priority_id => '6',
|
Chris@0
|
683 :category_id => '1', # no change
|
Chris@0
|
684 :custom_field_values => { '2' => 'New custom value' }
|
Chris@0
|
685 }
|
Chris@0
|
686 end
|
Chris@0
|
687 end
|
Chris@0
|
688 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
689 issue.reload
|
Chris@0
|
690 assert_equal 'New custom value', issue.custom_value_for(2).value
|
Chris@0
|
691
|
Chris@0
|
692 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
693 assert_kind_of TMail::Mail, mail
|
Chris@0
|
694 assert mail.body.include?("Searchable field changed from 125 to New custom value")
|
Chris@0
|
695 end
|
Chris@0
|
696
|
Chris@0
|
697 def test_put_update_with_status_and_assignee_change
|
Chris@0
|
698 issue = Issue.find(1)
|
Chris@0
|
699 assert_equal 1, issue.status_id
|
Chris@0
|
700 @request.session[:user_id] = 2
|
Chris@0
|
701 assert_difference('TimeEntry.count', 0) do
|
Chris@0
|
702 put :update,
|
Chris@0
|
703 :id => 1,
|
Chris@0
|
704 :issue => { :status_id => 2, :assigned_to_id => 3 },
|
Chris@0
|
705 :notes => 'Assigned to dlopper',
|
Chris@0
|
706 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
|
Chris@0
|
707 end
|
Chris@0
|
708 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
709 issue.reload
|
Chris@0
|
710 assert_equal 2, issue.status_id
|
Chris@0
|
711 j = Journal.find(:first, :order => 'id DESC')
|
Chris@0
|
712 assert_equal 'Assigned to dlopper', j.notes
|
Chris@0
|
713 assert_equal 2, j.details.size
|
Chris@0
|
714
|
Chris@0
|
715 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
716 assert mail.body.include?("Status changed from New to Assigned")
|
Chris@0
|
717 # subject should contain the new status
|
Chris@0
|
718 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
|
Chris@0
|
719 end
|
Chris@0
|
720
|
Chris@0
|
721 def test_put_update_with_note_only
|
Chris@0
|
722 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
|
Chris@0
|
723 # anonymous user
|
Chris@0
|
724 put :update,
|
Chris@0
|
725 :id => 1,
|
Chris@0
|
726 :notes => notes
|
Chris@0
|
727 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
728 j = Journal.find(:first, :order => 'id DESC')
|
Chris@0
|
729 assert_equal notes, j.notes
|
Chris@0
|
730 assert_equal 0, j.details.size
|
Chris@0
|
731 assert_equal User.anonymous, j.user
|
Chris@0
|
732
|
Chris@0
|
733 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
734 assert mail.body.include?(notes)
|
Chris@0
|
735 end
|
Chris@0
|
736
|
Chris@0
|
737 def test_put_update_with_note_and_spent_time
|
Chris@0
|
738 @request.session[:user_id] = 2
|
Chris@0
|
739 spent_hours_before = Issue.find(1).spent_hours
|
Chris@0
|
740 assert_difference('TimeEntry.count') do
|
Chris@0
|
741 put :update,
|
Chris@0
|
742 :id => 1,
|
Chris@0
|
743 :notes => '2.5 hours added',
|
Chris@0
|
744 :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
|
Chris@0
|
745 end
|
Chris@0
|
746 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
747
|
Chris@0
|
748 issue = Issue.find(1)
|
Chris@0
|
749
|
Chris@0
|
750 j = Journal.find(:first, :order => 'id DESC')
|
Chris@0
|
751 assert_equal '2.5 hours added', j.notes
|
Chris@0
|
752 assert_equal 0, j.details.size
|
Chris@0
|
753
|
Chris@0
|
754 t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
|
Chris@0
|
755 assert_not_nil t
|
Chris@0
|
756 assert_equal 2.5, t.hours
|
Chris@0
|
757 assert_equal spent_hours_before + 2.5, issue.spent_hours
|
Chris@0
|
758 end
|
Chris@0
|
759
|
Chris@0
|
760 def test_put_update_with_attachment_only
|
Chris@0
|
761 set_tmp_attachments_directory
|
Chris@0
|
762
|
Chris@0
|
763 # Delete all fixtured journals, a race condition can occur causing the wrong
|
Chris@0
|
764 # journal to get fetched in the next find.
|
Chris@0
|
765 Journal.delete_all
|
Chris@0
|
766
|
Chris@0
|
767 # anonymous user
|
Chris@0
|
768 put :update,
|
Chris@0
|
769 :id => 1,
|
Chris@0
|
770 :notes => '',
|
Chris@0
|
771 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
|
Chris@0
|
772 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
773 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
|
Chris@0
|
774 assert j.notes.blank?
|
Chris@0
|
775 assert_equal 1, j.details.size
|
Chris@0
|
776 assert_equal 'testfile.txt', j.details.first.value
|
Chris@0
|
777 assert_equal User.anonymous, j.user
|
Chris@0
|
778
|
Chris@0
|
779 mail = ActionMailer::Base.deliveries.last
|
Chris@0
|
780 assert mail.body.include?('testfile.txt')
|
Chris@0
|
781 end
|
Chris@0
|
782
|
Chris@0
|
783 def test_put_update_with_attachment_that_fails_to_save
|
Chris@0
|
784 set_tmp_attachments_directory
|
Chris@0
|
785
|
Chris@0
|
786 # Delete all fixtured journals, a race condition can occur causing the wrong
|
Chris@0
|
787 # journal to get fetched in the next find.
|
Chris@0
|
788 Journal.delete_all
|
Chris@0
|
789
|
Chris@0
|
790 # Mock out the unsaved attachment
|
Chris@0
|
791 Attachment.any_instance.stubs(:create).returns(Attachment.new)
|
Chris@0
|
792
|
Chris@0
|
793 # anonymous user
|
Chris@0
|
794 put :update,
|
Chris@0
|
795 :id => 1,
|
Chris@0
|
796 :notes => '',
|
Chris@0
|
797 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
|
Chris@0
|
798 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
799 assert_equal '1 file(s) could not be saved.', flash[:warning]
|
Chris@0
|
800
|
Chris@0
|
801 end if Object.const_defined?(:Mocha)
|
Chris@0
|
802
|
Chris@0
|
803 def test_put_update_with_no_change
|
Chris@0
|
804 issue = Issue.find(1)
|
Chris@0
|
805 issue.journals.clear
|
Chris@0
|
806 ActionMailer::Base.deliveries.clear
|
Chris@0
|
807
|
Chris@0
|
808 put :update,
|
Chris@0
|
809 :id => 1,
|
Chris@0
|
810 :notes => ''
|
Chris@0
|
811 assert_redirected_to :action => 'show', :id => '1'
|
Chris@0
|
812
|
Chris@0
|
813 issue.reload
|
Chris@0
|
814 assert issue.journals.empty?
|
Chris@0
|
815 # No email should be sent
|
Chris@0
|
816 assert ActionMailer::Base.deliveries.empty?
|
Chris@0
|
817 end
|
Chris@0
|
818
|
Chris@0
|
819 def test_put_update_should_send_a_notification
|
Chris@0
|
820 @request.session[:user_id] = 2
|
Chris@0
|
821 ActionMailer::Base.deliveries.clear
|
Chris@0
|
822 issue = Issue.find(1)
|
Chris@0
|
823 old_subject = issue.subject
|
Chris@0
|
824 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
|
Chris@0
|
825
|
Chris@0
|
826 put :update, :id => 1, :issue => {:subject => new_subject,
|
Chris@0
|
827 :priority_id => '6',
|
Chris@0
|
828 :category_id => '1' # no change
|
Chris@0
|
829 }
|
Chris@0
|
830 assert_equal 1, ActionMailer::Base.deliveries.size
|
Chris@0
|
831 end
|
Chris@0
|
832
|
Chris@0
|
833 def test_put_update_with_invalid_spent_time
|
Chris@0
|
834 @request.session[:user_id] = 2
|
Chris@0
|
835 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
|
Chris@0
|
836
|
Chris@0
|
837 assert_no_difference('Journal.count') do
|
Chris@0
|
838 put :update,
|
Chris@0
|
839 :id => 1,
|
Chris@0
|
840 :notes => notes,
|
Chris@0
|
841 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
|
Chris@0
|
842 end
|
Chris@0
|
843 assert_response :success
|
Chris@0
|
844 assert_template 'edit'
|
Chris@0
|
845
|
Chris@0
|
846 assert_tag :textarea, :attributes => { :name => 'notes' },
|
Chris@0
|
847 :content => notes
|
Chris@0
|
848 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
|
Chris@0
|
849 end
|
Chris@0
|
850
|
Chris@0
|
851 def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
|
Chris@0
|
852 issue = Issue.find(2)
|
Chris@0
|
853 @request.session[:user_id] = 2
|
Chris@0
|
854
|
Chris@0
|
855 put :update,
|
Chris@0
|
856 :id => issue.id,
|
Chris@0
|
857 :issue => {
|
Chris@0
|
858 :fixed_version_id => 4
|
Chris@0
|
859 }
|
Chris@0
|
860
|
Chris@0
|
861 assert_response :redirect
|
Chris@0
|
862 issue.reload
|
Chris@0
|
863 assert_equal 4, issue.fixed_version_id
|
Chris@0
|
864 assert_not_equal issue.project_id, issue.fixed_version.project_id
|
Chris@0
|
865 end
|
Chris@0
|
866
|
Chris@0
|
867 def test_put_update_should_redirect_back_using_the_back_url_parameter
|
Chris@0
|
868 issue = Issue.find(2)
|
Chris@0
|
869 @request.session[:user_id] = 2
|
Chris@0
|
870
|
Chris@0
|
871 put :update,
|
Chris@0
|
872 :id => issue.id,
|
Chris@0
|
873 :issue => {
|
Chris@0
|
874 :fixed_version_id => 4
|
Chris@0
|
875 },
|
Chris@0
|
876 :back_url => '/issues'
|
Chris@0
|
877
|
Chris@0
|
878 assert_response :redirect
|
Chris@0
|
879 assert_redirected_to '/issues'
|
Chris@0
|
880 end
|
Chris@0
|
881
|
Chris@0
|
882 def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
|
Chris@0
|
883 issue = Issue.find(2)
|
Chris@0
|
884 @request.session[:user_id] = 2
|
Chris@0
|
885
|
Chris@0
|
886 put :update,
|
Chris@0
|
887 :id => issue.id,
|
Chris@0
|
888 :issue => {
|
Chris@0
|
889 :fixed_version_id => 4
|
Chris@0
|
890 },
|
Chris@0
|
891 :back_url => 'http://google.com'
|
Chris@0
|
892
|
Chris@0
|
893 assert_response :redirect
|
Chris@0
|
894 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
|
Chris@0
|
895 end
|
Chris@0
|
896
|
Chris@0
|
897 def test_get_bulk_edit
|
Chris@0
|
898 @request.session[:user_id] = 2
|
Chris@0
|
899 get :bulk_edit, :ids => [1, 2]
|
Chris@0
|
900 assert_response :success
|
Chris@0
|
901 assert_template 'bulk_edit'
|
Chris@0
|
902
|
Chris@0
|
903 # Project specific custom field, date type
|
Chris@0
|
904 field = CustomField.find(9)
|
Chris@0
|
905 assert !field.is_for_all?
|
Chris@0
|
906 assert_equal 'date', field.field_format
|
Chris@0
|
907 assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
|
Chris@0
|
908
|
Chris@0
|
909 # System wide custom field
|
Chris@0
|
910 assert CustomField.find(1).is_for_all?
|
Chris@0
|
911 assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'}
|
Chris@0
|
912 end
|
Chris@0
|
913
|
Chris@14
|
914 def test_bulk_update
|
Chris@0
|
915 @request.session[:user_id] = 2
|
Chris@0
|
916 # update issues priority
|
Chris@14
|
917 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
|
Chris@0
|
918 :issue => {:priority_id => 7,
|
Chris@0
|
919 :assigned_to_id => '',
|
Chris@0
|
920 :custom_field_values => {'2' => ''}}
|
Chris@0
|
921
|
Chris@0
|
922 assert_response 302
|
Chris@0
|
923 # check that the issues were updated
|
Chris@0
|
924 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
|
Chris@0
|
925
|
Chris@0
|
926 issue = Issue.find(1)
|
Chris@0
|
927 journal = issue.journals.find(:first, :order => 'created_on DESC')
|
Chris@0
|
928 assert_equal '125', issue.custom_value_for(2).value
|
Chris@0
|
929 assert_equal 'Bulk editing', journal.notes
|
Chris@0
|
930 assert_equal 1, journal.details.size
|
Chris@0
|
931 end
|
Chris@0
|
932
|
Chris@14
|
933 def test_bullk_update_should_send_a_notification
|
Chris@0
|
934 @request.session[:user_id] = 2
|
Chris@0
|
935 ActionMailer::Base.deliveries.clear
|
Chris@14
|
936 post(:bulk_update,
|
Chris@0
|
937 {
|
Chris@0
|
938 :ids => [1, 2],
|
Chris@0
|
939 :notes => 'Bulk editing',
|
Chris@0
|
940 :issue => {
|
Chris@0
|
941 :priority_id => 7,
|
Chris@0
|
942 :assigned_to_id => '',
|
Chris@0
|
943 :custom_field_values => {'2' => ''}
|
Chris@0
|
944 }
|
Chris@0
|
945 })
|
Chris@0
|
946
|
Chris@0
|
947 assert_response 302
|
Chris@0
|
948 assert_equal 2, ActionMailer::Base.deliveries.size
|
Chris@0
|
949 end
|
Chris@0
|
950
|
Chris@14
|
951 def test_bulk_update_status
|
Chris@0
|
952 @request.session[:user_id] = 2
|
Chris@0
|
953 # update issues priority
|
Chris@14
|
954 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status',
|
Chris@0
|
955 :issue => {:priority_id => '',
|
Chris@0
|
956 :assigned_to_id => '',
|
Chris@0
|
957 :status_id => '5'}
|
Chris@0
|
958
|
Chris@0
|
959 assert_response 302
|
Chris@0
|
960 issue = Issue.find(1)
|
Chris@0
|
961 assert issue.closed?
|
Chris@0
|
962 end
|
Chris@0
|
963
|
Chris@14
|
964 def test_bulk_update_custom_field
|
Chris@0
|
965 @request.session[:user_id] = 2
|
Chris@0
|
966 # update issues priority
|
Chris@14
|
967 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field',
|
Chris@0
|
968 :issue => {:priority_id => '',
|
Chris@0
|
969 :assigned_to_id => '',
|
Chris@0
|
970 :custom_field_values => {'2' => '777'}}
|
Chris@0
|
971
|
Chris@0
|
972 assert_response 302
|
Chris@0
|
973
|
Chris@0
|
974 issue = Issue.find(1)
|
Chris@0
|
975 journal = issue.journals.find(:first, :order => 'created_on DESC')
|
Chris@0
|
976 assert_equal '777', issue.custom_value_for(2).value
|
Chris@0
|
977 assert_equal 1, journal.details.size
|
Chris@0
|
978 assert_equal '125', journal.details.first.old_value
|
Chris@0
|
979 assert_equal '777', journal.details.first.value
|
Chris@0
|
980 end
|
Chris@0
|
981
|
Chris@14
|
982 def test_bulk_update_unassign
|
Chris@0
|
983 assert_not_nil Issue.find(2).assigned_to
|
Chris@0
|
984 @request.session[:user_id] = 2
|
Chris@0
|
985 # unassign issues
|
Chris@14
|
986 post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
|
Chris@0
|
987 assert_response 302
|
Chris@0
|
988 # check that the issues were updated
|
Chris@0
|
989 assert_nil Issue.find(2).assigned_to
|
Chris@0
|
990 end
|
Chris@0
|
991
|
Chris@14
|
992 def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject
|
Chris@0
|
993 @request.session[:user_id] = 2
|
Chris@0
|
994
|
Chris@14
|
995 post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4}
|
Chris@0
|
996
|
Chris@0
|
997 assert_response :redirect
|
Chris@0
|
998 issues = Issue.find([1,2])
|
Chris@0
|
999 issues.each do |issue|
|
Chris@0
|
1000 assert_equal 4, issue.fixed_version_id
|
Chris@0
|
1001 assert_not_equal issue.project_id, issue.fixed_version.project_id
|
Chris@0
|
1002 end
|
Chris@0
|
1003 end
|
Chris@0
|
1004
|
Chris@14
|
1005 def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter
|
Chris@0
|
1006 @request.session[:user_id] = 2
|
Chris@14
|
1007 post :bulk_update, :ids => [1,2], :back_url => '/issues'
|
Chris@0
|
1008
|
Chris@0
|
1009 assert_response :redirect
|
Chris@0
|
1010 assert_redirected_to '/issues'
|
Chris@0
|
1011 end
|
Chris@0
|
1012
|
Chris@14
|
1013 def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
|
Chris@0
|
1014 @request.session[:user_id] = 2
|
Chris@14
|
1015 post :bulk_update, :ids => [1,2], :back_url => 'http://google.com'
|
Chris@0
|
1016
|
Chris@0
|
1017 assert_response :redirect
|
Chris@0
|
1018 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
|
Chris@0
|
1019 end
|
Chris@0
|
1020
|
Chris@0
|
1021 def test_destroy_issue_with_no_time_entries
|
Chris@0
|
1022 assert_nil TimeEntry.find_by_issue_id(2)
|
Chris@0
|
1023 @request.session[:user_id] = 2
|
Chris@0
|
1024 post :destroy, :id => 2
|
Chris@0
|
1025 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
|
Chris@0
|
1026 assert_nil Issue.find_by_id(2)
|
Chris@0
|
1027 end
|
Chris@0
|
1028
|
Chris@0
|
1029 def test_destroy_issues_with_time_entries
|
Chris@0
|
1030 @request.session[:user_id] = 2
|
Chris@0
|
1031 post :destroy, :ids => [1, 3]
|
Chris@0
|
1032 assert_response :success
|
Chris@0
|
1033 assert_template 'destroy'
|
Chris@0
|
1034 assert_not_nil assigns(:hours)
|
Chris@0
|
1035 assert Issue.find_by_id(1) && Issue.find_by_id(3)
|
Chris@0
|
1036 end
|
Chris@0
|
1037
|
Chris@0
|
1038 def test_destroy_issues_and_destroy_time_entries
|
Chris@0
|
1039 @request.session[:user_id] = 2
|
Chris@0
|
1040 post :destroy, :ids => [1, 3], :todo => 'destroy'
|
Chris@0
|
1041 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
|
Chris@0
|
1042 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
|
Chris@0
|
1043 assert_nil TimeEntry.find_by_id([1, 2])
|
Chris@0
|
1044 end
|
Chris@0
|
1045
|
Chris@0
|
1046 def test_destroy_issues_and_assign_time_entries_to_project
|
Chris@0
|
1047 @request.session[:user_id] = 2
|
Chris@0
|
1048 post :destroy, :ids => [1, 3], :todo => 'nullify'
|
Chris@0
|
1049 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
|
Chris@0
|
1050 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
|
Chris@0
|
1051 assert_nil TimeEntry.find(1).issue_id
|
Chris@0
|
1052 assert_nil TimeEntry.find(2).issue_id
|
Chris@0
|
1053 end
|
Chris@0
|
1054
|
Chris@0
|
1055 def test_destroy_issues_and_reassign_time_entries_to_another_issue
|
Chris@0
|
1056 @request.session[:user_id] = 2
|
Chris@0
|
1057 post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
|
Chris@0
|
1058 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
|
Chris@0
|
1059 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
|
Chris@0
|
1060 assert_equal 2, TimeEntry.find(1).issue_id
|
Chris@0
|
1061 assert_equal 2, TimeEntry.find(2).issue_id
|
Chris@0
|
1062 end
|
Chris@0
|
1063
|
Chris@0
|
1064 def test_default_search_scope
|
Chris@0
|
1065 get :index
|
Chris@0
|
1066 assert_tag :div, :attributes => {:id => 'quick-search'},
|
Chris@0
|
1067 :child => {:tag => 'form',
|
Chris@0
|
1068 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}
|
Chris@0
|
1069 end
|
Chris@0
|
1070 end
|