Rails 应用程序 Ruby 中的 Rspec 应该测试什么测试

Question

我是 Rspec 的初学者，实际上我们要求我对一些已经构建但从未测试过的方法进行 Rspec 测试（他们之前没有编写测试构建方法）。

现在我很想知道如何测试我的方法，下面是示例：

 class ConnectorJob < ActiveJob::Base

  queue_as :connector

  def perform(tenant_id = nil, debug = false)
    tenant_to_sync = Tenant.to_sync(tenant_id)
    return if tenant_to_sync.empty?
    tenant_to_sync.each do |tenant|
      service = MyAPP::ContactToSync.new(tenant, debug).call
      if service.success?
        ConnectorService::Synchronization.new(
          tenant, service.data, debug
        ).call
      end
    end
  end
end

我应该测试什么？我应该测试 return 值是否正确，或者其他方法是否正确调用？

这是我尝试做的

    require "rails_helper"

RSpec.describe ConnectorJob, type: :job do
  it 'is in connector queue' do
    expect(ConnectorJob.new.queue_name).to eq('connector')
  end

  describe 'perform' do
    let (:tenant) { create(:tenant) }
    let (:job) { ConnectorJob.new.perform(tenant.id) }

  context 'with empty tenant' do
      it { expect(ConnectorJob.new.perform(@tenant.id)).to eq nil }
    end

    context 'with tenant' do
      it { expect(ConnectorJob.new.perform(tenant.id)).to eq job }
    end
  end
end

如您所见，我的最后一个测试没有任何意义，但我不知道我应该在 Rspec 上写些什么来预测此方法的结果。

如果我检查我的 Rspec 覆盖率，Rspec 告诉我我覆盖了 100% 的方法，但我不确定这是正确的。

我希望我说清楚了，请随时问我更多细节。

谢谢大家

Answer 1

我认为你应该测试最终结果，我的意思是调用后的结果 ConnectorService::Synchronization.new(...).call 并测试三种情况，例如如果这个调用创建新用户，你应该测试它：

如果tenant_to_sync.empty? == true

 context 'with empty tenant' do
   it { expect(ConnectorJob.new.perform(@tenant.id)).to change(User.count).by(0) }
 end

如果service.success? == false

context 'MyAPP::ContactToSync return false' do
   it { expect(ConnectorJob.new.perform(@tenant.id)).to change(User.count).by(0) }
 end

如果service.success? == true

context 'success' do
   it { expect(ConnectorJob.new.perform(@tenant.id)).to change(User.count).by(1) }
 end

应该足够覆盖所有场景了

Rails 应用程序 Ruby 中的 Rspec 应该测试什么测试

What a test should test with Rspec in Ruby on Rails application

ruby-on-rails

ruby

testing

rspec

tdd