Chris@0
|
1 # Redmine - project management software
|
Chris@1295
|
2 # Copyright (C) 2006-2013 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@909
|
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@909
|
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@119
|
18 require File.expand_path('../../test_helper', __FILE__)
|
Chris@0
|
19 require 'issues_controller'
|
Chris@0
|
20
|
Chris@0
|
21 class IssuesControllerTransactionTest < ActionController::TestCase
|
Chris@1295
|
22 tests IssuesController
|
Chris@0
|
23 fixtures :projects,
|
Chris@0
|
24 :users,
|
Chris@0
|
25 :roles,
|
Chris@0
|
26 :members,
|
Chris@0
|
27 :member_roles,
|
Chris@0
|
28 :issues,
|
Chris@0
|
29 :issue_statuses,
|
Chris@0
|
30 :versions,
|
Chris@0
|
31 :trackers,
|
Chris@0
|
32 :projects_trackers,
|
Chris@0
|
33 :issue_categories,
|
Chris@0
|
34 :enabled_modules,
|
Chris@0
|
35 :enumerations,
|
Chris@0
|
36 :attachments,
|
Chris@0
|
37 :workflows,
|
Chris@0
|
38 :custom_fields,
|
Chris@0
|
39 :custom_values,
|
Chris@0
|
40 :custom_fields_projects,
|
Chris@0
|
41 :custom_fields_trackers,
|
Chris@0
|
42 :time_entries,
|
Chris@0
|
43 :journals,
|
Chris@0
|
44 :journal_details,
|
Chris@0
|
45 :queries
|
Chris@0
|
46
|
Chris@0
|
47 self.use_transactional_fixtures = false
|
Chris@909
|
48
|
Chris@0
|
49 def setup
|
Chris@0
|
50 User.current = nil
|
Chris@0
|
51 end
|
Chris@909
|
52
|
Chris@1115
|
53 def test_update_stale_issue_should_not_update_the_issue
|
Chris@0
|
54 issue = Issue.find(2)
|
Chris@0
|
55 @request.session[:user_id] = 2
|
Chris@0
|
56
|
Chris@0
|
57 assert_no_difference 'Journal.count' do
|
Chris@0
|
58 assert_no_difference 'TimeEntry.count' do
|
Chris@1115
|
59 put :update,
|
Chris@1115
|
60 :id => issue.id,
|
Chris@1115
|
61 :issue => {
|
Chris@1115
|
62 :fixed_version_id => 4,
|
Chris@1115
|
63 :notes => 'My notes',
|
Chris@1115
|
64 :lock_version => (issue.lock_version - 1)
|
Chris@1115
|
65 },
|
Chris@1115
|
66 :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id }
|
Chris@1115
|
67 end
|
Chris@1115
|
68 end
|
Chris@1115
|
69
|
Chris@1115
|
70 assert_response :success
|
Chris@1115
|
71 assert_template 'edit'
|
Chris@1115
|
72
|
Chris@1115
|
73 assert_select 'div.conflict'
|
Chris@1115
|
74 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'overwrite'
|
Chris@1115
|
75 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'add_notes'
|
Chris@1115
|
76 assert_select 'label' do
|
Chris@1115
|
77 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'cancel'
|
Chris@1115
|
78 assert_select 'a[href=/issues/2]'
|
Chris@1115
|
79 end
|
Chris@1115
|
80 end
|
Chris@1115
|
81
|
Chris@1115
|
82 def test_update_stale_issue_should_save_attachments
|
Chris@1115
|
83 set_tmp_attachments_directory
|
Chris@1115
|
84 issue = Issue.find(2)
|
Chris@1115
|
85 @request.session[:user_id] = 2
|
Chris@1115
|
86
|
Chris@1115
|
87 assert_no_difference 'Journal.count' do
|
Chris@1115
|
88 assert_no_difference 'TimeEntry.count' do
|
Chris@1115
|
89 assert_difference 'Attachment.count' do
|
Chris@0
|
90 put :update,
|
Chris@0
|
91 :id => issue.id,
|
Chris@0
|
92 :issue => {
|
Chris@0
|
93 :fixed_version_id => 4,
|
Chris@1115
|
94 :notes => 'My notes',
|
Chris@0
|
95 :lock_version => (issue.lock_version - 1)
|
Chris@0
|
96 },
|
Chris@0
|
97 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}},
|
Chris@0
|
98 :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id }
|
Chris@0
|
99 end
|
Chris@0
|
100 end
|
Chris@0
|
101 end
|
Chris@909
|
102
|
Chris@0
|
103 assert_response :success
|
Chris@0
|
104 assert_template 'edit'
|
Chris@1115
|
105 attachment = Attachment.first(:order => 'id DESC')
|
Chris@1115
|
106 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
|
Chris@1295
|
107 assert_tag 'input', :attributes => {:name => 'attachments[p0][filename]', :value => 'testfile.txt'}
|
Chris@1115
|
108 end
|
Chris@1115
|
109
|
Chris@1115
|
110 def test_update_stale_issue_without_notes_should_not_show_add_notes_option
|
Chris@1115
|
111 issue = Issue.find(2)
|
Chris@1115
|
112 @request.session[:user_id] = 2
|
Chris@1115
|
113
|
Chris@1115
|
114 put :update, :id => issue.id,
|
Chris@1115
|
115 :issue => {
|
Chris@1115
|
116 :fixed_version_id => 4,
|
Chris@1115
|
117 :notes => '',
|
Chris@1115
|
118 :lock_version => (issue.lock_version - 1)
|
Chris@1115
|
119 }
|
Chris@1115
|
120
|
Chris@1115
|
121 assert_tag 'div', :attributes => {:class => 'conflict'}
|
Chris@1115
|
122 assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'overwrite'}
|
Chris@1115
|
123 assert_no_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'add_notes'}
|
Chris@1115
|
124 assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'cancel'}
|
Chris@1115
|
125 end
|
Chris@1115
|
126
|
Chris@1115
|
127 def test_update_stale_issue_should_show_conflicting_journals
|
Chris@1115
|
128 @request.session[:user_id] = 2
|
Chris@1115
|
129
|
Chris@1115
|
130 put :update, :id => 1,
|
Chris@1115
|
131 :issue => {
|
Chris@1115
|
132 :fixed_version_id => 4,
|
Chris@1115
|
133 :notes => '',
|
Chris@1115
|
134 :lock_version => 2
|
Chris@1115
|
135 },
|
Chris@1115
|
136 :last_journal_id => 1
|
Chris@1115
|
137
|
Chris@1115
|
138 assert_not_nil assigns(:conflict_journals)
|
Chris@1115
|
139 assert_equal 1, assigns(:conflict_journals).size
|
Chris@1115
|
140 assert_equal 2, assigns(:conflict_journals).first.id
|
Chris@1115
|
141 assert_tag 'div', :attributes => {:class => 'conflict'},
|
Chris@1115
|
142 :descendant => {:content => /Some notes with Redmine links/}
|
Chris@1115
|
143 end
|
Chris@1115
|
144
|
Chris@1115
|
145 def test_update_stale_issue_without_previous_journal_should_show_all_journals
|
Chris@1115
|
146 @request.session[:user_id] = 2
|
Chris@1115
|
147
|
Chris@1115
|
148 put :update, :id => 1,
|
Chris@1115
|
149 :issue => {
|
Chris@1115
|
150 :fixed_version_id => 4,
|
Chris@1115
|
151 :notes => '',
|
Chris@1115
|
152 :lock_version => 2
|
Chris@1115
|
153 },
|
Chris@1115
|
154 :last_journal_id => ''
|
Chris@1115
|
155
|
Chris@1115
|
156 assert_not_nil assigns(:conflict_journals)
|
Chris@1115
|
157 assert_equal 2, assigns(:conflict_journals).size
|
Chris@1115
|
158 assert_tag 'div', :attributes => {:class => 'conflict'},
|
Chris@1115
|
159 :descendant => {:content => /Some notes with Redmine links/}
|
Chris@1115
|
160 assert_tag 'div', :attributes => {:class => 'conflict'},
|
Chris@1115
|
161 :descendant => {:content => /Journal notes/}
|
Chris@1115
|
162 end
|
Chris@1115
|
163
|
Chris@1115
|
164 def test_update_stale_issue_should_show_private_journals_with_permission_only
|
Chris@1115
|
165 journal = Journal.create!(:journalized => Issue.find(1), :notes => 'Privates notes', :private_notes => true, :user_id => 1)
|
Chris@1115
|
166
|
Chris@1115
|
167 @request.session[:user_id] = 2
|
Chris@1115
|
168 put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => ''
|
Chris@1115
|
169 assert_include journal, assigns(:conflict_journals)
|
Chris@1115
|
170
|
Chris@1115
|
171 Role.find(1).remove_permission! :view_private_notes
|
Chris@1115
|
172 put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => ''
|
Chris@1115
|
173 assert_not_include journal, assigns(:conflict_journals)
|
Chris@1115
|
174 end
|
Chris@1115
|
175
|
Chris@1115
|
176 def test_update_stale_issue_with_overwrite_conflict_resolution_should_update
|
Chris@1115
|
177 @request.session[:user_id] = 2
|
Chris@1115
|
178
|
Chris@1115
|
179 assert_difference 'Journal.count' do
|
Chris@1115
|
180 put :update, :id => 1,
|
Chris@1115
|
181 :issue => {
|
Chris@1115
|
182 :fixed_version_id => 4,
|
Chris@1115
|
183 :notes => 'overwrite_conflict_resolution',
|
Chris@1115
|
184 :lock_version => 2
|
Chris@1115
|
185 },
|
Chris@1115
|
186 :conflict_resolution => 'overwrite'
|
Chris@1115
|
187 end
|
Chris@1115
|
188
|
Chris@1115
|
189 assert_response 302
|
Chris@1115
|
190 issue = Issue.find(1)
|
Chris@1115
|
191 assert_equal 4, issue.fixed_version_id
|
Chris@1115
|
192 journal = Journal.first(:order => 'id DESC')
|
Chris@1115
|
193 assert_equal 'overwrite_conflict_resolution', journal.notes
|
Chris@1115
|
194 assert journal.details.any?
|
Chris@1115
|
195 end
|
Chris@1115
|
196
|
Chris@1115
|
197 def test_update_stale_issue_with_add_notes_conflict_resolution_should_update
|
Chris@1115
|
198 @request.session[:user_id] = 2
|
Chris@1115
|
199
|
Chris@1115
|
200 assert_difference 'Journal.count' do
|
Chris@1115
|
201 put :update, :id => 1,
|
Chris@1115
|
202 :issue => {
|
Chris@1115
|
203 :fixed_version_id => 4,
|
Chris@1115
|
204 :notes => 'add_notes_conflict_resolution',
|
Chris@1115
|
205 :lock_version => 2
|
Chris@1115
|
206 },
|
Chris@1115
|
207 :conflict_resolution => 'add_notes'
|
Chris@1115
|
208 end
|
Chris@1115
|
209
|
Chris@1115
|
210 assert_response 302
|
Chris@1115
|
211 issue = Issue.find(1)
|
Chris@1115
|
212 assert_nil issue.fixed_version_id
|
Chris@1115
|
213 journal = Journal.first(:order => 'id DESC')
|
Chris@1115
|
214 assert_equal 'add_notes_conflict_resolution', journal.notes
|
Chris@1115
|
215 assert journal.details.empty?
|
Chris@1115
|
216 end
|
Chris@1115
|
217
|
Chris@1115
|
218 def test_update_stale_issue_with_cancel_conflict_resolution_should_redirect_without_updating
|
Chris@1115
|
219 @request.session[:user_id] = 2
|
Chris@1115
|
220
|
Chris@1115
|
221 assert_no_difference 'Journal.count' do
|
Chris@1115
|
222 put :update, :id => 1,
|
Chris@1115
|
223 :issue => {
|
Chris@1115
|
224 :fixed_version_id => 4,
|
Chris@1115
|
225 :notes => 'add_notes_conflict_resolution',
|
Chris@1115
|
226 :lock_version => 2
|
Chris@1115
|
227 },
|
Chris@1115
|
228 :conflict_resolution => 'cancel'
|
Chris@1115
|
229 end
|
Chris@1115
|
230
|
Chris@1115
|
231 assert_redirected_to '/issues/1'
|
Chris@1115
|
232 issue = Issue.find(1)
|
Chris@1115
|
233 assert_nil issue.fixed_version_id
|
Chris@1115
|
234 end
|
Chris@1115
|
235
|
Chris@1115
|
236 def test_put_update_with_spent_time_and_failure_should_not_add_spent_time
|
Chris@1115
|
237 @request.session[:user_id] = 2
|
Chris@1115
|
238
|
Chris@1115
|
239 assert_no_difference('TimeEntry.count') do
|
Chris@1115
|
240 put :update,
|
Chris@1115
|
241 :id => 1,
|
Chris@1115
|
242 :issue => { :subject => '' },
|
Chris@1115
|
243 :time_entry => { :hours => '2.5', :comments => 'should not be added', :activity_id => TimeEntryActivity.first.id }
|
Chris@1115
|
244 assert_response :success
|
Chris@1115
|
245 end
|
Chris@1115
|
246
|
Chris@1115
|
247 assert_select 'input[name=?][value=?]', 'time_entry[hours]', '2.5'
|
Chris@1115
|
248 assert_select 'input[name=?][value=?]', 'time_entry[comments]', 'should not be added'
|
Chris@1115
|
249 assert_select 'select[name=?]', 'time_entry[activity_id]' do
|
Chris@1115
|
250 assert_select 'option[value=?][selected=selected]', TimeEntryActivity.first.id
|
Chris@1115
|
251 end
|
Chris@1115
|
252 end
|
Chris@1115
|
253
|
Chris@1115
|
254 def test_index_should_rescue_invalid_sql_query
|
Chris@1295
|
255 IssueQuery.any_instance.stubs(:statement).returns("INVALID STATEMENT")
|
Chris@1115
|
256
|
Chris@1115
|
257 get :index
|
Chris@1115
|
258 assert_response 500
|
Chris@1115
|
259 assert_tag 'p', :content => /An error occurred/
|
Chris@1115
|
260 assert_nil session[:query]
|
Chris@1115
|
261 assert_nil session[:issues_index_sort]
|
Chris@0
|
262 end
|
Chris@0
|
263 end
|