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