Adds accessors for report_last_commit.

git-svn-id: http://svn.redmine.org/redmine/trunk@15885 e93f8b46-1217-0410-a6f0-8f06a7374b81

3 files changed, 18 insertions, 17 deletions

diff --git a/test/functional/repositories_controller_test.rb b/test/functional/repositories_controller_test.rb
index f55867f5b..ebe6c0da8 100644
--- a/test/functional/repositories_controller_test.rb
+++ b/test/functional/repositories_controller_test.rb
@@ -49,6 +49,16 @@ class RepositoriesControllerTest < Redmine::ControllerTest
       assert_select 'option[value=Git]:not([selected])'
     end
   end
+ 
+  def test_get_new_with_type
+    @request.session[:user_id] = 1
+    get :new, :project_id => 'subproject1', :repository_scm => 'Git'
+    assert_response :success
+
+    assert_select 'select[name=?]', 'repository_scm' do
+      assert_select 'option[value=?][selected=selected]', 'Git'
+    end
+  end
 
   def test_create
     @request.session[:user_id] = 1
diff --git a/test/functional/repositories_git_controller_test.rb b/test/functional/repositories_git_controller_test.rb
index c76a5165e..093e985b8 100644
--- a/test/functional/repositories_git_controller_test.rb
+++ b/test/functional/repositories_git_controller_test.rb
@@ -60,22 +60,22 @@ class RepositoriesGitControllerTest < Redmine::ControllerTest
                        :url => '/test',
                        :is_default => '0',
                        :identifier => 'test-create',
-                       :extra_report_last_commit => '1',
+                       :report_last_commit => '1',
                      }
     end
     assert_response 302
     repository = Repository.order('id DESC').first
     assert_kind_of Repository::Git, repository
     assert_equal '/test', repository.url
-    assert_equal true, repository.extra_report_last_commit
+    assert_equal true, repository.report_last_commit
 
     put :update, :id => repository.id,
                  :repository => {
-                     :extra_report_last_commit => '0'
+                   :report_last_commit => '0'
                  }
     assert_response 302
     repo2 = Repository.find(repository.id)
-    assert_equal false, repo2.extra_report_last_commit
+    assert_equal false, repo2.report_last_commit
   end
 
   if File.directory?(REPOSITORY_PATH)
diff --git a/test/unit/repository_git_test.rb b/test/unit/repository_git_test.rb
index 15b7ea53b..7b56b4a79 100644
--- a/test/unit/repository_git_test.rb
+++ b/test/unit/repository_git_test.rb
@@ -243,22 +243,13 @@ class RepositoryGitTest < ActiveSupport::TestCase
       assert_equal h1, h2
     end
 
-    def test_keep_extra_report_last_commit_in_clear_changesets
+    def test_clear_changesets_should_keep_report_last_commit
       assert_nil @repository.extra_info
-      h = {}
-      h["extra_report_last_commit"] = "1"
-      @repository.merge_extra_info(h)
+      @repository.report_last_commit = "1"
       @repository.save
-      @project.reload
-
-      assert_equal 0, @repository.changesets.count
-      @repository.fetch_changesets
-      @project.reload
-
-      assert_equal NUM_REV, @repository.changesets.count
       @repository.send(:clear_changesets)
-      assert_equal 1, @repository.extra_info.size
-      assert_equal "1", @repository.extra_info["extra_report_last_commit"]
+
+      assert_equal true, @repository.report_last_commit
     end
 
     def test_refetch_after_clear_changesets