spree_contact_us 1.0.0

data/.gitignore

@@ -0,0 +1,13 @@
+\#*
+*~
+.#*
+.DS_Store
+.idea
+.project
+coverage
+Gemfile.lock
+tmp
+nbproject
+pkg
+*.swp
+spec/dummy

data/.rspec

@@ -0,0 +1 @@
+--colour

data/.travis.yml

@@ -0,0 +1,11 @@
+before_script:
+  - 'bundle exec rake test_app'
+script: 'bundle exec rspec spec'
+rvm:
+  - 1.9.2
+  - 1.9.3
+  - ruby-head
+language: ruby
+notifications:
+  recipients:
+    - JDutil@BurlingtonWebApps.com

data/Gemfile

@@ -0,0 +1,3 @@
+source 'http://rubygems.org'
+
+gemspec

data/LICENSE

@@ -0,0 +1,26 @@
+Copyright (c) 2012 Jeff Dutil
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without modification,
+are permitted provided that the following conditions are met:
+
+    * Redistributions of source code must retain the above copyright notice,
+      this list of conditions and the following disclaimer.
+    * Redistributions in binary form must reproduce the above copyright notice,
+      this list of conditions and the following disclaimer in the documentation
+      and/or other materials provided with the distribution.
+    * Neither the name Spree nor the names of its contributors may be used to
+      endorse or promote products derived from this software without specific
+      prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

data/README.md

@@ -0,0 +1,98 @@
+# SpreeContactUs [![Build Status](https://secure.travis-ci.org/jdutil/spree_contact_us.png)](http://travis-ci.org/jdutil/spree_contact_us) [![Dependency Status](https://gemnasium.com/jdutil/spree_contact_us.png?travis)](https://gemnasium.com/jdutil/contact_us) [![Click here to lend your support to: ContactUs and make a donation at www.pledgie.com !](https://www.pledgie.com/campaigns/17259.png?skin_name=chrome)][pledgie]
+A Rails 3+ Engine providing a basic contact form as a Spree Extension.
+
+[travis]: http://travis-ci.org/jdutil/spree_contact_us
+[gemnasium]: https://gemnasium.com/jdutil/spree_contact_us
+[pledgie]: http://www.pledgie.com/campaigns/17259
+
+Modified version of the `contact_us` gem to work with Spree.
+
+For more information please see https://github.com/jdutil/contact_us
+
+## REQUIREMENTS
+
+SpreeContactUs requires:
+
+* Spree Core 1.1.0.rc2 or greater
+* Ruby 1.9.2 or greater.
+
+If you would like to run on Spree 1.0.x or Ruby 1.8.7 everything should just work by modifying your forks gemspec.  I do not intend to support these older versions though.
+
+## INSTALLATION
+
+In your `Gemfile`, add the following dependencies:
+
+To install from edge:
+
+    gem 'spree_contact_us', :git => 'https://github.com/jdutil/spree_contact_us'
+
+Or install from latest stable version:
+
+    gem 'spree_contact_us', '~> 1.0.0'
+
+From `Rails.root` run:
+
+    $ bundle
+    $ bundle exec rails g spree_contact_us:install
+
+In `config/initializers/spree_contact_us.rb` modify:
+
+    config.mailer_to = "contact@please-change-me.com"
+
+Change to the email address you would like to receive the form submissions at for example:
+
+    config.mailer_to = "contact@yourdomain.com"
+
+By default the emails from field will be the email entered by the user to easily reply, but this may not be allowed if your required to verify your sending email addresses.
+You may also specify an email address for the notification emails from field:
+
+    config.mailer_from = "dontreply@yourdomain.com"
+
+## CONFIGURATION
+
+If you would like to add a name or subject field to the form you may simply set the options to true within the spree_contact_us initializer located at `config/initializers/spree_contact_us.rb`:
+
+    config.name = true
+    config.subject = true
+
+You may also update your locales under `config/locales/spree_contact_us.en.yml` or create your own.  Please feel free to submit your own locales so that other users will hopefully find this gem more useful.
+
+## USAGE
+
+Visit your website and navigate to `/contact_us` to see the form in action.
+
+## ISSUES
+
+Please report any bugs or feature requests to the Github issues page @ https://github.com/jdutil/spree_contact_us/issues
+
+## CONTRIBUTING
+
+In the spirit of [free software](http://www.fsf.org/licensing/essays/free-sw.html), **everyone** is encouraged to help improve this project.
+
+Here are some ways *you* can contribute:
+
+* by using prerelease versions
+* by reporting bugs
+* by suggesting new features
+* by [translating to a new language](https://github.com/jdutil/spree_contact_us/tree/master/config/locales)
+* by writing or editing documentation
+* by writing specifications
+* by writing code (**no patch is too small**: fix typos, add comments, clean up inconsistent whitespace)
+* by refactoring code
+* by resolving [issues](https://github.com/jdutil/spree_contact_us/issues)
+* by reviewing patches
+
+## Testing
+
+Be sure to bundle your dependencies and then create a dummy test app for the specs to run against.
+
+    $ bundle
+    $ bundle exec rake test_app
+    $ bundle exec rspec spec
+
+## TODO
+
+* Modify settings within the admin
+* Add new language translations
+
+Copyright (c) 2012 Jeff Dutil, released under the [New BSD License](https://github.com/jdutil/spree_contact_us/tree/master/LICENSE).

data/Rakefile

@@ -0,0 +1,15 @@
+require 'bundler'
+Bundler::GemHelper.install_tasks
+
+require 'rspec/core/rake_task'
+require 'spree/core/testing_support/common_rake'
+
+RSpec::Core::RakeTask.new
+
+task :default => [:spec]
+
+desc "Generates a dummy app for testing"
+task :test_app do
+  ENV['LIB_NAME'] = 'spree_contact_us'
+  Rake::Task['common:test_app'].invoke
+end

data/Versionfile

@@ -0,0 +1,11 @@
+# This file is used to designate compatibilty with different versions of Spree
+# Please see http://spreecommerce.com/documentation/extensions.html#versionfile for details
+
+# Examples
+#
+# "0.70.x" => { :branch => "master"}
+# "0.60.x" => { :branch => "0-60-stable" }
+# "0.40.x" => { :tag => "v1.0.0", :version => "1.0.0" }
+
+"1.1.x"  => { :branch => "master" }
+"1.0.x"  => { :branch => "master" }

data/app/assets/javascripts/admin/spree_contact_us.js

@@ -0,0 +1 @@
+//= require admin/spree_core

data/app/assets/javascripts/store/spree_contact_us.js

@@ -0,0 +1 @@
+//= require store/spree_core

data/app/assets/stylesheets/admin/spree_contact_us.css

@@ -0,0 +1,3 @@
+/*
+ *= require admin/spree_core
+*/

data/app/assets/stylesheets/store/spree_contact_us.css

@@ -0,0 +1,3 @@
+/*
+ *= require store/spree_core
+*/

data/app/controllers/spree/contact_us/contacts_controller.rb

@@ -0,0 +1,23 @@
+class Spree::ContactUs::ContactsController < Spree::BaseController
+
+  def create
+    @contact = Spree::ContactUs::Contact.new(params[:contact_us_contact])
+
+    if @contact.save
+      redirect_to('/', :notice => t('spree.contact_us.notices.success'))
+    else
+      render :new
+    end
+  end
+
+  def new
+    @contact = Spree::ContactUs::Contact.new
+  end
+
+  private
+
+  def accurate_title
+    t('spree.contact_us.contacts.new.contact_us')
+  end
+
+end

data/app/mailers/spree/contact_us/contact_mailer.rb

@@ -0,0 +1,9 @@
+class Spree::ContactUs::ContactMailer < ActionMailer::Base
+  def contact_email(contact)
+    @contact = contact
+
+    mail :from    => (SpreeContactUs.mailer_from || contact.email),
+         :subject => (SpreeContactUs.require_subject ? @contact.subject : t('spree.contact_us.contact_mailer.contact_email.subject', :email => contact.email)),
+         :to      => SpreeContactUs.mailer_to
+  end
+end

data/app/models/spree/contact_us/contact.rb

@@ -0,0 +1,36 @@
+module Spree
+  module ContactUs
+    class Contact
+
+      include ActiveModel::Conversion
+      include ActiveModel::Validations
+
+      attr_accessor :email, :message, :name, :subject
+
+      validates :email,   :format => { :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i },
+                          :presence => true
+      validates :message, :presence => true
+      validates :name,    :presence => {:if => Proc.new{SpreeContactUs.require_name}}
+      validates :subject, :presence => {:if => Proc.new{SpreeContactUs.require_subject}}
+
+      def initialize(attributes = {})
+        attributes.each do |key, value|
+          self.send("#{key}=", value)
+        end
+      end
+
+      def save
+        if self.valid?
+          Spree::ContactUs::ContactMailer.contact_email(self).deliver
+          return true
+        end
+        return false
+      end
+  
+      def persisted?
+        false
+      end
+
+    end
+  end
+end

data/app/overrides/contact_us_in_footer.rb

@@ -0,0 +1,5 @@
+# If SpreeStaticContent extension is also being used we list a Contact Us link with the other footer page links.
+Deface::Override.new(:virtual_path => "spree/layouts/spree_application",
+                     :name => "contact_us_in_footer",
+                     :insert_bottom => "#footer-pages ul",
+                     :text => "<li class='<%= (request.fullpath.gsub('//','/') == '/contact_us') ? 'current' : 'not'%>'><%= link_to t('spree.contact_us.contacts.new.contact_us'), '/contact_us'  %></li>")

data/app/overrides/contact_us_in_header.rb

@@ -0,0 +1,4 @@
+Deface::Override.new(:virtual_path => "spree/layouts/spree_application",
+                     :name => "contact_us_in_header",
+                     :insert_bottom => "#main-nav-bar",
+                     :text => "<li class='<%= (request.fullpath.gsub('//','/') == '/contact_us') ? 'current' : 'not'%>'><%= link_to t('spree.contact_us.contacts.new.contact_us'), '/contact_us'  %></li>")

data/app/views/spree/contact_us/contact_mailer/contact_email.html.erb

@@ -0,0 +1,3 @@
+<p><%= @contact.message %></p>
+<p>---------------------</p>
+<p><%= SpreeContactUs.require_name ? t('.sent_by_name', :email => @contact.email, :name => @contact.name) : t('.sent_by_contact_form', :email => @contact.email) %></p>

data/app/views/spree/contact_us/contact_mailer/contact_email.text.plain.erb

@@ -0,0 +1,4 @@
+<%= @contact.message %>
+
+-------------------------------------
+<%= SpreeContactUs.require_name ? t('.sent_by_name', :email => @contact.email, :name => @contact.name) : t('.sent_by_contact_form', :email => @contact.email) %>

data/app/views/spree/contact_us/contacts/new.html.erb

@@ -0,0 +1,44 @@
+<% content_for :head do -%>
+  <meta name="title" content="Contact Us">
+<% end -%>
+
+<% content_for :sidebar do %>
+  <% if "products" == @current_controller && @taxon %>
+    <%= render :partial => "spree/shared/filters" %>
+  <% else %>
+    <%= render :partial => "spree/shared/taxonomies" %>
+  <% end %>
+<% end %>
+
+<%= render :partial => 'spree/shared/error_messages', :locals => { :target => @contact } %>
+
+<h1><%= t('.contact_us') %></h1>
+<%= form_for @contact, :url => contacts_path do |f| %>
+  <% if SpreeContactUs.require_name %>
+    <p>
+      <%= f.label :name %>
+      <br/>
+      <%= f.text_field :name %>
+    </p>
+  <% end %>
+  <p>
+    <%= f.label :email %>
+    <br/>
+    <%= f.email_field :email %>
+  </p>
+  <% if SpreeContactUs.require_subject %>
+    <p>
+      <%= f.label :subject %>
+      <br/>
+      <%= f.text_field :subject %>
+    </p>
+  <% end %>
+  <p>
+    <%= f.label :message %>
+    <br/>
+    <%= f.text_area :message %>
+  </p>
+  <p>
+    <%= f.submit t('.submit'), :alt => t('.submit'), :id => 'contact_us_contact_submit', :title => t('.submit') %>
+  </p>
+<% end %>

data/config/locales/spree_contact_us.en.yml

@@ -0,0 +1,21 @@
+# Sample localization file for English. Add more files in this directory for other locales.
+# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
+
+en:
+  spree:
+    contact_us:
+      contact_mailer:
+        contact_email:
+          sent_by_contact_form: "Sent by contact form from %{email}"
+          sent_by_name: "Sent by %{name} from %{email}"
+          subject: "Contact Us message from %{email}"
+      contacts:
+        new:
+          contact_us: "Contact Us"
+          email: "Email"
+          message: "Message"
+          name: "Name"
+          subject: "Subject"
+          submit: "Submit"
+      notices:
+        success: "Contact email was successfully sent."

data/config/routes.rb

@@ -0,0 +1,6 @@
+Spree::Core::Engine.routes.draw do
+  resources :contacts,
+    :controller => 'contact_us/contacts',
+    :only       => [:new, :create]
+  match 'contact_us' => 'contact_us/contacts#new'
+end

data/lib/generators/spree_contact_us/install/install_generator.rb

@@ -0,0 +1,14 @@
+module SpreeContactUs
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+
+      argument :file_name, :type => :string, :desc => 'rails app_path', :default => '.'
+      source_root File.expand_path('../../templates', __FILE__)
+
+      def copy_initializer_file
+        template 'spree_contact_us.rb', "#{file_name}/config/initializers/spree_contact_us.rb"
+      end
+
+    end
+  end
+end

data/lib/generators/spree_contact_us/templates/spree_contact_us.rb

@@ -0,0 +1,23 @@
+# Use this initializer to configure the contact mailer.
+
+SpreeContactUs.setup do |config|
+
+  # ==> Mailer Configuration
+
+  # Configure the e-mail address which email notifications should be sent from.  If emails must be sent from a verified email address you may set it here.
+  # Example:
+  # config.mailer_from = "contact@please-change-me.com"
+  config.mailer_from = nil
+
+  # Configure the e-mail address which should receive the contact form email notifications.
+  config.mailer_to = "contact@please-change-me.com"
+
+  # ==> Form Configuration
+
+  # Configure the form to ask for the users name.
+  config.require_name = false
+
+  # Configure the form to ask for a subject.
+  config.require_subject = false
+
+end

data/lib/spree_contact_us.rb

@@ -0,0 +1,24 @@
+require 'spree_core'
+require 'spree_contact_us/engine'
+
+module SpreeContactUs
+
+  # Address ContactUs email notifications are sent from.
+  mattr_accessor :mailer_from
+
+  # Address to send ContactUs email notifications to.
+  mattr_accessor :mailer_to
+
+  # Enable or Disable name field.
+  mattr_accessor :require_name
+
+  # Enable or Disable subject field.
+  mattr_accessor :require_subject
+
+  # Default way to setup ContactUs. Run rake contact_us:install to create
+  # a fresh initializer with all configuration values.
+  def self.setup
+    yield self
+  end
+
+end

data/lib/spree_contact_us/engine.rb

@@ -0,0 +1,12 @@
+module SpreeContactUs
+  class Engine < Rails::Engine
+    engine_name 'spree_contact_us'
+
+    config.autoload_paths += %W(#{config.root}/lib)
+
+    # use rspec for tests
+    config.generators do |g|
+      g.test_framework :rspec
+    end
+  end
+end

data/script/rails

@@ -0,0 +1,7 @@
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+ENGINE_ROOT = File.expand_path('../..', __FILE__)
+ENGINE_PATH = File.expand_path('../../lib/spree_contact_us/engine', __FILE__)
+
+require 'rails/all'
+require 'rails/engine/commands'

data/spec/lib/generators/spree_contact_us/dummy/config/initializers/spree_contact_us.rb

@@ -0,0 +1,23 @@
+# Use this initializer to configure the contact mailer.
+
+SpreeContactUs.setup do |config|
+
+  # ==> Mailer Configuration
+
+  # Configure the e-mail address which email notifications should be sent from.  If emails must be sent from a verified email address you may set it here.
+  # Example:
+  # config.mailer_from = "contact@please-change-me.com"
+  config.mailer_from = nil
+
+  # Configure the e-mail address which should receive the contact form email notifications.
+  config.mailer_to = "contact@please-change-me.com"
+
+  # ==> Form Configuration
+
+  # Configure the form to ask for the users name.
+  config.require_name = false
+
+  # Configure the form to ask for a subject.
+  config.require_subject = false
+
+end

data/spec/lib/generators/spree_contact_us/install/install_generator_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+require 'generator_spec/test_case'
+require 'generators/spree_contact_us/install/install_generator'
+
+describe SpreeContactUs::Generators::InstallGenerator do
+
+  include GeneratorSpec::TestCase
+  destination File.expand_path("../../dummy", __FILE__)
+
+  before do
+    prepare_destination
+    run_generator
+  end
+
+  it "creates initializer file" do
+    assert_file "config/initializers/spree_contact_us.rb"
+  end
+
+end

data/spec/lib/spree_contact_us_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe SpreeContactUs do
+
+  after do
+    SpreeContactUs.mailer_from = nil
+    SpreeContactUs.mailer_to = nil
+    SpreeContactUs.require_name = false
+    SpreeContactUs.require_subject = false
+  end
+
+  it "should be valid" do
+    SpreeContactUs.should be_a(Module)
+  end
+
+  describe 'setup block' do
+    it 'should yield self' do
+      SpreeContactUs.setup do |config|
+        SpreeContactUs.should eql(config)
+      end
+    end
+  end
+
+  describe 'mailer_from' do
+    it 'should be configurable' do
+      SpreeContactUs.mailer_from = "contact@please-change-me.com"
+      SpreeContactUs.mailer_from.should eql("contact@please-change-me.com")
+    end
+  end
+
+  describe 'mailer_to' do
+    it 'should be configurable' do
+      SpreeContactUs.mailer_to = "contact@please-change-me.com"
+      SpreeContactUs.mailer_to.should eql("contact@please-change-me.com")
+    end
+  end
+
+  describe 'require_name' do
+    it 'should be configurable' do
+      SpreeContactUs.require_name = true
+      SpreeContactUs.require_name.should eql(true)
+    end
+  end
+
+  describe 'require_subject' do
+    it 'should be configurable' do
+      SpreeContactUs.require_subject = true
+      SpreeContactUs.require_subject.should eql(true)
+    end
+  end
+
+end

data/spec/mailers/spree/contact_us/contact_mailer_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe Spree::ContactUs::ContactMailer do
+
+  describe "#contact_email" do
+
+    before do
+      SpreeContactUs.mailer_to = "contact@please-change-me.com"
+      @contact = Spree::ContactUs::Contact.new(:email => 'test@email.com', :message => 'Thanks!')
+    end
+
+    it "should render successfully" do
+      lambda { Spree::ContactUs::ContactMailer.contact_email(@contact) }.should_not raise_error
+    end
+
+    it "should use the ContactUs.mailer_from setting when it is set" do
+      SpreeContactUs.mailer_from = "contact@please-change-me.com"
+      @mailer = Spree::ContactUs::ContactMailer.contact_email(@contact)
+      @mailer.from.should eql([SpreeContactUs.mailer_from])
+      SpreeContactUs.mailer_from = nil
+    end
+
+    describe "rendered without error" do
+
+      before do
+        @mailer = Spree::ContactUs::ContactMailer.contact_email(@contact)
+      end
+
+      it "should have the initializers to address" do
+        @mailer.to.should eql([SpreeContactUs.mailer_to])
+      end
+
+      it "should use the users email in the from field when ContactUs.mailer_from is not set" do
+        @mailer.from.should eql([@contact.email])
+      end
+
+      it "should have users email in the subject line" do
+        @mailer.subject.should eql("Contact Us message from #{@contact.email}")
+      end
+
+      it "should have the message in the body" do
+        @mailer.body.should match("<p>Thanks!</p>")
+      end
+
+      it "should deliver successfully" do
+        lambda { Spree::ContactUs::ContactMailer.contact_email(@contact).deliver }.should_not raise_error
+      end
+
+      describe "and delivered" do
+
+        it "should be added to the delivery queue" do
+          lambda { Spree::ContactUs::ContactMailer.contact_email(@contact).deliver }.should change(ActionMailer::Base.deliveries,:size).by(1)
+        end
+
+      end
+
+    end
+
+  end
+
+end

data/spec/models/spree/contact_us/contact_spec.rb

@@ -0,0 +1,66 @@
+require 'spec_helper'
+
+describe Spree::ContactUs::Contact do
+
+  it_should_behave_like 'ActiveModel'
+
+  describe "Validations" do
+
+    it {should validate_presence_of(:email)}
+    it {should validate_presence_of(:message)}
+    it {should_not validate_presence_of(:name)}
+    it {should_not validate_presence_of(:subject)}
+
+    context 'with name and subject settings' do
+
+      after do
+        SpreeContactUs.require_name = false
+        SpreeContactUs.require_subject = false
+      end
+
+      before do
+        SpreeContactUs.require_name = true
+        SpreeContactUs.require_subject =true
+      end
+
+      it {should validate_presence_of(:name)}
+      it {should validate_presence_of(:subject)}
+
+    end
+
+  end
+
+  describe 'Methods' do
+
+    describe '#read_attribute_for_validation' do
+      it 'should return attributes set during initialization' do
+        contact = Spree::ContactUs::Contact.new(:email => "Valid@Email.com", :message => "Test")
+        contact.read_attribute_for_validation(:email).should eql("Valid@Email.com")
+        contact.read_attribute_for_validation(:message).should eql("Test")
+      end
+    end
+
+    describe '#save' do
+
+      it 'should return false if records invalid' do
+        contact = Spree::ContactUs::Contact.new(:email => "Valid@Email.com", :message => "")
+        contact.save.should eql(false)
+      end
+
+      it 'should send email and return true if records valid' do
+        mail = Mail.new(:from=>"Valid@Email.com", :to => "test@test.com")
+        mail.stub(:deliver).and_return(true)
+        contact = Spree::ContactUs::Contact.new(:email => "Valid@Email.com", :message => "Test")
+        Spree::ContactUs::ContactMailer.should_receive(:contact_email).with(contact).and_return(mail)
+        contact.save.should eql(true)
+      end
+
+    end
+
+    describe '#to_key' do
+      it { subject.should respond_to(:to_key) }
+    end
+
+  end
+
+end

data/spec/requests/spree_contact_us_lint_spec.rb

@@ -0,0 +1,140 @@
+require 'spec_helper'
+
+describe 'Contact Us page' do
+
+  after do
+    ActionMailer::Base.deliveries = []
+    SpreeContactUs.mailer_from = nil
+    SpreeContactUs.mailer_to = nil
+    SpreeContactUs.require_name = false
+    SpreeContactUs.require_subject = false
+  end
+
+  before do
+    ActionMailer::Base.deliveries = []
+    SpreeContactUs.mailer_to = 'test@test.com'
+  end
+
+  it 'displays default contact form properly' do
+    visit spree.contact_us_path
+    within "form#new_contact_us_contact" do
+      page.should have_selector "input#contact_us_contact_email"
+      page.should have_selector "textarea#contact_us_contact_message"
+      page.should_not have_selector "input#contact_us_contact_name"
+      page.should_not have_selector "input#contact_us_contact_subject"
+      page.should have_selector "input#contact_us_contact_submit"
+    end
+  end
+
+  context "Submitting the form" do
+
+    before do
+      visit spree.contact_us_path
+    end
+
+    context "when valid" do
+      before do
+        fill_in 'Email', :with => 'test@example.com'
+        fill_in 'Message', :with => 'howdy'
+        click_button 'Submit'
+      end
+
+      it "I should be redirected to the homepage" do
+        current_path.should == "/"
+      end
+
+      it "An email should have been sent" do
+        ActionMailer::Base.deliveries.size.should == 1
+      end
+
+      it "The email should have the correct attributes" do
+        mail = ActionMailer::Base.deliveries.last
+        mail.to.should == ['test@test.com']
+        mail.from.should == ['test@example.com']
+        mail.body.should match 'howdy'
+      end
+    end
+
+    context "when invalid" do
+      context "Email and message are invalid" do
+        before do
+          fill_in 'Email', :with => 'a'
+          fill_in 'Message', :with => ''
+          click_button 'Submit'
+        end
+
+        it "I should see two error messages" do
+          within '#errorExplanation' do
+            page.should have_content "Email is invalid"
+            page.should have_content "Message can't be blank"
+          end
+        end
+
+        it "An email should not have been sent" do
+          ActionMailer::Base.deliveries.size.should == 0
+        end
+      end
+    end
+  end
+
+  context 'with name and subject configuration' do
+
+    before do
+      SpreeContactUs.require_name = true
+      SpreeContactUs.require_subject = true
+      visit spree.contact_us_path
+    end
+
+    it "displays an input for name and subject" do
+      page.should have_selector "input#contact_us_contact_name"
+      page.should have_selector "input#contact_us_contact_subject"
+    end
+
+    context "Submitting the form" do
+      context "when valid" do
+        before do
+          fill_in 'Email', :with => 'test@example.com'
+          fill_in 'Message', :with => 'howdy'
+          fill_in 'contact_us_contact[name]', :with => 'Jeff'
+          fill_in 'contact_us_contact[subject]', :with => 'Testing contact form.'
+          click_button 'Submit'
+        end
+
+        it "I should be redirected to the homepage" do
+          current_path.should == "/"
+        end
+
+        it "An email should have been sent" do
+          ActionMailer::Base.deliveries.size.should == 1
+        end
+
+        it "The email should have the correct attributes" do
+          mail = ActionMailer::Base.deliveries.last
+          mail.body.should match 'howdy'
+          mail.body.should match 'Jeff'
+          mail.from.should == ['test@example.com']
+          mail.subject.should match 'Testing contact form.'
+          mail.to.should == ['test@test.com']
+        end
+      end
+
+      context "when name and subject are blank" do
+        before do
+          click_button 'Submit'
+        end
+
+        it "I should see error messages" do
+          within '#errorExplanation' do
+            page.should have_content "Email can't be blank"
+            page.should have_content "Message can't be blank"
+          end
+        end
+
+        it "An email should not have been sent" do
+          ActionMailer::Base.deliveries.size.should == 0
+        end
+      end
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,38 @@
+# Configure simeplecov for test coverage report
+require 'simplecov'
+SimpleCov.start do
+  add_filter '/config/'
+  add_group 'Controllers', 'app/controllers'
+  add_group 'Helpers', 'app/helpers'
+  add_group 'Mailers', 'app/mailers'
+  add_group 'Models', 'app/models'
+  add_group 'Overrides', 'app/overrides'
+  add_group 'Libraries', 'lib'
+  add_group 'Specs', 'spec'
+end
+
+# Configure Rails Environment
+ENV["RAILS_ENV"] = "test"
+
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+
+# Configure capybara for integration testing
+require "capybara/rails"
+Capybara.default_driver   = :rack_test
+Capybara.default_selector = :css
+
+require 'ffaker'
+require 'rspec/rails'
+require 'shoulda-matchers'
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.join(File.dirname(__FILE__), "support/**/*.rb")].each {|f| require f }
+
+# Requires factories defined in spree_core
+require 'spree/core/testing_support/factories'
+require 'spree/core/url_helpers'
+
+RSpec.configure do |config|
+  config.include Spree::Core::UrlHelpers
+end

data/spec/support/active_model_lint.rb

@@ -0,0 +1,17 @@
+# adapted from rspec-rails:
+# http://github.com/rspec/rspec-rails/blob/master/spec/rspec/rails/mocks/mock_model_spec.rb
+
+shared_examples_for 'ActiveModel' do
+  include ActiveModel::Lint::Tests
+
+  # to_s is to support ruby-1.9
+  ActiveModel::Lint::Tests.public_instance_methods.map(&:to_s).grep(/^test/).each do |m|
+    example m.gsub('_', ' ') do
+      send m
+    end
+  end
+
+  def model
+    subject
+  end
+end

data/spree_contact_us.gemspec

@@ -0,0 +1,29 @@
+# encoding: UTF-8
+Gem::Specification.new do |s|
+  s.platform    = Gem::Platform::RUBY
+  s.name        = 'spree_contact_us'
+  s.version     = '1.0.0'
+  s.summary     = 'Reworked the contact_us gem to work with Spree.'
+  s.description = 'Reworked the contact_us gem to work with Spree.'
+  s.required_ruby_version = '>= 1.9.2'
+
+  s.author            = 'Jeff Dutil'
+  s.email             = 'jdutil@burlingtonwebapps.com'
+  s.homepage          = 'http://github.com/jdutil/spree_contact_us'
+
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.require_path = 'lib'
+  s.requirements << 'none'
+
+  s.add_dependency 'spree_core', '>= 1.1.0.rc2'
+
+  s.add_development_dependency 'capybara',         '~> 1.1'
+  s.add_development_dependency 'factory_girl',     '~> 2.6'
+  s.add_development_dependency 'ffaker',           '~> 1.12'
+  s.add_development_dependency 'generator_spec',   '~> 0.8'
+  s.add_development_dependency 'rspec-rails',      '~> 2.9'
+  s.add_development_dependency 'shoulda-matchers', '~> 1.1'
+  s.add_development_dependency 'simplecov',        '~> 0.6.2'
+  s.add_development_dependency 'sqlite3',          '~> 1.3.6'
+end

metadata

@@ -0,0 +1,236 @@
+--- !ruby/object:Gem::Specification
+name: spree_contact_us
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+  prerelease: 
+platform: ruby
+authors:
+- Jeff Dutil
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2012-04-28 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: spree_core
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.1.0.rc2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.1.0.rc2
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.1'
+- !ruby/object:Gem::Dependency
+  name: factory_girl
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.6'
+- !ruby/object:Gem::Dependency
+  name: ffaker
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.12'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.12'
+- !ruby/object:Gem::Dependency
+  name: generator_spec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.9'
+- !ruby/object:Gem::Dependency
+  name: shoulda-matchers
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.1'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.6.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.6.2
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.6
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.6
+description: Reworked the contact_us gem to work with Spree.
+email: jdutil@burlingtonwebapps.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- .gitignore
+- .rspec
+- .travis.yml
+- Gemfile
+- LICENSE
+- README.md
+- Rakefile
+- Versionfile
+- app/assets/javascripts/admin/spree_contact_us.js
+- app/assets/javascripts/store/spree_contact_us.js
+- app/assets/stylesheets/admin/spree_contact_us.css
+- app/assets/stylesheets/store/spree_contact_us.css
+- app/controllers/spree/contact_us/contacts_controller.rb
+- app/mailers/spree/contact_us/contact_mailer.rb
+- app/models/spree/contact_us/contact.rb
+- app/overrides/contact_us_in_footer.rb
+- app/overrides/contact_us_in_header.rb
+- app/views/spree/contact_us/contact_mailer/contact_email.html.erb
+- app/views/spree/contact_us/contact_mailer/contact_email.text.plain.erb
+- app/views/spree/contact_us/contacts/new.html.erb
+- config/locales/spree_contact_us.en.yml
+- config/routes.rb
+- lib/generators/spree_contact_us/install/install_generator.rb
+- lib/generators/spree_contact_us/templates/spree_contact_us.rb
+- lib/spree_contact_us.rb
+- lib/spree_contact_us/engine.rb
+- script/rails
+- spec/lib/generators/spree_contact_us/dummy/config/initializers/spree_contact_us.rb
+- spec/lib/generators/spree_contact_us/install/install_generator_spec.rb
+- spec/lib/spree_contact_us_spec.rb
+- spec/mailers/spree/contact_us/contact_mailer_spec.rb
+- spec/models/spree/contact_us/contact_spec.rb
+- spec/requests/spree_contact_us_lint_spec.rb
+- spec/spec_helper.rb
+- spec/support/active_model_lint.rb
+- spree_contact_us.gemspec
+homepage: http://github.com/jdutil/spree_contact_us
+licenses: []
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: 1.9.2
+required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: -4355211758008457708
+requirements:
+- none
+rubyforge_project: 
+rubygems_version: 1.8.19
+signing_key: 
+specification_version: 3
+summary: Reworked the contact_us gem to work with Spree.
+test_files:
+- spec/lib/generators/spree_contact_us/dummy/config/initializers/spree_contact_us.rb
+- spec/lib/generators/spree_contact_us/install/install_generator_spec.rb
+- spec/lib/spree_contact_us_spec.rb
+- spec/mailers/spree/contact_us/contact_mailer_spec.rb
+- spec/models/spree/contact_us/contact_spec.rb
+- spec/requests/spree_contact_us_lint_spec.rb
+- spec/spec_helper.rb
+- spec/support/active_model_lint.rb