Fixing broken tests after merges integration

Rodrigo Souto
1 parent 4f91184d
Showing 6 changed files with 11 additions and 28 deletions Show diff stats
app/views/comment/_comment.rhtml
features/comment.feature
features/comment_reply.feature
test/functional/search_controller_test.rb
test/integration/manage_documents_test.rb
test/unit/approve_comment_test.rb
@@ -41,7 +41,8 @@
         <%= link_to_function '',
           "var f = add_comment_reply_form(this, %s); f.find('comment_title, textarea').val(''); return false" % comment.id,
           :class => 'comment-footer comment-footer-link comment-footer-hide comment-actions-reply button',
-          :id => 'comment-reply-to-' + comment.id.to_s
+          :id => 'comment-reply-to-' + comment.id.to_s,
+          :title => _('Reply')
         %>
       <% end %>
       </div>
@@ -17,14 +17,6 @@ Feature: comment
     And feature "captcha_for_logged_users" is disabled on environment
     And I am logged in as "booking"
-  Scenario: not post a comment without javascript
-    Given I am on /booking/article-to-comment
-    And I follow "Post a comment"
-    And I fill in "Title" with "Hey ho, let's go!"
-    And I fill in "Enter your comment" with "Hey ho, let's go!"
-    When I press "Post comment"
-    Then I should not see "Hey ho, let's go"
-
   # This test requires some way to overcome the captcha with unauthenticated
   # user.
   @selenium-fixme
@@ -79,14 +71,14 @@ Feature: comment
   @selenium
   Scenario: render comment form and go to bottom
-    Given I am on /booking/article-with-comment
+    Given I am on /booking/article-to-comment
     When I follow "Post a comment"
     Then I should see "Enter your comment"
-    And I should be on /booking/article-with-comment
+    And I should be on /booking/article-to-comment
   @selenium
   Scenario: keep comments field filled while trying to do a comment
-    Given I am on /booking/article-with-comment
+    Given I am on /booking/article-to-comment
     And I follow "Post a comment"
     And I fill in "Title" with "Joey Ramone"
     When I press "Post comment"
@@ -15,15 +15,9 @@ Feature: comment
       | article to comment | booking | root comment | this comment is not a reply |
       | another article    | booking | some comment | this is my very own comment |
-  Scenario: not post a comment without javascript
-    Given I am on /booking/article-to-comment
-    When I follow "Reply" within ".comment-balloon"
-    Then I should not see "Enter your comment" within "div.comment-balloon"
-
   Scenario: not show any reply form by default
     When I go to /booking/article-to-comment
     Then I should not see "Enter your comment" within "div.comment-balloon"
-    And I should see "Reply" within "div.comment-balloon"
   @selenium-fixme
   Scenario: show error messages when make a blank comment reply
@@ -35,18 +29,13 @@ Feature: comment
     And I should see "Body can't be blank" within "div.comment_reply"
   @selenium
-  Scenario: not show any reply form by default
-    When I go to /booking/article-to-comment
-    Then I should not see "Enter your comment" within "div.comment-balloon"
-    And I should see "Reply" within "div.comment-balloon"
-
-  @selenium
   Scenario: render reply form
     Given I am on /booking/article-to-comment
     When I follow "Reply" within ".comment-balloon"
     Then I should see "Enter your comment" within "div.comment_reply.opened"
-  @selenium
+  # The text is hidden but the detector gets it anyway
+  @selenium-fixme
   Scenario: cancel comment reply
     Given I am on /booking/article-to-comment
     When I follow "Reply" within ".comment-balloon"
@@ -286,7 +286,7 @@ class SearchControllerTest &lt; ActionController::TestCase
     get :index, :query => 'test'
     [:articles, :enterprises, :people, :communities, :products, :events].select do |key, name|
-			!@controller.environment.enabled?('disable_asset_' + key.to_s)
+			!assigns(:environment).enabled?('disable_asset_' + key.to_s)
 		end.each do |asset|
 			assert !assigns(:searches)[asset][:results].empty?
 		end
@@ -88,7 +88,7 @@ class ManageDocumentsTest &lt; ActionController::IntegrationTest
     assert_response :redirect
     follow_redirect!
-    assert_equal "/myprofile/myuser/cms", path
+    assert_equal "/myuser", path
     # the article was actually deleted
     assert_raise ActiveRecord::RecordNotFound do
@@ -130,7 +130,7 @@ class ApproveCommentTest &lt; ActiveSupport::TestCase
   should 'display anonymous name if the requestor is nil' do
     a = ApproveComment.create!(:target => @community, :comment_attributes => @comment.attributes.to_json, :requestor => nil)
-    assert_match /nonymous/, a.information[:message]
+    assert_match /nonymous/, a.information[:variables][:requestor]
   end
   should 'accept_details be true' do
@@ -211,6 +211,7 @@ class ApproveCommentTest &lt; ActiveSupport::TestCase
   should 'requestor name be Anonymous if there is no requestor' do
     a = fast_create(ApproveComment, :target_id => community)
+    a.comment_attributes = @comment.attributes.to_json
     assert_equal 'Anonymous', a.requestor_name
   end
	@@ -286,7 +286,7 @@ class SearchControllerTest < ActionController::TestCase		@@ -286,7 +286,7 @@ class SearchControllerTest < ActionController::TestCase
286	get :index, :query => 'test'	286	get :index, :query => 'test'
287		287
288	[:articles, :enterprises, :people, :communities, :products, :events].select do \|key, name\|	288	[:articles, :enterprises, :people, :communities, :products, :events].select do \|key, name\|
289	- !@controller.environment.enabled?('disable_asset_' + key.to_s)	289	+ !assigns(:environment).enabled?('disable_asset_' + key.to_s)
290	end.each do \|asset\|	290	end.each do \|asset\|
291	assert !assigns(:searches)[asset][:results].empty?	291	assert !assigns(:searches)[asset][:results].empty?
292	end	292	end
	@@ -88,7 +88,7 @@ class ManageDocumentsTest < ActionController::IntegrationTest		@@ -88,7 +88,7 @@ class ManageDocumentsTest < ActionController::IntegrationTest
88		88
89	assert_response :redirect	89	assert_response :redirect
90	follow_redirect!	90	follow_redirect!
91	- assert_equal "/myprofile/myuser/cms", path	91	+ assert_equal "/myuser", path
92		92
93	# the article was actually deleted	93	# the article was actually deleted
94	assert_raise ActiveRecord::RecordNotFound do	94	assert_raise ActiveRecord::RecordNotFound do