annotate .svn/pristine/0e/0e59eac074a412c3150c3135d0a1eec08baa4fba.svn-base @ 1327:287f201c2802 redmine-2.2-integration

Add italic
author Chris Cannam <chris.cannam@soundsoftware.ac.uk>
date Wed, 19 Jun 2013 20:56:22 +0100
parents 038ba2d95de8
children
rev   line source
Chris@1296 1 # Redmine - project management software
Chris@1296 2 # Copyright (C) 2006-2012 Jean-Philippe Lang
Chris@1296 3 #
Chris@1296 4 # This program is free software; you can redistribute it and/or
Chris@1296 5 # modify it under the terms of the GNU General Public License
Chris@1296 6 # as published by the Free Software Foundation; either version 2
Chris@1296 7 # of the License, or (at your option) any later version.
Chris@1296 8 #
Chris@1296 9 # This program is distributed in the hope that it will be useful,
Chris@1296 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@1296 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@1296 12 # GNU General Public License for more details.
Chris@1296 13 #
Chris@1296 14 # You should have received a copy of the GNU General Public License
Chris@1296 15 # along with this program; if not, write to the Free Software
Chris@1296 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@1296 17
Chris@1296 18 require File.expand_path('../../test_helper', __FILE__)
Chris@1296 19 require 'issues_controller'
Chris@1296 20
Chris@1296 21 class IssuesControllerTransactionTest < ActionController::TestCase
Chris@1296 22 fixtures :projects,
Chris@1296 23 :users,
Chris@1296 24 :roles,
Chris@1296 25 :members,
Chris@1296 26 :member_roles,
Chris@1296 27 :issues,
Chris@1296 28 :issue_statuses,
Chris@1296 29 :versions,
Chris@1296 30 :trackers,
Chris@1296 31 :projects_trackers,
Chris@1296 32 :issue_categories,
Chris@1296 33 :enabled_modules,
Chris@1296 34 :enumerations,
Chris@1296 35 :attachments,
Chris@1296 36 :workflows,
Chris@1296 37 :custom_fields,
Chris@1296 38 :custom_values,
Chris@1296 39 :custom_fields_projects,
Chris@1296 40 :custom_fields_trackers,
Chris@1296 41 :time_entries,
Chris@1296 42 :journals,
Chris@1296 43 :journal_details,
Chris@1296 44 :queries
Chris@1296 45
Chris@1296 46 self.use_transactional_fixtures = false
Chris@1296 47
Chris@1296 48 def setup
Chris@1296 49 @controller = IssuesController.new
Chris@1296 50 @request = ActionController::TestRequest.new
Chris@1296 51 @response = ActionController::TestResponse.new
Chris@1296 52 User.current = nil
Chris@1296 53 end
Chris@1296 54
Chris@1296 55 def test_update_stale_issue_should_not_update_the_issue
Chris@1296 56 issue = Issue.find(2)
Chris@1296 57 @request.session[:user_id] = 2
Chris@1296 58
Chris@1296 59 assert_no_difference 'Journal.count' do
Chris@1296 60 assert_no_difference 'TimeEntry.count' do
Chris@1296 61 put :update,
Chris@1296 62 :id => issue.id,
Chris@1296 63 :issue => {
Chris@1296 64 :fixed_version_id => 4,
Chris@1296 65 :notes => 'My notes',
Chris@1296 66 :lock_version => (issue.lock_version - 1)
Chris@1296 67 },
Chris@1296 68 :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id }
Chris@1296 69 end
Chris@1296 70 end
Chris@1296 71
Chris@1296 72 assert_response :success
Chris@1296 73 assert_template 'edit'
Chris@1296 74
Chris@1296 75 assert_select 'div.conflict'
Chris@1296 76 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'overwrite'
Chris@1296 77 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'add_notes'
Chris@1296 78 assert_select 'label' do
Chris@1296 79 assert_select 'input[name=?][value=?]', 'conflict_resolution', 'cancel'
Chris@1296 80 assert_select 'a[href=/issues/2]'
Chris@1296 81 end
Chris@1296 82 end
Chris@1296 83
Chris@1296 84 def test_update_stale_issue_should_save_attachments
Chris@1296 85 set_tmp_attachments_directory
Chris@1296 86 issue = Issue.find(2)
Chris@1296 87 @request.session[:user_id] = 2
Chris@1296 88
Chris@1296 89 assert_no_difference 'Journal.count' do
Chris@1296 90 assert_no_difference 'TimeEntry.count' do
Chris@1296 91 assert_difference 'Attachment.count' do
Chris@1296 92 put :update,
Chris@1296 93 :id => issue.id,
Chris@1296 94 :issue => {
Chris@1296 95 :fixed_version_id => 4,
Chris@1296 96 :notes => 'My notes',
Chris@1296 97 :lock_version => (issue.lock_version - 1)
Chris@1296 98 },
Chris@1296 99 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}},
Chris@1296 100 :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id }
Chris@1296 101 end
Chris@1296 102 end
Chris@1296 103 end
Chris@1296 104
Chris@1296 105 assert_response :success
Chris@1296 106 assert_template 'edit'
Chris@1296 107 attachment = Attachment.first(:order => 'id DESC')
Chris@1296 108 assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token}
Chris@1296 109 assert_tag 'span', :content => /testfile.txt/
Chris@1296 110 end
Chris@1296 111
Chris@1296 112 def test_update_stale_issue_without_notes_should_not_show_add_notes_option
Chris@1296 113 issue = Issue.find(2)
Chris@1296 114 @request.session[:user_id] = 2
Chris@1296 115
Chris@1296 116 put :update, :id => issue.id,
Chris@1296 117 :issue => {
Chris@1296 118 :fixed_version_id => 4,
Chris@1296 119 :notes => '',
Chris@1296 120 :lock_version => (issue.lock_version - 1)
Chris@1296 121 }
Chris@1296 122
Chris@1296 123 assert_tag 'div', :attributes => {:class => 'conflict'}
Chris@1296 124 assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'overwrite'}
Chris@1296 125 assert_no_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'add_notes'}
Chris@1296 126 assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'cancel'}
Chris@1296 127 end
Chris@1296 128
Chris@1296 129 def test_update_stale_issue_should_show_conflicting_journals
Chris@1296 130 @request.session[:user_id] = 2
Chris@1296 131
Chris@1296 132 put :update, :id => 1,
Chris@1296 133 :issue => {
Chris@1296 134 :fixed_version_id => 4,
Chris@1296 135 :notes => '',
Chris@1296 136 :lock_version => 2
Chris@1296 137 },
Chris@1296 138 :last_journal_id => 1
Chris@1296 139
Chris@1296 140 assert_not_nil assigns(:conflict_journals)
Chris@1296 141 assert_equal 1, assigns(:conflict_journals).size
Chris@1296 142 assert_equal 2, assigns(:conflict_journals).first.id
Chris@1296 143 assert_tag 'div', :attributes => {:class => 'conflict'},
Chris@1296 144 :descendant => {:content => /Some notes with Redmine links/}
Chris@1296 145 end
Chris@1296 146
Chris@1296 147 def test_update_stale_issue_without_previous_journal_should_show_all_journals
Chris@1296 148 @request.session[:user_id] = 2
Chris@1296 149
Chris@1296 150 put :update, :id => 1,
Chris@1296 151 :issue => {
Chris@1296 152 :fixed_version_id => 4,
Chris@1296 153 :notes => '',
Chris@1296 154 :lock_version => 2
Chris@1296 155 },
Chris@1296 156 :last_journal_id => ''
Chris@1296 157
Chris@1296 158 assert_not_nil assigns(:conflict_journals)
Chris@1296 159 assert_equal 2, assigns(:conflict_journals).size
Chris@1296 160 assert_tag 'div', :attributes => {:class => 'conflict'},
Chris@1296 161 :descendant => {:content => /Some notes with Redmine links/}
Chris@1296 162 assert_tag 'div', :attributes => {:class => 'conflict'},
Chris@1296 163 :descendant => {:content => /Journal notes/}
Chris@1296 164 end
Chris@1296 165
Chris@1296 166 def test_update_stale_issue_should_show_private_journals_with_permission_only
Chris@1296 167 journal = Journal.create!(:journalized => Issue.find(1), :notes => 'Privates notes', :private_notes => true, :user_id => 1)
Chris@1296 168
Chris@1296 169 @request.session[:user_id] = 2
Chris@1296 170 put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => ''
Chris@1296 171 assert_include journal, assigns(:conflict_journals)
Chris@1296 172
Chris@1296 173 Role.find(1).remove_permission! :view_private_notes
Chris@1296 174 put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => ''
Chris@1296 175 assert_not_include journal, assigns(:conflict_journals)
Chris@1296 176 end
Chris@1296 177
Chris@1296 178 def test_update_stale_issue_with_overwrite_conflict_resolution_should_update
Chris@1296 179 @request.session[:user_id] = 2
Chris@1296 180
Chris@1296 181 assert_difference 'Journal.count' do
Chris@1296 182 put :update, :id => 1,
Chris@1296 183 :issue => {
Chris@1296 184 :fixed_version_id => 4,
Chris@1296 185 :notes => 'overwrite_conflict_resolution',
Chris@1296 186 :lock_version => 2
Chris@1296 187 },
Chris@1296 188 :conflict_resolution => 'overwrite'
Chris@1296 189 end
Chris@1296 190
Chris@1296 191 assert_response 302
Chris@1296 192 issue = Issue.find(1)
Chris@1296 193 assert_equal 4, issue.fixed_version_id
Chris@1296 194 journal = Journal.first(:order => 'id DESC')
Chris@1296 195 assert_equal 'overwrite_conflict_resolution', journal.notes
Chris@1296 196 assert journal.details.any?
Chris@1296 197 end
Chris@1296 198
Chris@1296 199 def test_update_stale_issue_with_add_notes_conflict_resolution_should_update
Chris@1296 200 @request.session[:user_id] = 2
Chris@1296 201
Chris@1296 202 assert_difference 'Journal.count' do
Chris@1296 203 put :update, :id => 1,
Chris@1296 204 :issue => {
Chris@1296 205 :fixed_version_id => 4,
Chris@1296 206 :notes => 'add_notes_conflict_resolution',
Chris@1296 207 :lock_version => 2
Chris@1296 208 },
Chris@1296 209 :conflict_resolution => 'add_notes'
Chris@1296 210 end
Chris@1296 211
Chris@1296 212 assert_response 302
Chris@1296 213 issue = Issue.find(1)
Chris@1296 214 assert_nil issue.fixed_version_id
Chris@1296 215 journal = Journal.first(:order => 'id DESC')
Chris@1296 216 assert_equal 'add_notes_conflict_resolution', journal.notes
Chris@1296 217 assert journal.details.empty?
Chris@1296 218 end
Chris@1296 219
Chris@1296 220 def test_update_stale_issue_with_cancel_conflict_resolution_should_redirect_without_updating
Chris@1296 221 @request.session[:user_id] = 2
Chris@1296 222
Chris@1296 223 assert_no_difference 'Journal.count' do
Chris@1296 224 put :update, :id => 1,
Chris@1296 225 :issue => {
Chris@1296 226 :fixed_version_id => 4,
Chris@1296 227 :notes => 'add_notes_conflict_resolution',
Chris@1296 228 :lock_version => 2
Chris@1296 229 },
Chris@1296 230 :conflict_resolution => 'cancel'
Chris@1296 231 end
Chris@1296 232
Chris@1296 233 assert_redirected_to '/issues/1'
Chris@1296 234 issue = Issue.find(1)
Chris@1296 235 assert_nil issue.fixed_version_id
Chris@1296 236 end
Chris@1296 237
Chris@1296 238 def test_put_update_with_spent_time_and_failure_should_not_add_spent_time
Chris@1296 239 @request.session[:user_id] = 2
Chris@1296 240
Chris@1296 241 assert_no_difference('TimeEntry.count') do
Chris@1296 242 put :update,
Chris@1296 243 :id => 1,
Chris@1296 244 :issue => { :subject => '' },
Chris@1296 245 :time_entry => { :hours => '2.5', :comments => 'should not be added', :activity_id => TimeEntryActivity.first.id }
Chris@1296 246 assert_response :success
Chris@1296 247 end
Chris@1296 248
Chris@1296 249 assert_select 'input[name=?][value=?]', 'time_entry[hours]', '2.5'
Chris@1296 250 assert_select 'input[name=?][value=?]', 'time_entry[comments]', 'should not be added'
Chris@1296 251 assert_select 'select[name=?]', 'time_entry[activity_id]' do
Chris@1296 252 assert_select 'option[value=?][selected=selected]', TimeEntryActivity.first.id
Chris@1296 253 end
Chris@1296 254 end
Chris@1296 255
Chris@1296 256 def test_index_should_rescue_invalid_sql_query
Chris@1296 257 Query.any_instance.stubs(:statement).returns("INVALID STATEMENT")
Chris@1296 258
Chris@1296 259 get :index
Chris@1296 260 assert_response 500
Chris@1296 261 assert_tag 'p', :content => /An error occurred/
Chris@1296 262 assert_nil session[:query]
Chris@1296 263 assert_nil session[:issues_index_sort]
Chris@1296 264 end
Chris@1296 265 end