Fixed test: comment.feature

Rodrigo Souto
1 parent 9a98d064
Showing 6 changed files with 80 additions and 50 deletions Show diff stats
app/views/content_viewer/_comment_form.rhtml
app/views/content_viewer/view_page.rhtml
features/comment.feature
features/step_definitions/web_steps.rb
public/javascripts/comment_form.js
public/stylesheets/application.css
@@ -21,8 +21,6 @@ function submit_comment_form(button) {
 }
 </script>
-<% focus_on = logged_in? ? 'title' : 'name' %>
-
 <% if @comment && @comment.errors.any? && @comment.reply_of_id.blank? %>
   <%= error_messages_for :comment %>
   <script type="text/javascript">jQuery(function() { document.location.href = '#page-comment-form'; });</script>
@@ -32,17 +30,7 @@ function submit_comment_form(button) {
 <div class="post_comment_box <%= @form_div %>">
-<% if display_link %>
-  <h4 onclick="var d = jQuery(this).parent('.post_comment_box');
-               if (d.hasClass('closed')) {
-                 d.removeClass('closed');
-                 d.addClass('opened');
-                 d.find('input[name=comment[title]], textarea').val('');
-                 d.find('.comment_form input[name=comment[<%= focus_on %>]]').focus();
-               }">
-    <%= content_tag('a', '', :name => 'comment_form') + _('Post a comment') %>
-  </h4>
-<% end %>
+  <%= link_to(_('Post a comment'), '#', :class => 'display-comment-form') if display_link %>
 <% unless pass_without_comment_captcha? %>
   <div id="recaptcha-container" style="display: none">
@@ -62,8 +50,6 @@ function submit_comment_form(button) {
 <% end %>
 <% form_tag( url, { :class => 'comment_form' } ) do %>
-  <%= hidden_field_tag(:confirm, 'false') %>
-
   <%= required_fields_message %>
   <% unless logged_in? %>
@@ -84,10 +70,13 @@ function submit_comment_form(button) {
   <%= labelled_form_field(_('Title'), text_field(:comment, :title)) %>
   <%= required labelled_form_field(_('Enter your comment'), text_area(:comment, :body, :rows => 5)) %>
+  <%= hidden_field_tag(:confirm, 'false') %>
+  <%= hidden_field_tag(:view, params[:view])%>
+
   <% button_bar do %>
     <%= submit_button('add', _('Post comment'), :onclick => "submit_comment_form(this); return false") %>
     <% if cancel_triggers_hide %>
-      <%= button_to_function :cancel, _('Cancel'), "f=jQuery(this).parents('.post_comment_box'); f.removeClass('opened'); f.addClass('closed'); return false" %>
+      <%= button :cancel, _('Cancel'), '', :id => 'cancel-comment' %>
     <% else %>
       <%= button('cancel', _('Cancel'), {:action => 'view_page', :profile => profile.identifier, :page => @comment.article.explode_path})%>
     <% end %>
@@ -95,3 +84,5 @@ function submit_comment_form(button) {
 <% end %>
 </div><!-- end class="post_comment_box" -->
+
+<%= javascript_include_tag 'comment_form'%>
@@ -90,7 +90,7 @@
   <% end %>
   <% if @page.accept_comments? && @comments_count > 1 %>
-    <p class="post-comment-button"><a href="#comment_form" onclick="jQuery('#page-comment-form h4').first().trigger('click')"><%= _('Post a comment') %></a></p>
+    <%= link_to(_('Post a comment'), '#', :class => 'display-comment-form') %>
   <% end %>
   <ul class="article-comments-list">
@@ -14,19 +14,23 @@ Feature: comment
       | article              | author  | title | body         |
       | article with comment | booking | hi    | how are you? |
       | article with comment | booking | hello | i am fine    |
+    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 fill in "Name" with "Joey Ramone"
-    And I fill in "e-mail" with "joey@ramones.com"
+    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
   Scenario: post a comment while not authenticated
     Given I am on /booking/article-to-comment
+    And I follow "Post a comment"
     And I fill in "Name" with "Joey Ramone"
     And I fill in "e-mail" with "joey@ramones.com"
     And I fill in "Title" with "Hey ho, let's go!"
@@ -34,58 +38,57 @@ Feature: comment
     When I press "Post comment"
     Then I should see "Hey ho, let's go"
-  @selenium-fixme
+  @selenium
   Scenario: post comment while authenticated
-    Given I am logged in as "booking"
-    And I am on /booking/article-to-comment
+    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 see "Hey ho, let's go"
-  @selenium-fixme
+  @selenium
   Scenario: redirect to right place after comment a picture
     Given the following files
       | owner   | file      | mime      |
       | booking | rails.png | image/png |
-    Given I am logged in as "booking"
     And I am on /booking/rails.png?view=true
+    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 be exactly on /booking/rails.png?view=true
-  @selenium-fixme
+  @selenium
   Scenario: show error messages when make a blank comment
-    Given I am logged in as "booking"
-    And I am on /booking/article-to-comment
+    Given I am on /booking/article-to-comment
+    And I follow "Post a comment"
     When I press "Post comment"
-    Then I should see "Title can't be blank"
-    And I should see "Body can't be blank"
+    Then I should see "Body can't be blank"
   @selenium-fixme
   Scenario: disable post comment button
     Given I am on /booking/article-to-comment
-    And I fill in "Name" with "Joey Ramone"
-    And I fill in "e-mail" with "joey@ramones.com"
+    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 the "value.Post comment" button should not be enabled
-    And I should see "Hey ho, let's go"
+# Implement these steps...
+#    Then "Post comment" button should not be enabled
+#    And I should see "Hey ho, let's go"
-  @selenium-fixme
+  @selenium
   Scenario: render comment form and go to bottom
     Given I am on /booking/article-with-comment
-    When I follow "Post a comment" within ".post-comment-button"
-    Then I should see "Enter your comment" within "div#page-comment-form div.post_comment_box.opened"
-    And I should be exactly on /booking/article-with-comment
-    And I should be moved to anchor "comment_form"
+    When I follow "Post a comment"
+    Then I should see "Enter your comment"
+    And I should be on /booking/article-with-comment
-  @selenium-fixme
+  @selenium
   Scenario: keep comments field filled while trying to do a comment
     Given I am on /booking/article-with-comment
-    And I fill in "Name" with "Joey Ramone"
+    And I follow "Post a comment"
+    And I fill in "Title" with "Joey Ramone"
     When I press "Post comment"
-    Then the "Name" field should contain "Joey Ramone"
-    And I should see "errors prohibited"
+    Then the "Title" field should contain "Joey Ramone"
+    And I should see "Body can't be blank"
@@ -210,6 +210,16 @@ Then /^(?:|I )should be on (.+)$/ do |page_name|
   end
 end
+Then /^(?:|I )should be exactly on (.+)$/ do |page_name|
+  uri = URI.parse(current_url)
+  path_with_params = "#{uri.path}?#{uri.query}"
+  if path_with_params.respond_to? :should
+    path_with_params.should == path_to(page_name)
+  else
+    assert_equal path_to(page_name), path_with_params
+  end
+end
+
 Then /^(?:|I )should have the following query string:$/ do |expected_pairs|
   query = URI.parse(current_url).query
   actual_params = query ? CGI.parse(query) : {}
@@ -0,0 +1,23 @@
+jQuery('.display-comment-form').click(function(){
+  toggleBox('.post_comment_box');
+  jQuery('.display-comment-form').hide();
+  jQuery('form.comment_form input').first().focus();
+  return false;
+});
+
+jQuery('#cancel-comment').click(function(){
+  toggleBox('.post_comment_box');
+  jQuery('.display-comment-form').show();
+  return false
+})
+
+function toggleBox(div_selector){
+  div = jQuery(div_selector);
+  if(div.hasClass('opened')) {
+    div.removeClass('opened');
+    div.addClass('closed');
+  } else {
+    div.removeClass('closed');
+    div.addClass('opened');
+  }
+}
@@ -1106,8 +1106,7 @@ a.comment-picture {
 }
 #article .article-comments-list, #article .article-comments-list ul, #article .article-comments-list li {
   padding: 0;
-  margin: 0;
-  margin-bottom: 10px;
+  margin: 25px 0 10px 0;
   list-style: none;
 }
 .article-comment .button-bar {
@@ -1273,20 +1272,24 @@ a.comment-picture {
   background: transparent url(../images/loading-small.gif) no-repeat left center;
 }
+
 .post_comment_box {
   text-align: center;
   padding: 0px 15px 5px 15px;
-  margin: 10px auto;
 }
-.post-comment-button a, .post_comment_box h4 {
+
+#comments_list a.display-comment-form {
   padding: 1px 20px;
   margin: 0px;
   background: #eee;
-}
-.post-comment-button a, .post_comment_box.closed h4 {
-  display: inline;
   border: 1px solid #888;
-  cursor: pointer;
+  text-decoration: none;
+  color: black;
+  font-size: 14px;
+  font-weight: bold;
+  -moz-border-radius: 4px;
+  -webkit-border-radius: 4px;
+  border-radius: 4px;
 }
 .post_comment_box.opened {
   border: 1px solid #888;
	@@ -90,7 +90,7 @@		@@ -90,7 +90,7 @@
90	<% end %>	90	<% end %>
91		91
92	<% if @page.accept_comments? && @comments_count > 1 %>	92	<% if @page.accept_comments? && @comments_count > 1 %>
93	- <p class="post-comment-button"><a href="#comment_form" onclick="jQuery('#page-comment-form h4').first().trigger('click')"><%= _('Post a comment') %></a></p>	93	+ <%= link_to(_('Post a comment'), '#', :class => 'display-comment-form') %>
94	<% end %>	94	<% end %>
95		95
96	<ul class="article-comments-list">	96	<ul class="article-comments-list">
	@@ -210,6 +210,16 @@ Then /^(?:\|I )should be on (.+)$/ do \|page_name\|		@@ -210,6 +210,16 @@ Then /^(?:\|I )should be on (.+)$/ do \|page_name\|
210	end	210	end
211	end	211	end
212		212
		213	+Then /^(?:\|I )should be exactly on (.+)$/ do \|page_name\|
		214	+ uri = URI.parse(current_url)
		215	+ path_with_params = "#{uri.path}?#{uri.query}"
		216	+ if path_with_params.respond_to? :should
		217	+ path_with_params.should == path_to(page_name)
		218	+ else
		219	+ assert_equal path_to(page_name), path_with_params
		220	+ end
		221	+end
		222	+
213	Then /^(?:\|I )should have the following query string:$/ do \|expected_pairs\|	223	Then /^(?:\|I )should have the following query string:$/ do \|expected_pairs\|
214	query = URI.parse(current_url).query	224	query = URI.parse(current_url).query
215	actual_params = query ? CGI.parse(query) : {}	225	actual_params = query ? CGI.parse(query) : {}
@@ -0,0 +1,23 @@		@@ -0,0 +1,23 @@
	1	+jQuery('.display-comment-form').click(function(){
	2	+ toggleBox('.post_comment_box');
	3	+ jQuery('.display-comment-form').hide();
	4	+ jQuery('form.comment_form input').first().focus();
	5	+ return false;
	6	+});
	7	+
	8	+jQuery('#cancel-comment').click(function(){
	9	+ toggleBox('.post_comment_box');
	10	+ jQuery('.display-comment-form').show();
	11	+ return false
	12	+})
	13	+
	14	+function toggleBox(div_selector){
	15	+ div = jQuery(div_selector);
	16	+ if(div.hasClass('opened')) {
	17	+ div.removeClass('opened');
	18	+ div.addClass('closed');
	19	+ } else {
	20	+ div.removeClass('closed');
	21	+ div.addClass('opened');
	22	+ }
	23	+}