factory_girl 2.6.0 → 2.6.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    factory_girl (2.6.0)
+    factory_girl (2.6.1)
       activesupport (>= 2.3.9)
 
 GEM

data/gemfiles/2.3.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.6.0)
+    factory_girl (2.6.1)
       activesupport (>= 2.3.9)
 
 GEM

data/gemfiles/3.0.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.6.0)
+    factory_girl (2.6.1)
       activesupport (>= 2.3.9)
 
 GEM

data/gemfiles/3.1.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.6.0)
+    factory_girl (2.6.1)
       activesupport (>= 2.3.9)
 
 GEM

data/gemfiles/3.2.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/joshuaclayton/dev/gems/factory_girl
   specs:
-    factory_girl (2.6.0)
+    factory_girl (2.6.1)
       activesupport (>= 2.3.9)
 
 GEM

data/lib/factory_girl.rb

@@ -1,7 +1,8 @@
 require "active_support/core_ext/module/delegation"
 
 require 'factory_girl/errors'
-require 'factory_girl/association_runner'
+require 'factory_girl/factory_runner'
+require 'factory_girl/strategy_calculator'
 require 'factory_girl/strategy'
 require 'factory_girl/registry'
 require 'factory_girl/null_factory'
@@ -66,4 +67,8 @@ module FactoryGirl
   def self.trait_by_name(name)
     traits.find(name)
   end
+
+  def self.callback_names
+    [:after_build, :after_create, :after_stub].freeze
+  end
 end

data/lib/factory_girl/callback.rb

@@ -1,7 +1,5 @@
 module FactoryGirl
   class Callback
-    VALID_NAMES = [:after_build, :after_create, :after_stub].freeze
-
     attr_reader :name, :block
 
     def initialize(name, block)
@@ -26,9 +24,9 @@ module FactoryGirl
     private
 
     def check_name
-      unless VALID_NAMES.include?(name)
+      unless FactoryGirl.callback_names.include?(name)
         raise InvalidCallbackNameError, "#{name} is not a valid callback name. " +
-          "Valid callback names are #{VALID_NAMES.inspect}"
+          "Valid callback names are #{FactoryGirl.callback_names.inspect}"
       end
     end
   end

data/lib/factory_girl/definition_proxy.rb

@@ -85,6 +85,8 @@ module FactoryGirl
         @definition.declare_attribute(Declaration::Implicit.new(name, @definition, @ignore))
       elsif args.first.is_a?(Hash) && args.first.has_key?(:factory)
         association(name, *args)
+      elsif FactoryGirl.callback_names.include?(name)
+        @definition.add_callback(Callback.new(name, block))
       else
         add_attribute(name, *args, &block)
       end
@@ -138,18 +140,6 @@ module FactoryGirl
       @definition.declare_attribute(Declaration::Association.new(name, options))
     end
 
-    def after_build(&block)
-      @definition.add_callback(Callback.new(:after_build, block))
-    end
-
-    def after_create(&block)
-      @definition.add_callback(Callback.new(:after_create, block))
-    end
-
-    def after_stub(&block)
-      @definition.add_callback(Callback.new(:after_stub, block))
-    end
-
     def to_create(&block)
       @definition.to_create(&block)
     end

data/lib/factory_girl/evaluator.rb

@@ -36,9 +36,12 @@ module FactoryGirl
                          overrides[:method]
                        elsif overrides.has_key?(:strategy)
                          overrides[:strategy]
+                       else
+                         Strategy::Create
                        end
 
-      runner = AssociationRunner.new(factory_name, build_strategy, overrides.except(:method, :strategy))
+      build_strategy = StrategyCalculator.new(build_strategy).strategy
+      runner = FactoryRunner.new(factory_name, build_strategy, [overrides.except(:method, :strategy)])
       @build_strategy.association(runner)
     end
 

data/lib/factory_girl/factory_runner.rb

@@ -0,0 +1,26 @@
+module FactoryGirl
+  class FactoryRunner
+    def initialize(name, strategy, traits_and_overrides)
+      @name     = name
+      @strategy = strategy
+
+      @overrides = if traits_and_overrides.last.respond_to?(:has_key?)
+                    traits_and_overrides.pop
+                  else
+                    {}
+                  end
+      @traits = traits_and_overrides
+    end
+
+    def run(strategy_override = nil, &block)
+      strategy_override ||= @strategy
+      factory = FactoryGirl.factory_by_name(@name)
+
+      if @traits.any?
+        factory = factory.with_traits(@traits)
+      end
+
+      factory.run(strategy_override, @overrides, &block)
+    end
+  end
+end

data/lib/factory_girl/strategy_calculator.rb

@@ -0,0 +1,29 @@
+module FactoryGirl
+  class StrategyCalculator
+    def initialize(name_or_object)
+      @name_or_object = name_or_object
+    end
+
+    def strategy
+      if strategy_is_object?
+        @name_or_object
+      else
+        strategy_name_to_object
+      end
+    end
+
+    private
+
+    def strategy_is_object?
+      @name_or_object.is_a?(Class) && @name_or_object.ancestors.include?(::FactoryGirl::Strategy)
+    end
+
+    def strategy_name_to_object
+      case @name_or_object
+      when :build  then Strategy::Build
+      when :create then Strategy::Create
+      else raise "unrecognized method #{@name_or_object}"
+      end
+    end
+  end
+end

data/lib/factory_girl/syntax/generate.rb

@@ -42,23 +42,20 @@ module FactoryGirl
         module ClassMethods #:nodoc:
 
           def generate(overrides = {}, &block)
-            factory = FactoryGirl.factory_by_name(name.underscore)
-            instance = factory.run(Strategy::Build, overrides)
+            instance = FactoryRunner.new(name.underscore, Strategy::Build, [overrides]).run
             instance.save
             yield(instance) if block_given?
             instance
           end
 
           def generate!(overrides = {}, &block)
-            factory = FactoryGirl.factory_by_name(name.underscore)
-            instance = factory.run(Strategy::Create, overrides)
+            instance = FactoryRunner.new(name.underscore, Strategy::Create, [overrides]).run
             yield(instance) if block_given?
             instance
           end
 
           def spawn(overrides = {}, &block)
-            factory = FactoryGirl.factory_by_name(name.underscore)
-            instance = factory.run(Strategy::Build, overrides)
+            instance = FactoryRunner.new(name.underscore, Strategy::Build, [overrides]).run
             yield(instance) if block_given?
             instance
           end

data/lib/factory_girl/syntax/make.rb

@@ -28,11 +28,11 @@ module FactoryGirl
         module ClassMethods #:nodoc:
 
           def make(overrides = {})
-            FactoryGirl.factory_by_name(name.underscore).run(Strategy::Build, overrides)
+            FactoryRunner.new(name.underscore, Strategy::Build, [overrides]).run
           end
 
           def make!(overrides = {})
-            FactoryGirl.factory_by_name(name.underscore).run(Strategy::Create, overrides)
+            FactoryRunner.new(name.underscore, Strategy::Create, [overrides]).run
           end
 
         end

data/lib/factory_girl/syntax/methods.rb

@@ -18,7 +18,7 @@ module FactoryGirl
       # A set of attributes that can be used to build an instance of the class
       # this factory generates.
       def attributes_for(name, *traits_and_overrides, &block)
-        run_factory_girl_strategy(name, traits_and_overrides, Strategy::AttributesFor, &block)
+        FactoryRunner.new(name, Strategy::AttributesFor, traits_and_overrides).run(&block)
       end
 
       # Generates and returns an instance from this factory. Attributes can be
@@ -37,7 +37,7 @@ module FactoryGirl
       # An instance of the class this factory generates, with generated attributes
       # assigned.
       def build(name, *traits_and_overrides, &block)
-        run_factory_girl_strategy(name, traits_and_overrides, Strategy::Build, &block)
+        FactoryRunner.new(name, Strategy::Build, traits_and_overrides).run(&block)
       end
 
       # Generates, saves, and returns an instance from this factory. Attributes can
@@ -60,7 +60,7 @@ module FactoryGirl
       # A saved instance of the class this factory generates, with generated
       # attributes assigned.
       def create(name, *traits_and_overrides, &block)
-        run_factory_girl_strategy(name, traits_and_overrides, Strategy::Create, &block)
+        FactoryRunner.new(name, Strategy::Create, traits_and_overrides).run(&block)
       end
 
       # Generates and returns an object with all attributes from this factory
@@ -79,7 +79,7 @@ module FactoryGirl
       # Returns: +Object+
       # An object with generated attributes stubbed out.
       def build_stubbed(name, *traits_and_overrides, &block)
-        run_factory_girl_strategy(name, traits_and_overrides, Strategy::Stub, &block)
+        FactoryRunner.new(name, Strategy::Stub, traits_and_overrides).run(&block)
       end
 
       # Builds and returns multiple instances from this factory as an array. Attributes can be
@@ -131,24 +131,6 @@ module FactoryGirl
       def generate(name)
         FactoryGirl.sequence_by_name(name).next
       end
-
-      private
-
-      def run_factory_girl_strategy(name, traits_and_overrides, strategy, &block)
-        overrides = if traits_and_overrides.last.respond_to?(:has_key?)
-                      traits_and_overrides.pop
-                    else
-                      {}
-                    end
-
-        factory = FactoryGirl.factory_by_name(name)
-
-        if traits_and_overrides.any?
-          factory = factory.with_traits(traits_and_overrides)
-        end
-
-        factory.run(strategy, overrides, &block)
-      end
     end
   end
 end

data/lib/factory_girl/version.rb

@@ -1,4 +1,4 @@
 module FactoryGirl
-  VERSION = "2.6.0"
+  VERSION = "2.6.1"
 end
 

data/spec/factory_girl/callback_spec.rb

@@ -28,7 +28,7 @@ describe FactoryGirl::Callback do
   end
 
   it "allows valid callback names to be assigned" do
-    FactoryGirl::Callback::VALID_NAMES.each do |callback_name|
+    FactoryGirl.callback_names.each do |callback_name|
       expect { FactoryGirl::Callback.new(callback_name, lambda {}) }.
         to_not raise_error(FactoryGirl::InvalidCallbackNameError)
     end

data/spec/factory_girl/strategy_calculator_spec.rb

@@ -0,0 +1,33 @@
+require "spec_helper"
+
+describe FactoryGirl::StrategyCalculator, "with a FactoryGirl::Strategy object" do
+  let(:strategy) { FactoryGirl::Strategy::Build }
+
+  it "returns the strategy object" do
+    FactoryGirl::StrategyCalculator.new(strategy).strategy.should == strategy
+  end
+end
+
+describe FactoryGirl::StrategyCalculator, "with a non-FactoryGirl::Strategy object" do
+  before { define_class "MyAwesomeStrategy" }
+
+  let(:strategy) { MyAwesomeStrategy }
+
+  it "returns the strategy object" do
+    expect { FactoryGirl::StrategyCalculator.new(strategy).strategy }.to raise_error "unrecognized method MyAwesomeStrategy"
+  end
+end
+
+describe FactoryGirl::StrategyCalculator do
+  it "returns the correct strategy object for :build" do
+    FactoryGirl::StrategyCalculator.new(:build).strategy.should == FactoryGirl::Strategy::Build
+  end
+
+  it "returns the correct strategy object for :create" do
+    FactoryGirl::StrategyCalculator.new(:create).strategy.should == FactoryGirl::Strategy::Create
+  end
+
+  it "raises when passing a bogus strategy" do
+    expect { FactoryGirl::StrategyCalculator.new(:bogus_strategy).strategy }.to raise_error "unrecognized method bogus_strategy"
+  end
+end

data/spec/spec_helper.rb

@@ -18,8 +18,6 @@ RSpec.configure do |config|
   config.include DeclarationMatchers
 
   config.after do
-    FactoryGirl.factories.clear
-    FactoryGirl.sequences.clear
-    FactoryGirl.traits.clear
+    FactoryGirl.reload
   end
 end

data/spec/support/shared_examples/strategy.rb

@@ -2,7 +2,7 @@ shared_examples_for "strategy without association support" do
   let(:attribute) { FactoryGirl::Attribute::Association.new(:user, :user, {}) }
 
   def association_named(name, overrides)
-    runner = FactoryGirl::AssociationRunner.new(name, :build, overrides)
+    runner = FactoryGirl::FactoryRunner.new(name, FactoryGirl::Strategy::Build, [overrides])
     subject.association(runner)
   end
 
@@ -21,7 +21,7 @@ shared_examples_for "strategy with association support" do |factory_girl_strateg
   let(:factory) { stub("associate_factory") }
 
   def association_named(name, strategy, overrides)
-    runner = FactoryGirl::AssociationRunner.new(name, strategy, overrides)
+    runner = FactoryGirl::FactoryRunner.new(name, strategy, [overrides])
     subject.association(runner)
   end
 
@@ -45,8 +45,8 @@ shared_examples_for "strategy with :strategy => :build" do |factory_girl_strateg
   let(:factory) { stub("associate_factory") }
 
   def association_named(name, overrides)
-    strategy = overrides[:strategy] || overrides[:method]
-    runner = FactoryGirl::AssociationRunner.new(name, strategy, overrides.except(:strategy, :method))
+    strategy = FactoryGirl::StrategyCalculator.new(overrides[:strategy] || overrides[:method]).strategy
+    runner = FactoryGirl::FactoryRunner.new(name, strategy, [overrides.except(:strategy, :method)])
     subject.association(runner)
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: factory_girl
 version: !ruby/object:Gem::Version
-  version: 2.6.0
+  version: 2.6.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-17 00:00:00.000000000 Z
+date: 2012-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70097037615520 !ruby/object:Gem::Requirement
+  requirement: &70301653289480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 2.3.9
   type: :runtime
   prerelease: false
-  version_requirements: *70097037615520
+  version_requirements: *70301653289480
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70097037615000 !ruby/object:Gem::Requirement
+  requirement: &70301653288800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '2.0'
   type: :development
   prerelease: false
-  version_requirements: *70097037615000
+  version_requirements: *70301653288800
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &70097037614460 !ruby/object:Gem::Requirement
+  requirement: &70301653287980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70097037614460
+  version_requirements: *70301653287980
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &70097037614020 !ruby/object:Gem::Requirement
+  requirement: &70301653285420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037614020
+  version_requirements: *70301653285420
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70097037613320 !ruby/object:Gem::Requirement
+  requirement: &70301653284860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037613320
+  version_requirements: *70301653284860
 - !ruby/object:Gem::Dependency
   name: aruba
-  requirement: &70097037612640 !ruby/object:Gem::Requirement
+  requirement: &70301653284360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037612640
+  version_requirements: *70301653284360
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &70097037611900 !ruby/object:Gem::Requirement
+  requirement: &70301653283880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037611900
+  version_requirements: *70301653283880
 - !ruby/object:Gem::Dependency
   name: bourne
-  requirement: &70097037611440 !ruby/object:Gem::Requirement
+  requirement: &70301653283400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037611440
+  version_requirements: *70301653283400
 - !ruby/object:Gem::Dependency
   name: appraisal
-  requirement: &70097037610900 !ruby/object:Gem::Requirement
+  requirement: &70301653282780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -109,10 +109,10 @@ dependencies:
         version: 0.3.8
   type: :development
   prerelease: false
-  version_requirements: *70097037610900
+  version_requirements: *70301653282780
 - !ruby/object:Gem::Dependency
   name: sqlite3-ruby
-  requirement: &70097037610400 !ruby/object:Gem::Requirement
+  requirement: &70301653282340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037610400
+  version_requirements: *70301653282340
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70097037609600 !ruby/object:Gem::Requirement
+  requirement: &70301653281820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037609600
+  version_requirements: *70301653281820
 - !ruby/object:Gem::Dependency
   name: bluecloth
-  requirement: &70097037608540 !ruby/object:Gem::Requirement
+  requirement: &70301653280800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,7 +142,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70097037608540
+  version_requirements: *70301653280800
 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."
@@ -183,7 +183,6 @@ files:
 - gemfiles/3.2.gemfile.lock
 - lib/factory_girl.rb
 - lib/factory_girl/aliases.rb
-- lib/factory_girl/association_runner.rb
 - lib/factory_girl/attribute.rb
 - lib/factory_girl/attribute/association.rb
 - lib/factory_girl/attribute/dynamic.rb
@@ -205,6 +204,7 @@ files:
 - lib/factory_girl/evaluator.rb
 - lib/factory_girl/evaluator_class_definer.rb
 - lib/factory_girl/factory.rb
+- lib/factory_girl/factory_runner.rb
 - lib/factory_girl/find_definitions.rb
 - lib/factory_girl/null_factory.rb
 - lib/factory_girl/null_object.rb
@@ -218,6 +218,7 @@ files:
 - lib/factory_girl/strategy/build.rb
 - lib/factory_girl/strategy/create.rb
 - lib/factory_girl/strategy/stub.rb
+- lib/factory_girl/strategy_calculator.rb
 - lib/factory_girl/syntax.rb
 - lib/factory_girl/syntax/blueprint.rb
 - lib/factory_girl/syntax/default.rb
@@ -259,7 +260,6 @@ files:
 - spec/acceptance/traits_spec.rb
 - spec/acceptance/transient_attributes_spec.rb
 - spec/factory_girl/aliases_spec.rb
-- spec/factory_girl/association_runner_spec.rb
 - spec/factory_girl/attribute/association_spec.rb
 - spec/factory_girl/attribute/dynamic_spec.rb
 - spec/factory_girl/attribute/sequence_spec.rb
@@ -283,6 +283,7 @@ files:
 - spec/factory_girl/strategy/build_spec.rb
 - spec/factory_girl/strategy/create_spec.rb
 - spec/factory_girl/strategy/stub_spec.rb
+- spec/factory_girl/strategy_calculator_spec.rb
 - spec/factory_girl/strategy_spec.rb
 - spec/factory_girl_spec.rb
 - spec/spec_helper.rb
@@ -364,7 +365,6 @@ test_files:
 - spec/acceptance/traits_spec.rb
 - spec/acceptance/transient_attributes_spec.rb
 - spec/factory_girl/aliases_spec.rb
-- spec/factory_girl/association_runner_spec.rb
 - spec/factory_girl/attribute/association_spec.rb
 - spec/factory_girl/attribute/dynamic_spec.rb
 - spec/factory_girl/attribute/sequence_spec.rb
@@ -388,6 +388,7 @@ test_files:
 - spec/factory_girl/strategy/build_spec.rb
 - spec/factory_girl/strategy/create_spec.rb
 - spec/factory_girl/strategy/stub_spec.rb
+- spec/factory_girl/strategy_calculator_spec.rb
 - spec/factory_girl/strategy_spec.rb
 - spec/factory_girl_spec.rb
 - spec/spec_helper.rb

data/lib/factory_girl/association_runner.rb

@@ -1,50 +0,0 @@
-module FactoryGirl
-  class AssociationRunner
-    def initialize(factory_name, strategy_name_or_object, overrides)
-      @factory_name            = factory_name
-      @strategy_name_or_object = strategy_name_or_object
-      @overrides               = overrides
-    end
-
-    def run(strategy_override = nil)
-      strategy_name_or_object = strategy_override || @strategy_name_or_object
-      strategy = StrategyCalculator.new(strategy_name_or_object).strategy
-      factory.run(strategy, @overrides)
-    end
-
-    private
-
-    def factory
-      FactoryGirl.factory_by_name(@factory_name)
-    end
-
-    class StrategyCalculator
-      def initialize(name_or_object)
-        @name_or_object = name_or_object
-      end
-
-      def strategy
-        if strategy_is_object?
-          @name_or_object
-        else
-          strategy_name_to_object
-        end
-      end
-
-      private
-
-      def strategy_is_object?
-        @name_or_object.is_a?(Class) && @name_or_object.ancestors.include?(::FactoryGirl::Strategy)
-      end
-
-      def strategy_name_to_object
-        case @name_or_object
-        when :build  then Strategy::Build
-        when :create then Strategy::Create
-        when nil     then Strategy::Create
-        else raise "unrecognized method #{@name_or_object}"
-        end
-      end
-    end
-  end
-end

data/spec/factory_girl/association_runner_spec.rb

@@ -1,31 +0,0 @@
-require "spec_helper"
-
-describe FactoryGirl::AssociationRunner do
-  let(:factory)  { stub("factory", :run => instance) }
-  let(:instance) { stub("instance") }
-
-  before do
-    FactoryGirl.stubs(:factory_by_name => factory)
-  end
-
-  it "passes all overrides to the factory" do
-    FactoryGirl::AssociationRunner.new(:user, FactoryGirl::Strategy::Build, :strategy => :build, :name => "John").run
-    factory.should have_received(:run).with(FactoryGirl::Strategy::Build, :strategy => :build, :name => "John")
-  end
-
-  it "runs a strategy inferred by name based on a factory name" do
-    FactoryGirl::AssociationRunner.new(:user, :build, :strategy => :build, :name => "John").run
-    factory.should have_received(:run).with(FactoryGirl::Strategy::Build, :strategy => :build, :name => "John")
-  end
-
-  it "allows overriding strategy" do
-    FactoryGirl::AssociationRunner.new(:user, :build, :strategy => :build, :name => "John").run(FactoryGirl::Strategy::Create)
-    factory.should have_received(:run).with(FactoryGirl::Strategy::Create, :strategy => :build, :name => "John")
-  end
-
-  it "raises if the strategy cannot be inferred" do
-    expect do
-      FactoryGirl::AssociationRunner.new(:user, :bogus_strategy, :strategy => :build, :name => "John").run
-    end.to raise_error("unrecognized method bogus_strategy")
-  end
-end