diff --git a/app/controllers/discourse_donations/charges_controller.rb b/app/controllers/discourse_donations/charges_controller.rb index 3741678..0a1f2d6 100644 --- a/app/controllers/discourse_donations/charges_controller.rb +++ b/app/controllers/discourse_donations/charges_controller.rb @@ -14,6 +14,7 @@ module DiscourseDonations else payment = DiscourseDonations::Stripe.new(secret_key, stripe_options) response = payment.charge(email, params) + response['message'] = response['outcome']['seller_message'] end response['rewards'] = [] diff --git a/app/services/discourse_donations/stripe.rb b/app/services/discourse_donations/stripe.rb index 0a1b4f1..f11a092 100644 --- a/app/services/discourse_donations/stripe.rb +++ b/app/services/discourse_donations/stripe.rb @@ -20,7 +20,6 @@ module DiscourseDonations description: description, currency: currency ) - @charge[:message] = @charge[:outcome][:seller_message] if @charge[:outcome] @charge end diff --git a/spec/controllers/discourse_donations/charges_controller_spec.rb b/spec/controllers/discourse_donations/charges_controller_spec.rb index 8df8f62..febc398 100644 --- a/spec/controllers/discourse_donations/charges_controller_spec.rb +++ b/spec/controllers/discourse_donations/charges_controller_spec.rb @@ -79,7 +79,7 @@ module DiscourseDonations end it 'has no rewards' do - stripe.expects(:create).returns({ outcome: { seller_message: 'bummer' } }) + stripe.expects(:create).returns({ 'outcome' => { 'seller_message' => 'bummer' } }) post :create expect(response_rewards).to be_empty end diff --git a/spec/services/discourse_donations/stripe_spec.rb b/spec/services/discourse_donations/stripe_spec.rb index cc7f30c..d7f6746 100644 --- a/spec/services/discourse_donations/stripe_spec.rb +++ b/spec/services/discourse_donations/stripe_spec.rb @@ -34,13 +34,6 @@ module DiscourseDonations subject.charge(email, params) end - it 'has a message' do - ::Stripe::Customer.expects(:create).returns(customer) - ::Stripe::Charge.expects(:create).returns({ outcome: { seller_message: 'yay!' } }) - response = subject.charge(email, params) - expect(response[:message]).to eq 'yay!' - end - describe '.successful?' do let(:charge_options) { { customer: customer.id, amount: params[:amount], description: stripe_options[:description], currency: stripe_options[:currency] } }