Chris@1494: # Redmine - project management software Chris@1494: # Copyright (C) 2006-2014 Jean-Philippe Lang Chris@1494: # Chris@1494: # This program is free software; you can redistribute it and/or Chris@1494: # modify it under the terms of the GNU General Public License Chris@1494: # as published by the Free Software Foundation; either version 2 Chris@1494: # of the License, or (at your option) any later version. Chris@1494: # Chris@1494: # This program is distributed in the hope that it will be useful, Chris@1494: # but WITHOUT ANY WARRANTY; without even the implied warranty of Chris@1494: # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the Chris@1494: # GNU General Public License for more details. Chris@1494: # Chris@1494: # You should have received a copy of the GNU General Public License Chris@1494: # along with this program; if not, write to the Free Software Chris@1494: # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. Chris@1494: Chris@1494: require File.expand_path('../../test_helper', __FILE__) Chris@1494: require 'issues_controller' Chris@1494: Chris@1494: class IssuesControllerTransactionTest < ActionController::TestCase Chris@1494: tests IssuesController Chris@1494: fixtures :projects, Chris@1494: :users, Chris@1494: :roles, Chris@1494: :members, Chris@1494: :member_roles, Chris@1494: :issues, Chris@1494: :issue_statuses, Chris@1494: :versions, Chris@1494: :trackers, Chris@1494: :projects_trackers, Chris@1494: :issue_categories, Chris@1494: :enabled_modules, Chris@1494: :enumerations, Chris@1494: :attachments, Chris@1494: :workflows, Chris@1494: :custom_fields, Chris@1494: :custom_values, Chris@1494: :custom_fields_projects, Chris@1494: :custom_fields_trackers, Chris@1494: :time_entries, Chris@1494: :journals, Chris@1494: :journal_details, Chris@1494: :queries Chris@1494: Chris@1494: self.use_transactional_fixtures = false Chris@1494: Chris@1494: def setup Chris@1494: User.current = nil Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_should_not_update_the_issue Chris@1494: issue = Issue.find(2) Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_no_difference 'Journal.count' do Chris@1494: assert_no_difference 'TimeEntry.count' do Chris@1494: put :update, Chris@1494: :id => issue.id, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => 'My notes', Chris@1494: :lock_version => (issue.lock_version - 1) Chris@1494: }, Chris@1494: :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id } Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: assert_response :success Chris@1494: assert_template 'edit' Chris@1494: Chris@1494: assert_select 'div.conflict' Chris@1494: assert_select 'input[name=?][value=?]', 'conflict_resolution', 'overwrite' Chris@1494: assert_select 'input[name=?][value=?]', 'conflict_resolution', 'add_notes' Chris@1494: assert_select 'label' do Chris@1494: assert_select 'input[name=?][value=?]', 'conflict_resolution', 'cancel' Chris@1494: assert_select 'a[href=/issues/2]' Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_should_save_attachments Chris@1494: set_tmp_attachments_directory Chris@1494: issue = Issue.find(2) Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_no_difference 'Journal.count' do Chris@1494: assert_no_difference 'TimeEntry.count' do Chris@1494: assert_difference 'Attachment.count' do Chris@1494: put :update, Chris@1494: :id => issue.id, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => 'My notes', Chris@1494: :lock_version => (issue.lock_version - 1) Chris@1494: }, Chris@1494: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}, Chris@1494: :time_entry => { :hours => '2.5', :comments => '', :activity_id => TimeEntryActivity.first.id } Chris@1494: end Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: assert_response :success Chris@1494: assert_template 'edit' Chris@1494: attachment = Attachment.first(:order => 'id DESC') Chris@1494: assert_tag 'input', :attributes => {:name => 'attachments[p0][token]', :value => attachment.token} Chris@1494: assert_tag 'input', :attributes => {:name => 'attachments[p0][filename]', :value => 'testfile.txt'} Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_without_notes_should_not_show_add_notes_option Chris@1494: issue = Issue.find(2) Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: put :update, :id => issue.id, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => '', Chris@1494: :lock_version => (issue.lock_version - 1) Chris@1494: } Chris@1494: Chris@1494: assert_tag 'div', :attributes => {:class => 'conflict'} Chris@1494: assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'overwrite'} Chris@1494: assert_no_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'add_notes'} Chris@1494: assert_tag 'input', :attributes => {:name => 'conflict_resolution', :value => 'cancel'} Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_should_show_conflicting_journals Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: put :update, :id => 1, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => '', Chris@1494: :lock_version => 2 Chris@1494: }, Chris@1494: :last_journal_id => 1 Chris@1494: Chris@1494: assert_not_nil assigns(:conflict_journals) Chris@1494: assert_equal 1, assigns(:conflict_journals).size Chris@1494: assert_equal 2, assigns(:conflict_journals).first.id Chris@1494: assert_tag 'div', :attributes => {:class => 'conflict'}, Chris@1494: :descendant => {:content => /Some notes with Redmine links/} Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_without_previous_journal_should_show_all_journals Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: put :update, :id => 1, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => '', Chris@1494: :lock_version => 2 Chris@1494: }, Chris@1494: :last_journal_id => '' Chris@1494: Chris@1494: assert_not_nil assigns(:conflict_journals) Chris@1494: assert_equal 2, assigns(:conflict_journals).size Chris@1494: assert_tag 'div', :attributes => {:class => 'conflict'}, Chris@1494: :descendant => {:content => /Some notes with Redmine links/} Chris@1494: assert_tag 'div', :attributes => {:class => 'conflict'}, Chris@1494: :descendant => {:content => /Journal notes/} Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_should_show_private_journals_with_permission_only Chris@1494: journal = Journal.create!(:journalized => Issue.find(1), :notes => 'Privates notes', :private_notes => true, :user_id => 1) Chris@1494: Chris@1494: @request.session[:user_id] = 2 Chris@1494: put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => '' Chris@1494: assert_include journal, assigns(:conflict_journals) Chris@1494: Chris@1494: Role.find(1).remove_permission! :view_private_notes Chris@1494: put :update, :id => 1, :issue => {:fixed_version_id => 4, :lock_version => 2}, :last_journal_id => '' Chris@1494: assert_not_include journal, assigns(:conflict_journals) Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_with_overwrite_conflict_resolution_should_update Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_difference 'Journal.count' do Chris@1494: put :update, :id => 1, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => 'overwrite_conflict_resolution', Chris@1494: :lock_version => 2 Chris@1494: }, Chris@1494: :conflict_resolution => 'overwrite' Chris@1494: end Chris@1494: Chris@1494: assert_response 302 Chris@1494: issue = Issue.find(1) Chris@1494: assert_equal 4, issue.fixed_version_id Chris@1494: journal = Journal.first(:order => 'id DESC') Chris@1494: assert_equal 'overwrite_conflict_resolution', journal.notes Chris@1494: assert journal.details.any? Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_with_add_notes_conflict_resolution_should_update Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_difference 'Journal.count' do Chris@1494: put :update, :id => 1, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => 'add_notes_conflict_resolution', Chris@1494: :lock_version => 2 Chris@1494: }, Chris@1494: :conflict_resolution => 'add_notes' Chris@1494: end Chris@1494: Chris@1494: assert_response 302 Chris@1494: issue = Issue.find(1) Chris@1494: assert_nil issue.fixed_version_id Chris@1494: journal = Journal.first(:order => 'id DESC') Chris@1494: assert_equal 'add_notes_conflict_resolution', journal.notes Chris@1494: assert journal.details.empty? Chris@1494: end Chris@1494: Chris@1494: def test_update_stale_issue_with_cancel_conflict_resolution_should_redirect_without_updating Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_no_difference 'Journal.count' do Chris@1494: put :update, :id => 1, Chris@1494: :issue => { Chris@1494: :fixed_version_id => 4, Chris@1494: :notes => 'add_notes_conflict_resolution', Chris@1494: :lock_version => 2 Chris@1494: }, Chris@1494: :conflict_resolution => 'cancel' Chris@1494: end Chris@1494: Chris@1494: assert_redirected_to '/issues/1' Chris@1494: issue = Issue.find(1) Chris@1494: assert_nil issue.fixed_version_id Chris@1494: end Chris@1494: Chris@1494: def test_put_update_with_spent_time_and_failure_should_not_add_spent_time Chris@1494: @request.session[:user_id] = 2 Chris@1494: Chris@1494: assert_no_difference('TimeEntry.count') do Chris@1494: put :update, Chris@1494: :id => 1, Chris@1494: :issue => { :subject => '' }, Chris@1494: :time_entry => { :hours => '2.5', :comments => 'should not be added', :activity_id => TimeEntryActivity.first.id } Chris@1494: assert_response :success Chris@1494: end Chris@1494: Chris@1494: assert_select 'input[name=?][value=?]', 'time_entry[hours]', '2.5' Chris@1494: assert_select 'input[name=?][value=?]', 'time_entry[comments]', 'should not be added' Chris@1494: assert_select 'select[name=?]', 'time_entry[activity_id]' do Chris@1494: assert_select 'option[value=?][selected=selected]', TimeEntryActivity.first.id Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: def test_index_should_rescue_invalid_sql_query Chris@1494: IssueQuery.any_instance.stubs(:statement).returns("INVALID STATEMENT") Chris@1494: Chris@1494: get :index Chris@1494: assert_response 500 Chris@1494: assert_tag 'p', :content => /An error occurred/ Chris@1494: assert_nil session[:query] Chris@1494: assert_nil session[:issues_index_sort] Chris@1494: end Chris@1494: end