Tidy HelpControllerSpec

1 files changed, 41 insertions, 29 deletions

diff --git a/spec/controllers/help_controller_spec.rb b/spec/controllers/help_controller_spec.rb
index cc024f840..d552127ba 100644
--- a/spec/controllers/help_controller_spec.rb
+++ b/spec/controllers/help_controller_spec.rb
@@ -1,48 +1,60 @@
 # -*- coding: utf-8 -*-
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
-describe HelpController, "when using help" do
+describe HelpController do
     render_views
 
-    it "shows the about page" do
-        get :about
-    end
+    describe :about do
 
-    it "shows contact form" do
-        get :contact
-    end
+        it 'shows the about page' do
+            get :about
+        end
 
-    it "sends a contact message" do
-        post :contact, { :contact => {
-                :name => "Vinny Vanilli",
-                :email => "vinny@localhost",
-                :subject => "Why do I have such an ace name?",
-                :message => "You really should know!!!\n\nVinny",
-            }, :submitted_contact_form => 1
-        }
-        response.should redirect_to(:controller => 'general', :action => 'frontpage')
-
-        deliveries = ActionMailer::Base.deliveries
-        deliveries.size.should  == 1
-        deliveries[0].body.should include("really should know")
-        deliveries.clear
     end
 
-    describe 'when requesting a page in a supported locale ' do
+    describe 'GET contact' do
 
-        before do
-            # Prepend our fixture templates
-            fixture_theme_path = File.join(Rails.root, 'spec', 'fixtures', 'theme_views', 'theme_one')
-            controller.prepend_view_path fixture_theme_path
+        it 'shows contact form' do
+            get :contact
         end
 
-        it 'should render the locale-specific template if available' do
-            get :contact, {:locale => 'es'}
-            response.body.should match('contáctenos theme one')
+        describe 'when requesting a page in a supported locale' do
+
+            before do
+                # Prepend our fixture templates
+                fixture_theme_path = File.join(Rails.root, 'spec', 'fixtures', 'theme_views', 'theme_one')
+                controller.prepend_view_path fixture_theme_path
+            end
+
+            it 'should render the locale-specific template if available' do
+                get :contact, {:locale => 'es'}
+                response.body.should match('contáctenos theme one')
+            end
+
         end
 
     end
 
+    describe 'POST contact' do
+
+        it 'sends a contact message' do
+            post :contact, { :contact => {
+                    :name => 'Vinny Vanilli',
+                    :email => 'vinny@localhost',
+                    :subject => 'Why do I have such an ace name?',
+                    :comment => '',
+                    :message => "You really should know!!!\n\nVinny",
+                }, :submitted_contact_form => 1
+            }
+            response.should redirect_to(:controller => 'general', :action => 'frontpage')
+
+            deliveries = ActionMailer::Base.deliveries
+            deliveries.size.should == 1
+            deliveries[0].body.should include('really should know')
+            deliveries.clear
+        end
+
+    end
 
 end