Question

我有以下型号： subscription，user和events

user has_one subscription
subscription belongs_to一个user
user has_many events
event belongs_to一个user

到目前为止，我已经能够使用Capybara和RSpec成功创建验收测试。这让我可以升级＆＃39;用户帐户（添加不同的角色）。我还能够进行验收测试，用户取消订阅并确保删除其角色。

但是，现在我想确保取消任何用户的公开活动。这就是我被卡住的地方。实际上，我甚至没有做到这一点，因为我试图甚至破坏订阅时遇到了麻烦。

因此，我创建了一个名为subscriptions_controller_spec.rb的控制器规范。在此规范中，有一项测试可确保destroy操作按预期工作。这是失败的，因为在我的控制器中它会检索不存在的客户和订阅，并返回Stripe::InvalidRequestError。

为了解决这个问题，我尝试使用stripe-ruby-mock来模拟条带服务器。但是，我不确定我是如何在控制器规范中使用它的，我真的很困惑。下面是我的控制器和我的控制器规格。关于我应该如何攻击这一点的任何建议都会非常感激。

subscriptions_controller_spec.rb

require 'rails_helper'

RSpec.describe SubscriptionsController, :type => :controller do

  let(:stripe_helper) { StripeMock.create_test_helper }
  before { StripeMock.start }
  after { StripeMock.stop }

  # ... omitted 

  describe 'DELETE destroy' do
    before :each do
      sign_in_trainer
      @subscription = create(:subscription, user: subject.current_user)
      plan = stripe_helper.create_plan(:id => 'Standard')
      customer = Stripe::Customer.create({
                                            email: 'johnny@appleseed.com',
                                            source: stripe_helper.generate_card_token,
                                            plan: 'Standard'
                                        })
      @subscription.customer_id = customer.id
      @subscription.stripe_sub_id = customer.subscriptions.data.first.id
    end

    it 'destroys the requested subscription' do
      expect {
        delete :destroy, {:id => @subscription.to_param}
      }.to change(Subscription, :count).by(-1)
    end

    # ... omitted

  end
end

subscriptions_controller.rb

class SubscriptionsController < ApplicationController
  before_action :set_subscription, only: [:update, :destroy]

  # ... ommitted

  # DELETE /cancel-subscriptions/1
  def destroy
    begin
      customer = Stripe::Customer.retrieve(@subscription.customer_id)
      customer.subscriptions.retrieve(@subscription.stripe_sub_id).delete
    rescue Stripe::CardError => e
      # User's card was declined for many magnitude of reasons
      redirect_to user_dashboard_path, alert: 'There was a problem cancelling your subscription' and return
    rescue Stripe::APIConnectionError => e
      # Stripe network issues
      redirect_to user_dashboard_path, alert: 'Network issue. Please try again later' and return
    rescue Stripe::APIError => e
      # Stripe network issues
      redirect_to user_dashboard_path, alert: 'Network issue. Please try again later' and return
    rescue Stripe::InvalidRequestError => e
      # This is something that we screwed up in our programming. This should literally never happen.
      redirect_to user_dashboard_path, alert: 'There was a problem cancelling your subscription.' and return
    rescue => e
      logger.error e.message
      logger.error e.backtrace.join("\n")
      redirect_to user_dashboard_path, alert: 'There was a problem cancelling your subscription.' and return
    end

    if current_user.events
      @events = current_user.events
      @events.open.each do |event|
        event.cancel
      end
    end

    current_user.remove_role 'trainer'
    current_user.add_role 'user'
    current_user.save
    @subscription.destroy
    respond_to do |format|
      format.html { redirect_to user_dashboard_path, notice: 'Subscription cancelled. All your open events have been cancelled.' }
      format.json { head :no_content }
    end
  end

  private
  # Use callbacks to share common setup or constraints between actions.
  def set_subscription
    @subscription = Subscription.find(params[:id])
  end

  # Never trust parameters from the scary internet, only allow the white list through.
  def subscription_params
    params[:subscription]
  end
end

Answer 1

我认为你已经在这方面达成了一席之地，在控制器规范中难以测试的事实表明，这可能是考虑将行为转移到服务类的好时机。

我要做的是设置集成测试以用作反馈循环，然后重构并恢复绿色。完成后，开始重构您的服务类，并从那里构建您的规范。

Answer 2

简单地模仿Stripe不起作用，例如：

require 'rails_helper'

RSpec.describe SubscriptionsController, :type => :controller do

  # ... omitted 

  describe 'DELETE destroy' do
    before :each do
      sign_in_trainer
      @subscription = create(:subscription, user: subject.current_user)
    end

    it 'destroys the requested subscription' do
      # just mock stripe to pass back the customer you expect - as though it Just Works
      expect(Stripe::Customer).to receive(:retreive).and_return(subscription.customer)

      expect {
        delete :destroy, {:id => @subscription.to_param}
      }.to change(Subscription, :count).by(-1)
    end


    it 'does not destroy it if we got a card error' do
      # likewise you can mock up what happens when an error is raised
      expect(Stripe::Customer).to receive(:retreive).and_raise(Stripe::CardError)

      expect {
        delete :destroy, {:id => @subscription.to_param}
      }.not_to change(Subscription, :count)
    end
    # ... omitted

  end
end

如何有效地测试条带和控制器？

2 个答案: