Reopen issues if necessary when creating a report.
This commit is contained in:
parent
e23ff48dce
commit
3a386f185c
2 changed files with 22 additions and 1 deletions
|
@ -17,7 +17,7 @@ class ReportsController < ApplicationController
|
|||
|
||||
if @report.save
|
||||
@report.issue.save
|
||||
# FIXME: reopen issue if necessary
|
||||
@report.issue.reopen! unless @report.issue.open?
|
||||
redirect_to root_path, :notice => t("issues.create.successful_report")
|
||||
else
|
||||
redirect_to new_report_path(:reportable_type => @report.issue.reportable_type, :reportable_id => @report.issue.reportable_id), :notice => t("issues.create.provide_details")
|
||||
|
|
|
@ -28,4 +28,25 @@ class ReportDiaryEntryTest < ApplicationSystemTestCase
|
|||
|
||||
assert page.has_content? "Your report has been registered sucessfully"
|
||||
end
|
||||
|
||||
def test_it_reopens_issue
|
||||
issue = create(:issue, :reportable => @diary_entry)
|
||||
issue.resolve!
|
||||
|
||||
sign_in_as(create(:user))
|
||||
visit diary_entry_path(@diary_entry.user.display_name, @diary_entry)
|
||||
assert page.has_content? @diary_entry.title
|
||||
|
||||
click_on "\u2690"
|
||||
assert page.has_content? "Report"
|
||||
assert page.has_content? I18n.t("issues.new.disclaimer.intro")
|
||||
|
||||
choose I18n.t("reports.categories.DiaryEntry.spam")
|
||||
fill_in "report_details", :with => "This is advertising"
|
||||
click_on "Save changes"
|
||||
|
||||
issue.reload
|
||||
assert !issue.resolved?
|
||||
assert issue.open?
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Add table
Reference in a new issue