尝试通过向 rspec 传递特定数字来测试我的随机生成器

Question

我目前正在尝试为以下代码创建测试。我先编写了测试，然后编写了代码，然后意识到我的测试没有做我想要它做的事情。

class Weather

  def initialize
    @weather = 'sunny'
  end

  def weather_randomiser
    case rand(1..10)
    when 1..8 then @weather = 'sunny'
    when 9..10 then @weather = 'stormy'
    end
  end

  def stormy?
    weather_randomiser
    @weather == 'stormy'
  end
end

我的测试如下：

require 'weather.rb'

describe Weather do
  it 'randomises weather and sets the weather to the randomised result' do
    weather = Weather.new
    allow(weather).to receive(:weather_randomiser).and_return(9)
    expect(weather).to be_stormy
    allow(weather).to receive(:weather_randomiser).and_return(2)
    expect(weather).to_not be_stormy
  end
end

您能解释一下我如何测试weather_randomiser和stormy吗？与此类似的方法/我的代码是否可行？

您能否进一步解释一下您是如何到达您所做的事情的，因为我想了解而不仅仅是一个答案。

预先非常感谢您:)

Answer 1

你的weather_randomiser做了太多的事情：

1. 它生成一个随机数
2. 它将一个随机数映射到一个字符串值
3. 它更新一个实例变量

为了使事情更具可测试性，你可以将它们分开，例如：

def weather_randomiser
  rand(1..10)
end

def set_weather(value)
  @weather = case value
    when 1..8 then 'sunny'
    when 9..10 then 'stormy'
  end
end

def stormy?
  set_weather weather_randomiser
  @weather == 'stormy'
end

它确实引入了随机生成器和设置器之间的尴尬耦合；改变前者需要考虑后者...... ymmv

Answer 2

最简单的解决方案是这样的。不需要使用 any_instance_of，但也不建议在 subject 上存根任何内容。

describe Weather do
  subject(:weather) { described_class.new }

  before(:each) do
    allow(weather).to receive(:rand).with(1..10).and_return(random_value)
  end

  context 'when random value is in 1..8' do
    let(:random_value) { 8 }

    it { is_expected.not_to be_stormy }
  end

  context 'when random value is in 9..10' do
    let(:random_value) { 10 }

    it { is_expected.to be_stormy }
  end
end

另一种不暴露私有实现细节的编写方法如下：

describe Weather do
  subject(:weather) { described_class.new }

  let(:stormy_weathers) do
    (1..10).map do |val|
      allow(weather).to receive(:rand).and_return(val)
      weather.stormy?
    end
  end

  it 'is stormy 20% of the time and sunny 80% of the time' do
    expect(stormy_weathers)
      .to match_array(Array.new(2, true) + Array.new(8, false))
  end
end

即使您将内部实现更改为分别使用 1..2 和 3..10 作为晴天和暴风雨范围，此版本也会通过。只要比例仍然是80/20。

Answer 3

我相信您可以使用以下代码片段。在这里，您想要对 rand 函数本身进行存根。我真的不建议这样做。如果您的类可能被称为“Randomizer”并且它包含“random”类，我建议您这样做。像 Object 这样的存根类根本就不好。

#!/usr/bin/env ruby
require_relative '../weather.rb'

RSpec.describe Weather do
  it 'randomises weather return stormy' do
    allow_any_instance_of(Object).to receive(:rand).and_return(9)
    weather = Weather.new
    expect(weather).to be_stormy
  end

  it 'randomises weather return sunny' do
    allow_any_instance_of(Object).to receive(:rand).and_return(4)
    weather = Weather.new
    expect(weather).to_not be_stormy
  end
end