X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/9401e451d18d34ad8fd27161e66496171dbed54f..bb561d3ea9441b9a1c9032a80d751b640ddd2a23:/test/system/issues_test.rb?ds=sidebyside diff --git a/test/system/issues_test.rb b/test/system/issues_test.rb index 2a9862ed9..327b888bc 100644 --- a/test/system/issues_test.rb +++ b/test/system/issues_test.rb @@ -5,7 +5,7 @@ class IssuesTest < ApplicationSystemTestCase def test_view_issues_not_logged_in visit issues_path - assert_content I18n.t("sessions.new.title") + assert_content "Log in" end def test_view_issues_normal_user @@ -19,7 +19,7 @@ class IssuesTest < ApplicationSystemTestCase sign_in_as(create(:moderator_user)) visit issues_path - assert_content I18n.t("issues.index.issues_not_found") + assert_content I18n.t("issues.page.issues_not_found") end def test_view_issues @@ -41,6 +41,23 @@ class IssuesTest < ApplicationSystemTestCase assert_selector "strong", :text => "with kramdown" end + def test_view_issue_rich_text_container + sign_in_as(create(:moderator_user)) + issue = create(:issue, :assigned_role => "moderator") + issue.reports << create(:report, :details => "paragraph one\n\n---\n\nparagraph two") + + visit issue_path(issue) + assert_content I18n.t("issues.show.reports", :count => 1) + richtext = find "div.richtext" + richtext_elements = richtext.all "*" + assert_equal 3, richtext_elements.size + assert_equal "p", richtext_elements[0].tag_name + assert_equal "paragraph one", richtext_elements[0].text + assert_equal "hr", richtext_elements[1].tag_name + assert_equal "p", richtext_elements[2].tag_name + assert_equal "paragraph two", richtext_elements[2].text + end + def test_view_issues_with_no_reported_user sign_in_as(create(:moderator_user)) anonymous_note = create(:note_with_comments) @@ -64,22 +81,22 @@ class IssuesTest < ApplicationSystemTestCase visit issues_path fill_in "search_by_user", :with => good_user.display_name click_on "Search" - assert_no_content I18n.t("issues.index.user_not_found") - assert_content I18n.t("issues.index.issues_not_found") + assert_no_content I18n.t("issues.page.user_not_found") + assert_content I18n.t("issues.page.issues_not_found") # User doesn't exist visit issues_path fill_in "search_by_user", :with => "Nonexistent User" click_on "Search" - assert_content I18n.t("issues.index.user_not_found") - assert_content I18n.t("issues.index.issues_not_found") + assert_content I18n.t("issues.page.user_not_found") + assert_no_content I18n.t("issues.page.issues_not_found") # Find Issue against bad_user visit issues_path fill_in "search_by_user", :with => bad_user.display_name click_on "Search" - assert_no_content I18n.t("issues.index.user_not_found") - assert_no_content I18n.t("issues.index.issues_not_found") + assert_no_content I18n.t("issues.page.user_not_found") + assert_no_content I18n.t("issues.page.issues_not_found") end def test_commenting @@ -108,10 +125,27 @@ class IssuesTest < ApplicationSystemTestCase check :reassign click_on "Add Comment" + assert_content "and the issue was reassigned" + assert_current_path issues_path(:status => "open") + issue.reload assert_equal "moderator", issue.assigned_role end + def test_reassign_issue_as_super_user + issue = create(:issue) + sign_in_as(create(:super_user)) + + visit issue_path(issue) + + fill_in :issue_comment_body, :with => "reassigning to moderators" + check :reassign + click_on "Add Comment" + + assert_content "and the issue was reassigned" + assert_current_path issue_path(issue) + end + def test_issue_index_with_multiple_roles user1 = create(:user) user2 = create(:user) @@ -124,7 +158,50 @@ class IssuesTest < ApplicationSystemTestCase visit issues_path - assert_link I18n.t("issues.index.reports_count", :count => issue1.reports_count), :href => issue_path(issue1) - assert_link I18n.t("issues.index.reports_count", :count => issue2.reports_count), :href => issue_path(issue2) + assert_link I18n.t("issues.page.reports_count", :count => issue1.reports_count), :href => issue_path(issue1) + assert_link I18n.t("issues.page.reports_count", :count => issue2.reports_count), :href => issue_path(issue2) + end + + def test_issues_pagination + 1.upto(8).each do |n| + user = create(:user, :display_name => "extra_#{n}") + create(:issue, :reportable => user, :reported_user => user, :assigned_role => "administrator") + end + + sign_in_as(create(:administrator_user)) + + visit issues_path(:limit => 5) + + # First Page + assert_no_content I18n.t("issues.page.user_not_found") + assert_no_content I18n.t("issues.page.issues_not_found") + 4.upto(8).each do |n| + assert_content(/extra_#{n}[^\d]/i, :count => 2) + end + 1.upto(3).each do |n| + assert_no_content(/extra_#{n}[^\d]/i) + end + + # Second Page + click_on "Older Issues" + assert_no_content I18n.t("issues.page.user_not_found") + assert_no_content I18n.t("issues.page.issues_not_found") + 4.upto(8).each do |n| + assert_no_content(/extra_#{n}[^\d]/i) + end + 1.upto(3).each do |n| + assert_content(/extra_#{n}[^\d]/i, :count => 2) + end + + # Back to First Page + click_on "Newer Issues" + assert_no_content I18n.t("issues.page.user_not_found") + assert_no_content I18n.t("issues.page.issues_not_found") + 4.upto(8).each do |n| + assert_content(/extra_#{n}[^\d]/i, :count => 2) + end + 1.upto(3).each do |n| + assert_no_content(/extra_#{n}[^\d]/i) + end end end