has_many при соблюдении стратегии сборки в factory_girl - PullRequest
20 голосов
/ 02 октября 2009

Положение

# Models
class User < ActiveRecord::Base
  has_many :items 
end 

class Items < ActiveRecord::Base
  belongs_to :user 
  validates_presence_of :user_id 
end 

# Factories
Factory.define(:user) do |u| 
  u.name "foo" 
end 

Factory.define(:user_with_items, :parent => :user) do |u| 
  u.items {|items| [items.association(:item), items.association(:item)]} 
end

Factory.define(:item) do |i| 
  i.color "red" 
end 

Factory.define(:item_with_user, :parent => :user) do |i| 
  i.association(:user) 
end

Проблема

Если вы запустите @user = Factory(:user_with_items), тогда @user.items содержит два элемента. Проблема в том, что элементы не связаны с пользователем в базе данных. Если вы перезагрузите ассоциацию @user.items(true), вы получите пустой массив. Я знаю, что вы можете построить их вручную или создайте вспомогательные методы самостоятельно для построения графа объектов, но я бы хотел этого избежать.

Вопрос

Итак, мой вопрос: как вы можете построить отношение has_many в factory_girl, соблюдая при этом стратегию сборки?

Ответы [ 3 ]

18 голосов
/ 17 октября 2009

Я написал это правильно с наследованием и все такое. Мои коммиты объединены в здесь и здесь .

Это теперь в FactoryGirl 1.2.3, woot!

11 голосов
/ 02 октября 2009

В итоге я исправил фабричную девушку, чтобы разрешить after_build и after_create обратные вызовы.

Осуществление

Factory.class_eval do
  def run (proxy_class, overrides) #:nodoc:
    proxy = proxy_class.new(build_class)
    proxy.callbacks = @callbacks
    overrides = symbolize_keys(overrides)
    overrides.each {|attr, val| proxy.set(attr, val) }
    passed_keys = overrides.keys.collect {|k| Factory.aliases_for(k) }.flatten
    @attributes.each do |attribute|
      unless passed_keys.include?(attribute.name)
        attribute.add_to(proxy)
      end
    end
    proxy.result
  end

  def after_create(&block)
    @callbacks ||= {}
    @callbacks[:after_create] = block
  end

  def after_build(&block)
    @callbacks ||= {}
    @callbacks[:after_build] = block
  end
end

Factory::Proxy.class_eval do
  attr_accessor :callbacks

  def run_callback(name)
    callbacks && callbacks[name] && callbacks[name].call(@instance)
  end
end

Factory::Proxy::Build.class_eval do
  def result
    run_callback(:after_build)
    @instance
  end
end

Factory::Proxy::Create.class_eval do
  def result
    run_callback(:after_build)
    @instance.save!
    run_callback(:after_create)
    @instance
  end
end

Это может быть злой близнец или просто нужное вам расширение.

Пример использования

# Models
class User < ActiveRecord::Base
  has_many :items
end

class Items < ActiveRecord::Base
  belongs_to :user
  validates_presence_of :user_id
end

# Factories
Factory.define(:user) do |u|
  u.name "foo"
end

Factory.define(:user_with_items, :parent => :user) do |u|
  u.after_build do |o|
    o.items = [Factory.build(:item, :user => o), Factory.build(:item, :user => o)]
  end
end

Factory.define(:item) do |i|
  i.color "red"
end

Factory.define(:item_with_user, :parent => :user) do |i|
  i.association(:user)
end

# Run
user = Factory(:user_with_items)
user.items(true) # Shows the two saved items

Надеюсь, это поможет кому-то в будущем. Я, вероятно, попытаюсь передать это парням в thinkbot, но в их баг-трекере уже есть пара устаревших билетов на эту тему.

5 голосов
/ 02 марта 2011

Я обычно люблю разделять построение и создание, чтобы я мог строить объект, не заходя в базу данных.

Factory.define(:user_with_items, :parent => :user) do |u|
  u.after_build do |u|
    u.items = (1..2).map {Factory.build(:item, :user => u)}
  end
  u.after_create do |u|
    u.items.each {|i| i.save!}
  end
end
...