Merge remote-tracking branch 'upstream/pull/4169'

[rails.git] / test / controllers / reports_controller_test.rb

diff --git a/test/controllers/reports_controller_test.rb b/test/controllers/reports_controller_test.rb
index d9bc190d8b1d9a5495177c2dd5f1cfa318ad6f09..8374e76d9f5d0efefe9fe39239cb8d7d8e4fbeeb 100644 (file)
--- a/test/controllers/reports_controller_test.rb
+++ b/test/controllers/reports_controller_test.rb
@@ -1,9 +1,9 @@
 require "test_helper"
 
 require "test_helper"
 

-class ReportsControllerTest < ActionController::TestCase
+class ReportsControllerTest < ActionDispatch::IntegrationTest

   def test_new_report_without_login
     target_user = create(:user)
   def test_new_report_without_login
     target_user = create(:user)

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :redirect
     assert_redirected_to login_path(:referer => new_report_path(:reportable_id => target_user.id, :reportable_type => "User"))
   end
     assert_response :redirect
     assert_redirected_to login_path(:referer => new_report_path(:reportable_id => target_user.id, :reportable_type => "User"))
   end


@@ -11,28 +11,22 @@ class ReportsControllerTest < ActionController::TestCase
   def test_new_report_after_login
     target_user = create(:user)
 
   def test_new_report_after_login
     target_user = create(:user)
 

-    session[:user] = create(:user).id
-
-    assert_equal 0, Issue.count
+    session_for(create(:user))

 
     # Create an Issue and a report
 
     # Create an Issue and a report

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"
     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"

-      post :create,
-           :params => {
-             :report => {
-               :details => details,
-               :category => category,
-               :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
-             }
-           }
+      post reports_path(:report => {
+                          :details => details,
+                          :category => category,
+                          :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
+                        })

     end
     end

-    assert_equal 1, Issue.count

     assert_response :redirect
     assert_response :redirect

-    assert_redirected_to user_path(target_user.display_name)
+    assert_redirected_to user_path(target_user)

   end
 
   def test_new_report_with_incomplete_details
   end
 
   def test_new_report_with_incomplete_details


@@ -40,45 +34,43 @@ class ReportsControllerTest < ActionController::TestCase
     target_user = create(:user)
 
     # Login
     target_user = create(:user)
 
     # Login

-    session[:user] = create(:user).id
-
-    assert_equal 0, Issue.count
+    session_for(create(:user))

 
     # Create an Issue and a report
 
     # Create an Issue and a report

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"
     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"

-      post :create,
-           :params => {
-             :report => {
-               :details => details,
-               :category => category,
-               :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
-             }
-           }
+      post reports_path(:report => {
+                          :details => details,
+                          :category => category,
+                          :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
+                        })

     end
     end

-    assert_equal 1, Issue.count

     assert_response :redirect
     assert_response :redirect

-    assert_redirected_to user_path(target_user.display_name)
+    assert_redirected_to user_path(target_user)
+
+    issue = Issue.last

 
 

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    assert_equal 1, issue.reports.count
+
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :success
 
     # Report without details
     assert_no_difference "Issue.count" do
       category = "other"
     assert_response :success
 
     # Report without details
     assert_no_difference "Issue.count" do
       category = "other"

-      post :create,
-           :params => {
-             :report => {
-               :category => category,
-               :issue => { :reportable_id => 1, :reportable_type => "User" }
-             }
-           }
+      post reports_path(:report => {
+                          :category => category,
+                          :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
+                        })

     end
     end

-    assert_response :redirect
-    assert_equal 1, Issue.find_by(:reportable_id => target_user.id, :reportable_type => "User").reports.count
+    assert_response :success
+    assert_template :new
+    assert_match(/Please provide the required details/, flash[:notice])
+
+    assert_equal 1, issue.reports.count

   end
 
   def test_new_report_with_complete_details
   end
 
   def test_new_report_with_complete_details


@@ -86,46 +78,41 @@ class ReportsControllerTest < ActionController::TestCase
     target_user = create(:user)
 
     # Login
     target_user = create(:user)
 
     # Login

-    session[:user] = create(:user).id
-
-    assert_equal 0, Issue.count
+    session_for(create(:user))

 
     # Create an Issue and a report
 
     # Create an Issue and a report

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"
     assert_response :success
     assert_difference "Issue.count", 1 do
       details = "Details of a report"
       category = "other"

-      post :create,
-           :params => {
-             :report => {
-               :details => details,
-               :category => category,
-               :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
-             }
-           }
+      post reports_path(:report => {
+                          :details => details,
+                          :category => category,
+                          :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
+                        })

     end
     end

-    assert_equal 1, Issue.count

     assert_response :redirect
     assert_response :redirect

-    assert_redirected_to user_path(target_user.display_name)
+    assert_redirected_to user_path(target_user)
+
+    issue = Issue.last
+
+    assert_equal 1, issue.reports.count

 
     # Create a report for an existing Issue
 
     # Create a report for an existing Issue

-    get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
+    get new_report_path(:reportable_id => target_user.id, :reportable_type => "User")

     assert_response :success
     assert_no_difference "Issue.count" do
       details = "Details of another report under the same issue"
       category = "other"
     assert_response :success
     assert_no_difference "Issue.count" do
       details = "Details of another report under the same issue"
       category = "other"

-      post :create,
-           :params => {
-             :report => {
-               :details => details,
-               :category => category,
-               :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
-             }
-           }
+      post reports_path(:report => {
+                          :details => details,
+                          :category => category,
+                          :issue => { :reportable_id => target_user.id, :reportable_type => "User" }
+                        })

     end
     assert_response :redirect
     end
     assert_response :redirect

-    report_count = Issue.find_by(:reportable_id => target_user.id, :reportable_type => "User").reports.count
-    assert_equal 2, report_count
+
+    assert_equal 2, issue.reports.count

   end
 end
   end
 end