Refactor some controller tests to use factories.

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

diff --git a/test/controllers/redactions_controller_test.rb b/test/controllers/redactions_controller_test.rb
index 44a76069468d3c423217b8128cf64fe85e2a91d5..5a72830e4805c85ff39f03aefaad4e5be4ce2cfa 100644 (file)
--- a/test/controllers/redactions_controller_test.rb
+++ b/test/controllers/redactions_controller_test.rb
@@ -1,5 +1,5 @@
-require 'test_helper'
-require 'redactions_controller'
+require "test_helper"
+require "redactions_controller"

 
 class RedactionsControllerTest < ActionController::TestCase
   api_fixtures
 
 class RedactionsControllerTest < ActionController::TestCase
   api_fixtures


@@ -37,40 +37,77 @@ class RedactionsControllerTest < ActionController::TestCase
     )
   end
 
     )
   end
 

-  def test_moderators_can_create
-    session[:user] = users(:moderator_user).id
+  def test_index
+    get :index
+    assert_response :success
+    assert_template :index
+    assert_select "ul#redaction_list", 1 do
+      assert_select "li", Redaction.count
+    end
+  end
+
+  def test_new
+    get :new
+    assert_response :redirect
+    assert_redirected_to login_path(:referer => new_redaction_path)
+  end
+
+  def test_new_moderator
+    session[:user] = create(:moderator_user).id
+
+    get :new
+    assert_response :success
+    assert_template :new
+  end
+
+  def test_new_non_moderator
+    session[:user] = create(:user).id
+
+    get :new
+    assert_response :redirect
+    assert_redirected_to redactions_path
+  end
+
+  def test_create_moderator
+    session[:user] = create(:moderator_user).id

 
     post :create, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :redirect
 
     post :create, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :redirect

-    assert_redirected_to(redaction_path(Redaction.find_by_title("Foo")))
+    assert_redirected_to(redaction_path(Redaction.find_by(:title => "Foo")))

   end
 
   end
 

-  def test_non_moderators_cant_create
-    session[:user] = users(:public_user).id
+  def test_create_moderator_invalid
+    session[:user] = create(:moderator_user).id
+
+    post :create, :redaction => { :title => "Foo", :description => "" }
+    assert_response :success
+    assert_template :new
+  end
+
+  def test_create_non_moderator
+    session[:user] = create(:user).id

 
     post :create, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :forbidden
   end
 
 
     post :create, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :forbidden
   end
 

-  def test_moderators_can_delete_empty
-    session[:user] = users(:moderator_user).id
+  def test_destroy_moderator_empty
+    session[:user] = create(:moderator_user).id

 
 

-    # remove all elements from the redaction
-    redaction = redactions(:example)
-    redaction.old_nodes.each     { |n| n.redaction = nil; n.save! }
-    redaction.old_ways.each      { |w| w.redaction = nil; w.save! }
-    redaction.old_relations.each { |r| r.redaction = nil; r.save! }
+    # create an empty redaction
+    redaction = create(:redaction)

 
     delete :destroy, :id => redaction.id
     assert_response :redirect
     assert_redirected_to(redactions_path)
   end
 
 
     delete :destroy, :id => redaction.id
     assert_response :redirect
     assert_redirected_to(redactions_path)
   end
 

-  def test_moderators_cant_delete_nonempty
-    session[:user] = users(:moderator_user).id
+  def test_destroy_moderator_non_empty
+    session[:user] = create(:moderator_user).id

 
 

-    # leave elements in the redaction
-    redaction = redactions(:example)
+    # create elements in the redaction
+    redaction = create(:redaction)
+    create(:old_node, :redaction => redaction)

 
     delete :destroy, :id => redaction.id
     assert_response :redirect
 
     delete :destroy, :id => redaction.id
     assert_response :redirect


@@ -78,42 +115,60 @@ class RedactionsControllerTest < ActionController::TestCase
     assert_match /^Redaction is not empty/, flash[:error]
   end
 
     assert_match /^Redaction is not empty/, flash[:error]
   end
 

-  def test_non_moderators_cant_delete
-    session[:user] = users(:public_user).id
+  def test_delete_non_moderator
+    session[:user] = create(:user).id

 
 

-    delete :destroy, :id => redactions(:example).id
+    delete :destroy, :id => create(:redaction).id

     assert_response :forbidden
   end
 
     assert_response :forbidden
   end
 

-  def test_moderators_can_edit
-    session[:user] = users(:moderator_user).id
+  def test_edit
+    redaction = create(:redaction)
+
+    get :edit, :id => redaction.id
+    assert_response :redirect
+    assert_redirected_to login_path(:referer => edit_redaction_path(redaction))
+  end

 
 

-    get :edit, :id => redactions(:example).id
+  def test_edit_moderator
+    session[:user] = create(:moderator_user).id
+
+    get :edit, :id => create(:redaction).id

     assert_response :success
   end
 
     assert_response :success
   end
 

-  def test_non_moderators_cant_edit
-    session[:user] = users(:public_user).id
+  def test_edit_non_moderator
+    session[:user] = create(:user).id

 
 

-    get :edit, :id => redactions(:example).id
+    get :edit, :id => create(:redaction).id

     assert_response :redirect
     assert_redirected_to(redactions_path)
   end
 
     assert_response :redirect
     assert_redirected_to(redactions_path)
   end
 

-  def test_moderators_can_update
-    session[:user] = users(:moderator_user).id
+  def test_update_moderator
+    session[:user] = create(:moderator_user).id

 
 

-    redaction = redactions(:example)
+    redaction = create(:redaction)

 
     put :update, :id => redaction.id, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :redirect
     assert_redirected_to(redaction_path(redaction))
   end
 
 
     put :update, :id => redaction.id, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :redirect
     assert_redirected_to(redaction_path(redaction))
   end
 

-  def test_non_moderators_cant_update
-    session[:user] = users(:public_user).id
+  def test_update_moderator_invalid
+    session[:user] = create(:moderator_user).id
+
+    redaction = create(:redaction)
+
+    put :update, :id => redaction.id, :redaction => { :title => "Foo", :description => "" }
+    assert_response :success
+    assert_template :edit
+  end
+
+  def test_updated_non_moderator
+    session[:user] = create(:user).id

 
 

-    redaction = redactions(:example)
+    redaction = create(:redaction)

 
     put :update, :id => redaction.id, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :forbidden
 
     put :update, :id => redaction.id, :redaction => { :title => "Foo", :description => "Description here." }
     assert_response :forbidden