ruby on rails—避免过多的rspec嵌套subject、let和alternative参数

Question

我试图做一些model_spec测试，但有麻烦，不必进一步嵌套我的rspec代码。如果在这种情况下，我可以有一组"它"，而不是每次我想要切换变量var时都必须添加上下文，那将是伟大的。下面是代码:

describe "#some_method" do
subject { course.some_method(var) }
context 'given a project' do
  let(:var) {random[1]}
  it 'returns the one after' do
    is_expected.to eq(random[2])
  end
  context 'being the last' do
    let(:vars) {random.last}
    it 'returns nil' do
      is_expected.to be_nil
    end
  end
  context '...you get the point, being something else' do
    let(:vars) { something.else }
    it 'returns nil' do
      is_expected.to.to be_nil
    end
  end
end
end

也许我只是陷入了错误的思维模式，有人能想出一个更好的方法吗?我的上司建议我一定要用这个话题。

起初，我不同意，认为它变得有点繁重，但后来我发现保持主题并让(:var)应用于它是非常有用的…

Answer 1

RSpecs主题是一个工具，可以用来使测试更简洁。在许多情况下，使用主语是有意义的:

RSpec.describe User do
  # with the help of shoulda-matchers
  it { should validate_uniqueness_of :username } # implicit subject
end
RSpec.describe UsersController do
  describe '#show' do
    it 'is successful' do
       get :show
       expect(response).to have_http_status :success
    end
    it 'renders template show' do
       get :show
       expect(response).to render_template :show
    end
  end
  #vs 
  describe '#show' do
    subject { response }
    before { get :show }
    it { should have_http_status :success }
    it { should render_template :success }
  end      
end

在某些情况下，使用subject会损害测试的可读性和灵敏度。

你的学校坚持要你总是用subject，这是完全错误的。

一个好的手规则是，如果你需要一个it块，那么你不应该使用主题或is_expected。

如果你在描述一个方法的调用签名，你应该在你的规范中以与你在现实生活中相同的方式调用它。

let(:decorator){ described_class.new(user) }
describe "#link" do
  it 'takes a class option' do
    expect(decorator.link(class: 'button')).to match /class="button/
  end
end

我建议使用--format documentation选项运行rspec并检查输出是否实际有意义。当你有上百个规范时，这一点非常重要，因为很难记住规范实际上涵盖了什么行为。

Answer 2

这样写怎么样?expect(subject.call(foo))不是很漂亮，但它摆脱了嵌套。

describe "#some_method" do
  subject { course.method(:some_method) }
  it 'returns the one after if given a project' do
    expect(subject.call(random[1])).to eq(random[2])
  end
  it 'returns nil when it is the last' do
    expect(subject.call(random.last)).to be_nil
  end
  it 'returns nil...' do
    expect(subject.call(something.else)).to be_nil
  end
end