jro_apps/redmine Commit - r2233:cd83f72da47f

Slight changes in functional tests....

Jean-Philippe Lang -

r2233:cd83f72da47f

parent child

Context file:

r2233:cd83f72da47f

Collapse all files

test/functional/admin_controller_test.rb +1 -1

               def test_test_email
                 get :test_email
-                assert_redirected_to 'settings/edit'
+                assert_redirected_to '/settings/edit?tab=notifications'
                 mail = ActionMailer::Base.deliveries.last
                 assert_kind_of TMail::Mail, mail
                 user = User.find(1)

test/functional/attachments_controller_test.rb +2 -2

               def test_anonymous_on_private_private
                 get :download, :id => 7
-                assert_redirected_to 'account/login'
+                assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdownload%2F7'
               end
               def test_destroy_issue_attachment
               def test_destroy_without_permission
                 post :destroy, :id => 3
-                assert_redirected_to '/login'
+                assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdestroy%2F3'
                 assert Attachment.find_by_id(3)
               end
             end

test/functional/issues_controller_test.rb +5 -4

                                       :priority_id => 5,
                                       :estimated_hours => '',
                                       :custom_field_values => {'2' => 'Value for field 2'}}
-                assert_redirected_to 'issues/show'
+                assert_redirected_to :controller => 'issues', :action => 'show'
                 issue = Issue.find_by_subject('This is the test_new issue')
                 assert_not_nil issue
                                       :subject => 'This is the test_new issue',
                                       :description => 'This is the description',
                                       :priority_id => 5}
-                assert_redirected_to 'issues/show'
+                assert_redirected_to :controller => 'issues', :action => 'show'
               end
               def test_post_new_with_required_custom_field_and_without_custom_fields_param
                                         :priority_id => 5,
                                         :watcher_user_ids => ['2', '3']}
                 end
-                assert_redirected_to 'issues/show'
                 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
                 # Watchers added
                 assert_equal [2, 3], issue.watcher_user_ids.sort
                 assert issue.watched_by?(User.find(3))

test/functional/timelog_controller_test.rb +1 -1

                 r.permissions_will_change!
                 r.save
                 get :report
-                assert_redirected_to '/account/login'
+                assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Ftimelog%2Freport'
               end
               def test_report_all_projects_one_criteria

test/functional/users_controller_test.rb +2 -2

               def test_edit_membership
                 post :edit_membership, :id => 2, :membership_id => 1,
                                        :membership => { :role_id => 2}
-                assert_redirected_to 'users/edit/2'
+                assert_redirected_to '/users/edit/2?tab=memberships'
                 assert_equal 2, Member.find(1).role_id
               end
               def test_destroy_membership
                 post :destroy_membership, :id => 2, :membership_id => 1
-                assert_redirected_to 'users/edit/2'
+                assert_redirected_to '/users/edit/2?tab=memberships'
                 assert_nil Member.find_by_id(1)
               end
             end

test/functional/versions_controller_test.rb +2 -2

                 post :edit, :id => 2,
                             :version => { :name => 'New version name',
                                           :effective_date => Date.today.strftime("%Y-%m-%d")}
-                assert_redirected_to 'projects/settings/ecookbook'
+                assert_redirected_to '/projects/settings/ecookbook?tab=versions'
                 version = Version.find(2)
                 assert_equal 'New version name', version.name
                 assert_equal Date.today, version.effective_date
               def test_destroy
                 @request.session[:user_id] = 2
                 post :destroy, :id => 3
-                assert_redirected_to 'projects/settings/ecookbook'
+                assert_redirected_to '/projects/settings/ecookbook?tab=versions'
                 assert_nil Version.find_by_id(3)
               end

test/functional/wiki_controller_test.rb +1 -1

                 assert page.protected?
                 @request.session[:user_id] = 2
                 post :protect, :id => 1, :page => page.title, :protected => '0'
-                assert_redirected_to 'wiki/ecookbook'
+                assert_redirected_to '/wiki/ecookbook/CookBook_documentation'
                 assert !page.reload.protected?
               end

test/functional/wikis_controller_test.rb +1 -1

               def test_destroy
                 @request.session[:user_id] = 1
                 post :destroy, :id => 1, :confirm => 1
-                assert_redirected_to 'projects/settings/ecookbook'
+                assert_redirected_to '/projects/settings/ecookbook?tab=wiki'
                 assert_nil Project.find(1).wiki
               end

test/functional/workflows_controller_test.rb +1 -1

               def test_post_edit
                 post :edit, :role_id => 2, :tracker_id => 1, :issue_status => {'4' => ['5'], '3' => ['1', '2']}
-                assert_redirected_to 'workflows/edit'
+                assert_redirected_to '/workflows/edit?role_id=2&tracker_id=1'
                 assert_equal 3, Workflow.count(:conditions => {:tracker_id => 1, :role_id => 2})
                 assert_not_nil  Workflow.find(:first, :conditions => {:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 2})

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages