Factory Girl Fehler mit has_many Beziehung

Question

Ich habe folgende Fabriken:

Factory.define :email do |email| 
    email.email {"infomcburney.cowan.com"} 
end 

Factory.define :lead do |lead| 
    lead.emails {|emails| [emails.association(:email)]} 
end 

, die die folgenden Klassen modelliert werden

class Lead < ActiveRecord::Base 
    has_many :emails 
end 

class Email < ActiveRecord::Base 
    belongs_to :lead, :class_name => "Lead", :foreign_key => "lead_id" 
end 

Wenn ich die diesen Test durch shoulda laufen:

should "capture emails" do 
     lead = Factory.build(:lead) 
     assert_equal(1, lead.emails.size) 
    end 

Ich erhalte den folgenden Fehler:

Factory::AttributeDefinitionError: Attribute already defined: emails

Ich bin völlig darauf fest, kann mir jemand in die richtige Richtung zeigen. Ich benutze factory_girl 1.3.2.

Answer 1

Ich würde empfehlen, has_many Beziehungsdaten zu Ihren Fabriken hinzuzufügen. Der Grund dafür ist, dass Ihre Lead-Factory jetzt davon abhängt, diese Verbindung zu füllen, und sie fügt mehr Kopplung hinzu und möglicherweise einige Verwirrung auf der Straße, wenn sich die Verbindung ändert.

Wenn Sie diese Beziehung testen möchten (und ich empfehle es Ihnen), gibt es ein großes Juwel namens Shoulda, das Unit Test-Makros hinzufügt, um sicherzustellen, dass die Beziehungen richtig eingerichtet sind. Ich habe es nicht mit dem eingebauten in Rails-Test :: Unit verwendet, aber ein RSpec Beispiel würde in etwa so aussehen:

describe Lead do 
    it { should have_many(:emails) } 
end 

Wenn Sie wirklich, diese Beziehung testen wollen, sollten Sie es in der Spezifikation tun. Entfernen Sie die E-Mails Vereinigung von Ihrem Lead Factory und erstellen Sie ein Lead-Objekt und versuchen Sie es Objekte ein paar E-Mail wie so weitergeben müssen:

lead = Factory.build(:lead) 
2.times do { lead.emails << Factory.build(:email, :lead => lead) } 

Dann sollte es ein paar E-Mails zusammen mit ihm haben. Sie sollten jedoch Vertrauen in ActiveRecord setzen und nur Dinge testen, die über das hinausgehen, was Rails bereits für Sie tut. Dies ist, wo Shoulda kommt.

Ein weiterer Kommentar, den ich habe, ist auf Ihre E-Mail-Beziehungen gehört. Da Sie nur die Standardkonventionen verwenden, weiß rails, was zu tun ist.

class Email < ActiveRecord::Base 
    belongs_to :lead 
end 

Answer 2

Dies ist ein interessanter Artikel, die hilfreich sein könnten:

http://icelab.com.au/articles/factorygirl-and-has-many-associations/

FactoryGirl.define do 
     factory :venue_with_gigs, :parent => :venue do 
     after_create do |venue| 
      FactoryGirl.create(:gig, :venue => venue) 
     end 
     end 
    end