Mercurial > hg > soundsoftware-site
diff test/unit/helpers/application_helper_test.rb @ 1298:4f746d8966dd redmine_2.3_integration
Merge from redmine-2.3 branch to create new branch redmine-2.3-integration
| author | Chris Cannam |
|---|---|
| date | Fri, 14 Jun 2013 09:28:30 +0100 |
| parents | 0a574315af3e 622f24f53b42 |
| children |
line wrap: on
line diff
--- a/test/unit/helpers/application_helper_test.rb Fri Jun 14 09:07:32 2013 +0100 +++ b/test/unit/helpers/application_helper_test.rb Fri Jun 14 09:28:30 2013 +0100 @@ -1,7 +1,7 @@ # encoding: utf-8 # # Redmine - project management software -# Copyright (C) 2006-2012 Jean-Philippe Lang +# Copyright (C) 2006-2013 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 @@ -21,6 +21,7 @@ class ApplicationHelperTest < ActionView::TestCase include ERB::Util + include Rails.application.routes.url_helpers fixtures :projects, :roles, :enabled_modules, :users, :repositories, :changesets, @@ -83,7 +84,11 @@ # escaping 'http://foo"bar' => '<a class="external" href="http://foo"bar">http://foo"bar</a>', # wrap in angle brackets - '<http://foo.bar>' => '<<a class="external" href="http://foo.bar">http://foo.bar</a>>' + '<http://foo.bar>' => '<<a class="external" href="http://foo.bar">http://foo.bar</a>>', + # invalid urls + 'http://' => 'http://', + 'www.' => 'www.', + 'test-www.bar.com' => 'test-www.bar.com', } to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text) } end @@ -100,8 +105,11 @@ end def test_auto_mailto - assert_equal '<p><a class="email" href="mailto:test@foo.bar">test@foo.bar</a></p>', - textilizable('test@foo.bar') + to_test = { + 'test@foo.bar' => '<a class="email" href="mailto:test@foo.bar">test@foo.bar</a>', + 'test@www.foo.bar' => '<a class="email" href="mailto:test@www.foo.bar">test@www.foo.bar</a>', + } + to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text) } end def test_inline_images @@ -131,14 +139,14 @@ def test_attached_images to_test = { - 'Inline image: !logo.gif!' => 'Inline image: <img src="/attachments/download/3" title="This is a logo" alt="This is a logo" />', - 'Inline image: !logo.GIF!' => 'Inline image: <img src="/attachments/download/3" title="This is a logo" alt="This is a logo" />', + 'Inline image: !logo.gif!' => 'Inline image: <img src="/attachments/download/3/logo.gif" title="This is a logo" alt="This is a logo" />', + 'Inline image: !logo.GIF!' => 'Inline image: <img src="/attachments/download/3/logo.gif" title="This is a logo" alt="This is a logo" />', 'No match: !ogo.gif!' => 'No match: <img src="ogo.gif" alt="" />', 'No match: !ogo.GIF!' => 'No match: <img src="ogo.GIF" alt="" />', # link image - '!logo.gif!:http://foo.bar/' => '<a href="http://foo.bar/"><img src="/attachments/download/3" title="This is a logo" alt="This is a logo" /></a>', + '!logo.gif!:http://foo.bar/' => '<a href="http://foo.bar/"><img src="/attachments/download/3/logo.gif" title="This is a logo" alt="This is a logo" /></a>', } - attachments = Attachment.find(:all) + attachments = Attachment.all to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text, :attachments => attachments) } end @@ -182,13 +190,13 @@ to_test = { 'Inline image: !testtest.jpg!' => - 'Inline image: <img src="/attachments/download/' + a1.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a1.id.to_s + '/testtest.JPG" alt="" />', 'Inline image: !testtest.jpeg!' => - 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '/testtest.jpeg" alt="" />', 'Inline image: !testtest.jpe!' => - 'Inline image: <img src="/attachments/download/' + a3.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a3.id.to_s + '/testtest.JPE" alt="" />', 'Inline image: !testtest.bmp!' => - 'Inline image: <img src="/attachments/download/' + a4.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a4.id.to_s + '/Testtest.BMP" alt="" />', } attachments = [a1, a2, a3, a4] @@ -211,9 +219,9 @@ to_test = { 'Inline image: !testfile.png!' => - 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '/testfile.PNG" alt="" />', 'Inline image: !Testfile.PNG!' => - 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '" alt="" />', + 'Inline image: <img src="/attachments/download/' + a2.id.to_s + '/testfile.PNG" alt="" />', } attachments = [a1, a2] to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text, :attachments => attachments) } @@ -252,15 +260,19 @@ def test_redmine_links issue_link = link_to('#3', {:controller => 'issues', :action => 'show', :id => 3}, - :class => 'issue status-1 priority-4 priority-lowest overdue', :title => 'Error 281 when updating a recipe (New)') + :class => Issue.find(3).css_classes, :title => 'Error 281 when updating a recipe (New)') note_link = link_to('#3', {:controller => 'issues', :action => 'show', :id => 3, :anchor => 'note-14'}, - :class => 'issue status-1 priority-4 priority-lowest overdue', :title => 'Error 281 when updating a recipe (New)') + :class => Issue.find(3).css_classes, :title => 'Error 281 when updating a recipe (New)') - changeset_link = link_to('r1', {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 1}, - :class => 'changeset', :title => 'My very first commit') - changeset_link2 = link_to('r2', {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 2}, + revision_link = link_to('r1', {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 1}, + :class => 'changeset', :title => 'My very first commit do not escaping #<>&') + revision_link2 = link_to('r2', {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 2}, :class => 'changeset', :title => 'This commit fixes #1, #2 and references #1 & #3') + changeset_link2 = link_to('691322a8eb01e11fd7', + {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 1}, + :class => 'changeset', :title => 'My very first commit do not escaping #<>&') + document_link = link_to('Test document', {:controller => 'documents', :action => 'show', :id => 1}, :class => 'document') @@ -279,11 +291,13 @@ source_url_with_rev = '/projects/ecookbook/repository/revisions/52/entry/some/file' source_url_with_ext = '/projects/ecookbook/repository/entry/some/file.ext' source_url_with_rev_and_ext = '/projects/ecookbook/repository/revisions/52/entry/some/file.ext' + source_url_with_branch = '/projects/ecookbook/repository/revisions/branch/entry/some/file' export_url = '/projects/ecookbook/repository/raw/some/file' export_url_with_rev = '/projects/ecookbook/repository/revisions/52/raw/some/file' export_url_with_ext = '/projects/ecookbook/repository/raw/some/file.ext' export_url_with_rev_and_ext = '/projects/ecookbook/repository/revisions/52/raw/some/file.ext' + export_url_with_branch = '/projects/ecookbook/repository/revisions/branch/raw/some/file' to_test = { # tickets @@ -294,10 +308,11 @@ # should not ignore leading zero '#03' => '#03', # changesets - 'r1' => changeset_link, - 'r1.' => "#{changeset_link}.", - 'r1, r2' => "#{changeset_link}, #{changeset_link2}", - 'r1,r2' => "#{changeset_link},#{changeset_link2}", + 'r1' => revision_link, + 'r1.' => "#{revision_link}.", + 'r1, r2' => "#{revision_link}, #{revision_link2}", + 'r1,r2' => "#{revision_link},#{revision_link2}", + 'commit:691322a8eb01e11fd7' => changeset_link2, # documents 'document#1' => document_link, 'document:"Test document"' => document_link, @@ -314,6 +329,7 @@ 'source:/some/file.ext. ' => link_to('source:/some/file.ext', source_url_with_ext, :class => 'source') + ".", 'source:/some/file, ' => link_to('source:/some/file', source_url, :class => 'source') + ",", 'source:/some/file@52' => link_to('source:/some/file@52', source_url_with_rev, :class => 'source'), + 'source:/some/file@branch' => link_to('source:/some/file@branch', source_url_with_branch, :class => 'source'), 'source:/some/file.ext@52' => link_to('source:/some/file.ext@52', source_url_with_rev_and_ext, :class => 'source'), 'source:/some/file#L110' => link_to('source:/some/file#L110', source_url + "#L110", :class => 'source'), 'source:/some/file.ext#L110' => link_to('source:/some/file.ext#L110', source_url_with_ext + "#L110", :class => 'source'), @@ -323,6 +339,7 @@ 'export:/some/file.ext' => link_to('export:/some/file.ext', export_url_with_ext, :class => 'source download'), 'export:/some/file@52' => link_to('export:/some/file@52', export_url_with_rev, :class => 'source download'), 'export:/some/file.ext@52' => link_to('export:/some/file.ext@52', export_url_with_rev_and_ext, :class => 'source download'), + 'export:/some/file@branch' => link_to('export:/some/file@branch', export_url_with_branch, :class => 'source download'), # forum 'forum#2' => link_to('Discussion', board_url, :class => 'board'), 'forum:Discussion' => link_to('Discussion', board_url, :class => 'board'), @@ -553,9 +570,8 @@ end def test_attachment_links - attachment_link = link_to('error281.txt', {:controller => 'attachments', :action => 'download', :id => '1'}, :class => 'attachment') to_test = { - 'attachment:error281.txt' => attachment_link + 'attachment:error281.txt' => '<a href="/attachments/download/1/error281.txt" class="attachment">error281.txt</a>' } to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text, :attachments => Issue.find(3).attachments), "#{text} failed" } end @@ -565,7 +581,7 @@ a1 = Attachment.generate!(:filename => "test.txt", :created_on => 1.hour.ago) a2 = Attachment.generate!(:filename => "test.txt") - assert_equal %(<p><a href="/attachments/download/#{a2.id}" class="attachment">test.txt</a></p>), + assert_equal %(<p><a href="/attachments/download/#{a2.id}/test.txt" class="attachment">test.txt</a></p>), textilizable('attachment:test.txt', :attachments => [a1, a2]) end @@ -741,7 +757,7 @@ expected = <<-EXPECTED <p><a href="/projects/ecookbook/wiki/CookBook_documentation" class="wiki-page">CookBook documentation</a></p> -<p><a href="/issues/1" class="issue status-1 priority-4 priority-lowest" title="Can't print recipes (New)">#1</a></p> +<p><a href="/issues/1" class="#{Issue.find(1).css_classes}" title="Can't print recipes (New)">#1</a></p> <pre> [[CookBook documentation]] @@ -1076,6 +1092,26 @@ assert_equal ::I18n.t(:label_user_anonymous), t end + def test_link_to_attachment + a = Attachment.find(3) + assert_equal '<a href="/attachments/3/logo.gif">logo.gif</a>', + link_to_attachment(a) + assert_equal '<a href="/attachments/3/logo.gif">Text</a>', + link_to_attachment(a, :text => 'Text') + assert_equal '<a href="/attachments/3/logo.gif" class="foo">logo.gif</a>', + link_to_attachment(a, :class => 'foo') + assert_equal '<a href="/attachments/download/3/logo.gif">logo.gif</a>', + link_to_attachment(a, :download => true) + assert_equal '<a href="http://test.host/attachments/3/logo.gif">logo.gif</a>', + link_to_attachment(a, :only_path => false) + end + + def test_thumbnail_tag + a = Attachment.find(3) + assert_equal '<a href="/attachments/3/logo.gif" title="logo.gif"><img alt="3" src="/attachments/thumbnail/3" /></a>', + thumbnail_tag(a) + end + def test_link_to_project project = Project.find(1) assert_equal %(<a href="/projects/ecookbook">eCookbook</a>), @@ -1088,6 +1124,17 @@ link_to_project(project, {:action => 'settings'}, :class => "project") end + def test_link_to_project_settings + project = Project.find(1) + assert_equal '<a href="/projects/ecookbook/settings">eCookbook</a>', link_to_project_settings(project) + + project.status = Project::STATUS_CLOSED + assert_equal '<a href="/projects/ecookbook">eCookbook</a>', link_to_project_settings(project) + + project.status = Project::STATUS_ARCHIVED + assert_equal 'eCookbook', link_to_project_settings(project) + end + def test_link_to_legacy_project_with_numerical_identifier_should_use_id # numeric identifier are no longer allowed Project.update_all "identifier=25", "id=1" @@ -1163,19 +1210,4 @@ def test_javascript_include_tag_for_plugin_should_pick_the_plugin_javascript assert_match 'src="/plugin_assets/foo/javascripts/scripts.js"', javascript_include_tag("scripts", :plugin => :foo) end - - def test_per_page_links_should_show_usefull_values - set_language_if_valid 'en' - stubs(:link_to).returns("[link]") - - with_settings :per_page_options => '10, 25, 50, 100' do - assert_nil per_page_links(10, 3) - assert_nil per_page_links(25, 3) - assert_equal "Per page: 10, [link]", per_page_links(10, 22) - assert_equal "Per page: [link], 25", per_page_links(25, 22) - assert_equal "Per page: [link], [link], 50", per_page_links(50, 22) - assert_equal "Per page: [link], 25, [link]", per_page_links(25, 26) - assert_equal "Per page: [link], 25, [link], [link]", per_page_links(25, 120) - end - end end