Merge branch 'pull/5627'

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

diff --git a/test/controllers/notes_controller_test.rb b/test/controllers/notes_controller_test.rb
index 5a920349a077903c0372b1f1fbe4c176ab849bcb..024380de15e88dece8ec71aa089b84374cc88334 100644 (file)
--- a/test/controllers/notes_controller_test.rb
+++ b/test/controllers/notes_controller_test.rb
@@ -40,28 +40,30 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
       create(:note_comment, :note => note, :author => second_user)
     end
 
       create(:note_comment, :note => note, :author => second_user)
     end
 

-    get user_notes_path(:display_name => first_user.display_name)
+    get user_notes_path(first_user)

     assert_response :success
     assert_response :success

+    assert_select ".content-heading a[href='#{user_path first_user}']", :text => first_user.display_name

     assert_select "table.note_list tbody tr", :count => 1
 
     assert_select "table.note_list tbody tr", :count => 1
 

-    get user_notes_path(:display_name => second_user.display_name)
+    get user_notes_path(second_user)

     assert_response :success
     assert_response :success

+    assert_select ".content-heading a[href='#{user_path second_user}']", :text => second_user.display_name

     assert_select "table.note_list tbody tr", :count => 1
 
     assert_select "table.note_list tbody tr", :count => 1
 

-    get user_notes_path(:display_name => "non-existent")
+    get user_notes_path("non-existent")

     assert_response :not_found
 
     session_for(moderator_user)
 
     assert_response :not_found
 
     session_for(moderator_user)
 

-    get user_notes_path(:display_name => first_user.display_name)
+    get user_notes_path(first_user)

     assert_response :success
     assert_select "table.note_list tbody tr", :count => 1
 
     assert_response :success
     assert_select "table.note_list tbody tr", :count => 1
 

-    get user_notes_path(:display_name => second_user.display_name)
+    get user_notes_path(second_user)

     assert_response :success
     assert_select "table.note_list tbody tr", :count => 2
 
     assert_response :success
     assert_select "table.note_list tbody tr", :count => 2
 

-    get user_notes_path(:display_name => "non-existent")
+    get user_notes_path("non-existent")

     assert_response :not_found
   end
 
     assert_response :not_found
   end
 


@@ -72,18 +74,27 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
       create(:note_comment, :note => note, :author => user)
     end
 
       create(:note_comment, :note => note, :author => user)
     end
 

-    get user_notes_path(:display_name => user.display_name)
+    get user_notes_path(user)

     assert_response :success
     assert_select "table.note_list tbody tr", :count => 10
 
     assert_response :success
     assert_select "table.note_list tbody tr", :count => 10
 

-    get user_notes_path(:display_name => user.display_name, :page => 2)
+    get user_notes_path(user, :page => 2)

     assert_response :success
     assert_select "table.note_list tbody tr", :count => 10
   end
 
     assert_response :success
     assert_select "table.note_list tbody tr", :count => 10
   end
 

+  def test_index_invalid_paged
+    user = create(:user)
+
+    %w[-1 0 fred].each do |page|
+      get user_notes_path(user, :page => page)
+      assert_redirected_to :controller => :errors, :action => :bad_request
+    end
+  end
+

   def test_empty_page
     user = create(:user)
   def test_empty_page
     user = create(:user)

-    get user_notes_path(:display_name => user.display_name)
+    get user_notes_path(user)

     assert_response :success
     assert_select "h4", :html => "No notes"
   end
     assert_response :success
     assert_select "h4", :html => "No notes"
   end


@@ -91,25 +102,25 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
   def test_read_note
     open_note = create(:note_with_comments)
 
   def test_read_note
     open_note = create(:note_with_comments)
 

-    browse_check :note_path, open_note.id, "notes/show"
+    sidebar_browse_check :note_path, open_note.id, "notes/show"

   end
 
   def test_read_hidden_note
     hidden_note_with_comment = create(:note_with_comments, :status => "hidden")
 
   end
 
   def test_read_hidden_note
     hidden_note_with_comment = create(:note_with_comments, :status => "hidden")
 

-    get note_path(:id => hidden_note_with_comment)
+    get note_path(hidden_note_with_comment)

     assert_response :not_found
     assert_template "browse/not_found"
     assert_template :layout => "map"
 
     assert_response :not_found
     assert_template "browse/not_found"
     assert_template :layout => "map"
 

-    get note_path(:id => hidden_note_with_comment), :xhr => true
+    get note_path(hidden_note_with_comment), :xhr => true

     assert_response :not_found
     assert_template "browse/not_found"
     assert_template :layout => "xhr"
 
     session_for(create(:moderator_user))
 
     assert_response :not_found
     assert_template "browse/not_found"
     assert_template :layout => "xhr"
 
     session_for(create(:moderator_user))
 

-    browse_check :note_path, hidden_note_with_comment.id, "notes/show"
+    sidebar_browse_check :note_path, hidden_note_with_comment.id, "notes/show"

   end
 
   def test_read_note_hidden_comments
   end
 
   def test_read_note_hidden_comments


@@ -117,12 +128,12 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
       create(:note_comment, :note => note, :visible => false)
     end
 
       create(:note_comment, :note => note, :visible => false)
     end
 

-    browse_check :note_path, note_with_hidden_comment.id, "notes/show"
+    sidebar_browse_check :note_path, note_with_hidden_comment.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 1
 
     session_for(create(:moderator_user))
 
     assert_select "div.note-comments ul li", :count => 1
 
     session_for(create(:moderator_user))
 

-    browse_check :note_path, note_with_hidden_comment.id, "notes/show"
+    sidebar_browse_check :note_path, note_with_hidden_comment.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 2
   end
 
     assert_select "div.note-comments ul li", :count => 2
   end
 


@@ -132,20 +143,29 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
       create(:note_comment, :note => note, :author => hidden_user)
     end
 
       create(:note_comment, :note => note, :author => hidden_user)
     end
 

-    browse_check :note_path, note_with_hidden_user_comment.id, "notes/show"
+    sidebar_browse_check :note_path, note_with_hidden_user_comment.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 1
 
     session_for(create(:moderator_user))
 
     assert_select "div.note-comments ul li", :count => 1
 
     session_for(create(:moderator_user))
 

-    browse_check :note_path, note_with_hidden_user_comment.id, "notes/show"
+    sidebar_browse_check :note_path, note_with_hidden_user_comment.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 1
   end
 
     assert_select "div.note-comments ul li", :count => 1
   end
 

+  def test_read_note_hidden_opener
+    hidden_user = create(:user, :deleted)
+    note_with_hidden_opener = create(:note)
+    create(:note_comment, :author => hidden_user, :note => note_with_hidden_opener)
+
+    sidebar_browse_check :note_path, note_with_hidden_opener.id, "notes/show"
+    assert_select "div.note-comments ul li", :count => 0
+  end
+

   def test_read_closed_note
     user = create(:user)
     closed_note = create(:note_with_comments, :closed, :closed_by => user, :comments_count => 2)
 
   def test_read_closed_note
     user = create(:user)
     closed_note = create(:note_with_comments, :closed, :closed_by => user, :comments_count => 2)
 

-    browse_check :note_path, closed_note.id, "notes/show"
+    sidebar_browse_check :note_path, closed_note.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 2
     assert_select "div.details", /Resolved by #{user.display_name}/
 
     assert_select "div.note-comments ul li", :count => 2
     assert_select "div.details", /Resolved by #{user.display_name}/
 


@@ -153,52 +173,58 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
 
     reset!
 
 
     reset!
 

-    browse_check :note_path, closed_note.id, "notes/show"
+    sidebar_browse_check :note_path, closed_note.id, "notes/show"

     assert_select "div.note-comments ul li", :count => 1
     assert_select "div.details", /Resolved by deleted/
   end
 
     assert_select "div.note-comments ul li", :count => 1
     assert_select "div.details", /Resolved by deleted/
   end
 

+  def test_new_note_anonymous
+    get new_note_path
+    assert_response :success
+    assert_template "notes/new"
+    assert_select "#sidebar_content a[href='#{login_path(:referer => new_note_path)}']", :count => 1
+  end
+

   def test_new_note
   def test_new_note

+    session_for(create(:user))
+

     get new_note_path
     assert_response :success
     assert_template "notes/new"
     get new_note_path
     assert_response :success
     assert_template "notes/new"

+    assert_select "#sidebar_content a[href='#{login_path(:referer => new_note_path)}']", :count => 0

   end
 
   end
 

-  private
+  def test_index_filter_by_status
+    user = create(:user)
+    other_user = create(:user)

 
 

-  # This is a convenience method for most of the above checks
-  # First we check that when we don't have an id, it will correctly return a 404
-  # then we check that we get the correct 404 when a non-existant id is passed
-  # then we check that it will get a successful response, when we do pass an id
-  def browse_check(path, id, template)
-    path_method = method(path)
+    open_note = create(:note, :status => "open")
+    create(:note_comment, :note => open_note, :author => user)

 
 

-    assert_raise ActionController::UrlGenerationError do
-      get path_method.call
-    end
+    closed_note = create(:note, :status => "closed")
+    create(:note_comment, :note => closed_note, :author => user)

 
 

-    # assert_raise ActionController::UrlGenerationError do
-    #   get path_method.call(:id => -10) # we won't have an id that's negative
-    # end
+    hidden_note = create(:note, :status => "hidden")
+    create(:note_comment, :note => hidden_note, :author => user)

 
 

-    get path_method.call(:id => 0)
-    assert_response :not_found
-    assert_template "browse/not_found"
-    assert_template :layout => "map"
+    commented_note = create(:note, :status => "open")
+    create(:note_comment, :note => commented_note, :author => other_user)
+    create(:note_comment, :note => commented_note, :author => user)

 
 

-    get path_method.call(:id => 0), :xhr => true
-    assert_response :not_found
-    assert_template "browse/not_found"
-    assert_template :layout => "xhr"
+    get user_notes_path(user, :status => "all")
+    assert_response :success
+    assert_select "table.note_list tbody tr", :count => 3

 
 

-    get path_method.call(:id => id)
+    get user_notes_path(user, :status => "open")

     assert_response :success
     assert_response :success

-    assert_template template
-    assert_template :layout => "map"
+    assert_select "table.note_list tbody tr", :count => 2

 
 

-    get path_method.call(:id => id), :xhr => true
+    get user_notes_path(user, :status => "closed")

     assert_response :success
     assert_response :success

-    assert_template template
-    assert_template :layout => "xhr"
+    assert_select "table.note_list tbody tr", :count => 1
+
+    get user_notes_path(user)
+    assert_response :success
+    assert_select "table.note_list tbody tr", :count => 3

   end
 end
   end
 end