paid_up 0.9.13 → 0.9.14

data/spec/dummy/config/initializers/devise.rb

@@ -4,13 +4,15 @@ Devise.setup do |config|
   # The secret key used by Devise. Devise uses this key to generate
   # random tokens. Changing this key will render invalid all existing
   # confirmation, reset password and unlock tokens in the database.
-  # config.secret_key = '79c56bbc3a599ca06df00c2f65aaa9edac01f37daffb886f5983a5eb03341f5d8eedb4431a557033b04c33ceb7fb3c19e2f367aa3434ac9f58f0af4e903bb0e3'
+  # config.secret_key = '79c56bbc3a599ca06df00c2f65aaa9edac01f37daffb886f5983a'\
+  #                       '5eb03341f5d8eedb4431a557033b04c33ceb7fb3c19e2f367aa'\
+  #                       '3434ac9f58f0af4e903bb0e3'
 
   # ==> Mailer Configuration
   # Configure the e-mail address which will be shown in Devise::Mailer,
   # note that it will be overwritten if you use your own mailer class
-  # with default "from" parameter.
-  config.mailer_sender = 'please-change-me-at-config-initializers-devise@example.com'
+  # with default 'from' parameter.
+  config.mailer_sender = 'do-not-reply@example.com'
 
   # Configure the class responsible to send e-mails.
   # config.mailer = 'Devise::Mailer'
@@ -26,39 +28,44 @@ Devise.setup do |config|
   # just :email. You can configure it to use [:username, :subdomain], so for
   # authenticating a user, both parameters are required. Remember that those
   # parameters are used only when authenticating and not when retrieving from
-  # session. If you need permissions, you should implement that in a before filter.
+  # session. If you need permissions, you should implement that in a before
+  # filter.
   # You can also supply a hash where the value is a boolean determining whether
   # or not authentication should be aborted when the value is not present.
   # config.authentication_keys = [ :email ]
 
-  # Configure parameters from the request object used for authentication. Each entry
-  # given should be a request method and it will automatically be passed to the
-  # find_for_authentication method and considered in your model lookup. For instance,
-  # if you set :request_keys to [:subdomain], :subdomain will be used on authentication.
-  # The same considerations mentioned for authentication_keys also apply to request_keys.
+  # Configure parameters from the request object used for authentication. Each
+  # entry given should be a request method and it will automatically be passed
+  # to the find_for_authentication method and considered in your model lookup.
+  # For instance, if you set :request_keys to [:subdomain], :subdomain will be
+  # used on authentication. The same considerations mentioned for
+  # authentication_keys also apply to request_keys.
   # config.request_keys = []
 
   # Configure which authentication keys should be case-insensitive.
   # These keys will be downcased upon creating or modifying a user and when used
   # to authenticate or find a user. Default is :email.
-  config.case_insensitive_keys = [ :email ]
+  config.case_insensitive_keys = [:email]
 
   # Configure which authentication keys should have whitespace stripped.
   # These keys will have whitespace before and after removed upon creating or
-  # modifying a user and when used to authenticate or find a user. Default is :email.
-  config.strip_whitespace_keys = [ :email ]
+  # modifying a user and when used to authenticate or find a user.
+  # Default is :email.
+  config.strip_whitespace_keys = [:email]
 
   # Tell if authentication through request.params is enabled. True by default.
-  # It can be set to an array that will enable params authentication only for the
-  # given strategies, for example, `config.params_authenticatable = [:database]` will
-  # enable it only for database (email + password) authentication.
+  # It can be set to an array that will enable params authentication only for
+  # the given strategies, for example,
+  # `config.params_authenticatable = [:database]` will enable it only for
+  # database (email + password) authentication.
   # config.params_authenticatable = true
 
   # Tell if authentication through HTTP Auth is enabled. False by default.
   # It can be set to an array that will enable http authentication only for the
-  # given strategies, for example, `config.http_authenticatable = [:database]` will
-  # enable it only for database authentication. The supported strategies are:
-  # :database      = Support basic authentication with authentication key + password
+  # given strategies, for example, `config.http_authenticatable = [:database]`
+  # will enable it only for database authentication. The supported strategies
+  # are:
+  # :database = Support basic authentication with authentication key + password
   # config.http_authenticatable = false
 
   # If 401 status code should be returned for AJAX requests. True by default.
@@ -87,17 +94,21 @@ Devise.setup do |config|
 
   # ==> Configuration for :database_authenticatable
   # For bcrypt, this is the cost for hashing the password and defaults to 10. If
-  # using other encryptors, it sets how many times you want the password re-encrypted.
+  # using other encryptors, it sets how many times you want the password
+  # re-encrypted.
   #
-  # Limiting the stretches to just one in testing will increase the performance of
-  # your test suite dramatically. However, it is STRONGLY RECOMMENDED to not use
-  # a value less than 10 in other environments. Note that, for bcrypt (the default
-  # encryptor), the cost increases exponentially with the number of stretches (e.g.
-  # a value of 20 is already extremely slow: approx. 60 seconds for 1 calculation).
+  # Limiting the stretches to just one in testing will increase the performance
+  # of your test suite dramatically. However, it is STRONGLY RECOMMENDED to not
+  # use a value less than 10 in other environments. Note that, for bcrypt
+  # (the default encryptor), the cost increases exponentially with the number of
+  # stretches (e.g. a value of 20 is already extremely slow: approx. 60 seconds
+  # for 1 calculation).
   config.stretches = Rails.env.test? ? 1 : 10
 
   # Setup a pepper to generate the encrypted password.
-  # config.pepper = 'bb8297994955970522d4c4d11d015f3ad97d00cf168464ba0542934583ce1d1a960a5111f385b1176a311bd9c44c6c5eea1804828485457eb2931d6cdbd2fcdf'
+  # config.pepper = 'bb8297994955970522d4c4d11d015f3ad97d00cf168464ba054293458'\
+  #                   '3ce1d1a960a5111f385b1176a311bd9c44c6c5eea1804828485457e'\
+  #                   'b2931d6cdbd2fcdf'
 
   # ==> Configuration for :confirmable
   # A period that the user is allowed to access the website even without
@@ -116,9 +127,10 @@ Devise.setup do |config|
   # config.confirm_within = 3.days
 
   # If true, requires any email changes to be confirmed (exactly the same way as
-  # initial account confirmation) to be applied. Requires additional unconfirmed_email
-  # db field (see migrations). Until confirmed, new email is stored in
-  # unconfirmed_email column, and copied to email column on successful confirmation.
+  # initial account confirmation) to be applied. Requires additional
+  # unconfirmed_email db field (see migrations). Until confirmed, new email is
+  # stored in unconfirmed_email column, and copied to email column on successful
+  # confirmation.
   config.reconfirmable = true
 
   # Defines which key will be used when confirming an account
@@ -157,7 +169,8 @@ Devise.setup do |config|
 
   # ==> Configuration for :lockable
   # Defines which strategy will be used to lock an account.
-  # :failed_attempts = Locks an account after a number of failed attempts to sign in.
+  # :failed_attempts = Locks an account after a number of failed attempts to
+  # sign in.
   # :none            = No lock strategy. You should handle locking by yourself.
   # config.lock_strategy = :failed_attempts
 
@@ -166,7 +179,8 @@ Devise.setup do |config|
 
   # Defines which strategy will be used to unlock an account.
   # :email = Sends an unlock link to the user email
-  # :time  = Re-enables login after a certain amount of time (see :unlock_in below)
+  # :time  = Re-enables login after a certain amount of time (see :unlock_in
+  # below)
   # :both  = Enables both strategies
   # :none  = No unlock strategy. You should handle unlocking by yourself.
   # config.unlock_strategy = :both
@@ -192,19 +206,19 @@ Devise.setup do |config|
   config.reset_password_within = 6.hours
 
   # ==> Configuration for :encryptable
-  # Allow you to use another encryption algorithm besides bcrypt (default). You can use
-  # :sha1, :sha512 or encryptors from others authentication tools as :clearance_sha1,
-  # :authlogic_sha512 (then you should set stretches above to 20 for default behavior)
-  # and :restful_authentication_sha1 (then you should set stretches to 10, and copy
-  # REST_AUTH_SITE_KEY to pepper).
+  # Allow you to use another encryption algorithm besides bcrypt (default).
+  # You can use :sha1, :sha512 or encryptors from others authentication tools as
+  # :clearance_sha1, :authlogic_sha512 (then you should set stretches above to
+  # 20 for default behavior) and :restful_authentication_sha1 (then you should
+  # set stretches to 10, and copy REST_AUTH_SITE_KEY to pepper).
   #
   # Require the `devise-encryptable` gem when using anything other than bcrypt
   # config.encryptor = :sha512
 
   # ==> Scopes configuration
-  # Turn scoped views on. Before rendering "sessions/new", it will first check for
-  # "users/sessions/new". It's turned off by default because it's slower if you
-  # are using only default views.
+  # Turn scoped views on. Before rendering 'sessions/new', it will first check
+  # for "users/sessions/new". It's turned off by default because it's slower if
+  # you are using only default views.
   # config.scoped_views = false
 
   # Configure the default scope given to Warden. By default it's the first
@@ -223,7 +237,7 @@ Devise.setup do |config|
   # If you have any extra navigational formats, like :iphone or :mobile, you
   # should add them to the navigational formats lists.
   #
-  # The "*/*" below is required to match Internet Explorer requests.
+  # The '*/*' below is required to match Internet Explorer requests.
   # config.navigational_formats = ['*/*', :html]
 
   # The default HTTP method used to sign out a resource. Default is :delete.
@@ -236,7 +250,8 @@ Devise.setup do |config|
 
   # ==> Warden configuration
   # If you want to use other strategies, that are not supported by Devise, or
-  # change the failure app, you can configure them inside the config.warden block.
+  # change the failure app, you can configure them inside the config.warden
+  # block.
   #
   # config.warden do |manager|
   #   manager.intercept_401 = false
@@ -244,8 +259,9 @@ Devise.setup do |config|
   # end
 
   # ==> Mountable engine configurations
-  # When using Devise inside an engine, let's call it `MyEngine`, and this engine
-  # is mountable, there are some extra configurations to be taken into account.
+  # When using Devise inside an engine, let's call it `MyEngine`, and this
+  # engine is mountable, there are some extra configurations to be taken into
+  # account.
   # The following options are available, assuming the engine is mounted as:
   #
   #     mount MyEngine, at: '/my_engine'

data/spec/dummy/config/initializers/high_voltage.rb

@@ -1,3 +1,3 @@
 HighVoltage.configure do |config|
   config.home_page = 'index'
-end
+end

data/spec/dummy/config/initializers/mime_types.rb

@@ -1,4 +1,4 @@
 # Be sure to restart your server when you modify this file.
 
 # Add new mime types for use in respond_to blocks:
-# Mime::Type.register "text/richtext", :rtf
+# Mime::Type.register 'text/richtext', :rtf

data/spec/dummy/config/initializers/paid_up.rb

@@ -4,22 +4,24 @@ PaidUp.configure do |config|
   config.free_plan_stripe_id = 'free-plan'
 
   PaidUp.add_feature(
-      slug: 'ad_free',
-      title: 'Ad Free',
-      description: 'Are ads removed from the site with this plan?',
-      setting_type: 'boolean'
+    slug: 'ad_free',
+    title: 'Ad Free',
+    description: 'Are ads removed from the site with this plan?',
+    setting_type: 'boolean'
   )
   PaidUp.add_feature(
-      slug: 'groups',
-      title: 'Groups',
-      description: 'How many groups are allowed with this plan?',
-      setting_type: 'rolify_rows'  # Enables table row counting that is enabled by a positive value
-# for the PaidUp::PlanFeatureSetting.setting associated with this PaidUp::Feature
+    slug: 'groups',
+    title: 'Groups',
+    description: 'How many groups are allowed with this plan?',
+    # Enables table row counting that is enabled by a positive value
+    # for the PaidUp::PlanFeatureSetting.setting associated with this
+    # PaidUp::Feature
+    setting_type: 'rolify_rows'
   )
   PaidUp.add_feature(
-      slug: 'doodads',
-      title: 'Doodads',
-      description: 'How many doodads included with this plan?',
-      setting_type: 'table_rows'
+    slug: 'doodads',
+    title: 'Doodads',
+    description: 'How many doodads included with this plan?',
+    setting_type: 'table_rows'
   )
-end
+end

data/spec/dummy/config/initializers/rolify.rb

@@ -2,6 +2,7 @@ Rolify.configure do |config|
   # By default ORM adapter is ActiveRecord. uncomment to use mongoid
   # config.use_mongoid
 
-  # Dynamic shortcuts for User class (user.is_admin? like methods). Default is: false
+  # Dynamic shortcuts for User class (user.is_admin? like methods).
+  # Default is: false
   # config.use_dynamic_shortcuts
-end
+end

data/spec/dummy/config/initializers/wrap_parameters.rb

@@ -3,7 +3,8 @@
 # This file contains settings for ActionController::ParamsWrapper which
 # is enabled by default.
 
-# Enable parameter wrapping for JSON. You can disable this by setting :format to an empty array.
+# Enable parameter wrapping for JSON. You can disable this by setting
+# :format to an empty array.
 ActiveSupport.on_load(:action_controller) do
   wrap_parameters format: [:json] if respond_to?(:wrap_parameters)
 end

data/spec/dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Rails.application.routes.draw do
   devise_for :users
-  mount PaidUp::Engine => '/', :as => 'paid_up'
-end
+  mount PaidUp::Engine => '/', as: 'paid_up'
+end

data/spec/dummy/db/migrate/20150523010827_add_devise_to_users.rb

@@ -2,8 +2,8 @@ class AddDeviseToUsers < ActiveRecord::Migration
   def self.up
     change_table(:users) do |t|
       ## Database authenticatable
-      t.string :email,              null: false, default: ""
-      t.string :encrypted_password, null: false, default: ""
+      t.string :email,              null: false, default: ''
+      t.string :encrypted_password, null: false, default: ''
 
       ## Recoverable
       t.string   :reset_password_token

data/spec/dummy/db/migrate/20150523010837_rolify_create_roles.rb

@@ -2,12 +2,12 @@ class RolifyCreateRoles < ActiveRecord::Migration
   def change
     create_table(:roles) do |t|
       t.string :name
-      t.references :resource, :polymorphic => true
+      t.references :resource, polymorphic: true
 
       t.timestamps
     end
 
-    create_table(:users_roles, :id => false) do |t|
+    create_table(:users_roles, id: false) do |t|
       t.references :user
       t.references :role
     end

data/spec/dummy/db/migrate/20160207184112_create_paid_up_plans_table.paid_up.rb

@@ -9,7 +9,7 @@ class CreatePaidUpPlansTable < ActiveRecord::Migration
 
       t.timestamps
     end
-    add_index :paid_up_plans, :title, :unique => true
-    add_index :paid_up_plans, :stripe_id, :unique => true
+    add_index :paid_up_plans, :title, unique: true
+    add_index :paid_up_plans, :stripe_id, unique: true
   end
-end
+end

data/spec/dummy/db/schema.rb

@@ -13,77 +13,77 @@
 
 ActiveRecord::Schema.define(version: 20160210165341) do
 
-  create_table "doodads", force: :cascade do |t|
-    t.string "user_id"
-    t.string "name"
-    t.text   "description"
+  create_table 'doodads', force: :cascade do |t|
+    t.string 'user_id'
+    t.string 'name'
+    t.text   'description'
   end
 
-  add_index "doodads", ["user_id"], name: "index_doodads_on_user_id"
+  add_index "doodads", ['user_id'], name: "index_doodads_on_user_id"
 
-  create_table "groups", force: :cascade do |t|
-    t.string   "title"
-    t.text     "description"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'groups', force: :cascade do |t|
+    t.string   'title'
+    t.text     'description'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  create_table "paid_up_plan_feature_settings", force: :cascade do |t|
-    t.integer "paid_up_plan_id"
-    t.string  "feature"
-    t.integer "setting"
+  create_table 'paid_up_plan_feature_settings', force: :cascade do |t|
+    t.integer 'paid_up_plan_id'
+    t.string  'feature'
+    t.integer 'setting'
   end
 
-  add_index "paid_up_plan_feature_settings", ["feature"], name: "index_paid_up_plan_feature_settings_on_feature"
-  add_index "paid_up_plan_feature_settings", ["paid_up_plan_id"], name: "index_paid_up_plan_feature_settings_on_paid_up_plan_id"
+  add_index "paid_up_plan_feature_settings", ['feature'], name: "index_paid_up_plan_feature_settings_on_feature"
+  add_index "paid_up_plan_feature_settings", ['paid_up_plan_id'], name: "index_paid_up_plan_feature_settings_on_paid_up_plan_id"
 
-  create_table "paid_up_plans", force: :cascade do |t|
-    t.string   "stripe_id"
-    t.string   "title"
-    t.text     "description"
-    t.integer  "sort_order"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'paid_up_plans', force: :cascade do |t|
+    t.string   'stripe_id'
+    t.string   'title'
+    t.text     'description'
+    t.integer  'sort_order'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  add_index "paid_up_plans", ["stripe_id"], name: "index_paid_up_plans_on_stripe_id", unique: true
-  add_index "paid_up_plans", ["title"], name: "index_paid_up_plans_on_title", unique: true
+  add_index "paid_up_plans", ["stripe_id"], name: 'index_paid_up_plans_on_stripe_id', unique: true
+  add_index "paid_up_plans", ["title"], name: 'index_paid_up_plans_on_title', unique: true
 
-  create_table "roles", force: :cascade do |t|
-    t.string   "name"
-    t.integer  "resource_id"
-    t.string   "resource_type"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'roles', force: :cascade do |t|
+    t.string   'name'
+    t.integer  'resource_id'
+    t.string   'resource_type'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  add_index "roles", ["name", "resource_type", "resource_id"], name: "index_roles_on_name_and_resource_type_and_resource_id"
-  add_index "roles", ["name"], name: "index_roles_on_name"
+  add_index "roles", ["name", "resource_type", 'resource_id'], name: "index_roles_on_name_and_resource_type_and_resource_id"
+  add_index "roles", ['name'], name: "index_roles_on_name"
 
-  create_table "users", force: :cascade do |t|
-    t.string   "name"
-    t.string   "email",                  default: "", null: false
-    t.string   "encrypted_password",     default: "", null: false
-    t.string   "reset_password_token"
-    t.datetime "reset_password_sent_at"
-    t.datetime "remember_created_at"
-    t.integer  "sign_in_count",          default: 0,  null: false
-    t.datetime "current_sign_in_at"
-    t.datetime "last_sign_in_at"
-    t.string   "current_sign_in_ip"
-    t.string   "last_sign_in_ip"
-    t.string   "stripe_id"
-    t.string   "coupon_code"
+  create_table 'users', force: :cascade do |t|
+    t.string   'name'
+    t.string   "email",                  default: '', null: false
+    t.string   "encrypted_password",     default: '', null: false
+    t.string   'reset_password_token'
+    t.datetime 'reset_password_sent_at'
+    t.datetime 'remember_created_at'
+    t.integer  'sign_in_count',          default: 0,  null: false
+    t.datetime 'current_sign_in_at'
+    t.datetime 'last_sign_in_at'
+    t.string   'current_sign_in_ip'
+    t.string   'last_sign_in_ip'
+    t.string   'stripe_id'
+    t.string   'coupon_code'
   end
 
-  add_index "users", ["email"], name: "index_users_on_email", unique: true
-  add_index "users", ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
+  add_index "users", ["email"], name: 'index_users_on_email', unique: true
+  add_index "users", ["reset_password_token"], name: 'index_users_on_reset_password_token', unique: true
 
-  create_table "users_roles", id: false, force: :cascade do |t|
-    t.integer "user_id"
-    t.integer "role_id"
+  create_table 'users_roles', id: false, force: :cascade do |t|
+    t.integer 'user_id'
+    t.integer 'role_id'
   end
 
-  add_index "users_roles", ["user_id", "role_id"], name: "index_users_roles_on_user_id_and_role_id"
+  add_index "users_roles", ["user_id", 'role_id'], name: "index_users_roles_on_user_id_and_role_id"
 
 end

data/spec/dummy/db/seeds.rb

@@ -4,27 +4,23 @@
 
 Stripe::Plan.find_or_create_by_id(
   'anonymous-plan',
-  {
-    amount: 0,
-    interval: 'month',
-    name: 'Anonymous Plan',
-    currency: 'usd'
-  }
-)
-anonymous_plan = PaidUp::Plan.create(
+  amount: 0,
+  interval: 'month',
+  name: 'Anonymous Plan',
+  currency: 'usd'
+)
+PaidUp::Plan.create(
   title: 'Anonymous',
   stripe_id: 'anonymous-plan',
-  description: "What you can do without logging in.",
+  description: 'What you can do without logging in.',
   sort_order: -1
 )
 Stripe::Plan.find_or_create_by_id(
   'free-plan',
-  {
-    amount: 0,
-    interval: 'month',
-    name: 'Free Plan',
-    currency: 'usd'
-  }
+  amount: 0,
+  interval: 'month',
+  name: 'Free Plan',
+  currency: 'usd'
 )
 free_plan = PaidUp::Plan.create(
   title: 'Free',
@@ -34,47 +30,42 @@ free_plan = PaidUp::Plan.create(
 )
 Stripe::Plan.find_or_create_by_id(
   'no-ads-plan',
-  {
-    amount: 100,
-    interval: 'month',
-    name: 'No Ads Plan',
-    currency: 'usd'
-  }
+  amount: 100,
+  interval: 'month',
+  name: 'No Ads Plan',
+  currency: 'usd'
 )
 no_ads_plan = PaidUp::Plan.create(
   title: 'No Ads',
   stripe_id: 'no-ads-plan',
-  description: "No frills, just removes the ads.",
+  description: 'No frills, just removes the ads.',
   sort_order: 1
 )
 Stripe::Plan.find_or_create_by_id(
   'group-leader-plan',
-  {
-    amount: 500,
-    interval: 'month',
-    name: 'Group Leader Plan',
-    currency: 'usd'
-  }
+  amount: 500,
+  interval: 'month',
+  name: 'Group Leader Plan',
+  currency: 'usd'
 )
 group_leader_plan = PaidUp::Plan.create(
   title: 'Group Leader',
   stripe_id: 'group-leader-plan',
-  description: "For leaders of single groups, with configuration.",
+  description: 'For leaders of single groups, with configuration.',
   sort_order: 2
 )
 Stripe::Plan.find_or_create_by_id(
   'professional-plan',
-  {
-    amount: 1000,
-    interval: 'month',
-    name: 'Professional Plan',
-    currency: 'usd'
-  }
+  amount: 1000,
+  interval: 'month',
+  name: 'Professional Plan',
+  currency: 'usd'
 )
 professional_plan = PaidUp::Plan.create(
   title: 'Professional',
   stripe_id: 'professional-plan',
-  description: "Designed for professionals with unlimited groups, a calendar and configuration.",
+  description: 'Designed for professionals with unlimited groups, a calendar '\
+                 'and configuration.',
   sort_order: 3
 )
 
@@ -83,21 +74,17 @@ professional_plan = PaidUp::Plan.create(
 ###############
 
 Stripe::Coupon.find_or_create_by_id(
-    '25OFF',
-    {
-        percent_off: 25,
-        currency: 'usd',
-        duration: 'forever'
-    }
+  '25OFF',
+  percent_off: 25,
+  currency: 'usd',
+  duration: 'forever'
 )
 
 Stripe::Coupon.find_or_create_by_id(
-    'MINUS25',
-    {
-        amount_off: 25,
-        currency: 'usd',
-        duration: 'forever'
-    }
+  'MINUS25',
+  amount_off: 25,
+  currency: 'usd',
+  duration: 'forever'
 )
 
 ######################
@@ -105,10 +92,8 @@ Stripe::Coupon.find_or_create_by_id(
 ######################
 Stripe::Customer.find_or_create_by_id(
   'anonymous-customer',
-  {
-    description: 'Anonymous Customer',
-    plan: 'anonymous-plan'
-  }
+  description: 'Anonymous Customer',
+  plan: 'anonymous-plan'
 )
 #######################
 # PlanFeatureSettings #
@@ -159,13 +144,13 @@ PaidUp::PlanFeatureSetting.create(
 # Users     #
 ###############
 
-free_subscriber = FactoryGirl.create(
+FactoryGirl.create(
   :user,
   name: 'Free Subscriber',
   plan: free_plan
 )
 
-no_ads_subscriber = FactoryGirl.create(
+FactoryGirl.create(
   :user,
   name: 'No Ads Subscriber',
   plan: no_ads_plan
@@ -189,7 +174,7 @@ professional_subscriber = FactoryGirl.create(
   plan: professional_plan
 )
 
-blank_subscriber = FactoryGirl.create(
+FactoryGirl.create(
   :user,
   name: 'Blank Subscriber',
   plan: professional_plan
@@ -238,4 +223,4 @@ FactoryGirl.create(
   :group,
   title: 'Past Due Group',
   owner: past_due_subscriber
-)
+)

data/spec/dummy/lib/tasks/system.rake

@@ -7,6 +7,6 @@ namespace :system do
     Rake::Task['db:migrate'].invoke
     Rake::Task['db:seed'].invoke
     Rake::Task['db:test:prepare'].invoke
-    puts "========== Completed in #{(Time.now-time).to_s} s =========="
+    puts "========== Completed in #{(Time.now - time)} s =========="
   end
-end
+end

data/spec/factories/group.rb

@@ -2,7 +2,7 @@ FactoryGirl.define do
   factory :group do
     title 'Test Title'
     transient do
-      owner { User.order("RANDOM()").first }
+      owner { User.order('RANDOM()').first }
     end
     # the after(:create) yields two values; the user instance itself and the
     # evaluator, which stores all values from the factory, including transient