fix source indent of RepositoriesMercurialControllerTest

git-svn-id: http://svn.redmine.org/redmine/trunk@20102 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Toshi MARUYAMA
2020-10-02 12:58:50 +00:00
parent 9b61637e4a
commit 98a25c61fa

View File

@@ -33,11 +33,12 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
super super
User.current = nil User.current = nil
@project = Project.find(PRJ_ID) @project = Project.find(PRJ_ID)
@repository = Repository::Mercurial.create( @repository =
:project => @project, Repository::Mercurial.create(
:url => REPOSITORY_PATH, :project => @project,
:path_encoding => 'ISO-8859-1' :url => REPOSITORY_PATH,
) :path_encoding => 'ISO-8859-1'
)
assert @repository assert @repository
@diff_c_support = true @diff_c_support = true
end end
@@ -52,10 +53,13 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_get_new def test_get_new
@request.session[:user_id] = 1 @request.session[:user_id] = 1
@project.repository.destroy @project.repository.destroy
get :new, :params => { get(
:new,
:params => {
:project_id => 'subproject1', :project_id => 'subproject1',
:repository_scm => 'Mercurial' :repository_scm => 'Mercurial'
} }
)
assert_response :success assert_response :success
assert_select 'select[name=?]', 'repository_scm' do assert_select 'select[name=?]', 'repository_scm' do
assert_select 'option[value=?][selected=selected]', 'Mercurial' assert_select 'option[value=?][selected=selected]', 'Mercurial'
@@ -67,9 +71,12 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@repository.fetch_changesets @repository.fetch_changesets
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID :id => PRJ_ID
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody' do assert_select 'table.entries tbody' do
@@ -90,11 +97,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@repository.fetch_changesets @repository.fetch_changesets
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['images'])[:param] :path => repository_path_hash(['images'])[:param]
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody' do assert_select 'table.entries tbody' do
@@ -114,12 +124,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
[0, '0', '0885933ad4f6'].each do |r1| [0, '0', '0885933ad4f6'].each do |r1|
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['images'])[:param], :path => repository_path_hash(['images'])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody' do assert_select 'table.entries tbody' do
@@ -139,12 +152,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
[13, '13', '3a330eb32958'].each do |r1| [13, '13', '3a330eb32958'].each do |r1|
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sql_escape', 'percent%dir'])[:param], :path => repository_path_hash(['sql_escape', 'percent%dir'])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody' do assert_select 'table.entries tbody' do
@@ -168,12 +184,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
[21, '21', 'adf805632193'].each do |r1| [21, '21', 'adf805632193'].each do |r1|
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['latin-1-dir'])[:param], :path => repository_path_hash(['latin-1-dir'])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody' do assert_select 'table.entries tbody' do
@@ -197,9 +216,12 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def show_should_show_branch_selection_form def show_should_show_branch_selection_form
@repository.fetch_changesets @repository.fetch_changesets
@project.reload @project.reload
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID :id => PRJ_ID
} }
)
assert_select 'form#revision_selector[action=?]', '/projects/subproject1/repository/show' do assert_select 'form#revision_selector[action=?]', '/projects/subproject1/repository/show' do
assert_select 'select[name=branch]' do assert_select 'select[name=branch]' do
assert_select 'option[value=?]', 'test-branch-01' assert_select 'option[value=?]', 'test-branch-01'
@@ -220,11 +242,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
'test_branch.latin-1', 'test_branch.latin-1',
'test-branch-00', 'test-branch-00',
].each do |bra| ].each do |bra|
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => bra :rev => bra
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody tr' assert_select 'table.entries tbody tr'
@@ -242,11 +267,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
'tag_test.00', 'tag_test.00',
'tag-init-revision' 'tag-init-revision'
].each do |tag| ].each do |tag|
get :show, :params => { get(
:show,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => tag :rev => tag
} }
)
assert_response :success assert_response :success
assert_select 'table.entries tbody tr' assert_select 'table.entries tbody tr'
@@ -255,21 +283,27 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
end end
def test_changes def test_changes
get :changes, :params => { get(
:changes,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['images', 'edit.png'])[:param] :path => repository_path_hash(['images', 'edit.png'])[:param]
} }
)
assert_response :success assert_response :success
assert_select 'h2', :text => /edit.png/ assert_select 'h2', :text => /edit.png/
end end
def test_entry_show def test_entry_show
get :entry, :params => { get(
:entry,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param] :path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param]
} }
)
assert_response :success assert_response :success
# Line 10 # Line 10
assert_select 'tr#L10 td.line-code', :text => /WITHOUT ANY WARRANTY/ assert_select 'tr#L10 td.line-code', :text => /WITHOUT ANY WARRANTY/
@@ -277,12 +311,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_entry_show_latin_1_path def test_entry_show_latin_1_path
[21, '21', 'adf805632193'].each do |r1| [21, '21', 'adf805632193'].each do |r1|
get :entry, :params => { get(
:entry,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['latin-1-dir', "test-Ü-2.txt"])[:param], :path => repository_path_hash(['latin-1-dir', "test-Ü-2.txt"])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select 'tr#L1 td.line-code', :text => /Mercurial is a distributed version control system/ assert_select 'tr#L1 td.line-code', :text => /Mercurial is a distributed version control system/
end end
@@ -291,12 +328,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_entry_show_latin_1_contents def test_entry_show_latin_1_contents
with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do
[27, '27', '7bbf4c738e71'].each do |r1| [27, '27', '7bbf4c738e71'].each do |r1|
get :entry, :params => { get(
:entry,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['latin-1-dir', "test-Ü.txt"])[:param], :path => repository_path_hash(['latin-1-dir', "test-Ü.txt"])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select 'tr#L1 td.line-code', :text => /test-Ü.txt/ assert_select 'tr#L1 td.line-code', :text => /test-Ü.txt/
end end
@@ -304,12 +344,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
end end
def test_entry_download def test_entry_download
get :entry, :params => { get(
:entry,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param], :path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param],
:format => 'raw' :format => 'raw'
} }
)
assert_response :success assert_response :success
# File content # File content
assert @response.body.include?('WITHOUT ANY WARRANTY') assert @response.body.include?('WITHOUT ANY WARRANTY')
@@ -320,11 +363,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
end end
def test_directory_entry def test_directory_entry
get :entry, :params => { get(
:entry,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sources'])[:param] :path => repository_path_hash(['sources'])[:param]
} }
)
assert_response :success assert_response :success
assert_select 'h2 a', :text => 'sources' assert_select 'h2 a', :text => 'sources'
assert_select 'table.entries tbody' assert_select 'table.entries tbody'
@@ -338,12 +384,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
[4, '4', 'def6d2f1254a'].each do |r1| [4, '4', 'def6d2f1254a'].each do |r1|
# Full diff of changeset 4 # Full diff of changeset 4
['inline', 'sbs'].each do |dt| ['inline', 'sbs'].each do |dt|
get :diff, :params => { get(
:diff,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r1, :rev => r1,
:type => dt :type => dt
} }
)
assert_response :success assert_response :success
if @diff_c_support if @diff_c_support
# Line 22 removed # Line 22 removed
@@ -362,13 +411,16 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
[2, '400bb8672109', '400', 400].each do |r1| [2, '400bb8672109', '400', 400].each do |r1|
[4, 'def6d2f1254a'].each do |r2| [4, 'def6d2f1254a'].each do |r2|
['inline', 'sbs'].each do |dt| ['inline', 'sbs'].each do |dt|
get :diff, :params => { get(
:diff,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r1, :rev => r1,
:rev_to => r2, :rev_to => r2,
:type => dt :type => dt
} }
)
assert_response :success assert_response :success
assert_select 'h2', :text => /4:def6d2f1254a 2:400bb8672109/ assert_select 'h2', :text => /4:def6d2f1254a 2:400bb8672109/
end end
@@ -380,12 +432,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do
[21, 'adf805632193'].each do |r1| [21, 'adf805632193'].each do |r1|
['inline', 'sbs'].each do |dt| ['inline', 'sbs'].each do |dt|
get :diff, :params => { get(
:diff,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r1, :rev => r1,
:type => dt :type => dt
} }
)
assert_response :success assert_response :success
assert_select 'table' do assert_select 'table' do
assert_select 'thead th.filename', :text => /latin-1-dir\/test-Ü-2.txt/ assert_select 'thead th.filename', :text => /latin-1-dir\/test-Ü-2.txt/
@@ -397,33 +452,42 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
end end
def test_diff_should_show_modified_filenames def test_diff_should_show_modified_filenames
get :diff, :params => { get(
:diff,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => '400bb8672109', :rev => '400bb8672109',
:type => 'inline' :type => 'inline'
} }
)
assert_response :success assert_response :success
assert_select 'th.filename', :text => 'sources/watchers_controller.rb' assert_select 'th.filename', :text => 'sources/watchers_controller.rb'
end end
def test_diff_should_show_deleted_filenames def test_diff_should_show_deleted_filenames
get :diff, :params => { get(
:diff,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => 'b3a615152df8', :rev => 'b3a615152df8',
:type => 'inline' :type => 'inline'
} }
)
assert_response :success assert_response :success
assert_select 'th.filename', :text => 'sources/welcome_controller.rb' assert_select 'th.filename', :text => 'sources/welcome_controller.rb'
end end
def test_annotate def test_annotate
get :annotate, :params => { get(
:annotate,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param] :path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param]
} }
)
assert_response :success assert_response :success
# Line 22, revision 4:def6d2f1254a # Line 22, revision 4:def6d2f1254a
@@ -440,11 +504,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@repository.fetch_changesets @repository.fetch_changesets
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
get :annotate, :params => { get(
:annotate,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['sources', 'welcome_controller.rb'])[:param] :path => repository_path_hash(['sources', 'welcome_controller.rb'])[:param]
} }
)
assert_response 404 assert_response 404
assert_select_error /was not found/ assert_select_error /was not found/
end end
@@ -455,12 +522,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
[2, '400bb8672109', '400', 400].each do |r1| [2, '400bb8672109', '400', 400].each do |r1|
get :annotate, :params => { get(
:annotate,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r1, :rev => r1,
:path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param] :path => repository_path_hash(['sources', 'watchers_controller.rb'])[:param]
} }
)
assert_response :success assert_response :success
assert_select 'h2', :text => /@ 2:400bb8672109/ assert_select 'h2', :text => /@ 2:400bb8672109/
end end
@@ -468,12 +538,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_annotate_latin_1_path def test_annotate_latin_1_path
[21, '21', 'adf805632193'].each do |r1| [21, '21', 'adf805632193'].each do |r1|
get :annotate, :params => { get(
:annotate,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['latin-1-dir', "test-Ü-2.txt"])[:param], :path => repository_path_hash(['latin-1-dir', "test-Ü-2.txt"])[:param],
:rev => r1 :rev => r1
} }
)
assert_response :success assert_response :success
assert_select "th.line-num", :text => '1' do assert_select "th.line-num", :text => '1' do
assert_select "+ td.revision" do assert_select "+ td.revision" do
@@ -490,12 +563,15 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_annotate_latin_1_contents def test_annotate_latin_1_contents
with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do
[27, '7bbf4c738e71'].each do |r1| [27, '7bbf4c738e71'].each do |r1|
get :annotate, :params => { get(
:annotate,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:path => repository_path_hash(['latin-1-dir', "test-Ü.txt"])[:param], :path => repository_path_hash(['latin-1-dir', "test-Ü.txt"])[:param],
:rev => r1 :rev => r1
} }
)
assert_select 'tr#L1 td.line-code', :text => /test-Ü.txt/ assert_select 'tr#L1 td.line-code', :text => /test-Ü.txt/
end end
end end
@@ -508,11 +584,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
['1', '9d5b5b', '9d5b5b004199'].each do |r| ['1', '9d5b5b', '9d5b5b004199'].each do |r|
with_settings :default_language => "en" do with_settings :default_language => "en" do
get :revision, :params => { get(
:revision,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r :rev => r
} }
)
assert_response :success assert_response :success
assert_select 'title', assert_select 'title',
:text => 'Revision 1:9d5b5b004199 - Added 2 files and modified one. - eCookbook Subproject 1 - Redmine' :text => 'Revision 1:9d5b5b004199 - Added 2 files and modified one. - eCookbook Subproject 1 - Redmine'
@@ -526,11 +605,14 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
@project.reload @project.reload
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
['', ' ', nil].each do |r| ['', ' ', nil].each do |r|
get :revision, :params => { get(
:revision,
:params => {
:id => PRJ_ID, :id => PRJ_ID,
:repository_id => @repository.id, :repository_id => @repository.id,
:rev => r :rev => r
} }
)
assert_response 404 assert_response 404
assert_select_error /was not found/ assert_select_error /was not found/
end end
@@ -543,9 +625,12 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
assert_equal NUM_REV, @repository.changesets.count assert_equal NUM_REV, @repository.changesets.count
assert_difference 'Repository.count', -1 do assert_difference 'Repository.count', -1 do
delete :destroy, :params => { delete(
:destroy,
:params => {
:id => @repository.id :id => @repository.id
} }
)
end end
assert_response 302 assert_response 302
@project.reload @project.reload
@@ -555,18 +640,22 @@ class RepositoriesMercurialControllerTest < Redmine::RepositoryControllerTest
def test_destroy_invalid_repository def test_destroy_invalid_repository
@request.session[:user_id] = 1 # admin @request.session[:user_id] = 1 # admin
@project.repository.destroy @project.repository.destroy
@repository = Repository::Mercurial.create!( @repository =
:project => Project.find(PRJ_ID), Repository::Mercurial.create!(
:url => "/invalid", :project => Project.find(PRJ_ID),
:path_encoding => 'ISO-8859-1' :url => "/invalid",
) :path_encoding => 'ISO-8859-1'
)
@repository.fetch_changesets @repository.fetch_changesets
assert_equal 0, @repository.changesets.count assert_equal 0, @repository.changesets.count
assert_difference 'Repository.count', -1 do assert_difference 'Repository.count', -1 do
delete :destroy, :params => { delete(
:destroy,
:params => {
:id => @repository.id :id => @repository.id
} }
)
end end
assert_response 302 assert_response 302
@project.reload @project.reload