RubyGems - pointless_feedback - Versions diffs - 1.0.0 - Mend

pointless_feedback 1.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

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