From 6bd2e5e335258edb45a755f6031733e66b354728 Mon Sep 17 00:00:00 2001
From: Rishabh Saxena <orishabh1@gmail.com>
Date: Wed, 10 Aug 2016 23:19:36 +0530
Subject: [PATCH] refactor tests

---
 spec/features/ticket_purchases_spec.rb | 11 -----------
 spec/models/payment_spec.rb            |  6 ------
 2 files changed, 17 deletions(-)

diff --git a/spec/features/ticket_purchases_spec.rb b/spec/features/ticket_purchases_spec.rb
index 1d67c285..9ac82a63 100644
--- a/spec/features/ticket_purchases_spec.rb
+++ b/spec/features/ticket_purchases_spec.rb
@@ -1,5 +1,4 @@
 require 'spec_helper'
-require 'stripe_mock'
 
 feature Registration do
   let!(:ticket) { create(:ticket) }
@@ -9,16 +8,12 @@ feature Registration do
   context 'as a participant' do
     before(:each) do
       sign_in participant
-      StripeMock.start
     end
 
     after(:each) do
-      StripeMock.stop
       sign_out
     end
 
-    let(:stripe_helper) { StripeMock.create_test_helper }
-
     context 'who is not registered' do
 
       scenario 'purchases and pays for a ticket succcessfully', feature: true, js: true do
@@ -37,12 +32,6 @@ feature Registration do
         expect(flash).to eq('Please pay here to purchase tickets.')
         purchase = TicketPurchase.where(user_id: participant.id, ticket_id: ticket.id).first
         expect(purchase.quantity).to eq(2)
-
-        # token = stripe_helper.generate_card_token
-        # merge token, email and submit form
-        # page.execute_script("$('form').submit()")
-
-        expect(current_path).to eq(conference_conference_registration_path(conference.short_title))
       end
     end
   end
diff --git a/spec/models/payment_spec.rb b/spec/models/payment_spec.rb
index 71277146..6af28679 100644
--- a/spec/models/payment_spec.rb
+++ b/spec/models/payment_spec.rb
@@ -90,7 +90,6 @@ describe Payment do
       context 'when the connection to Stripe drops' do
         it 'raises exception' do
           StripeMock.prepare_error(Stripe::APIConnectionError.new)
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::APIConnectionError)
           expect{ payment.purchase }.not_to raise_error
         end
       end
@@ -98,7 +97,6 @@ describe Payment do
       context 'when there is a Stripe API Error' do
         it 'raises exception' do
           StripeMock.prepare_error(Stripe::APIError.new)
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::APIError)
           expect{ payment.purchase }.not_to raise_error
         end
       end
@@ -106,7 +104,6 @@ describe Payment do
       context 'when there is authentication error' do
         it 'raises exception' do
           StripeMock.prepare_error(Stripe::AuthenticationError.new)
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::AuthenticationError)
           expect{ payment.purchase }.not_to raise_error
         end
       end
@@ -114,7 +111,6 @@ describe Payment do
       context 'when there is a card error' do
         it 'raises exception' do
           StripeMock.prepare_card_error(:card_declined)
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::CardError)
           expect{ payment.purchase }.not_to raise_error
         end
       end
@@ -122,7 +118,6 @@ describe Payment do
       context 'when the request to Stripe is invalid' do
         it 'raises exception' do
           StripeMock.prepare_error(Stripe::InvalidRequestError.new('Your request is invalid.', code: 402))
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::InvalidRequestError)
           expect{ payment.purchase }.not_to raise_error
         end
       end
@@ -130,7 +125,6 @@ describe Payment do
       context 'when Stripe rate limit exceeds' do
         it 'raises exception' do
           StripeMock.prepare_error(Stripe::RateLimitError.new)
-          expect{ Stripe::Charge.create }.to raise_error(Stripe::RateLimitError)
           expect{ payment.purchase }.not_to raise_error
         end
       end