vitalish-factory_girl 1.2.5 → 1.2.6

data/spec/factory_girl/proxy/attributes_for_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
 
-describe FactoryGirl::Proxy::AttributesFor do
+describe Factory::Proxy::AttributesFor do
   before do
-    @proxy = FactoryGirl::Proxy::AttributesFor.new(@class)
+    @proxy = Factory::Proxy::AttributesFor.new(@class)
   end
 
   describe "when asked to associate with another factory" do

data/spec/factory_girl/proxy/build_spec.rb

@@ -1,16 +1,18 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
 
-describe FactoryGirl::Proxy::Build do
+describe Factory::Proxy::Build do
   before do
     @class       = Class.new
     @instance    = "built-instance"
+    @association = "associated-instance"
 
     stub(@class).new { @instance }
     stub(@instance).attribute { 'value' }
+    stub(Factory).create { @association }
     stub(@instance, :attribute=)
     stub(@instance, :owner=)
 
-    @proxy = FactoryGirl::Proxy::Build.new(@class)
+    @proxy = Factory::Proxy::Build.new(@class)
   end
 
   it "should instantiate the class" do
@@ -19,16 +21,11 @@ describe FactoryGirl::Proxy::Build do
 
   describe "when asked to associate with another factory" do
     before do
-      @association = "associated-instance"
-      @associated_factory = "associated-factory"
-      stub(FactoryGirl).factory_by_name { @associated_factory }
-      stub(@associated_factory).run { @association }
-      @overrides = { 'attr' => 'value' }
-      @proxy.associate(:owner, :user, @overrides)
+      @proxy.associate(:owner, :user, {})
     end
 
     it "should create the associated instance" do
-      @associated_factory.should have_received.run(FactoryGirl::Proxy::Create, @overrides)
+      Factory.should have_received.create(:user, {})
     end
 
     it "should set the associated instance" do
@@ -36,14 +33,12 @@ describe FactoryGirl::Proxy::Build do
     end
   end
 
-  it "should run create when building an association" do
-    association = "associated-instance"
-    associated_factory = "associated-factory"
-    stub(FactoryGirl).factory_by_name { associated_factory }
-    stub(associated_factory).run { association }
-    overrides = { 'attr' => 'value' }
-    @proxy.association(:user, overrides).should == association
-    associated_factory.should have_received.run(FactoryGirl::Proxy::Create, overrides)
+  it "should call Factory.create when building an association" do
+    association = 'association'
+    attribs     = { :first_name => 'Billy' }
+    stub(Factory).create { association }
+    @proxy.association(:user, attribs).should == association
+    Factory.should have_received.create(:user, attribs)
   end
 
   it "should return the built instance when asked for the result" do

data/spec/factory_girl/proxy/create_spec.rb

@@ -1,17 +1,19 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
 
-describe FactoryGirl::Proxy::Create do
+describe Factory::Proxy::Create do
   before do
     @class       = Class.new
     @instance    = "built-instance"
+    @association = "associated-instance"
 
     stub(@class).new { @instance }
+    stub(Factory).create { @association }
     stub(@instance).attribute { 'value' }
     stub(@instance, :attribute=)
     stub(@instance, :owner=)
     stub(@instance).save!
 
-    @proxy = FactoryGirl::Proxy::Create.new(@class)
+    @proxy = Factory::Proxy::Create.new(@class)
   end
 
   it "should instantiate the class" do
@@ -20,16 +22,11 @@ describe FactoryGirl::Proxy::Create do
 
   describe "when asked to associate with another factory" do
     before do
-      @association = "associated-instance"
-      @associated_factory = "associated-factory"
-      stub(FactoryGirl).factory_by_name { @associated_factory }
-      stub(@associated_factory).run { @association }
-      @overrides = { 'attr' => 'value' }
-      @proxy.associate(:owner, :user, @overrides)
+      @proxy.associate(:owner, :user, {})
     end
 
     it "should create the associated instance" do
-      @associated_factory.should have_received.run(FactoryGirl::Proxy::Create, @overrides)
+      Factory.should have_received.create(:user, {})
     end
 
     it "should set the associated instance" do
@@ -37,14 +34,12 @@ describe FactoryGirl::Proxy::Create do
     end
   end
 
-  it "should run create when building an association" do
-    association = "associated-instance"
-    associated_factory = "associated-factory"
-    stub(FactoryGirl).factory_by_name { associated_factory }
-    stub(associated_factory).run { association }
-    overrides = { 'attr' => 'value' }
-    @proxy.association(:user, overrides).should == association
-    associated_factory.should have_received.run(FactoryGirl::Proxy::Create, overrides)
+  it "should call Factory.create when building an association" do
+    association = 'association'
+    attribs     = { :first_name => 'Billy' }
+    stub(Factory).create { association }
+    @proxy.association(:user, attribs).should == association
+    Factory.should have_received.create(:user, attribs)
   end
 
   describe "when asked for the result" do

data/spec/factory_girl/proxy/stub_spec.rb

@@ -1,6 +1,6 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
 
-describe FactoryGirl::Proxy::Stub do
+describe Factory::Proxy::Stub do
   before do
     @class = "class"
     @instance = "instance"
@@ -9,7 +9,7 @@ describe FactoryGirl::Proxy::Stub do
     stub(@instance).id { 42 }
     stub(@instance).reload { @instance.connection.reload }
 
-    @stub = FactoryGirl::Proxy::Stub.new(@class)
+    @stub = Factory::Proxy::Stub.new(@class)
   end
 
   it "should not be a new record" do
@@ -23,14 +23,13 @@ describe FactoryGirl::Proxy::Stub do
   describe "when a user factory exists" do
     before do
       @user = "user"
-      stub(FactoryGirl).factory_by_name { @associated_factory }
-      @associated_factory = 'associate-factory'
+      stub(Factory).stub(:user, {}) { @user }
     end
 
     describe "when asked to associate with another factory" do
       before do
         stub(@instance).owner { @user }
-        mock(@associated_factory).run(FactoryGirl::Proxy::Stub, {}) { @user }
+        mock(Factory).stub(:user, {}) { @user }
         mock(@stub).set(:owner, @user)
 
         @stub.associate(:owner, :user, {})
@@ -42,7 +41,7 @@ describe FactoryGirl::Proxy::Stub do
     end
 
     it "should return the association when building one" do
-      mock(@associated_factory).run(FactoryGirl::Proxy::Stub, {}) { @user }
+      mock(Factory).create.never
       @stub.association(:user).should == @user
     end
 

data/spec/factory_girl/proxy_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
 
-describe FactoryGirl::Proxy do
+describe Factory::Proxy do
   before do
-    @proxy = FactoryGirl::Proxy.new(Class.new)
+    @proxy = Factory::Proxy.new(Class.new)
   end
 
   it "should do nothing when asked to set an attribute to a value" do

data/spec/factory_girl/sequence_spec.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
 
-describe FactoryGirl::Sequence do
-  describe "a basic sequence" do
+describe Factory::Sequence do
+  describe "a sequence" do
     before do
-      @sequence = FactoryGirl::Sequence.new {|n| "=#{n}" }
+      @sequence = Factory::Sequence.new {|n| "=#{n}" }
     end
 
     it "should start with a value of 1" do
@@ -20,24 +20,47 @@ describe FactoryGirl::Sequence do
       end
     end
   end
-  
-  describe "a custom sequence" do
+
+  describe "defining a sequence" do
     before do
-      @sequence = FactoryGirl::Sequence.new("A") {|n| "=#{n}" }
+      @sequence = "sequence"
+      @name     = :count
+      stub(Factory::Sequence).new { @sequence }
     end
 
-    it "should start with a value of A" do
-      @sequence.next.should == "=A"
+    it "should create a new sequence" do
+      mock(Factory::Sequence).new() { @sequence }
+      Factory.sequence(@name)
     end
 
-    describe "after being called" do
-      before do
-        @sequence.next
-      end
+    it "should use the supplied block as the sequence generator" do
+      stub(Factory::Sequence).new.yields(1)
+      yielded = false
+      Factory.sequence(@name) {|n| yielded = true }
+      (yielded).should be
+    end
+  end
 
-      it "should use the next value" do
-        @sequence.next.should == "=B"
-      end
+  describe "after defining a sequence" do
+    before do
+      @sequence = "sequence"
+      @name     = :test
+      @value    = '1 2 5'
+
+      stub(@sequence).next { @value }
+      stub(Factory::Sequence).new { @sequence }
+
+      Factory.sequence(@name) {}
+    end
+
+    it "should call next on the sequence when sent next" do
+      mock(@sequence).next
+
+      Factory.next(@name)
+    end
+
+    it "should return the value from the sequence" do
+      Factory.next(@name).should == @value
     end
   end
 end

data/spec/factory_girl/syntax/blueprint_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+require 'factory_girl/syntax/blueprint'
+
+describe "a blueprint" do
+  before do
+    Factory.sequence(:email) { |n| "somebody#{n}@example.com" }
+    User.blueprint do
+      first_name { 'Bill'               }
+      last_name  { 'Nye'                }
+      email      { Factory.next(:email) }
+    end
+  end
+
+  after do
+    Factory.factories.clear
+    Factory.sequences.clear
+  end
+
+  describe "after making an instance" do
+    before do
+      @instance = Factory(:user, :last_name => 'Rye')
+    end
+
+    it "should use attributes from the blueprint" do
+      @instance.first_name.should == 'Bill'
+    end
+
+    it "should evaluate attribute blocks for each instance" do
+      @instance.email.should =~ /somebody\d+@example.com/
+      Factory(:user).email.should_not == @instance.email
+    end
+  end
+end

data/spec/factory_girl/syntax/generate_spec.rb

@@ -0,0 +1,57 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+require 'factory_girl/syntax/generate'
+
+describe "a factory using generate syntax" do
+  before do
+    Factory.define :user do |factory|
+      factory.first_name 'Bill'
+      factory.last_name  'Nye'
+      factory.email      'science@guys.net'
+    end
+  end
+
+  after do
+    Factory.factories.clear
+  end
+
+  it "should not raise an error when generating an invalid instance" do
+    lambda { User.generate(:first_name => nil) }.should_not raise_error
+  end
+
+  it "should raise an error when forcefully generating an invalid instance" do
+    lambda { User.generate!(:first_name => nil) }.should raise_error(ActiveRecord::RecordInvalid)
+  end
+
+  %w(generate generate! spawn).each do |method|
+    it "should yield a generated instance when using #{method} with a block" do
+      saved_instance = nil
+      User.send(method) {|instance| saved_instance = instance }
+      saved_instance.should be_kind_of(User)
+    end
+
+    describe "after generating an instance using #{method}" do
+      before do
+        @instance = User.send(method, :last_name => 'Rye')
+      end
+
+      it "should use attributes from the factory" do
+        @instance.first_name.should == 'Bill'
+      end
+
+      it "should use attributes passed to generate" do
+        @instance.last_name.should == 'Rye'
+      end
+
+      if method == 'spawn'
+        it "should not save the record" do
+          @instance.should be_new_record
+        end
+      else
+        it "should save the record" do
+          @instance.should_not be_new_record
+        end
+      end
+    end
+  end
+end

data/spec/factory_girl/syntax/make_spec.rb

@@ -0,0 +1,35 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+require 'factory_girl/syntax/make'
+
+describe "a factory using make syntax" do
+  before do
+    Factory.define :user do |factory|
+      factory.first_name 'Bill'
+      factory.last_name  'Nye'
+      factory.email      'science@guys.net'
+    end
+  end
+
+  after do
+    Factory.factories.clear
+  end
+
+  describe "after making an instance" do
+    before do
+      @instance = User.make(:last_name => 'Rye')
+    end
+
+    it "should use attributes from the factory" do
+      @instance.first_name.should == 'Bill'
+    end
+
+    it "should use attributes passed to make" do
+      @instance.last_name.should == 'Rye'
+    end
+
+    it "should save the record" do
+      @instance.should_not be_new_record
+    end
+  end
+end

data/spec/factory_girl/syntax/sham_spec.rb

@@ -0,0 +1,35 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+require 'factory_girl/syntax/sham'
+
+describe "a factory using sham syntax" do
+  before do
+    Sham.name  { "Name" }
+    Sham.email { "somebody#{rand(5)}@example.com" }
+
+    Factory.define :user do |factory|
+      factory.first_name { Sham.name }
+      factory.last_name  { Sham.name }
+      factory.email      { Sham.email }
+    end
+  end
+
+  after do
+    Factory.factories.clear
+    Factory.sequences.clear
+  end
+
+  describe "after making an instance" do
+    before do
+      @instance = Factory(:user, :last_name => 'Rye')
+    end
+
+    it "should support a sham called 'name'" do
+      @instance.first_name.should == 'Name'
+    end
+
+    it "should use the sham for the email" do
+      @instance.email.should =~ /somebody\d@example.com/
+    end
+  end
+end

data/spec/integration_spec.rb

@@ -0,0 +1,304 @@
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+
+describe "integration" do
+  before do
+    Factory.define :user, :class => 'user' do |f|
+      f.first_name 'Jimi'
+      f.last_name  'Hendrix'
+      f.admin       false
+      f.email {|a| "#{a.first_name}.#{a.last_name}@example.com".downcase }
+    end
+
+    Factory.define Post, :default_strategy => :attributes_for do |f|
+      f.name   'Test Post'
+      f.association :author, :factory => :user
+    end
+
+    Factory.define :admin, :class => User do |f|
+      f.first_name 'Ben'
+      f.last_name  'Stein'
+      f.admin       true
+      f.sequence(:username) { |n| "username#{n}" }
+      f.email { Factory.next(:email) }
+    end
+
+    Factory.define :sequence_abuser, :class => User do |f|
+      f.first_name { Factory.sequence(:email) }
+    end
+
+    Factory.define :guest, :parent => :user do |f|
+      f.last_name 'Anonymous'
+      f.username  'GuestUser'
+    end
+
+    Factory.define :user_with_callbacks, :parent => :user do |f|
+      f.after_stub   {|u| u.first_name = 'Stubby' }
+      f.after_build  {|u| u.first_name = 'Buildy' }
+      f.after_create {|u| u.last_name  = 'Createy' }
+    end
+
+    Factory.define :user_with_inherited_callbacks, :parent => :user_with_callbacks do |f|
+      f.callback(:after_stub) {|u| u.last_name = 'Double-Stubby' }
+    end
+
+    Factory.define :business do |f|
+      f.name 'Supplier of Awesome'
+      f.association :owner, :factory => :user
+    end
+
+    Factory.sequence :email do |n|
+      "somebody#{n}@example.com"
+    end
+  end
+
+  after do
+    Factory.factories.clear
+  end
+
+  describe "a generated attributes hash" do
+
+    before do
+      @attrs = Factory.attributes_for(:user, :first_name => 'Bill')
+    end
+
+    it "should assign all attributes" do
+      expected_attrs = [:admin, :email, :first_name, :last_name]
+      actual_attrs = @attrs.keys.sort {|a, b| a.to_s <=> b.to_s }
+      actual_attrs.should == expected_attrs
+    end
+
+    it "should correctly assign lazy, dependent attributes" do
+      @attrs[:email].should == "bill.hendrix@example.com"
+    end
+
+    it "should override attrbutes" do
+      @attrs[:first_name].should == 'Bill'
+    end
+
+    it "should not assign associations" do
+      Factory.attributes_for(:post)[:author].should be_nil
+    end
+
+  end
+
+  describe "a built instance" do
+
+    before do
+      @instance = Factory.build(:post)
+    end
+
+    it "should not be saved" do
+      @instance.should be_new_record
+    end
+
+    it "should assign associations" do
+      @instance.author.should be_kind_of(User)
+    end
+
+    it "should save associations" do
+      @instance.author.should_not be_new_record
+    end
+
+    it "should not assign both an association and its foreign key" do
+      Factory.build(:post, :author_id => 1).author_id.should == 1
+    end
+
+  end
+
+  describe "a created instance" do
+
+    before do
+      @instance = Factory.create('post')
+    end
+
+    it "should be saved" do
+      @instance.should_not be_new_record
+    end
+
+    it "should assign associations" do
+      @instance.author.should be_kind_of(User)
+    end
+
+    it "should save associations" do
+      @instance.author.should_not be_new_record
+    end
+
+  end
+
+  describe "a generated stub instance" do
+
+    before do
+      @stub = Factory.stub(:user, :first_name => 'Bill')
+    end
+
+    it "should assign all attributes" do
+      [:admin, :email, :first_name, :last_name].each do |attr|
+        @stub.send(attr).should_not be_nil
+      end
+    end
+
+    it "should correctly assign attributes" do
+      @stub.email.should == "bill.hendrix@example.com"
+    end
+
+    it "should override attributes" do
+      @stub.first_name.should == 'Bill'
+    end
+
+    it "should assign associations" do
+      Factory.stub(:post).author.should_not be_nil
+    end
+
+    it "should have an id" do
+      @stub.id.should > 0
+    end
+
+    it "should have unique IDs" do
+      @other_stub = Factory.stub(:user)
+      @stub.id.should_not == @other_stub.id
+    end
+
+    it "should not be considered a new record" do
+      @stub.should_not be_new_record
+    end
+
+    it "should raise exception if connection to the database is attempted" do
+      lambda { @stub.connection }.should raise_error(RuntimeError)
+      lambda { @stub.update_attribute(:first_name, "Nick") }.should raise_error(RuntimeError)
+      lambda { @stub.reload }.should raise_error(RuntimeError)
+      lambda { @stub.destroy }.should raise_error(RuntimeError)
+      lambda { @stub.save }.should raise_error(RuntimeError)
+      lambda { @stub.increment!(:age) }.should raise_error(RuntimeError)
+    end
+  end
+
+  describe "an instance generated by a factory with a custom class name" do
+
+    before do
+      @instance = Factory.create(:admin)
+    end
+
+    it "should use the correct class name" do
+      @instance.should be_kind_of(User)
+    end
+
+    it "should use the correct factory definition" do
+      @instance.should be_admin
+    end
+
+  end
+
+  describe "an instance generated by a factory that inherits from another factory" do
+    before do
+      @instance = Factory.create(:guest)
+    end
+
+    it "should use the class name of the parent factory" do
+      @instance.should be_kind_of(User)
+    end
+
+    it "should have attributes of the parent" do
+      @instance.first_name.should == 'Jimi'
+    end
+
+    it "should have attributes defined in the factory itself" do
+      @instance.username.should == 'GuestUser'
+    end
+
+    it "should have attributes that have been overriden" do
+      @instance.last_name.should == 'Anonymous'
+    end
+  end
+
+  describe "an attribute generated by a sequence" do
+
+    before do
+      @email = Factory.attributes_for(:admin)[:email]
+    end
+
+    it "should match the correct format" do
+      @email.should =~ /^somebody\d+@example\.com$/
+    end
+
+    describe "after the attribute has already been generated once" do
+
+      before do
+        @another_email = Factory.attributes_for(:admin)[:email]
+      end
+
+      it "should match the correct format" do
+        @email.should =~ /^somebody\d+@example\.com$/
+      end
+
+      it "should not be the same as the first generated value" do
+        @another_email.should_not == @email
+      end
+
+    end
+
+  end
+
+  describe "an attribute generated by an in-line sequence" do
+
+    before do
+      @username = Factory.attributes_for(:admin)[:username]
+    end
+
+    it "should match the correct format" do
+      @username.should =~ /^username\d+$/
+    end
+
+    describe "after the attribute has already been generated once" do
+
+      before do
+        @another_username = Factory.attributes_for(:admin)[:username]
+      end
+
+      it "should match the correct format" do
+        @username.should =~ /^username\d+$/
+      end
+
+      it "should not be the same as the first generated value" do
+        @another_username.should_not == @username
+      end
+
+    end
+
+  end
+
+  describe "a factory with a default strategy specified" do
+    it "should generate instances according to the strategy" do
+      Factory(:post).should be_kind_of(Hash)
+    end
+  end
+
+  it "should raise Factory::SequenceAbuseError" do
+    lambda {
+      Factory(:sequence_abuser)
+    }.should raise_error(Factory::SequenceAbuseError)
+  end
+
+  describe "an instance with callbacks" do
+    it "should run the after_stub callback when stubbing" do
+      @user = Factory.stub(:user_with_callbacks)
+      @user.first_name.should == 'Stubby'
+    end
+
+    it "should run the after_build callback when building" do
+      @user = Factory.build(:user_with_callbacks)
+      @user.first_name.should == 'Buildy'
+    end
+
+    it "should run both the after_build and after_create callbacks when creating" do
+      @user = Factory(:user_with_callbacks)
+      @user.first_name.should == 'Buildy'
+      @user.last_name.should == 'Createy'
+    end
+
+    it "should run both the after_stub callback on the factory and the inherited after_stub callback" do
+      @user = Factory.stub(:user_with_inherited_callbacks)
+      @user.first_name.should == 'Stubby'
+      @user.last_name.should == 'Double-Stubby'
+    end
+  end
+end