/test/functional/issues_controller_test.rb - Annotate - SoundSoftware Code Site

To check out this repository please hg clone the following URL, or open the URL using EasyMercurial or your preferred Mercurial client.

Mercurial

Statistics Download as Zip

root / test / functional / issues_controller_test.rb @ 441:cbce1fd3b1b7

History | View | Annotate | Download (50.3 KB)

-:513646585e45
+# Redmine - project management software
-:cbce1fd3b1b7
+# Copyright (C) 2006-2011  Jean-Philippe Lang
-:513646585e45
+            Chris
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
 # as published by the Free Software Foundation; either version 2
 # of the License, or (at your option) any later version.
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+# This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+# You should have received a copy of the GNU General Public License
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
-:8661b858af72
+require File.expand_path('../../test_helper', __FILE__)
-:513646585e45
+require 'issues_controller'
 # Re-raise errors caught by the controller.
 class IssuesController; def rescue_action(e) raise e end; end
 class IssuesControllerTest < ActionController::TestCase
   fixtures :projects,
            :users,
            :roles,
            :members,
            :member_roles,
            :issues,
            :issue_statuses,
            :versions,
            :trackers,
            :projects_trackers,
            :issue_categories,
            :enabled_modules,
            :enumerations,
            :attachments,
            :workflows,
            :custom_fields,
            :custom_values,
            :custom_fields_projects,
            :custom_fields_trackers,
            :time_entries,
            :journals,
            :journal_details,
            :queries
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def setup
     @controller = IssuesController.new
     @request    = ActionController::TestRequest.new
     @response   = ActionController::TestResponse.new
     User.current = nil
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index
     Setting.default_language = 'en'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :index
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_nil assigns(:project)
     assert_tag :tag => 'a', :content => /Can't print recipes/
     assert_tag :tag => 'a', :content => /Subproject issue/
     # private projects hidden
     assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
     assert_no_tag :tag => 'a', :content => /Issue on project 2/
     # project column
     assert_tag :tag => 'th', :content => /Project/
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_should_not_list_issues_when_module_disabled
     EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
     get :index
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_nil assigns(:project)
     assert_no_tag :tag => 'a', :content => /Can't print recipes/
     assert_tag :tag => 'a', :content => /Subproject issue/
   end
   def test_index_should_not_list_issues_when_module_disabled
     EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
     get :index
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_nil assigns(:project)
     assert_no_tag :tag => 'a', :content => /Can't print recipes/
     assert_tag :tag => 'a', :content => /Subproject issue/
   end
-:cbce1fd3b1b7
+            Chris
   def test_index_should_list_visible_issues_only
     get :index, :per_page => 100
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_nil assigns(:issues).detect {|issue| !issue.visible?}
   end
-:513646585e45
+  def test_index_with_project
     Setting.display_subprojects_issues = 0
     get :index, :project_id => 1
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_tag :tag => 'a', :content => /Can't print recipes/
     assert_no_tag :tag => 'a', :content => /Subproject issue/
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_project_and_subprojects
     Setting.display_subprojects_issues = 1
     get :index, :project_id => 1
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_tag :tag => 'a', :content => /Can't print recipes/
     assert_tag :tag => 'a', :content => /Subproject issue/
     assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_project_and_subprojects_should_show_private_subprojects
     @request.session[:user_id] = 2
     Setting.display_subprojects_issues = 1
     get :index, :project_id => 1
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_tag :tag => 'a', :content => /Can't print recipes/
     assert_tag :tag => 'a', :content => /Subproject issue/
     assert_tag :tag => 'a', :content => /Issue of a private subproject/
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  def test_index_with_project_and_default_filter
-:513646585e45
+    get :index, :project_id => 1, :set_filter => 1
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    query = assigns(:query)
     assert_not_nil query
     # default filter
     assert_equal({'status_id' => {:operator => 'o', :values => ['']}}, query.filters)
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  def test_index_with_project_and_filter
-:cbce1fd3b1b7
+    get :index, :project_id => 1, :set_filter => 1,
       :f => ['tracker_id'],
       :op => {'tracker_id' => '='},
       :v => {'tracker_id' => ['1']}
-:94944d00e43c
+    assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    query = assigns(:query)
     assert_not_nil query
     assert_equal({'tracker_id' => {:operator => '=', :values => ['1']}}, query.filters)
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  def test_index_with_project_and_empty_filters
     get :index, :project_id => 1, :set_filter => 1, :fields => ['']
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    query = assigns(:query)
     assert_not_nil query
     # no filter
     assert_equal({}, query.filters)
-:513646585e45
+  end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_query
     get :index, :project_id => 1, :query_id => 5
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_nil assigns(:issue_count_by_group)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_query_grouped_by_tracker
     get :index, :project_id => 1, :query_id => 6
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_not_nil assigns(:issue_count_by_group)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_query_grouped_by_list_custom_field
     get :index, :project_id => 1, :query_id => 9
     assert_response :success
     assert_template 'index.rhtml'
     assert_not_nil assigns(:issues)
     assert_not_nil assigns(:issue_count_by_group)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_sort_by_field_not_included_in_columns
     Setting.issue_list_default_columns = %w(subject author)
     get :index, :sort => 'tracker'
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_csv_with_project
     Setting.default_language = 'en'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :index, :format => 'csv'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_equal 'text/csv', @response.content_type
     assert @response.body.starts_with?("#,")
     get :index, :project_id => 1, :format => 'csv'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_equal 'text/csv', @response.content_type
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_pdf
     get :index, :format => 'pdf'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_equal 'application/pdf', @response.content_type
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :index, :project_id => 1, :format => 'pdf'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_equal 'application/pdf', @response.content_type
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_equal 'application/pdf', @response.content_type
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_pdf_with_query_grouped_by_list_custom_field
     get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
     assert_response :success
     assert_not_nil assigns(:issues)
     assert_not_nil assigns(:issue_count_by_group)
     assert_equal 'application/pdf', @response.content_type
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_sort
     get :index, :sort => 'tracker,id:desc'
     assert_response :success
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    sort_params = @request.session['issues_index_sort']
     assert sort_params.is_a?(String)
     assert_equal 'tracker,id:desc', sort_params
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issues = assigns(:issues)
     assert_not_nil issues
     assert !issues.empty?
     assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_index_with_columns
     columns = ['tracker', 'subject', 'assigned_to']
-:cbce1fd3b1b7
+    get :index, :set_filter => 1, :c => columns
-:513646585e45
+    assert_response :success
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # query should use specified columns
     query = assigns(:query)
     assert_kind_of Query, query
     assert_equal columns, query.column_names.map(&:to_s)
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # columns should be stored in session
     assert_kind_of Hash, session[:query]
     assert_kind_of Array, session[:query][:column_names]
     assert_equal columns, session[:query][:column_names].map(&:to_s)
   end
-:cbce1fd3b1b7
+  def test_index_with_custom_field_column
     columns = %w(tracker subject cf_2)
     get :index, :set_filter => 1, :c => columns
     assert_response :success
     # query should use specified columns
     query = assigns(:query)
     assert_kind_of Query, query
     assert_equal columns, query.column_names.map(&:to_s)
     assert_tag :td,
       :attributes => {:class => 'cf_2 string'},
       :ancestor => {:tag => 'table', :attributes => {:class => /issues/}}
   end
-:513646585e45
+  def test_show_by_anonymous
     get :show, :id => 1
     assert_response :success
     assert_template 'show.rhtml'
     assert_not_nil assigns(:issue)
     assert_equal Issue.find(1), assigns(:issue)
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # anonymous role is allowed to add a note
     assert_tag :tag => 'form',
                :descendant => { :tag => 'fieldset',
-:cbce1fd3b1b7
+                                :child => { :tag => 'legend',
-:513646585e45
+                                            :content => /Notes/ } }
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_show_by_manager
     @request.session[:user_id] = 2
     get :show, :id => 1
     assert_response :success
-:cbce1fd3b1b7
+            Chris
-:8661b858af72
+    assert_tag :tag => 'a',
       :content => /Quote/
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_tag :tag => 'form',
                :descendant => { :tag => 'fieldset',
-:cbce1fd3b1b7
+                                :child => { :tag => 'legend',
-:513646585e45
+                                            :content => /Change properties/ } },
                :descendant => { :tag => 'fieldset',
-:cbce1fd3b1b7
+                                :child => { :tag => 'legend',
-:513646585e45
+                                            :content => /Log time/ } },
                :descendant => { :tag => 'fieldset',
-:cbce1fd3b1b7
+                                :child => { :tag => 'legend',
-:513646585e45
+                                            :content => /Notes/ } }
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_show_should_deny_anonymous_access_without_permission
     Role.anonymous.remove_permission!(:view_issues)
     get :show, :id => 1
     assert_response :redirect
   end
-:cbce1fd3b1b7
+            Chris
   def test_show_should_deny_anonymous_access_to_private_issue
     Issue.update_all(["is_private = ?", true], "id = 1")
     get :show, :id => 1
     assert_response :redirect
   end
-:513646585e45
+  def test_show_should_deny_non_member_access_without_permission
     Role.non_member.remove_permission!(:view_issues)
     @request.session[:user_id] = 9
     get :show, :id => 1
     assert_response 403
   end
-:cbce1fd3b1b7
+            Chris
   def test_show_should_deny_non_member_access_to_private_issue
     Issue.update_all(["is_private = ?", true], "id = 1")
     @request.session[:user_id] = 9
     get :show, :id => 1
     assert_response 403
   end
-:513646585e45
+  def test_show_should_deny_member_access_without_permission
     Role.find(1).remove_permission!(:view_issues)
     @request.session[:user_id] = 2
     get :show, :id => 1
     assert_response 403
   end
-:cbce1fd3b1b7
+            Chris
   def test_show_should_deny_member_access_to_private_issue_without_permission
     Issue.update_all(["is_private = ?", true], "id = 1")
     @request.session[:user_id] = 3
     get :show, :id => 1
     assert_response 403
   end
   def test_show_should_allow_author_access_to_private_issue
     Issue.update_all(["is_private = ?, author_id = 3", true], "id = 1")
     @request.session[:user_id] = 3
     get :show, :id => 1
     assert_response :success
   end
   def test_show_should_allow_assignee_access_to_private_issue
     Issue.update_all(["is_private = ?, assigned_to_id = 3", true], "id = 1")
     @request.session[:user_id] = 3
     get :show, :id => 1
     assert_response :success
   end
   def test_show_should_allow_member_access_to_private_issue_with_permission
     Issue.update_all(["is_private = ?", true], "id = 1")
     User.find(3).roles_for_project(Project.find(1)).first.update_attribute :issues_visibility, 'all'
     @request.session[:user_id] = 3
     get :show, :id => 1
     assert_response :success
   end
-:513646585e45
+  def test_show_should_not_disclose_relations_to_invisible_issues
     Setting.cross_project_issue_relations = '1'
     IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
     # Relation to a private project issue
     IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :show, :id => 1
     assert_response :success
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_tag :div, :attributes => { :id => 'relations' },
                      :descendant => { :tag => 'a', :content => /#2$/ }
     assert_no_tag :div, :attributes => { :id => 'relations' },
                         :descendant => { :tag => 'a', :content => /#4$/ }
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_show_atom
     get :show, :id => 2, :format => 'atom'
     assert_response :success
-:1d32c0a0efbf
+    assert_template 'journals/index.rxml'
-:513646585e45
+    # Inline image
     assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_show_export_to_pdf
     get :show, :id => 3, :format => 'pdf'
     assert_response :success
     assert_equal 'application/pdf', @response.content_type
     assert @response.body.starts_with?('%PDF')
     assert_not_nil assigns(:issue)
   end
   def test_get_new
     @request.session[:user_id] = 2
     get :new, :project_id => 1, :tracker_id => 1
     assert_response :success
     assert_template 'new'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]',
                                                  :value => 'Default string' }
   end
   def test_get_new_without_tracker_id
     @request.session[:user_id] = 2
     get :new, :project_id => 1
     assert_response :success
     assert_template 'new'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = assigns(:issue)
     assert_not_nil issue
     assert_equal Project.find(1).trackers.first, issue.tracker
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_get_new_with_no_default_status_should_display_an_error
     @request.session[:user_id] = 2
     IssueStatus.delete_all
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :new, :project_id => 1
     assert_response 500
-:94944d00e43c
+    assert_error_tag :content => /No default issue/
-:513646585e45
+  end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_get_new_with_no_tracker_should_display_an_error
     @request.session[:user_id] = 2
     Tracker.delete_all
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    get :new, :project_id => 1
     assert_response 500
-:94944d00e43c
+    assert_error_tag :content => /No tracker/
-:513646585e45
+  end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_update_new_form
     @request.session[:user_id] = 2
-:1d32c0a0efbf
+    xhr :post, :new, :project_id => 1,
-:cbce1fd3b1b7
+                     :issue => {:tracker_id => 2,
-:513646585e45
+                                :subject => 'This is the test_new issue',
                                 :description => 'This is the description',
                                 :priority_id => 5}
     assert_response :success
     assert_template 'attributes'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = assigns(:issue)
     assert_kind_of Issue, issue
     assert_equal 1, issue.project_id
     assert_equal 2, issue.tracker_id
     assert_equal 'This is the test_new issue', issue.subject
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create
     @request.session[:user_id] = 2
     assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:513646585e45
+                 :issue => {:tracker_id => 3,
                             :status_id => 2,
                             :subject => 'This is the test_new issue',
                             :description => 'This is the description',
                             :priority_id => 5,
-:94944d00e43c
+                            :start_date => '2010-11-07',
-:513646585e45
+                            :estimated_hours => '',
                             :custom_field_values => {'2' => 'Value for field 2'}}
     end
     assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = Issue.find_by_subject('This is the test_new issue')
     assert_not_nil issue
     assert_equal 2, issue.author_id
     assert_equal 3, issue.tracker_id
     assert_equal 2, issue.status_id
-:94944d00e43c
+    assert_equal Date.parse('2010-11-07'), issue.start_date
-:513646585e45
+    assert_nil issue.estimated_hours
     v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
     assert_not_nil v
     assert_equal 'Value for field 2', v.value
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  def test_post_create_without_start_date
     @request.session[:user_id] = 2
     assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:94944d00e43c
+                 :issue => {:tracker_id => 3,
                             :status_id => 2,
                             :subject => 'This is the test_new issue',
                             :description => 'This is the description',
                             :priority_id => 5,
                             :start_date => '',
                             :estimated_hours => '',
                             :custom_field_values => {'2' => 'Value for field 2'}}
     end
     assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    issue = Issue.find_by_subject('This is the test_new issue')
     assert_not_nil issue
     assert_nil issue.start_date
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_and_continue
     @request.session[:user_id] = 2
-:cbce1fd3b1b7
+    post :create, :project_id => 1,
-:513646585e45
+               :issue => {:tracker_id => 3,
                           :subject => 'This is first issue',
                           :priority_id => 5},
                :continue => ''
-:40f7cfd4df19
+    assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook',
                          :issue => {:tracker_id => 3}
-:513646585e45
+  end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_without_custom_fields_param
     @request.session[:user_id] = 2
     assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:513646585e45
+                 :issue => {:tracker_id => 1,
                             :subject => 'This is the test_new issue',
                             :description => 'This is the description',
                             :priority_id => 5}
     end
     assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
   end
   def test_post_create_with_required_custom_field_and_without_custom_fields_param
     field = IssueCustomField.find_by_name('Database')
     field.update_attribute(:is_required, true)
     @request.session[:user_id] = 2
-:cbce1fd3b1b7
+    post :create, :project_id => 1,
-:513646585e45
+               :issue => {:tracker_id => 1,
                           :subject => 'This is the test_new issue',
                           :description => 'This is the description',
                           :priority_id => 5}
     assert_response :success
     assert_template 'new'
     issue = assigns(:issue)
     assert_not_nil issue
     assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_with_watchers
     @request.session[:user_id] = 2
     ActionMailer::Base.deliveries.clear
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_difference 'Watcher.count', 2 do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:513646585e45
+                 :issue => {:tracker_id => 1,
                             :subject => 'This is a new issue with watchers',
                             :description => 'This is the description',
                             :priority_id => 5,
                             :watcher_user_ids => ['2', '3']}
     end
     issue = Issue.find_by_subject('This is a new issue with watchers')
     assert_not_nil issue
     assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # Watchers added
     assert_equal [2, 3], issue.watcher_user_ids.sort
     assert issue.watched_by?(User.find(3))
     # Watchers notified
     mail = ActionMailer::Base.deliveries.last
     assert_kind_of TMail::Mail, mail
     assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_subissue
     @request.session[:user_id] = 2
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:513646585e45
+                 :issue => {:tracker_id => 1,
                             :subject => 'This is a child issue',
                             :parent_issue_id => 2}
     end
     issue = Issue.find_by_subject('This is a child issue')
     assert_not_nil issue
     assert_equal Issue.find(2), issue.parent
   end
-:8661b858af72
+            Chris
   def test_post_create_subissue_with_non_numeric_parent_id
     @request.session[:user_id] = 2
-:cbce1fd3b1b7
+            Chris
-:8661b858af72
+    assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:8661b858af72
+                 :issue => {:tracker_id => 1,
                             :subject => 'This is a child issue',
                             :parent_issue_id => 'ABC'}
     end
     issue = Issue.find_by_subject('This is a child issue')
     assert_not_nil issue
     assert_nil issue.parent
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_should_send_a_notification
     ActionMailer::Base.deliveries.clear
     @request.session[:user_id] = 2
     assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+      post :create, :project_id => 1,
-:513646585e45
+                 :issue => {:tracker_id => 3,
                             :subject => 'This is the test_new issue',
                             :description => 'This is the description',
                             :priority_id => 5,
                             :estimated_hours => '',
                             :custom_field_values => {'2' => 'Value for field 2'}}
     end
     assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_equal 1, ActionMailer::Base.deliveries.size
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_should_preserve_fields_values_on_validation_failure
     @request.session[:user_id] = 2
-:cbce1fd3b1b7
+    post :create, :project_id => 1,
-:513646585e45
+               :issue => {:tracker_id => 1,
                           # empty subject
                           :subject => '',
                           :description => 'This is a description',
                           :priority_id => 6,
                           :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
     assert_response :success
     assert_template 'new'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_tag :textarea, :attributes => { :name => 'issue[description]' },
                           :content => 'This is a description'
     assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
                         :child => { :tag => 'option', :attributes => { :selected => 'selected',
                                                                        :value => '6' },
-:cbce1fd3b1b7
+                                                      :content => 'High' }
-:513646585e45
+    # Custom fields
     assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
                         :child => { :tag => 'option', :attributes => { :selected => 'selected',
                                                                        :value => 'Oracle' },
-:cbce1fd3b1b7
+                                                      :content => 'Oracle' }
-:513646585e45
+    assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
                                         :value => 'Value for field 2'}
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_post_create_should_ignore_non_safe_attributes
     @request.session[:user_id] = 2
     assert_nothing_raised do
       post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
     end
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  context "without workflow privilege" do
     setup do
       Workflow.delete_all(["role_id = ?", Role.anonymous.id])
-:94944d00e43c
+      Role.anonymous.add_permission! :add_issues, :add_issue_notes
-:513646585e45
+    end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    context "#new" do
       should "propose default status only" do
         get :new, :project_id => 1
         assert_response :success
         assert_template 'new'
         assert_tag :tag => 'select',
           :attributes => {:name => 'issue[status_id]'},
           :children => {:count => 1},
           :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}}
       end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+      should "accept default status" do
         assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+          post :create, :project_id => 1,
-:513646585e45
+                     :issue => {:tracker_id => 1,
                                 :subject => 'This is an issue',
                                 :status_id => 1}
         end
         issue = Issue.last(:order => 'id')
         assert_equal IssueStatus.default, issue.status
       end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+      should "ignore unauthorized status" do
         assert_difference 'Issue.count' do
-:cbce1fd3b1b7
+          post :create, :project_id => 1,
-:513646585e45
+                     :issue => {:tracker_id => 1,
                                 :subject => 'This is an issue',
                                 :status_id => 3}
         end
         issue = Issue.last(:order => 'id')
         assert_equal IssueStatus.default, issue.status
       end
     end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    context "#update" do
       should "ignore status change" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
         end
         assert_equal 1, Issue.find(1).status_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "ignore attributes changes" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
         end
         issue = Issue.find(1)
         assert_equal "Can't print recipes", issue.subject
         assert_nil issue.assigned_to
       end
     end
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  context "with workflow privilege" do
     setup do
       Workflow.delete_all(["role_id = ?", Role.anonymous.id])
       Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3)
       Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4)
       Role.anonymous.add_permission! :add_issues, :add_issue_notes
     end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    context "#update" do
       should "accept authorized status" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
         end
         assert_equal 3, Issue.find(1).status_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "ignore unauthorized status" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
         end
         assert_equal 1, Issue.find(1).status_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "accept authorized attributes changes" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:assigned_to_id => 2}
         end
         issue = Issue.find(1)
         assert_equal 2, issue.assigned_to_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "ignore unauthorized attributes changes" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed'}
         end
         issue = Issue.find(1)
         assert_equal "Can't print recipes", issue.subject
       end
     end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    context "and :edit_issues permission" do
       setup do
         Role.anonymous.add_permission! :add_issues, :edit_issues
       end
       should "accept authorized status" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
         end
         assert_equal 3, Issue.find(1).status_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "ignore unauthorized status" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
         end
         assert_equal 1, Issue.find(1).status_id
       end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+      should "accept authorized attributes changes" do
         assert_difference 'Journal.count' do
           put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
         end
         issue = Issue.find(1)
         assert_equal "changed", issue.subject
         assert_equal 2, issue.assigned_to_id
       end
     end
-:513646585e45
+  end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_copy_issue
     @request.session[:user_id] = 2
     get :new, :project_id => 1, :copy_from => 1
     assert_template 'new'
     assert_not_nil assigns(:issue)
     orig = Issue.find(1)
     assert_equal orig.subject, assigns(:issue).subject
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_get_edit
     @request.session[:user_id] = 2
     get :edit, :id => 1
     assert_response :success
     assert_template 'edit'
     assert_not_nil assigns(:issue)
     assert_equal Issue.find(1), assigns(:issue)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_get_edit_with_params
     @request.session[:user_id] = 2
-:94944d00e43c
+    get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 },
         :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => TimeEntryActivity.first.id }
-:513646585e45
+    assert_response :success
     assert_template 'edit'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = assigns(:issue)
     assert_not_nil issue
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_equal 5, issue.status_id
     assert_tag :select, :attributes => { :name => 'issue[status_id]' },
-:cbce1fd3b1b7
+                        :child => { :tag => 'option',
-:513646585e45
+                                    :content => 'Closed',
                                     :attributes => { :selected => 'selected' } }
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_equal 7, issue.priority_id
     assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
-:cbce1fd3b1b7
+                        :child => { :tag => 'option',
-:513646585e45
+                                    :content => 'Urgent',
                                     :attributes => { :selected => 'selected' } }
-:94944d00e43c
+            chris
     assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => '2.5' }
     assert_tag :select, :attributes => { :name => 'time_entry[activity_id]' },
                         :child => { :tag => 'option',
                                     :attributes => { :selected => 'selected', :value => TimeEntryActivity.first.id } }
     assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => 'test_get_edit_with_params' }
-:513646585e45
+  end
   def test_update_edit_form
     @request.session[:user_id] = 2
-:1d32c0a0efbf
+    xhr :post, :new, :project_id => 1,
-:513646585e45
+                             :id => 1,
-:cbce1fd3b1b7
+                             :issue => {:tracker_id => 2,
-:513646585e45
+                                        :subject => 'This is the test_new issue',
                                         :description => 'This is the description',
                                         :priority_id => 5}
     assert_response :success
     assert_template 'attributes'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = assigns(:issue)
     assert_kind_of Issue, issue
     assert_equal 1, issue.id
     assert_equal 1, issue.project_id
     assert_equal 2, issue.tracker_id
     assert_equal 'This is the test_new issue', issue.subject
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_update_using_invalid_http_verbs
     @request.session[:user_id] = 2
     subject = 'Updated by an invalid http verb'
     get :update, :id => 1, :issue => {:subject => subject}
     assert_not_equal subject, Issue.find(1).subject
     post :update, :id => 1, :issue => {:subject => subject}
     assert_not_equal subject, Issue.find(1).subject
     delete :update, :id => 1, :issue => {:subject => subject}
     assert_not_equal subject, Issue.find(1).subject
   end
   def test_put_update_without_custom_fields_param
     @request.session[:user_id] = 2
     ActionMailer::Base.deliveries.clear
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = Issue.find(1)
     assert_equal '125', issue.custom_value_for(2).value
     old_subject = issue.subject
     new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_difference('Journal.count') do
       assert_difference('JournalDetail.count', 2) do
         put :update, :id => 1, :issue => {:subject => new_subject,
                                          :priority_id => '6',
                                          :category_id => '1' # no change
+                                        }
       end
     end
     assert_redirected_to :action => 'show', :id => '1'
     issue.reload
     assert_equal new_subject, issue.subject
     # Make sure custom fields were not cleared
     assert_equal '125', issue.custom_value_for(2).value
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    mail = ActionMailer::Base.deliveries.last
     assert_kind_of TMail::Mail, mail
     assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
     assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}")
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_with_custom_field_change
     @request.session[:user_id] = 2
     issue = Issue.find(1)
     assert_equal '125', issue.custom_value_for(2).value
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_difference('Journal.count') do
       assert_difference('JournalDetail.count', 3) do
         put :update, :id => 1, :issue => {:subject => 'Custom field change',
                                          :priority_id => '6',
                                          :category_id => '1', # no change
                                          :custom_field_values => { '2' => 'New custom value' }
+                                        }
       end
     end
     assert_redirected_to :action => 'show', :id => '1'
     issue.reload
     assert_equal 'New custom value', issue.custom_value_for(2).value
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    mail = ActionMailer::Base.deliveries.last
     assert_kind_of TMail::Mail, mail
     assert mail.body.include?("Searchable field changed from 125 to New custom value")
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_with_status_and_assignee_change
     issue = Issue.find(1)
     assert_equal 1, issue.status_id
     @request.session[:user_id] = 2
     assert_difference('TimeEntry.count', 0) do
       put :update,
            :id => 1,
            :issue => { :status_id => 2, :assigned_to_id => 3 },
            :notes => 'Assigned to dlopper',
            :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
     end
     assert_redirected_to :action => 'show', :id => '1'
     issue.reload
     assert_equal 2, issue.status_id
     j = Journal.find(:first, :order => 'id DESC')
     assert_equal 'Assigned to dlopper', j.notes
     assert_equal 2, j.details.size
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    mail = ActionMailer::Base.deliveries.last
     assert mail.body.include?("Status changed from New to Assigned")
     # subject should contain the new status
     assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_with_note_only
     notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
     # anonymous user
     put :update,
          :id => 1,
          :notes => notes
     assert_redirected_to :action => 'show', :id => '1'
     j = Journal.find(:first, :order => 'id DESC')
     assert_equal notes, j.notes
     assert_equal 0, j.details.size
     assert_equal User.anonymous, j.user
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    mail = ActionMailer::Base.deliveries.last
     assert mail.body.include?(notes)
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_with_note_and_spent_time
     @request.session[:user_id] = 2
     spent_hours_before = Issue.find(1).spent_hours
     assert_difference('TimeEntry.count') do
       put :update,
            :id => 1,
            :notes => '2.5 hours added',
            :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
     end
     assert_redirected_to :action => 'show', :id => '1'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = Issue.find(1)
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    j = Journal.find(:first, :order => 'id DESC')
     assert_equal '2.5 hours added', j.notes
     assert_equal 0, j.details.size
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
     assert_not_nil t
     assert_equal 2.5, t.hours
     assert_equal spent_hours_before + 2.5, issue.spent_hours
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_with_attachment_only
     set_tmp_attachments_directory
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # Delete all fixtured journals, a race condition can occur causing the wrong
     # journal to get fetched in the next find.
     Journal.delete_all
     # anonymous user
     put :update,
          :id => 1,
          :notes => '',
          :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
     assert_redirected_to :action => 'show', :id => '1'
     j = Issue.find(1).journals.find(:first, :order => 'id DESC')
     assert j.notes.blank?
     assert_equal 1, j.details.size
     assert_equal 'testfile.txt', j.details.first.value
     assert_equal User.anonymous, j.user
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    mail = ActionMailer::Base.deliveries.last
     assert mail.body.include?('testfile.txt')
   end
   def test_put_update_with_attachment_that_fails_to_save
     set_tmp_attachments_directory
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # Delete all fixtured journals, a race condition can occur causing the wrong
     # journal to get fetched in the next find.
     Journal.delete_all
     # Mock out the unsaved attachment
     Attachment.any_instance.stubs(:create).returns(Attachment.new)
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # anonymous user
     put :update,
          :id => 1,
          :notes => '',
          :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
     assert_redirected_to :action => 'show', :id => '1'
     assert_equal '1 file(s) could not be saved.', flash[:warning]
   end if Object.const_defined?(:Mocha)
   def test_put_update_with_no_change
     issue = Issue.find(1)
     issue.journals.clear
     ActionMailer::Base.deliveries.clear
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    put :update,
          :id => 1,
          :notes => ''
     assert_redirected_to :action => 'show', :id => '1'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue.reload
     assert issue.journals.empty?
     # No email should be sent
     assert ActionMailer::Base.deliveries.empty?
   end
   def test_put_update_should_send_a_notification
     @request.session[:user_id] = 2
     ActionMailer::Base.deliveries.clear
     issue = Issue.find(1)
     old_subject = issue.subject
     new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    put :update, :id => 1, :issue => {:subject => new_subject,
                                      :priority_id => '6',
                                      :category_id => '1' # no change
+                                    }
     assert_equal 1, ActionMailer::Base.deliveries.size
   end
-:cbce1fd3b1b7
+            Chris
   def test_put_update_with_invalid_spent_time_hours_only
-:513646585e45
+    @request.session[:user_id] = 2
     notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_no_difference('Journal.count') do
       put :update,
            :id => 1,
            :notes => notes,
            :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
     end
     assert_response :success
     assert_template 'edit'
-:cbce1fd3b1b7
+            Chris
     assert_error_tag :descendant => {:content => /Activity can't be blank/}
     assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes
-:513646585e45
+    assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
   end
-:cbce1fd3b1b7
+            Chris
   def test_put_update_with_invalid_spent_time_comments_only
     @request.session[:user_id] = 2
     notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
     assert_no_difference('Journal.count') do
       put :update,
            :id => 1,
            :notes => notes,
            :time_entry => {"comments"=>"this is my comment", "activity_id"=>"", "hours"=>""}
     end
     assert_response :success
     assert_template 'edit'
     assert_error_tag :descendant => {:content => /Activity can't be blank/}
     assert_error_tag :descendant => {:content => /Hours can't be blank/}
     assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes
     assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => "this is my comment" }
   end
-:513646585e45
+  def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
     issue = Issue.find(2)
     @request.session[:user_id] = 2
     put :update,
          :id => issue.id,
          :issue => {
            :fixed_version_id => 4
+         }
     assert_response :redirect
     issue.reload
     assert_equal 4, issue.fixed_version_id
     assert_not_equal issue.project_id, issue.fixed_version.project_id
   end
   def test_put_update_should_redirect_back_using_the_back_url_parameter
     issue = Issue.find(2)
     @request.session[:user_id] = 2
     put :update,
          :id => issue.id,
          :issue => {
            :fixed_version_id => 4
          },
          :back_url => '/issues'
     assert_response :redirect
     assert_redirected_to '/issues'
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
     issue = Issue.find(2)
     @request.session[:user_id] = 2
     put :update,
          :id => issue.id,
          :issue => {
            :fixed_version_id => 4
          },
          :back_url => 'http://google.com'
     assert_response :redirect
     assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_get_bulk_edit
     @request.session[:user_id] = 2
     get :bulk_edit, :ids => [1, 2]
     assert_response :success
     assert_template 'bulk_edit'
-:cbce1fd3b1b7
+            Chris
     assert_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
-:513646585e45
+    # Project specific custom field, date type
     field = CustomField.find(9)
     assert !field.is_for_all?
     assert_equal 'date', field.field_format
     assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    # System wide custom field
     assert CustomField.find(1).is_for_all?
     assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'}
   end
-:94944d00e43c
+  def test_get_bulk_edit_on_different_projects
     @request.session[:user_id] = 2
     get :bulk_edit, :ids => [1, 2, 6]
     assert_response :success
     assert_template 'bulk_edit'
-:cbce1fd3b1b7
+            Chris
     # Can not set issues from different projects as children of an issue
     assert_no_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
-:94944d00e43c
+    # Project specific custom field, date type
     field = CustomField.find(9)
     assert !field.is_for_all?
     assert !field.project_ids.include?(Issue.find(6).project_id)
     assert_no_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
   end
-:cbce1fd3b1b7
+  def test_get_bulk_edit_with_user_custom_field
     field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true)
     @request.session[:user_id] = 2
     get :bulk_edit, :ids => [1, 2]
     assert_response :success
     assert_template 'bulk_edit'
     assert_tag :select,
       :attributes => {:name => "issue[custom_field_values][#{field.id}]"},
       :children => {
         :only => {:tag => 'option'},
         :count => Project.find(1).users.count + 1
+      }
   end
   def test_get_bulk_edit_with_version_custom_field
     field = IssueCustomField.create!(:name => 'Affected version', :field_format => 'version', :is_for_all => true)
     @request.session[:user_id] = 2
     get :bulk_edit, :ids => [1, 2]
     assert_response :success
     assert_template 'bulk_edit'
     assert_tag :select,
       :attributes => {:name => "issue[custom_field_values][#{field.id}]"},
       :children => {
         :only => {:tag => 'option'},
         :count => Project.find(1).versions.count + 1
+      }
   end
-:1d32c0a0efbf
+  def test_bulk_update
-:513646585e45
+    @request.session[:user_id] = 2
     # update issues priority
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
-:513646585e45
+                                     :issue => {:priority_id => 7,
                                                 :assigned_to_id => '',
                                                 :custom_field_values => {'2' => ''}}
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_response 302
     # check that the issues were updated
     assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = Issue.find(1)
     journal = issue.journals.find(:first, :order => 'created_on DESC')
     assert_equal '125', issue.custom_value_for(2).value
     assert_equal 'Bulk editing', journal.notes
     assert_equal 1, journal.details.size
   end
-:94944d00e43c
+  def test_bulk_update_on_different_projects
     @request.session[:user_id] = 2
     # update issues priority
     post :bulk_update, :ids => [1, 2, 6], :notes => 'Bulk editing',
                                      :issue => {:priority_id => 7,
                                                 :assigned_to_id => '',
                                                 :custom_field_values => {'2' => ''}}
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    assert_response 302
     # check that the issues were updated
     assert_equal [7, 7, 7], Issue.find([1,2,6]).map(&:priority_id)
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+    issue = Issue.find(1)
     journal = issue.journals.find(:first, :order => 'created_on DESC')
     assert_equal '125', issue.custom_value_for(2).value
     assert_equal 'Bulk editing', journal.notes
     assert_equal 1, journal.details.size
   end
   def test_bulk_update_on_different_projects_without_rights
     @request.session[:user_id] = 3
     user = User.find(3)
     action = { :controller => "issues", :action => "bulk_update" }
     assert user.allowed_to?(action, Issue.find(1).project)
     assert ! user.allowed_to?(action, Issue.find(6).project)
     post :bulk_update, :ids => [1, 6], :notes => 'Bulk should fail',
                                      :issue => {:priority_id => 7,
                                                 :assigned_to_id => '',
                                                 :custom_field_values => {'2' => ''}}
     assert_response 403
     assert_not_equal "Bulk should fail", Journal.last.notes
   end
-:cbce1fd3b1b7
+            Chris
-:1d32c0a0efbf
+  def test_bullk_update_should_send_a_notification
-:513646585e45
+    @request.session[:user_id] = 2
     ActionMailer::Base.deliveries.clear
-:1d32c0a0efbf
+    post(:bulk_update,
-:513646585e45
+            Chris
            :ids => [1, 2],
            :notes => 'Bulk editing',
            :issue => {
              :priority_id => 7,
              :assigned_to_id => '',
              :custom_field_values => {'2' => ''}
+           }
          })
     assert_response 302
     assert_equal 2, ActionMailer::Base.deliveries.size
   end
-:1d32c0a0efbf
+  def test_bulk_update_status
-:513646585e45
+    @request.session[:user_id] = 2
     # update issues priority
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status',
-:513646585e45
+                                     :issue => {:priority_id => '',
                                                 :assigned_to_id => '',
                                                 :status_id => '5'}
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_response 302
     issue = Issue.find(1)
     assert issue.closed?
   end
-:cbce1fd3b1b7
+  def test_bulk_update_parent_id
     @request.session[:user_id] = 2
     post :bulk_update, :ids => [1, 3],
       :notes => 'Bulk editing parent',
       :issue => {:priority_id => '', :assigned_to_id => '', :status_id => '', :parent_issue_id => '2'}
     assert_response 302
     parent = Issue.find(2)
     assert_equal parent.id, Issue.find(1).parent_id
     assert_equal parent.id, Issue.find(3).parent_id
     assert_equal [1, 3], parent.children.collect(&:id).sort
   end
-:1d32c0a0efbf
+  def test_bulk_update_custom_field
-:513646585e45
+    @request.session[:user_id] = 2
     # update issues priority
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field',
-:513646585e45
+                                     :issue => {:priority_id => '',
                                                 :assigned_to_id => '',
                                                 :custom_field_values => {'2' => '777'}}
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    assert_response 302
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+    issue = Issue.find(1)
     journal = issue.journals.find(:first, :order => 'created_on DESC')
     assert_equal '777', issue.custom_value_for(2).value
     assert_equal 1, journal.details.size
     assert_equal '125', journal.details.first.old_value
     assert_equal '777', journal.details.first.value
   end
-:1d32c0a0efbf
+  def test_bulk_update_unassign
-:513646585e45
+    assert_not_nil Issue.find(2).assigned_to
     @request.session[:user_id] = 2
     # unassign issues
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
-:513646585e45
+    assert_response 302
     # check that the issues were updated
     assert_nil Issue.find(2).assigned_to
   end
-:cbce1fd3b1b7
+            Chris
-:1d32c0a0efbf
+  def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject
-:513646585e45
+    @request.session[:user_id] = 2
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4}
-:513646585e45
+            Chris
     assert_response :redirect
     issues = Issue.find([1,2])
     issues.each do |issue|
       assert_equal 4, issue.fixed_version_id
       assert_not_equal issue.project_id, issue.fixed_version.project_id
     end
   end
-:1d32c0a0efbf
+  def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter
-:513646585e45
+    @request.session[:user_id] = 2
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1,2], :back_url => '/issues'
-:513646585e45
+            Chris
     assert_response :redirect
     assert_redirected_to '/issues'
   end
-:1d32c0a0efbf
+  def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
-:513646585e45
+    @request.session[:user_id] = 2
-:1d32c0a0efbf
+    post :bulk_update, :ids => [1,2], :back_url => 'http://google.com'
-:513646585e45
+            Chris
     assert_response :redirect
     assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_destroy_issue_with_no_time_entries
     assert_nil TimeEntry.find_by_issue_id(2)
     @request.session[:user_id] = 2
     post :destroy, :id => 2
     assert_redirected_to :action => 'index', :project_id => 'ecookbook'
     assert_nil Issue.find_by_id(2)
   end
   def test_destroy_issues_with_time_entries
     @request.session[:user_id] = 2
     post :destroy, :ids => [1, 3]
     assert_response :success
     assert_template 'destroy'
     assert_not_nil assigns(:hours)
     assert Issue.find_by_id(1) && Issue.find_by_id(3)
   end
   def test_destroy_issues_and_destroy_time_entries
     @request.session[:user_id] = 2
     post :destroy, :ids => [1, 3], :todo => 'destroy'
     assert_redirected_to :action => 'index', :project_id => 'ecookbook'
     assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
     assert_nil TimeEntry.find_by_id([1, 2])
   end
   def test_destroy_issues_and_assign_time_entries_to_project
     @request.session[:user_id] = 2
     post :destroy, :ids => [1, 3], :todo => 'nullify'
     assert_redirected_to :action => 'index', :project_id => 'ecookbook'
     assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
     assert_nil TimeEntry.find(1).issue_id
     assert_nil TimeEntry.find(2).issue_id
   end
-:cbce1fd3b1b7
+            Chris
-:513646585e45
+  def test_destroy_issues_and_reassign_time_entries_to_another_issue
     @request.session[:user_id] = 2
     post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
     assert_redirected_to :action => 'index', :project_id => 'ecookbook'
     assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
     assert_equal 2, TimeEntry.find(1).issue_id
     assert_equal 2, TimeEntry.find(2).issue_id
   end
-:cbce1fd3b1b7
+            Chris
-:94944d00e43c
+  def test_destroy_issues_from_different_projects
     @request.session[:user_id] = 2
     post :destroy, :ids => [1, 2, 6], :todo => 'destroy'
     assert_redirected_to :controller => 'issues', :action => 'index'
     assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6))
   end
-:cbce1fd3b1b7
+            Chris
   def test_destroy_parent_and_child_issues
     parent = Issue.generate!(:project_id => 1, :tracker_id => 1)
     child = Issue.generate!(:project_id => 1, :tracker_id => 1, :parent_issue_id => parent.id)
     assert child.is_descendant_of?(parent.reload)
     @request.session[:user_id] = 2
     assert_difference 'Issue.count', -2 do
       post :destroy, :ids => [parent.id, child.id], :todo => 'destroy'
     end
     assert_response 302
   end
-:513646585e45
+  def test_default_search_scope
     get :index
     assert_tag :div, :attributes => {:id => 'quick-search'},
                      :child => {:tag => 'form',
                                 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}
   end
 end

SoundSoftware Project »

SoundSoftware Code Site

root / test / functional / issues_controller_test.rb @ 441:cbce1fd3b1b7