diff --git a/app/models/ticket.rb b/app/models/ticket.rb index 54f5c3d7..3dd2179f 100644 --- a/app/models/ticket.rb +++ b/app/models/ticket.rb @@ -9,7 +9,7 @@ class Ticket < ActiveRecord::Base # If we would allow different currencies per conference we also have to handle convertions between currencies! validate :tickets_of_conference_have_same_currency - validates :price_cents, :price_currency, :title, presence: true + validates :price_cents, :price_currency, :payment_mode, :title, presence: true validates_numericality_of :price_cents, greater_than: 0 diff --git a/db/migrate/20160526142456_add_payment_mode_to_tickets.rb b/db/migrate/20160526142456_add_payment_mode_to_tickets.rb new file mode 100644 index 00000000..df454d25 --- /dev/null +++ b/db/migrate/20160526142456_add_payment_mode_to_tickets.rb @@ -0,0 +1,5 @@ +class AddPaymentModeToTickets < ActiveRecord::Migration + def change + add_column :tickets, :payment_mode, :string + end +end diff --git a/db/schema.rb b/db/schema.rb index 00e93609..f0d1e01d 100644 --- a/db/schema.rb +++ b/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: 20160427104236) do +ActiveRecord::Schema.define(version: 20160526142456) do create_table "ahoy_events", force: :cascade do |t| t.uuid "visit_id", limit: 16 @@ -411,6 +411,7 @@ ActiveRecord::Schema.define(version: 20160427104236) do t.text "description" t.integer "price_cents", default: 0, null: false t.string "price_currency", default: "USD", null: false + t.string "payment_mode" end create_table "tracks", force: :cascade do |t| diff --git a/spec/factories/tickets.rb b/spec/factories/tickets.rb index 88532b86..206c8c44 100644 --- a/spec/factories/tickets.rb +++ b/spec/factories/tickets.rb @@ -3,5 +3,6 @@ FactoryGirl.define do title { "#{Faker::Hipster.word} Ticket" } price_cents 1000 price_currency 'USD' + payment_mode 'Online' end end diff --git a/spec/models/ticket_spec.rb b/spec/models/ticket_spec.rb index fc67ff3d..48145c2c 100644 --- a/spec/models/ticket_spec.rb +++ b/spec/models/ticket_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' describe Ticket do let(:conference) { create(:conference) } - let(:ticket) { create(:ticket, price: 50, price_currency: 'USD', conference: conference) } + let(:ticket) { create(:ticket, price: 50, price_currency: 'USD', payment_mode: 'Online', conference: conference) } let(:user) { create(:user) } describe 'validation' do @@ -22,6 +22,10 @@ describe Ticket do should validate_presence_of(:price_currency) end + it 'is not valid without a payment_mode' do + should validate_presence_of(:payment_mode) + end + it 'is not valid with a price_cents equals zero' do should_not allow_value(0).for(:price_cents) end