Проблема с Factory_girl, ассоциацией и after_initialize - PullRequest
21 голосов
/ 06 мая 2011

У меня определен семейный класс:

class Family < ActiveRecord::Base
  after_initialize :initialize_family
  belongs_to :user
  validates :user, 
       :presence => true

  validates :name,   
       :presence => true,          
       :length => { :maximum => 30 },
       :format => { :with => /\A[a-zA-Z0-9\-_\s\']+\z/i}

  def initialize_family
    if self.name.blank? && self.user
        self.name = "#{self.user.profile_full_name}'s Family"
    end
  end
end

На моих фабриках.rb у меня есть:

Factory.define :family do |f|
   f.association :user, :factory => :user
end

В моем family_spec.rb у меня есть

let(:family) { Factory(:family) }

Но это не с:

1) Family is valid with valid attributes
     Failure/Error: let(:family) { Factory(:family) }
     ActiveRecord::RecordInvalid:
       Validation failed: Name can't be blank, Name is invalid, Languages can't be blank, Languages is too short (minimum is 1 characters)
     # ./spec/models/family_spec.rb:8:in `block (2 levels) in <top (required)>'
     # ./spec/models/family_spec.rb:10:in `block (2 levels) in <top (required)>'

Используя отладчик, я вижу, что когда after_initialize вызывается, self.user равен nil. Почему это происходит? Если я называю семью с create или new, все работает нормально.

Спасибо за любую помощь.

Ответы [ 3 ]

24 голосов
/ 09 мая 2011

Это ответ, который я получил от Джо Ферриса:

factory_girl не передает аргументы конструктору.Он использует # user = в вашей модели и создает его без каких-либо аргументов.

и этот от Бена Хьюза:

Чтобы пояснить, что говорит Джо, методы after_initialize вызываются непосредственно для объектаинициализация, и это время действительно пользователь не был установлен.

Так, например, пока это будет работать:

family = Family.create!(:user => @user) # or @user.families.create ...

Это не будет (что делает factory_girl под капотом):

family = Family.new
family.user = @user
family.save!

Как правило, вы хотите быть очень осторожными, используя after_initialize, поскольку помните, что он вызывается для каждой инициализации объекта .Вызов Family.all для 1000 объектов вызовет его вызов 1000 раз.

Похоже, в этом случае вам лучше использовать before_validation вместо after_initialize.

Следующий синтаксис также работает для тестирования в rspec:

let (:family) { Family.create(:user => @user) }
16 голосов
/ 21 января 2015

Поскольку after_initialize запускается после создания новых объектов, и factory_girl создает экземпляры, вызывая new без каких-либо аргументов по умолчанию, вы должны использовать initialize_with для перезаписи сборки по умолчанию.

FactoryGirl.define do
  factory :family do
    initialize_with { new(user: build(:user)) }
  end
end
0 голосов
/ 09 мая 2011

Я считаю, что это потому, что ассоциация ленива, поэтому в "after_initialize" еще нет пользователя.

http://rdoc.info/github/thoughtbot/factory_girl/v1.3.3/file/README.rdoc

Возможно, вы можете напрямую вызывать одну фабрику из другой, но я не пробовал, например,

f.user Factory(:user)
...