diff test/functional/repositories_controller_test.rb @ 1115:433d4f72a19b redmine-2.2

Update to Redmine SVN revision 11137 on 2.2-stable branch

--- a/test/functional/repositories_controller_test.rb	Wed Jun 27 14:54:18 2012 +0100
+++ b/test/functional/repositories_controller_test.rb	Mon Jan 07 12:01:42 2013 +0000
@@ -1,5 +1,5 @@
 # Redmine - project management software
-# Copyright (C) 2006-2011  Jean-Philippe Lang
+# Copyright (C) 2006-2012  Jean-Philippe Lang
 #
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
@@ -22,7 +22,7 @@
 class RepositoriesController; def rescue_action(e) raise e end; end
 
 class RepositoriesControllerTest < ActionController::TestCase
-  fixtures :projects, :users, :roles, :members, :member_roles,
+  fixtures :projects, :users, :roles, :members, :member_roles, :enabled_modules,
            :repositories, :issues, :issue_statuses, :changesets, :changes,
            :issue_categories, :enumerations, :custom_fields, :custom_values, :trackers
 
@@ -33,13 +33,114 @@
     User.current = nil
   end
 
+  def test_new
+    @request.session[:user_id] = 1
+    get :new, :project_id => 'subproject1'
+    assert_response :success
+    assert_template 'new'
+    assert_kind_of Repository::Subversion, assigns(:repository)
+    assert assigns(:repository).new_record?
+    assert_tag 'input', :attributes => {:name => 'repository[url]', :disabled => nil}
+  end
+
+  def test_new_should_propose_enabled_scm_only
+    @request.session[:user_id] = 1
+    with_settings :enabled_scm => ['Mercurial', 'Git'] do
+      get :new, :project_id => 'subproject1'
+    end
+    assert_response :success
+    assert_template 'new'
+    assert_kind_of Repository::Mercurial, assigns(:repository)
+    assert_tag 'select', :attributes => {:name => 'repository_scm'},
+      :children => {:count => 3}
+    assert_tag 'select', :attributes => {:name => 'repository_scm'},
+      :child => {:tag => 'option', :attributes => {:value => 'Mercurial', :selected => 'selected'}}
+    assert_tag 'select', :attributes => {:name => 'repository_scm'},
+      :child => {:tag => 'option', :attributes => {:value => 'Git', :selected => nil}}
+  end
+
+  def test_create
+    @request.session[:user_id] = 1
+    assert_difference 'Repository.count' do
+      post :create, :project_id => 'subproject1',
+           :repository_scm => 'Subversion',
+           :repository => {:url => 'file:///test', :is_default => '1', :identifier => ''}
+    end
+    assert_response 302
+    repository = Repository.first(:order => 'id DESC')
+    assert_kind_of Repository::Subversion, repository
+    assert_equal 'file:///test', repository.url
+  end
+
+  def test_create_with_failure
+    @request.session[:user_id] = 1
+    assert_no_difference 'Repository.count' do
+      post :create, :project_id => 'subproject1',
+           :repository_scm => 'Subversion',
+           :repository => {:url => 'invalid'}
+    end
+    assert_response :success
+    assert_template 'new'
+    assert_kind_of Repository::Subversion, assigns(:repository)
+    assert assigns(:repository).new_record?
+  end
+
+  def test_edit
+    @request.session[:user_id] = 1
+    get :edit, :id => 11
+    assert_response :success
+    assert_template 'edit'
+    assert_equal Repository.find(11), assigns(:repository)
+    assert_tag 'input', :attributes => {:name => 'repository[url]', :value => 'svn://localhost/test', :disabled => 'disabled'}
+  end
+
+  def test_update
+    @request.session[:user_id] = 1
+    put :update, :id => 11, :repository => {:password => 'test_update'}
+    assert_response 302
+    assert_equal 'test_update', Repository.find(11).password
+  end
+
+  def test_update_with_failure
+    @request.session[:user_id] = 1
+    put :update, :id => 11, :repository => {:password => 'x'*260}
+    assert_response :success
+    assert_template 'edit'
+    assert_equal Repository.find(11), assigns(:repository)
+  end
+
+  def test_destroy
+    @request.session[:user_id] = 1
+    assert_difference 'Repository.count', -1 do
+      delete :destroy, :id => 11
+    end
+    assert_response 302
+    assert_nil Repository.find_by_id(11)
+  end
+
   def test_revisions
     get :revisions, :id => 1
     assert_response :success
     assert_template 'revisions'
+    assert_equal Repository.find(10), assigns(:repository)
     assert_not_nil assigns(:changesets)
   end
 
+  def test_revisions_for_other_repository
+    repository = Repository::Subversion.create!(:project_id => 1, :identifier => 'foo', :url => 'file:///foo')
+
+    get :revisions, :id => 1, :repository_id => 'foo'
+    assert_response :success
+    assert_template 'revisions'
+    assert_equal repository, assigns(:repository)
+    assert_not_nil assigns(:changesets)
+  end
+
+  def test_revisions_for_invalid_repository
+    get :revisions, :id => 1, :repository_id => 'foo'
+    assert_response 404
+  end
+
   def test_revision
     get :revision, :id => 1, :rev => 1
     assert_response :success
@@ -47,6 +148,14 @@
     assert_equal "1", assigns(:changeset).revision
   end
 
+  def test_revision_should_not_change_the_project_menu_link
+    get :revision, :id => 1, :rev => 1
+    assert_response :success
+
+    assert_tag 'a', :attributes => {:href => '/projects/ecookbook/repository', :class => /repository/},
+      :ancestor => {:attributes => {:id => 'main-menu'}}
+  end
+
   def test_revision_with_before_nil_and_afer_normal
     get :revision, {:id => 1, :rev => 1}
     assert_response :success
@@ -59,7 +168,51 @@
     }
   end
 
+  def test_add_related_issue
+    @request.session[:user_id] = 2
+    assert_difference 'Changeset.find(103).issues.size' do
+      xhr :post, :add_related_issue, :id => 1, :rev => 4, :issue_id => 2, :format => 'js'
+      assert_response :success
+      assert_template 'add_related_issue'
+      assert_equal 'text/javascript', response.content_type
+    end
+    assert_equal [2], Changeset.find(103).issue_ids
+    assert_include 'related-issues', response.body
+    assert_include 'Feature request #2', response.body
+  end
+
+  def test_add_related_issue_with_invalid_issue_id
+    @request.session[:user_id] = 2
+    assert_no_difference 'Changeset.find(103).issues.size' do
+      xhr :post, :add_related_issue, :id => 1, :rev => 4, :issue_id => 9999, :format => 'js'
+      assert_response :success
+      assert_template 'add_related_issue'
+      assert_equal 'text/javascript', response.content_type
+    end
+    assert_include 'alert("Issue is invalid")', response.body
+  end
+
+  def test_remove_related_issue
+    Changeset.find(103).issues << Issue.find(1)
+    Changeset.find(103).issues << Issue.find(2)
+
+    @request.session[:user_id] = 2
+    assert_difference 'Changeset.find(103).issues.size', -1 do
+      xhr :delete, :remove_related_issue, :id => 1, :rev => 4, :issue_id => 2, :format => 'js'
+      assert_response :success
+      assert_template 'remove_related_issue'
+      assert_equal 'text/javascript', response.content_type
+    end
+    assert_equal [1], Changeset.find(103).issue_ids
+    assert_include 'related-issue-2', response.body
+  end
+
   def test_graph_commits_per_month
+    # Make sure there's some data to display
+    latest = Project.find(1).repository.changesets.maximum(:commit_date)
+    assert_not_nil latest
+    Date.stubs(:today).returns(latest.to_date + 10)
+
     get :graph, :id => 1, :graph => 'commits_per_month'
     assert_response :success
     assert_equal 'image/svg+xml', @response.content_type
@@ -71,7 +224,7 @@
     assert_equal 'image/svg+xml', @response.content_type
   end
 
-  def test_committers
+  def test_get_committers
     @request.session[:user_id] = 2
     # add a commit with an unknown user
     Changeset.create!(
@@ -82,7 +235,7 @@
         :comments => 'Committed by foo.'
      )
 
-    get :committers, :id => 1
+    get :committers, :id => 10
     assert_response :success
     assert_template 'committers'
 
@@ -99,7 +252,7 @@
                                      :descendant => { :tag => 'option', :attributes => { :selected => 'selected' }}}
   end
 
-  def test_map_committers
+  def test_post_committers
     @request.session[:user_id] = 2
     # add a commit with an unknown user
     c = Changeset.create!(
@@ -110,8 +263,8 @@
             :comments => 'Committed by foo.'
           )
     assert_no_difference "Changeset.count(:conditions => 'user_id = 3')" do
-      post :committers, :id => 1, :committers => { '0' => ['foo', '2'], '1' => ['dlopper', '3']}
-      assert_redirected_to '/projects/ecookbook/repository/committers'
+      post :committers, :id => 10, :committers => { '0' => ['foo', '2'], '1' => ['dlopper', '3']}
+      assert_response 302
       assert_equal User.find(2), c.reload.user
     end
   end