factory_girl 2.4.1 → 2.4.2

data/Changelog

@@ -1,3 +1,6 @@
+2.4.2 (January 18, 2012)
+  Fix inline traits' interaction with defaults on the factory
+
 2.4.1 (January 17, 2012)
   Deprecate build_stubbed and attributes_for
   Fix inline traits

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/gemfiles/2.1.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/gemfiles/2.3.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/gemfiles/3.0.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/gemfiles/3.1.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/gemfiles/3.2.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.4.1)
+    factory_girl (2.4.2)
       activesupport
 
 GEM

data/lib/factory_girl/definition.rb

@@ -2,12 +2,13 @@ module FactoryGirl
   class Definition
     attr_reader :callbacks, :defined_traits, :declarations
 
-    def initialize(name = nil)
-      @declarations   = DeclarationList.new(name)
-      @callbacks      = []
-      @defined_traits = []
-      @to_create      = lambda {|instance| instance.save! }
-      @traits         = []
+    def initialize(name = nil, base_traits = [])
+      @declarations      = DeclarationList.new(name)
+      @callbacks         = []
+      @defined_traits    = []
+      @to_create         = lambda {|instance| instance.save! }
+      @base_traits       = base_traits
+      @additional_traits = []
     end
 
     delegate :declare_attribute, :to => :declarations
@@ -20,17 +21,17 @@ module FactoryGirl
       attributes
     end
 
+    def processing_order
+      base_traits + [self] + additional_traits
+    end
+
     def overridable
       declarations.overridable
       self
     end
 
-    def traits
-      @traits.reverse.map { |name| trait_by_name(name) }
-    end
-
     def inherit_traits(new_traits)
-      @traits += new_traits
+      @additional_traits += new_traits
     end
 
     def add_callback(callback)
@@ -51,6 +52,14 @@ module FactoryGirl
 
     private
 
+    def base_traits
+      @base_traits.map { |name| trait_by_name(name) }
+    end
+
+    def additional_traits
+      @additional_traits.map { |name| trait_by_name(name) }
+    end
+
     def trait_by_name(name)
       trait_for(name) || FactoryGirl.trait_by_name(name)
     end

data/lib/factory_girl/factory.rb

@@ -12,14 +12,12 @@ module FactoryGirl
       @aliases          = options[:aliases] || []
       @class_name       = options[:class]
       @default_strategy = options[:default_strategy]
-      @definition       = Definition.new(@name)
+      @definition       = Definition.new(@name, options[:traits] || [])
       @compiled         = false
-
-      inherit_traits(options[:traits] || [])
     end
 
     delegate :add_callback, :declare_attribute, :to_create, :define_trait,
-             :defined_traits, :traits, :inherit_traits, :to => :@definition
+             :defined_traits, :inherit_traits, :processing_order, :to => :@definition
 
     def factory_name
       $stderr.puts "DEPRECATION WARNING: factory.factory_name is deprecated; use factory.name instead."
@@ -127,10 +125,6 @@ module FactoryGirl
 
     private
 
-    def processing_order
-      [traits.reverse, @definition].flatten
-    end
-
     def assert_valid_options(options)
       options.assert_valid_keys(:class, :parent, :default_strategy, :aliases, :traits)
 

data/lib/factory_girl/version.rb

@@ -1,4 +1,4 @@
 module FactoryGirl
-  VERSION = "2.4.1"
+  VERSION = "2.4.2"
 end
 

data/spec/acceptance/traits_spec.rb

@@ -312,3 +312,51 @@ describe "applying inline traits" do
     FactoryGirl.create(:user, :with_post).posts.should_not be_empty
   end
 end
+
+describe "inline traits overriding existing attributes" do
+  before do
+    define_model("User", :status => :string)
+
+    FactoryGirl.define do
+      factory :user do
+        status "pending"
+
+        trait(:accepted) { status "accepted" }
+        trait(:declined) { status "declined" }
+
+        factory :declined_user, :traits => [:declined]
+        factory :extended_declined_user, :traits => [:declined] do
+          status "extended_declined"
+        end
+      end
+    end
+  end
+
+  it "returns the default status" do
+    FactoryGirl.build(:user).status.should == "pending"
+  end
+
+  it "prefers inline trait attributes over default attributes" do
+    FactoryGirl.build(:user, :accepted).status.should == "accepted"
+  end
+
+  it "prefers traits on a factory over default attributes" do
+    FactoryGirl.build(:declined_user).status.should == "declined"
+  end
+
+  it "prefers inline trait attributes over traits on a factory" do
+    FactoryGirl.build(:declined_user, :accepted).status.should == "accepted"
+  end
+
+  it "prefers attributes on factories over attributes from non-inline traits" do
+    FactoryGirl.build(:extended_declined_user).status.should == "extended_declined"
+  end
+
+  it "prefers inline traits over attributes on factories" do
+    FactoryGirl.build(:extended_declined_user, :accepted).status.should == "accepted"
+  end
+
+  it "prefers overridden attributes over attributes from traits, inline traits, or attributes on factories" do
+    FactoryGirl.build(:extended_declined_user, :accepted, :status => "completely overridden").status.should == "completely overridden"
+  end
+end

data/spec/factory_girl/definition_spec.rb

@@ -64,24 +64,41 @@ describe FactoryGirl::Definition, "#to_create" do
   end
 end
 
-describe FactoryGirl::Definition, "#traits" do
-  let(:female_trait) { stub("female trait", :name => :female) }
-  let(:admin_trait)  { stub("admin trait", :name => :admin) }
+describe FactoryGirl::Definition, "#processing_order" do
+  let(:female_trait) { FactoryGirl::Trait.new(:female) }
+  let(:admin_trait)  { FactoryGirl::Trait.new(:admin) }
 
   before do
     subject.define_trait(female_trait)
     FactoryGirl.stubs(:trait_by_name => admin_trait)
   end
 
-  its(:traits) { should be_empty }
+  context "without base traits" do
+    it "returns the definition without any traits" do
+      subject.processing_order.should == [subject]
+    end
 
-  it "finds the correct traits after inheriting" do
-    subject.inherit_traits([:female])
-    subject.traits.should == [female_trait]
+    it "finds the correct traits after inheriting" do
+      subject.inherit_traits([:female])
+      subject.processing_order.should == [subject, female_trait]
+    end
+
+    it "looks for the trait on FactoryGirl" do
+      subject.inherit_traits([:female, :admin])
+      subject.processing_order.should == [subject, female_trait, admin_trait]
+    end
   end
 
-  it "looks for the trait on FactoryGirl" do
-    subject.inherit_traits([:female, :admin])
-    subject.traits.should == [admin_trait, female_trait]
+  context "with base traits" do
+    subject { FactoryGirl::Definition.new("my definition", [:female]) }
+
+    it "returns the base traits and definition" do
+      subject.processing_order.should == [female_trait, subject]
+    end
+
+    it "finds the correct traits after inheriting" do
+      subject.inherit_traits([:admin])
+      subject.processing_order.should == [female_trait, subject, admin_trait]
+    end
   end
 end

data/spec/factory_girl/factory_spec.rb

@@ -305,11 +305,11 @@ describe FactoryGirl::Factory, "#with_traits" do
   end
 
   it "returns a factory with the correct traits" do
-    subject.with_traits([:admin, :female]).traits.should =~ [admin_trait, female_trait]
+    subject.with_traits([:admin, :female]).processing_order[1, 2].should == [admin_trait, female_trait]
   end
 
-  it "doesn't modify the original factory's traits" do
+  it "doesn't modify the original factory's processing order" do
     subject.with_traits([:admin, :female])
-    subject.traits.should be_empty
+    subject.processing_order.should == [subject.definition]
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: factory_girl
 version: !ruby/object:Gem::Version
-  version: 2.4.1
+  version: 2.4.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-17 00:00:00.000000000Z
+date: 2012-01-18 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70308695491200 !ruby/object:Gem::Requirement
+  requirement: &70205633314300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70308695491200
+  version_requirements: *70205633314300
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70308695490500 !ruby/object:Gem::Requirement
+  requirement: &70205633310180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '2.0'
   type: :development
   prerelease: false
-  version_requirements: *70308695490500
+  version_requirements: *70205633310180
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &70308695489400 !ruby/object:Gem::Requirement
+  requirement: &70205633309440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70308695489400
+  version_requirements: *70205633309440
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &70308695489000 !ruby/object:Gem::Requirement
+  requirement: &70205633308320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695489000
+  version_requirements: *70205633308320
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70308695488540 !ruby/object:Gem::Requirement
+  requirement: &70205633307580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695488540
+  version_requirements: *70205633307580
 - !ruby/object:Gem::Dependency
   name: aruba
-  requirement: &70308695488000 !ruby/object:Gem::Requirement
+  requirement: &70205633306600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695488000
+  version_requirements: *70205633306600
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &70308695487580 !ruby/object:Gem::Requirement
+  requirement: &70205633305800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695487580
+  version_requirements: *70205633305800
 - !ruby/object:Gem::Dependency
   name: bourne
-  requirement: &70308695481180 !ruby/object:Gem::Requirement
+  requirement: &70205633305240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695481180
+  version_requirements: *70205633305240
 - !ruby/object:Gem::Dependency
   name: appraisal
-  requirement: &70308695480520 !ruby/object:Gem::Requirement
+  requirement: &70205633304740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -109,10 +109,10 @@ dependencies:
         version: 0.3.8
   type: :development
   prerelease: false
-  version_requirements: *70308695480520
+  version_requirements: *70205633304740
 - !ruby/object:Gem::Dependency
   name: sqlite3-ruby
-  requirement: &70308695479860 !ruby/object:Gem::Requirement
+  requirement: &70205633304300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695479860
+  version_requirements: *70205633304300
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70308695479120 !ruby/object:Gem::Requirement
+  requirement: &70205633303840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695479120
+  version_requirements: *70205633303840
 - !ruby/object:Gem::Dependency
   name: bluecloth
-  requirement: &70308695478620 !ruby/object:Gem::Requirement
+  requirement: &70205633303340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,7 +142,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70308695478620
+  version_requirements: *70205633303340
 description: ! "factory_girl provides a framework and DSL for defining and\n                       using
   factories - less error-prone, more explicit, and\n                       all-around
   easier to work with than fixtures."