RubyGems - pointless_feedback - Versions diffs - 1.0.0 - Mend

pointless_feedback 1.0.0

Files changed (90) hide show

data/test/dummy/tmp/capybara/capybara-201308281055176538066301.html ADDED Viewed

@@ -0,0 +1,52 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Dummy</title>
+  <link href="/assets/application.css" media="all" rel="stylesheet" type="text/css" />
+  <script src="/assets/application.js" type="text/javascript"></script>
+</head>
+<body>
+<form accept-charset="UTF-8" action="/pointless_feedback/messages" class="new_message" id="new_message" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /></div>
+  <fieldset class="user-fields">
+    <legend>About You</legend>
+    <div class="field">
+      <label for="message_name">Name</label>
+      <input id="message_name" name="message[name]" size="30" type="text" />
+    </div>
+    <div class="field">
+      <label for="message_email_address">Email address</label>
+      <input id="message_email_address" name="message[email_address]" size="30" type="text" />
+    </div>
+  </fieldset>
+  <fieldset class="feedback-fields">
+    <legend>Leave Your Feedback</legend>
+    <div class="field">
+      <label for="message_topic">Topic</label>
+      <select id="message_topic" name="message[topic]"><option value="">Please select</option>
+<option value="Error on page">Error on page</option>
+<option value="Other">Other</option></select>
+    </div>
+    <div class="field">
+      <label for="message_description">Description</label>
+      <textarea cols="40" id="message_description" name="message[description]" rows="20">
+</textarea>
+    </div>
+    <div class="contact-input"></div>
+      <input id="message_contact_info" name="message[contact_info]" size="30" type="text" />
+    </div>
+  </fieldset>
+  <input name="commit" type="submit" value="Submit" />
+</form>
+</body>
+</html>

data/test/dummy/tmp/capybara/capybara-201311071553562925508230.html ADDED Viewed

@@ -0,0 +1,58 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Dummy</title>
+  <link href="/assets/application.css" media="all" rel="stylesheet" type="text/css" />
+  <script src="/assets/application.js" type="text/javascript"></script>
+</head>
+<body>
+<a href="http://www.example.com/">Homepage</a>
+<form accept-charset="UTF-8" action="/pointless_feedback/messages" class="new_message" id="new_message" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /></div>
+  <fieldset class="user-fields">
+    <legend>About You</legend>
+    <div class="field">
+      <label for="message_name">Name</label>
+      <input id="message_name" name="message[name]" size="30" type="text" />
+    </div>
+    <div class="field">
+      <label for="message_email_address">Email address</label>
+      <input id="message_email_address" name="message[email_address]" size="30" type="text" />
+    </div>
+  </fieldset>
+  <fieldset class="feedback-fields">
+    <legend>Leave Your Feedback</legend>
+    <div class="field">
+      <label for="message_topic">Topic</label>
+      <select id="message_topic" name="message[topic]"><option value="">Please select</option>
+<option value="Error on page">Error on page</option>
+<option value="Other">Other</option></select>
+    </div>
+    <div class="field">
+      <label for="message_description">Description</label>
+      <textarea cols="40" id="message_description" name="message[description]" rows="20">
+</textarea>
+    </div>
+    <!-- This is a honeypot field to keep the spammers away. Do not remove. -->
+    <!-- If this field is filled in, feedback will not be emailed -->
+    <div class="contact-input" style="position: absolute; left: -9999px;">
+      <input id="message_contact_info" name="message[contact_info]" size="30" type="text" />
+    </div>
+    <!-- End of honeypot field -->
+  </fieldset>
+  <input name="commit" type="submit" value="Submit" />
+</form>
+</body>
+</html>

data/test/factories/messages.rb ADDED Viewed

@@ -0,0 +1,8 @@
+FactoryGirl.define do
+  factory :message, :class => PointlessFeedback::Message do
+    name          'A Developer'
+    email_address 'developer@pointlesscorp.com'
+    description   'Site is broke'
+    topic         'Other'
+  end
+end

data/test/functional/pointless_feedback/feedback_mailer_test.rb ADDED Viewed

@@ -0,0 +1,25 @@
+require 'test_helper'
+module PointlessFeedback
+  class FeedbackMailerTest < ActionMailer::TestCase
+    before do
+      @message = FactoryGirl.create(:message)
+      @mail    = FeedbackMailer.feedback('to@example.com', @message)
+    end
+    test "feedback email" do
+      assert_equal "Pointless Feedback", @mail.subject
+      assert_equal ["to@example.com"], @mail.to
+      assert_equal ["feedback@pointlesscorp.com"], @mail.from
+    end
+    test "feedback email body" do
+      assert_match "You've got feedback!", @mail.body.encoded
+      assert_match "Name: #{@message.name}", @mail.body.encoded
+      assert_match "Email Address: #{@message.email_address}", @mail.body.encoded
+      assert_match "Topic: #{@message.topic}", @mail.body.encoded
+      assert_match "Description: #{@message.description}", @mail.body.encoded
+    end
+  end
+end

data/test/functional/pointless_feedback/messages_controller_test.rb ADDED Viewed

@@ -0,0 +1,61 @@
+require 'test_helper'
+module PointlessFeedback
+  class MessagesControllerTest < ActionController::TestCase
+    describe "getting new" do
+      setup { get :new, :use_route => :pointless_feedback }
+      it { assert_response :success }
+      it { assert_template :new }
+      it { assigns(:message).must_be_instance_of Message }
+    end
+    describe "posting to create" do
+      setup do
+        @valid_params = {
+          :message => {
+            :name          => 'Some Guy',
+            :email_address => 'some_guy@web.com',
+            :topic         => 'Other',
+            :description   => 'Yo website bork'
+          },
+          :use_route => :pointless_feedback
+        }
+      end
+      describe "with invalid params" do
+        setup do
+          post :create, @valid_params.merge(:message => { :name => ''})
+        end
+        it { assert_response :success }
+        it { assert_template :new }
+        it { assigns(:message).must_be :invalid? }
+      end
+      describe "with valid params" do
+        describe "with default after_message_create_path" do
+          setup do
+            post :create, @valid_params
+          end
+          it { assert_response :redirect }
+          it { assert_redirected_to '/' }
+          it { flash[:notice].must_equal 'Thanks for your feedback!' }
+        end
+        describe "with overridden after_message_create_path" do
+          setup do
+            @controller.instance_eval "def after_message_create_path; '/dashboard'; end"
+            post :create, @valid_params
+          end
+          it { assert_response :redirect }
+          it { assert_redirected_to '/dashboard' }
+          it { flash[:notice].must_equal 'Thanks for your feedback!' }
+        end
+      end
+    end
+  end
+end

data/test/generators/install_generator_test.rb ADDED Viewed

@@ -0,0 +1,14 @@
+require "test_helper"
+require "rails/generators/test_case"
+require "generators/pointless_feedback/install_generator"
+class InstallGeneratorTest < Rails::Generators::TestCase
+  tests PointlessFeedback::Generators::InstallGenerator
+  destination File.expand_path("../../tmp", __FILE__)
+  setup :prepare_destination
+  test "Assert all files are properly created" do
+    run_generator
+    assert_file "config/initializers/pointless_feedback.rb"
+  end
+end

data/test/generators/views_generator_test.rb ADDED Viewed

@@ -0,0 +1,14 @@
+require "test_helper"
+require "rails/generators/test_case"
+require "generators/pointless_feedback/views_generator"
+class ViewsGeneratorTest < Rails::Generators::TestCase
+  tests PointlessFeedback::Generators::ViewsGenerator
+  destination File.expand_path("../../tmp", __FILE__)
+  setup :prepare_destination
+  test "Assert all views are properly created" do
+    run_generator
+    assert_file 'app/views/pointless_feedback/messages/new.html.erb'
+  end
+end

data/test/integration/pointless_feedback/feedback_submission_test.rb ADDED Viewed

@@ -0,0 +1,54 @@
+require 'test_helper'
+class FeedbackSubmissionTest < ActionDispatch::IntegrationTest
+  fixtures :all
+  describe "submitting feedback" do
+    before do
+      visit "/pointless_feedback"
+      fill_in_fields
+      click_on "Submit"
+    end
+    test "redirects to the root path" do
+      assert_equal current_path, "/"
+    end
+    test "thanks message is displayed" do
+      within('body') do
+        assert_match 'Thanks for your feedback!', text
+      end
+    end
+  end
+  describe "submitting feedback with the honeypot field filled in" do
+    before do
+      visit "/pointless_feedback"
+      fill_in_fields
+      fill_in "message_contact_info", :with => "I'm a spam bot!"
+      click_on "Submit"
+    end
+    test "redirects to the root path" do
+      assert_equal current_path, "/"
+    end
+    test "thanks message is displayed" do
+      within('body') do
+        assert_match 'Thanks for your feedback!', text
+      end
+    end
+  end
+  def fill_in_fields
+    fill_in "Name",          :with => "Eli"
+    fill_in "Email address", :with => "eli@example.com"
+    select "Other",          :from => "Topic"
+    fill_in "Description",   :with => "This site is awful"
+  end
+end

data/test/integration/pointless_feedback/main_app_root_url_test.rb ADDED Viewed

@@ -0,0 +1,9 @@
+require 'test_helper'
+class MainAppRootUrlTest < ActionDispatch::IntegrationTest
+  test "root_url link points to main_app.root_url" do
+    visit "/pointless_feedback"
+    assert page.has_link?("Homepage", :href => current_host + "/")
+  end
+end

data/test/pointless_feedback_test.rb ADDED Viewed

@@ -0,0 +1,7 @@
+require 'test_helper'
+class PointlessFeedbackTest < ActiveSupport::TestCase
+  test "truth" do
+    assert_kind_of Module, PointlessFeedback
+  end
+end

data/test/test_helper.rb ADDED Viewed

@@ -0,0 +1,26 @@
+# Configure Rails Environment
+ENV["RAILS_ENV"] = "test"
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+require "rails/test_help"
+require "capybara/rails"
+require "factory_girl"
+require "test/unit"
+require "mocha/setup"
+require File.expand_path("../factories/messages.rb",  __FILE__)
+Rails.backtrace_cleaner.remove_silencers!
+# Load support files
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+# Load fixtures from the engine
+if ActiveSupport::TestCase.method_defined?(:fixture_path=)
+  ActiveSupport::TestCase.fixture_path = File.expand_path("../fixtures", __FILE__)
+end
+class ActionDispatch::IntegrationTest
+  # Make the Capybara DSL available in all integration tests
+  include Capybara::DSL
+end

data/test/tmp/app/views/pointless_feedback/messages/new.html.erb ADDED Viewed

@@ -0,0 +1,57 @@
+<%= form_for @message do |f| %>
+  <% if @message.errors.any? %>
+    <div class="error_messages">
+      <h2>
+        <%= I18n.t('activerecord.errors.header', :default => 'Invalid Fields') %>
+      </h2>
+      <p>
+        <%= I18n.t('activerecord.errors.message', :default => 'Correct the following errors and try again.') %>
+      </p>
+      <ul>
+        <% @message.errors.full_messages.each do |message| %>
+          <li><%= message.html_safe %></li>
+        <% end %>
+      </ul>
+    </div>
+  <% end %>
+  <fieldset class="user-fields">
+    <legend>About You</legend>
+    <div class="field">
+      <%= f.label :name %>
+      <%= f.text_field :name %>
+    </div>
+    <div class="field">
+      <%= f.label :email_address %>
+      <%= f.text_field :email_address %>
+    </div>
+  </fieldset>
+  <fieldset class="feedback-fields">
+    <legend>Leave Your Feedback</legend>
+    <div class="field">
+      <%= f.label :topic %>
+      <%= f.select :topic, PointlessFeedback.message_topics, { :prompt => true } %>
+    </div>
+    <div class="field">
+      <%= f.label :description %>
+      <%= f.text_area :description %>
+    </div>
+    <!-- This is a honeypot field to keep the spammers away. Do not remove. -->
+    <!-- If this field is filled in, feedback will not be emailed -->
+    <div class="contact-input" style="position: absolute; left: -9999px;">
+      <%= f.text_field :contact_info %>
+    </div>
+    <!-- End of honeypot field -->
+  </fieldset>
+  <%= f.submit 'Submit' %>
+<% end %>

data/test/unit/helpers/pointless_feedback/application_helper_test.rb ADDED Viewed

@@ -0,0 +1,39 @@
+require 'test_helper'
+module PointlessFeedback
+  class ApplicationHelperTest < ActionDispatch::IntegrationTest
+    describe "main app's named routes" do
+      class MainAppApplicationHelperMock
+        def foo_path
+          '/foo'
+        end
+        def foo_url
+          'http://host/foo'
+        end
+      end
+      class PointlessFeedbackApplicationHelperMock
+        include PointlessFeedback::ApplicationHelper
+        def main_app
+          MainAppApplicationHelperMock.new
+        end
+      end
+      let(:raw_helper) { PointlessFeedbackApplicationHelperMock.new }
+      it "outputs named path from the main app" do
+        raw_helper.foo_path.must_equal '/foo'
+      end
+      it "outputs named url from the main app" do
+        raw_helper.foo_url.must_equal 'http://host/foo'
+      end
+      it "raises exception with other routes" do
+        proc{ raw_helper.lolwut_path }.must_raise NoMethodError
+      end
+    end
+  end
+end

data/test/unit/pointless_feedback/message_test.rb ADDED Viewed

@@ -0,0 +1,96 @@
+require 'test_helper'
+module PointlessFeedback
+  class MessageTest < ActiveSupport::TestCase
+    subject { PointlessFeedback::Message.new }
+    %w(name email_address topic description).each do |field|
+      it "requires #{field}" do
+        subject.send("#{field}=", '')
+        subject.save
+        subject.errors[field.to_sym].must_include "can't be blank"
+      end
+    end
+    describe "validating topics" do
+      subject { FactoryGirl.build(:message) }
+      it "allows valid topic" do
+        subject.topic = 'Other'
+        assert subject.valid?
+      end
+      it "does not allow invalid topic" do
+        subject.topic = 'lolwut'
+        assert subject.invalid?
+        subject.errors[:topic].must_include "is not included in the list"
+      end
+    end
+    describe "validating email" do
+      subject { FactoryGirl.build(:message) }
+      it "allows a valid email address" do
+        subject.email_address = 'test@example.com'
+        assert subject.valid?
+      end
+      it "does not allow invalid emails address" do
+        subject.email_address = 'test@example'
+        assert subject.invalid?
+        subject.email_address = 'test.com'
+        assert subject.invalid?
+        subject.email_address = 'come on'
+        assert subject.invalid?
+      end
+    end
+    describe "export_feedback" do
+      describe "when PointlessFeedback.send_emails is true" do
+        before  do
+          PointlessFeedback.email_feedback = true
+          PointlessFeedback.to_emails = ['test1@example.com', 'test2@example.com']
+        end
+        subject { FactoryGirl.build(:message) }
+        it "sends mail after create" do
+          mailer = stub(:deliver => true)
+          ['test1@example.com', 'test2@example.com'].each do |email|
+            FeedbackMailer.expects(:feedback).with(email, subject).returns(mailer)
+          end
+          subject.save
+        end
+        describe "when the honeypot field is not nil" do
+          it "does not send mail after create" do
+            subject.contact_info = "I'm a spam bot!"
+            FeedbackMailer.expects(:feedback).never
+            subject.save
+          end
+        end
+      end
+      describe "when PointlessFeedback.send_emails is false" do
+        before  do
+          PointlessFeedback.email_feedback = false
+        end
+        subject { FactoryGirl.build(:message) }
+        it "does not send any mail" do
+          FeedbackMailer.expects(:feedback).times(0)
+          subject.save
+        end
+      end
+    end
+  end
+end