如何在 Factory Girl 或 Minifacture 中使用随机唯一数据制作 Ruby 测试工厂？

Question

我正在用一个典型的工厂测试一个典型的 Rails 模型：

# My model uses a 3-letter uppercase airport code,
# such as "ATL" for Atlanta, "BOS" for Boston, etc.

class Airport < ActiveRecord::Base
  validates :code, uniqueness: true

Factory.define :airport do |f|
  f.code { random_airport_code }  # Get a 3-letter uppercase code

我正在添加更多测试并开始在机场代码中看到冲突：例如，工厂使用代码“XYZ”创建了一个机场，然后对工厂的后续调用尝试使用相同的代码创建一个机场。

序列是解决此问题的一种方法。例如，使用 Factory Girl 序列、有序列表或预先计算的枚举，一些类似的方式来维护下一个可用代码的状态。

我的问题是：解决这个问题的非序列方法是什么？我想使用随机数据，并且不是一个序列。

我正在尝试一些想法，因为它们很实用 - 非常感谢您对这些想法的任何见解。

使用乐观锁的示例思路

while 
  airport = Factory.build :airport
  airport.save && return airport
end

优点：在实践中速度很快，因为很少发生碰撞；地方州。

缺点：语法笨拙；非本地工厂；保存可能由于碰撞以外的原因而失败。

使用事务的示例想法

Airport.transaction 
  while
    x = random_airport_code
    if Airport.exists?(code: x)
      next
    else
      Factory :airport, code: x
      break
    end
  end
end

优点：这是最接近我想要的；当地国家；确保没有碰撞。

缺点：冗长笨拙的语法。

赏金

Factory Girl 或 Minifacture 是否有更适合随机数据且不是序列的语法？

或者，如果有保存冲突，也许某种模式可以自动重新掷骰子？

一些开销对我来说很好。实际上，在具有数千个测试的持续集成设置中，每天大约会发生一次冲突。如果测试套件必须重新掷骰子几次，或者探测数据库中的现有值等，那很好。

cmets 询问为什么随机数据而不是序列。我更喜欢随机数据，因为我的经验是随机数据可以带来更好的测试、更好的长期可维护性以及更好的测试目标语义。另外，我使用 Faker 和 Forgery 而不是固定装置，以防万一了解。

要获得赏金，答案必须是即时随机的，而不是顺序。 （例如，我正在寻找的解决方案可能使用#sample 和/或无序集，并且可能不使用#shuffle 和/或有序集）

Answer 1

您可以使用回调。比如：

factory :airport do
  after(:build) do |airport|
    airport.code = loop do
      code = ('AAA'..'ZZZ').to_a.sample
      break code unless Airport.exists?(code: code)
    end
  end
end

您可能想将after(:build) 更改为before(:create)，这取决于您要如何使用工厂。

Answer 2

这应该可以，但它只允许创建 17576 个模型

CODES = ("AAA".."ZZZ").to_a.shuffle
Factory.define :airport do |f|
  f.code { CODES.pop }
end

Answer 3

是的，FactoryGirl 有一个功能可以让你做到这一点。请参阅序列文档的结尾：https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md#sequences 您可以将序列设置为任何知道如何在调用 #next 时返回自身递增版本的对象。因此，您可以编写一个知道如何返回唯一随机数据并实现#next 的类，例如

class AirportCode
  ALL = %w(AAA BBB CCC).shuffle

  attr_reader :index

  def initialize(index = rand(ALL.length))
    @index = index
  end

  def value
    ALL[@index]
  end

  def to_s
    value
  end

  # might need to explicitly delegate more methods to the value

  def method_missing(method, *args)
    value.send method, *args
  end

  def next
    AirportCode.new((index + 1) % ALL.length)
  end

end

（这个只有三个唯一值，但只是为了说明这一点），创建一个 FactoryGirl 序列并将其值设置为该类的一个实例。我没有尝试过 FactoryGirl 部分，所以请报告它是否有效 :)

Answer 4

与@GergoErdosi 的回答类似，我能够做到这一点：

CODES = ("AAA".."ZZZ").to_a.shuffle

factory :airport do after(:build) do |airport| if Airport.exists?(code: airport.code) new_code = ('AAA'..'ZZZ').to_a.sample airport.code = new_code end end code { CODES.rotate!.first } ... #other stuff for building Airports end