Stripe payments

Author: jordanbyron

.gitignore

@@ -9,9 +9,10 @@ config/initializers/omniauth.rb
 config/initializers/mail_settings.rb
 config/initializers/secret_token.rb
 config/initializers/cache_cooker_settings.rb
+config/initializers/stripe.rb
 lib/development_mail_interceptor.rb
 .sass-cache/
 public/assets
 coverage
 .kick
-.pow*
+.pow*

Gemfile

@@ -12,6 +12,8 @@ gem 'nokogiri'
 gem 'md_preview'
 gem 'md_emoji'
 
+gem 'stripe'
+
 gem 'will_paginate'
 gem 'haml'
 gem 'coffee-filter', '~> 0.1.1'
@@ -45,10 +47,9 @@ group :test do
   gem 'launchy'
   gem 'factory_girl_rails'
   gem 'mocha', :require => false
-  gem 'capybara-webkit'
+  gem 'capybara-webkit', "0.10.1"
   gem 'database_cleaner'
-  gem 'test_notifier' #, :git => "git://github.com/jordanbyron/test_notifier.git",
-                      # :branch => "minitest-errors"
+  gem 'test_notifier'
   gem 'turn', '~> 0.9.5'
   gem 'simplecov', :require => false
 end

Gemfile.lock

@@ -192,6 +192,8 @@ GEM
     rdoc (3.12)
       json (~> 1.4)
     redcarpet (2.1.1)
+    rest-client (1.6.7)
+      mime-types (>= 1.16)
     rubyzip (0.9.4)
     sass (3.1.19)
     sass-rails (3.2.5)
@@ -241,7 +243,7 @@ DEPENDENCIES
   capistrano
   capybara
   capybara-screenshot
-  capybara-webkit
+  capybara-webkit (= 0.10.1)
   coffee-filter (~> 0.1.1)
   coffee-rails (~> 3.2.0)
   compass-rails
@@ -273,6 +275,7 @@ DEPENDENCIES
   sass-rails (~> 3.2.0)
   sassy-buttons
   simplecov
+  stripe
   test_notifier
   turn (~> 0.9.5)
   uglifier

app/assets/images/payment/cvc.gif

@@ -0,0 +1,68 @@
+class PR.PaymentProcessor
+  constructor: (@key) ->
+    Stripe.setPublishableKey @key
+    @form = $("#payment-form");
+    @form.submit this.formSubmit
+
+    $('.card-number').focus()
+
+    $('a#show-cvc-help').click (e) ->
+      $.facebox { div: '#cvc-help' }, 'cvc-help'
+      e.preventDefault()
+
+  formSubmit: (event) =>
+    # disable the submit button to prevent repeated clicks
+    $('.submit-button').attr "disabled", "disabled"
+    $(".payment-errors").text ""
+
+    target  = document.getElementById('processing-spinner')
+
+    spinnerOpts = {
+      lines: 9,
+      length: 4,
+      width: 3,
+      radius: 5,
+      corners: 0.8,
+      hwaccel: true,
+      speed: 1.6
+    };
+
+    @spinner = new Spinner(spinnerOpts).spin(target)
+
+    Stripe.createToken {
+        number:    $('.card-number').val(),
+        cvc:       $('.card-cvc').val(),
+        exp_month: $('.card-expiry-month').val(),
+        exp_year:  $('.card-expiry-year').val()
+    }, this.responseHandler
+
+    # prevent the form from submitting with the default action
+    event.preventDefault();
+  responseHandler: (status, response) =>
+    if (response.error)
+      this.logError response.error.message
+    else
+      # token contains id, last4, and card type
+      token = response['id']
+      # insert the token into the form so it gets submitted to the server
+      @form.append "<input type='hidden' name='stripeToken' value='#{token}'/>"
+
+      if @couponCode == ""
+        this.submitPayment()
+      else
+        this.checkCoupon()
+
+  couponCode: =>
+    return $('#coupon').val()
+  checkCoupon: =>
+    $.getJSON '/registration/coupon_valid', { coupon: @couponCode }, (data) =>
+      if data.coupon_valid
+        this.submitPayment()
+      else
+        this.logError "Coupon code is not valid"
+  submitPayment: =>
+    @form.get(0).submit()
+  logError: (message) =>
+    @spinner.stop()
+    $(".payment-errors").text      message
+    $(".submit-button").removeAttr "disabled"

app/assets/images/payment/dolla_billz.jpg

@@ -26,3 +26,4 @@
 @import partials/settings
 @import partials/sample_articles
 @import partials/admin
+@import partials/payments

app/assets/javascripts/pr.payment_processor.coffee

@@ -140,3 +140,5 @@ $green: #3D6E3C
 =classy-button
   +sassy-button("matte", 5px, 1em, #f9f9f9, #d9d9d9, #333, "inset")
   font-family: inherit
+  &:disabled
+    color: #6E6E6E

app/assets/stylesheets/application.css.sass

@@ -0,0 +1,50 @@
+div.payment-errors
+  color: #9E4040
+  font-size: 1.5em
+
+form#payment-form
+  margin: 1em 0
+  label, input[type=text], select
+    font-size: 14px
+    font-weight: normal
+    line-height: 20px
+  label
+    display: block
+    margin-bottom: 5px
+  input[type=text], select
+    display: inline-block
+    height: 20px
+    padding: 4px 6px
+    margin-bottom: 9px
+    font-size: 14px
+    line-height: 20px
+    color: #555
+    +border-radius(3px)
+  input[type=text]
+    background-color: #fff
+    border: 1px solid #CCC
+    +single-box-shadow(rgba(0,0,0,0.075), 0, 1px, 1px, false, true)
+    &:focus
+      outline: 0
+      outline: thin dotted 9
+      border-color: $dark-blue
+  select
+    height: 30px
+    line-height: 30px
+    background-color: #fff
+    border: 1px solid #CCC
+  span#processing-spinner
+    display: inline-block
+    margin-left: 30px
+    height: 5px
+
+p.image img
+  +image-box
+
+#cvc-help
+  display: none
+#facebox .cvc-help
+  padding-top:  20px!important
+  img
+    display: block
+    margin: 0 auto

app/assets/stylesheets/partials/_mixins.sass

@@ -2,15 +2,18 @@ class ApplicationController < ActionController::Base
   include CacheCooker::Oven
 
   protect_from_forgery
+
   before_filter :authenticate_cache_cooker!
   before_filter :authenticate
   before_filter :authenticate_user
   before_filter :enable_notifications
 
   helper_method :current_user, :active_broadcasts
 
+  private
+
   def authenticate
-    current_authorization || (store_location && redirect_to("/auth/github"))
+    current_authorization || (store_location && redirect_to(login_path))
   end
 
   def authenticate_user
@@ -52,8 +55,6 @@ def redirect_back_or_default(default)
     session[:return_to] = nil
   end
 
-  private
-
   def active_broadcasts
     if current_user
       session[:dismissed_broadcasts] ||= [-1]
@@ -66,9 +67,9 @@ def active_broadcasts
   def enable_notifications
     current_user.try(:enable_notifications)
   end
-  
+
   def render_http_error(status)
-    render :file   => "public/#{status}", :layout  => false, 
+    render :file   => "public/#{status}", :layout  => false,
            :status => status,             :formats => [:html]
   end
 end

app/assets/stylesheets/partials/_payments.sass

@@ -1,12 +1,13 @@
 class RegistrationController < ApplicationController
   skip_before_filter :authenticate_user
-  before_filter :ye_shall_not_pass, :except => [:payment]
+  before_filter :ye_shall_not_pass, :except => [ :payment, :payment_pending,
+                                                 :create_payment, :complete ]
 
   def index
     path = case current_user.status
       when "authorized"           then {:action => :edit_profile }
       when "pending_confirmation" then {:action => :update_profile }
-      when "confirmed"            then {:action => :payment }
+      when "confirmed"            then {:action => :payment_pending }
       else library_path
     end
 
@@ -49,12 +50,35 @@ def confirm_email
       # TODO swtich this to confirmed one we are doing payment processing
       user.update_attribute(:status, "payment_pending")
 
-      return redirect_to(:action => :payment)
+      return redirect_to(:action => :payment_pending)
     end
   end
 
+  def payment_pending
+
+  end
+
   def payment
-    # TODO fancy payment stuffs
+    redirect_to(:action => :complete) unless current_user.status == "payment_pending"
+  end
+
+  def create_payment
+    payment_gateway = current_user.payment_gateway
+    payment_gateway.subscribe(params)
+
+    redirect_to :action => :complete
+  end
+
+  def complete
+
+  end
+
+  def coupon_valid
+    payment_gateway = current_user.payment_gateway
+
+    valid = payment_gateway.coupon_valid?(params[:coupon])
+
+    render :json => { :coupon_valid => valid }.to_json
   end
 
   private

app/controllers/application_controller.rb

@@ -2,6 +2,10 @@ class SessionsController < ApplicationController
   skip_before_filter :authenticate
   skip_before_filter :authenticate_user
 
+  def new
+    redirect_to '/auth/github'
+  end
+
   def create
     auth = request.env['omniauth.auth']
     github_uid = auth["uid"].to_s

app/controllers/registration_controller.rb

@@ -0,0 +1,5 @@
+module PaymentGateway
+  def self.for_user(user)
+    PaymentGateway::Stripe.new(user)
+  end
+end

app/controllers/sessions_controller.rb

@@ -0,0 +1,12 @@
+module PaymentGateway
+  class MailChimp
+
+    def initialize(user)
+      @user = user
+    end
+
+    def subscribe(params = {})
+      raise NotImplementedError
+    end
+  end
+end

app/models/payment_gateway.rb

@@ -0,0 +1,87 @@
+module PaymentGateway
+  class Stripe
+
+    def initialize(user)
+      @user            = user
+      ::Stripe.api_key = STRIPE_SECRET_KEY
+    end
+
+    def coupon_valid?(coupon)
+      return true if coupon.blank?
+
+      begin
+        !!::Stripe::Coupon.retrieve(coupon)
+      rescue ::Stripe::InvalidRequestError => e
+        false
+      end
+    end
+
+    def subscribe(params = {})
+      token  = params[:stripeToken]
+      coupon = params[:coupon]
+
+      customer = find_or_create_customer(token)
+
+      subscription_options = { :plan => "practicing-ruby-monthly" }
+
+      subscription_options[:coupon] = coupon unless coupon.blank?
+
+      subscription = customer.update_subscription(subscription_options)
+
+      PaymentLog.create(:user_id => user.id, :raw_data => subscription.to_json)
+
+      user.subscriptions.create(
+        :start_date         => Date.today,
+        :payment_provider   => 'stripe',
+        :monthly_rate_cents => subscription.plan.amount,
+        :coupon_code        => coupon
+      )
+
+      user.update_attributes(
+        :payment_provider    => 'stripe',
+        :payment_provider_id => customer.id
+      )
+
+      user.enable
+    end
+
+    def unsubscribe
+      customer = find_customer
+
+      begin
+        customer.cancel_subscription
+      rescue ::Stripe::InvalidRequestError => e
+        raise unless e.message[/No active subscription/]
+      end
+
+      user.disable
+    end
+
+    private
+
+    attr_reader :user
+
+    def find_or_create_customer(token)
+      find_customer || create_customer(token)
+    end
+
+    def find_customer
+      if user.payment_provider == 'stripe' && !user.payment_provider_id.blank?
+        customer = ::Stripe::Customer.retrieve(user.payment_provider_id)
+        customer unless customer["deleted"]
+      end
+    end
+
+    def create_customer(token)
+      customer = ::Stripe::Customer.create(
+        :card        => token,
+        :description => user.github_nickname,
+        :email       => user.contact_email
+      )
+
+      PaymentLog.create(:user_id => user.id, :raw_data => customer.to_json)
+
+      customer
+    end
+  end
+end

app/models/payment_gateway/mail_chimp.rb

@@ -0,0 +1,3 @@
+class PaymentLog < ActiveRecord::Base
+  belongs_to :user
+end

app/models/payment_gateway/stripe.rb

@@ -0,0 +1,11 @@
+class Subscription < ActiveRecord::Base
+  belongs_to :user
+
+  def self.active
+    where(:finish_date => nil).first
+  end
+
+  def self.cancel_account
+    active.update_attributes(:finish_date => Date.today) if active
+  end
+end

app/models/payment_log.rb

@@ -4,6 +4,8 @@ class User < ActiveRecord::Base
   ACTIVE_STATUSES = %w{active payment_pending}
 
   has_many :comments
+  has_many :subscriptions
+  has_many :payment_logs
 
   validates_uniqueness_of :contact_email, :on => :update
   validates :status,      :inclusion => {
@@ -33,6 +35,10 @@ def disabled?
     status == 'disabled'
   end
 
+  def payment_gateway
+    PaymentGateway.for_user(self)
+  end
+
   def name
     "#{first_name} #{last_name}"
   end
@@ -41,13 +47,21 @@ def disable
     self.notifications_enabled = false
     self.status                = 'disabled'
 
+    subscriptions.cancel_account
+
     save
   end
 
   def enable(mailchimp_web_id=nil)
-    self.notifications_enabled = true
-    self.mailchimp_web_id      = mailchimp_web_id unless mailchimp_web_id.blank?
+
+    # TODO Move this to PaymentGateway
+    unless mailchimp_web_id.blank?
+      self.payment_provider_id = mailchimp_web_id
+      self.payment_provider    = "mailchimp"
+    end
+
     self.status                = 'active'
+    self.notifications_enabled = true
 
     save
   end

app/models/subscription.rb

@@ -0,0 +1,6 @@
+- content_for(:title) { "Registration Complete" }
+- content_for(:header) { "You're done. Thanks for subscribing!" }
+
+%p.image= image_tag "payment/dolla_billz.jpg"
+
+%p Your account is now set up, which means that you can enjoy everything that Practicing Ruby has to offer. You can start by #{link_to 'browsing through our library', library_path} or reading a #{link_to 'randomly selected article', random_article_path}. If you have any questions, please send an email to #{link_to 'gregory@practicingruby.com', 'mailto:gregory@practicingruby.com'}.

app/models/user.rb

@@ -1,8 +1,43 @@
 - content_for(:header) { "Payment" }
 - content_for(:title) { "Payment" }
+- content_for :header_bottom do
+  = javascript_include_tag "https://js.stripe.com/v1/"
+  :coffeescript
+    $ -> new PR.PaymentProcessor('#{STRIPE_PUBLISHABLE_KEY}');
 
-%p Right now Practicing Ruby is transitioning payment providers. So what does that mean for you?
+%h3 You are so close. Just one last step.
 
-%p Until we are ready to make the switch, you get access to the full site free of charge. Once we start accepting payments again, we'll ask you to enter your payment information to continue using the service, but you won't be charged for accessing our content between now and then. Not a bad deal, right?
+%p All we need from you now is your payment information. We will bill this card $8.00 USD each month until you cancel your account, which you can do at any time.
 
-%p Your account is now set up, which means that you can enjoy everything that Practicing Ruby has to offer. You can start by #{link_to 'browsing through our library', library_path} or reading a #{link_to 'randomly selected article', random_article_path}. If you have any questions, please send an email to #{link_to 'gregory@practicingruby.com', 'mailto:gregory@practicingruby.com'}.
+.payment-errors= @errors
+
+= form_tag registration_create_payment_path, :id => "payment-form" do
+  .row
+    %label Card Number
+    %input.card-number{type: "text", size: 20, autocomplete: "off"}
+  .row
+    %label
+      CVC
+      %a#show-cvc-help{:href => "#cvc", :tabindex => -1} ?
+    %input.card-cvc{type:"text", size: 4, autocomplete: "off"}
+  .row
+    %label Card Expiration
+    = select_month nil, { add_month_numbers: true },
+      { name: nil, id: nil, class: "card-expiry-month" }
+    = select_year nil,
+      { start_year: Date.today.year, end_year: Date.today.year+15},
+      { name: nil, id: nil, class: "card-expiry-year"}
+
+  .row
+    = label_tag :coupon, "Coupon Code"
+    = text_field_tag :coupon, params[:coupon]
+  %hr
+  %p
+    = submit_tag "Submit Payment", :class => "submit-button"
+    %span#processing-spinner
+
+#cvc-help
+  %p
+    For added security, we verify your CVC, CVV, or CID code. This code isn't
+    stored and is checked before processing your payment.
+  %p= image_tag "payment/cvc.gif"

app/views/registration/complete.html.haml

@@ -0,0 +1,8 @@
+- content_for(:header) { "Payment" }
+- content_for(:title) { "Payment" }
+
+%p Right now Practicing Ruby is transitioning payment providers. So what does that mean for you?
+
+%p Until we are ready to make the switch, you get access to the full site free of charge. Once we start accepting payments again, we'll ask you to enter your payment information to continue using the service, but you won't be charged for accessing our content between now and then. Not a bad deal, right?
+
+%p Your account is now set up, which means that you can enjoy everything that Practicing Ruby has to offer. You can start by #{link_to 'browsing through our library', library_path} or reading a #{link_to 'randomly selected article', random_article_path}. If you have any questions, please send an email to #{link_to 'gregory@practicingruby.com', 'mailto:gregory@practicingruby.com'}.

app/views/registration/payment.html.haml

@@ -23,7 +23,8 @@
     "mail_settings.rb"         => "config/initializers/mail_settings.rb",
     "mailchimp_settings.rb"    => "config/initializers/mailchimp_settings.rb",
     "omniauth.rb"              => "config/initializers/omniauth.rb",
-    "cache_cooker_settings.rb" => "config/initializers/cache_cooker_settings.rb" }.
+    "cache_cooker_settings.rb" => "config/initializers/cache_cooker_settings.rb",
+    "stripe.rb"                => "config/initializers/stripe.rb" }.
   each do |from, to|
     run "ln -nfs #{shared_path}/#{from} #{release_path}/#{to}"
   end

app/views/registration/payment_pending.html.haml

@@ -1,4 +1,4 @@
-set :branch, "onboarding"
+set :branch, $1 if `git branch` =~ /\* (\S+)\s/m
 set :deploy_to, "/var/rapp/staging/#{application}"
 
 server "staging.practicingruby.com", :app, :web, :db, :primary => true

config/deploy.rb

@@ -0,0 +1,2 @@
+STRIPE_SECRET_KEY      = 'SECRET'
+STRIPE_PUBLISHABLE_KEY = 'PUBLISHABLE'

config/deploy/staging.rb

@@ -44,13 +44,17 @@
     get   'confirm_email/:secret' => 'registration#confirm_email',
       :as => 'confirmation'
     get   'payment'               => 'registration#payment'
+    get   'payment_pending'       => 'registration#payment_pending'
+    post  'create_payment'        => 'registration#create_payment'
+    get   'complete'              => 'registration#complete'
     get   'restart'               => 'registration#restart'
+    get   'coupon_valid'          => 'registration#coupon_valid'
   end
 
   match '/sessions/link/:secret' => 'sessions#link'
-  match '/auth/github/callback' => 'sessions#create'
-  match '/logout' => 'sessions#destroy', :as => 'logout'
-  match '/auth/github', :as => 'login'
+  match '/auth/github/callback'  => 'sessions#create'
+  match '/logout'                => 'sessions#destroy', :as => 'logout'
+  match '/login'                 => 'sessions#new',     :as => 'login'
 
   match '/dismiss_broadcasts' => 'announcements#dismiss', :as => 'dismiss_broadcasts'
 

config/initializers/stripe.rb.example

@@ -0,0 +1,45 @@
+class AddPaymentModels < ActiveRecord::Migration
+  def up
+    create_table :subscriptions do |t|
+      t.belongs_to :user
+      t.date       :start_date, :null => false
+      t.date       :finish_date
+      t.text       :payment_provider
+      t.integer    :monthly_rate_cents
+    end
+
+    create_table :payment_logs do |t|
+      t.belongs_to :user
+      t.text       :raw_data
+    end
+
+    add_column :users, :payment_provider,    :text
+    add_column :users, :payment_provider_id, :text
+
+    migrate_sql = %{UPDATE users
+      SET   payment_provider    = 'mailchimp',
+            payment_provider_id = users.mailchimp_web_id
+      WHERE mailchimp_web_id IS NOT NULL }
+
+    User.connection.execute(migrate_sql)
+
+    # TODO: Drop mailchimp_web_id column
+    #remove_column :users, :mailchimp_web_id
+  end
+
+  def down
+    drop_table :subscriptions
+    drop_table :payment_logs
+
+    #add_column :users, :mailchimp_web_id, :text
+
+    migrate_sql = %{UPDATE users
+      SET   mailchimp_web_id = users.payment_provider_id
+      WHERE payment_provider = 'mailchimp' }
+
+    User.connection.execute(migrate_sql)
+
+    remove_column :users, :payment_provider
+    remove_column :users, :payment_provider_id
+  end
+end

config/routes.rb

@@ -0,0 +1,5 @@
+class AddCouponCodeToSubscriptions < ActiveRecord::Migration
+  def change
+    add_column :subscriptions, :coupon_code, :text
+  end
+end

db/migrate/20120927154744_add_payment_models.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20120914175304) do
+ActiveRecord::Schema.define(:version => 20121011002134) do
 
   create_table "announcements", :force => true do |t|
     t.text     "title"
@@ -109,6 +109,11 @@
     t.datetime "updated_at"
   end
 
+  create_table "payment_logs", :force => true do |t|
+    t.integer "user_id"
+    t.text    "raw_data"
+  end
+
   create_table "shared_articles", :force => true do |t|
     t.integer  "user_id"
     t.integer  "article_id"
@@ -118,25 +123,36 @@
     t.datetime "updated_at"
   end
 
+  create_table "subscriptions", :force => true do |t|
+    t.integer "user_id"
+    t.date    "start_date",         :null => false
+    t.date    "finish_date"
+    t.text    "payment_provider"
+    t.integer "monthly_rate_cents"
+    t.text    "coupon_code"
+  end
+
   create_table "users", :force => true do |t|
     t.text     "first_name"
     t.text     "last_name"
     t.text     "email"
-    t.text     "mailchimp_web_id"
     t.datetime "created_at"
     t.datetime "updated_at"
     t.text     "github_nickname"
-    t.boolean  "admin",                 :default => false
-    t.boolean  "notify_conversations",  :default => true,  :null => false
-    t.boolean  "notify_mentions",       :default => true,  :null => false
-    t.boolean  "notify_comment_made",   :default => false, :null => false
-    t.boolean  "beta_tester",           :default => false
-    t.boolean  "account_disabled",      :default => false
-    t.boolean  "notifications_enabled", :default => false, :null => false
+    t.boolean  "admin",                       :default => false
+    t.boolean  "notify_conversations",        :default => true,  :null => false
+    t.boolean  "notify_mentions",             :default => true,  :null => false
+    t.boolean  "notify_comment_made",         :default => false, :null => false
+    t.boolean  "beta_tester",                 :default => false
+    t.boolean  "account_disabled",            :default => false
+    t.boolean  "notifications_enabled",       :default => false, :null => false
     t.string   "status"
     t.string   "contact_email"
     t.string   "access_token"
-    t.boolean  "notify_updates",        :default => true,  :null => false
+    t.boolean  "notify_updates",              :default => true,  :null => false
+    t.text     "mailchimp_web_id"
+    t.text     "payment_provider"
+    t.text     "payment_provider_id"
   end
 
   create_table "volumes", :force => true do |t|

db/migrate/20121011002134_add_coupon_code_to_subscriptions.rb

@@ -52,8 +52,10 @@ def profile
     end
 
     def find_user
-      User.find_by_mailchimp_web_id(params[:data][:web_id]) ||
-      User.where("LOWER(email) = ?", params[:data][:email].downcase).first
+      User.where(%{
+        (payment_provider = ? AND payment_provider_id = ?) OR LOWER(email) = ?},
+        'mailchimp', params[:data][:web_id], params[:data][:email].downcase
+      ).first
     end
   end
 end

db/schema.rb

@@ -6,16 +6,13 @@ namespace :travis do
 
     # Move configuration files into place
     #
-    initializers = Rails.root.join("config", "initializers")
+    initializer_dir = Rails.root.join("config", "initializers")
+    initializers    = %w{mailchimp_settings omniauth secret_token stripe}
 
-    FileUtils.cp initializers + "mailchimp_settings.rb.example",
-                 initializers + "mailchimp_settings.rb"
-
-    FileUtils.cp initializers + "omniauth.rb.example",
-                 initializers + "omniauth.rb"
-
-    FileUtils.cp initializers + "secret_token.rb.example",
-                 initializers + "secret_token.rb"
+    initializers.each do |initializer|
+      FileUtils.cp initializer_dir + "#{initializer}.rb.example",
+                   initializer_dir + "#{initializer}.rb"
+    end
 
     # Setup our database.yml file
     #

lib/mail_chimp/web_hooks.rb

@@ -1,6 +1,6 @@
 FactoryGirl.define do
   sequence(:email) { |n| "person#{n}@example.com" }
-  sequence(:mailchimp_web_id) { |n| "person_#{n}" }
+  sequence(:payment_provider_id) { |n| "person_#{n}" }
 
   factory :user do |u|
     u.first_name            'Frank'
@@ -9,7 +9,8 @@
     u.notifications_enabled  true
     u.email                  { FactoryGirl.generate(:email) }
     u.contact_email          { FactoryGirl.generate(:email) }
-    u.mailchimp_web_id       { FactoryGirl.generate(:mailchimp_web_id) }
+    u.payment_provider       'mailchimp'
+    u.payment_provider_id    { FactoryGirl.generate(:payment_provider_id) }
     u.status                 'active'
   end
 end

lib/tasks/travis.rake

@@ -0,0 +1,21 @@
+require 'test_helper'
+
+class PaymentTest < ActionDispatch::IntegrationTest
+  setup do
+    Capybara.current_driver = :webkit
+  end
+
+  test "valid payments activate user account" do
+    simulated_user do
+      register(Support::SimulatedUser.default)
+      make_stripe_payment
+    end
+  end
+
+  test "coupon codes can be applied to payments" do
+    simulated_user do
+      register(Support::SimulatedUser.default)
+      make_stripe_payment(:coupon => "test")
+    end
+  end
+end

test/factories/user_factory.rb

@@ -63,7 +63,7 @@ def click_link_within(scope, text)
     end
 
     def sign_user_in
-      visit "/auth/github"
+      visit login_path
     end
 
     def sign_out

test/integration/payment_test.rb

@@ -32,6 +32,7 @@ def register(params)
       create_profile(params)
       confirm_email
       make_payment
+      read_article
     end
 
     def create_profile(params={})
@@ -43,6 +44,8 @@ def create_profile(params={})
     end
 
     def confirm_email
+      browser { assert_current_path registration_update_profile_path }
+
       mail   = ActionMailer::Base.deliveries.pop
       base   = Rails.application.routes.url_helpers.
                  registration_confirmation_path(:secret => "")
@@ -55,7 +58,57 @@ def confirm_email
     end
 
     def make_payment
-      browser { assert_current_path registration_payment_path }
+      browser do
+        assert_current_path registration_payment_pending_path
+      end
+
+      # TODO Find a way to test this through the UI
+      #
+      @user.subscriptions.create(:start_date => Date.today)
+    end
+
+    def read_article
+      article = FactoryGirl.create(:article)
+
+      browser do
+        visit article_path(article)
+        assert_current_path article_path(article)
+      end
+
+      @browser.assert @user.subscriptions.active, "No active subscription"
+    end
+
+    def make_stripe_payment(params={})
+      @user.subscriptions.delete_all
+
+      browser do
+        skip_on_travis
+
+        Capybara.default_wait_time = 15
+
+        visit registration_payment_path
+
+        card  = find(:css, "input.card-number")
+        cvc   = find(:css, "input.card-cvc")
+        month = find(:css, "select.card-expiry-month")
+        year  = find(:css, "select.card-expiry-year")
+
+        card.set  "4242424242424242"
+        cvc.set   "123"
+        month.set "January"
+        year.set  Date.today.year + 1
+
+        fill_in "Coupon", :with => params.fetch(:coupon, "")
+
+        click_button "Submit Payment"
+
+        wait_until { current_path == registration_complete_path }
+
+        assert_content "Thanks for subscribing"
+
+        visit library_path
+        assert_current_path library_path
+      end
     end
 
     def payment_failure
@@ -82,6 +135,8 @@ def cancel_account
         visit library_path
         assert_current_path problems_sessions_path
       end
+
+      @browser.refute @user.subscriptions.active, "Subscription was not ended"
     end
 
     def restart_registration

test/support/integration.rb

@@ -41,3 +41,7 @@ class ActionDispatch::IntegrationTest
   end
 end
 
+def skip_on_travis
+  skip "Do not run this test on travis ci" if ENV["TRAVIS"]
+end
+

test/support/simulated_user.rb

@@ -138,7 +138,7 @@ def user_to_mailchimp_params(user, request_type)
     { :type => request_type,
       :data => {
         :email  => user.email,
-        :web_id => user.mailchimp_web_id,
+        :web_id => user.payment_provider_id,
         :merges => {
           :FNAME => user.first_name,
           :LNAME => user.last_name

test/test_helper.rb

@@ -37,7 +37,7 @@
 
 #facebox .close{
   position:absolute;
-  top:5px;
+  top:-3px;
   right:5px;
   padding:2px;
 /*  background:#fff;*/