emails as delayed jobs

Author: amygimma

Gemfile

@@ -11,6 +11,7 @@ gem 'bcrypt-ruby'
 gem 'dalli'
 gem 'sass-rails'
 gem 'pry'
+gem 'delayed_job_active_record'
 
 gem 'bourbon'
 gem 'neat'

Gemfile.lock

@@ -67,6 +67,11 @@ GEM
       rails (>= 3, < 5)
     dalli (2.7.1)
     database_cleaner (1.2.0)
+    delayed_job (4.0.2)
+      activesupport (>= 3.0, < 4.2)
+    delayed_job_active_record (4.0.1)
+      activerecord (>= 3.0, < 4.2)
+      delayed_job (>= 3.0, < 4.1)
     diff-lcs (1.2.5)
     email_validator (1.4.0)
       activemodel
@@ -236,6 +241,7 @@ DEPENDENCIES
   cucumber-rails
   dalli
   database_cleaner
+  delayed_job_active_record
   email_validator
   factory_girl_rails
   font-awesome-sass

Procfile

@@ -1 +1,2 @@
-web: bundle exec unicorn -c ./config/unicorn.rb
+web: bundle exec unicorn -c ./config/unicorn.rb
+worker:  bundle exec rake jobs:work

app/controllers/contact_requests_controller.rb

@@ -4,7 +4,7 @@ class ContactRequestsController < ApplicationController
   def create
     @contact_request = ContactRequest.new(contact_params)
     if @contact_request.save
-      ContactMailer.contact_request(@contact_request)
+      ContactRequest.delay.send_notification_email(@contact_request.id)
       render json: @contact_request, status: :created
     else
       render json: @contact_request.errors, status: :unprocessable_entity

app/models/contact_request.rb

@@ -1,4 +1,9 @@
 class ContactRequest < ActiveRecord::Base
   validates_presence_of :name, :message
   validates :email, presence: true, email: true
+
+  def self.send_notification_email(contact_request_id)
+    contact_request = find(contact_request_id)
+    ContactMailer.contact_request(contact_request)
+  end
 end

bin/delayed_job

@@ -0,0 +1,5 @@
+#!/usr/bin/env ruby
+
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'config', 'environment'))
+require 'delayed/command'
+Delayed::Command.new(ARGV).daemonize

config/initializers/delayed_job.rb

@@ -0,0 +1,3 @@
+Delayed::Worker.destroy_failed_jobs = false
+Delayed::Worker.max_attempts = 1
+Delayed::Worker.delay_jobs = !(Rails.env.test? || Rails.env.development?)

db/migrate/20140813185505_create_delayed_jobs.rb

@@ -0,0 +1,22 @@
+class CreateDelayedJobs < ActiveRecord::Migration
+  def self.up
+    create_table :delayed_jobs, :force => true do |table|
+      table.integer  :priority, :default => 0, :null => false # Allows some jobs to jump to the front of the queue
+      table.integer  :attempts, :default => 0, :null => false # Provides for retries, but still fail eventually.
+      table.text     :handler, :null => false                 # YAML-encoded string of the object that will do work
+      table.text     :last_error                              # reason for last failure (See Note below)
+      table.datetime :run_at                                  # When to run. Could be Time.zone.now for immediately, or sometime in the future.
+      table.datetime :locked_at                               # Set when a client is working on this object
+      table.datetime :failed_at                               # Set when all retries have failed (actually, by default, the record is deleted instead)
+      table.string   :locked_by                               # Who is working on this object (if locked)
+      table.string   :queue                                   # The name of the queue this job is in
+      table.timestamps
+    end
+
+    add_index :delayed_jobs, [:priority, :run_at], :name => 'delayed_jobs_priority'
+  end
+
+  def self.down
+    drop_table :delayed_jobs
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20140702212029) do
+ActiveRecord::Schema.define(version: 20140813185505) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -39,6 +39,22 @@
     t.boolean  "analytics"
   end
 
+  create_table "delayed_jobs", force: true do |t|
+    t.integer  "priority",   default: 0, null: false
+    t.integer  "attempts",   default: 0, null: false
+    t.text     "handler",                null: false
+    t.text     "last_error"
+    t.datetime "run_at"
+    t.datetime "locked_at"
+    t.datetime "failed_at"
+    t.string   "locked_by"
+    t.string   "queue"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
+  add_index "delayed_jobs", ["priority", "run_at"], name: "delayed_jobs_priority", using: :btree
+
   create_table "employees", force: true do |t|
     t.string   "name"
     t.string   "email"

features/contact_request.feature

@@ -0,0 +1,8 @@
+Feature: Contact Request Form
+
+  @javascript
+  Scenario: Filling out the contact form as a potential client
+    Given I am on the homepage
+    When I submit the contact request form correctly
+    Then I should see a confirmation message
+    And there should be mail delivered

features/step_definitions/contact_request_steps.rb

@@ -0,0 +1,19 @@
+Given(/^I am on the homepage$/) do
+  visit root_path
+end
+
+When(/^I submit the contact request form correctly$/) do
+  fill_in "Name", with: "Mando"
+  fill_in "Email", with: "[email protected]"
+  fill_in "Phone Number", with: "1234123412"
+  fill_in "What can we help you with?", with: "Help!"
+  click_on "Send Request!"
+end
+
+Then(/^I should see a confirmation message$/) do
+  expect(page).to have_content("We got the message.")
+end
+
+Then(/^there should be mail delivered$/) do
+  expect(ActionMailer::Base.deliveries.count).to eq(1)
+end

features/support/mailers.rb

@@ -0,0 +1 @@
+After { ActionMailer::Base.deliveries.clear }

spec/models/contact_request_spec.rb

@@ -11,4 +11,12 @@
     end
 
   end
+
+  describe ".send_mail_notification_email" do
+    it "sends the contact request info in an email" do
+      expect(ContactMailer).to receive(:contact_request)
+      request = ContactRequest.create(name: "Armando", email: "[email protected]", phone: "123-123-1234", message: "HELP!")
+      ContactRequest.send_notification_email(request.id)
+    end
+  end
 end