Put the expected value first in assert_equal

[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 8dc10238ecfad5a28730fe48be915a3dc5d949c6..9c39350f95dfce9b1cf223678dc01b7e91a941a9 100644 (file)
--- a/test/controllers/reports_controller_test.rb
+++ b/test/controllers/reports_controller_test.rb
@@ -13,7 +13,7 @@ class ReportsControllerTest < ActionController::TestCase
 
     session[:user] = create(:user).id
 
 
     session[:user] = create(:user).id
 

-    assert_equal Issue.count, 0
+    assert_equal 0, Issue.count

 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }


@@ -28,7 +28,7 @@ class ReportsControllerTest < ActionController::TestCase
              }
            }
     end
              }
            }
     end

-    assert_equal Issue.count, 1
+    assert_equal 1, Issue.count

     assert_response :redirect
     assert_redirected_to root_path
   end
     assert_response :redirect
     assert_redirected_to root_path
   end


@@ -40,7 +40,7 @@ class ReportsControllerTest < ActionController::TestCase
     # Login
     session[:user] = create(:user).id
 
     # Login
     session[:user] = create(:user).id
 

-    assert_equal Issue.count, 0
+    assert_equal 0, Issue.count

 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }


@@ -55,7 +55,7 @@ class ReportsControllerTest < ActionController::TestCase
              }
            }
     end
              }
            }
     end

-    assert_equal Issue.count, 1
+    assert_equal 1, Issue.count

     assert_response :redirect
     assert_redirected_to root_path
 
     assert_response :redirect
     assert_redirected_to root_path
 


@@ -74,7 +74,7 @@ class ReportsControllerTest < ActionController::TestCase
            }
     end
     assert_response :redirect
            }
     end
     assert_response :redirect

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

 
     # Report without details
     assert_no_difference "Issue.count" do
 
     # Report without details
     assert_no_difference "Issue.count" do


@@ -86,7 +86,7 @@ class ReportsControllerTest < ActionController::TestCase
            }
     end
     assert_response :redirect
            }
     end
     assert_response :redirect

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

   end
 
   def test_new_report_with_complete_details
   end
 
   def test_new_report_with_complete_details


@@ -96,7 +96,7 @@ class ReportsControllerTest < ActionController::TestCase
     # Login
     session[:user] = create(:user).id
 
     # Login
     session[:user] = create(:user).id
 

-    assert_equal Issue.count, 0
+    assert_equal 0, Issue.count

 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }
 
     # Create an Issue and a report
     get :new, :params => { :reportable_id => target_user.id, :reportable_type => "User" }


@@ -111,7 +111,7 @@ class ReportsControllerTest < ActionController::TestCase
              }
            }
     end
              }
            }
     end

-    assert_equal Issue.count, 1
+    assert_equal 1, Issue.count

     assert_response :redirect
     assert_redirected_to root_path
 
     assert_response :redirect
     assert_redirected_to root_path
 


@@ -130,6 +130,6 @@ class ReportsControllerTest < ActionController::TestCase
     end
     assert_response :redirect
     report_count = Issue.find_by(:reportable_id => target_user.id, :reportable_type => "User").reports.count
     end
     assert_response :redirect
     report_count = Issue.find_by(:reportable_id => target_user.id, :reportable_type => "User").reports.count

-    assert_equal report_count, 2
+    assert_equal 2, report_count

   end
 end
   end
 end