mirror of
https://github.com/thoughtbot/factory_bot.git
synced 2022-11-09 11:43:51 -05:00
![Andrew Peterson](/assets/img/avatar_default.png)
Why? Traits are another way to extend or modify behavior of a factory; because of this, applied traits may result in invalid factories, even if the base factory is valid. To enable this functionality, use: FactoryGirl.lint traits:true Note that, because FactoryGirl uses the `create` strategy when linting, this may greatly slow the time to lint factories.
163 lines
3.7 KiB
Ruby
163 lines
3.7 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe 'FactoryGirl.lint' do
|
|
it 'raises when a factory is invalid' do
|
|
define_model 'User', name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
define_model 'AlwaysValid'
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
factory :admin_user
|
|
end
|
|
|
|
factory :always_valid
|
|
end
|
|
|
|
error_message = <<-ERROR_MESSAGE.strip
|
|
The following factories are invalid:
|
|
|
|
* user - Validation failed: Name can't be blank (ActiveRecord::RecordInvalid)
|
|
* admin_user - Validation failed: Name can't be blank (ActiveRecord::RecordInvalid)
|
|
ERROR_MESSAGE
|
|
|
|
expect do
|
|
FactoryGirl.lint
|
|
end.to raise_error FactoryGirl::InvalidFactoryError, error_message
|
|
end
|
|
|
|
it 'does not raise when all factories are valid' do
|
|
define_model 'User', name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name 'assigned'
|
|
end
|
|
end
|
|
|
|
expect { FactoryGirl.lint }.not_to raise_error
|
|
end
|
|
|
|
it 'allows for selective linting' do
|
|
define_model 'InvalidThing', name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
define_model 'ValidThing', name: :string
|
|
|
|
FactoryGirl.define do
|
|
factory :valid_thing
|
|
factory :invalid_thing
|
|
end
|
|
|
|
expect do
|
|
only_valid_factories = FactoryGirl.factories.reject do |factory|
|
|
factory.name =~ /invalid/
|
|
end
|
|
|
|
FactoryGirl.lint only_valid_factories
|
|
end.not_to raise_error
|
|
end
|
|
|
|
it "allows for explicit factories and additional options" do
|
|
define_model "User", name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name "assigned"
|
|
end
|
|
end
|
|
|
|
FactoryGirl.lint FactoryGirl.factories, validate_traits: true
|
|
end
|
|
|
|
it "allows for additional options without explicit factories" do
|
|
define_model "User", name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name "assigned"
|
|
end
|
|
end
|
|
|
|
FactoryGirl.lint validate_traits: true
|
|
end
|
|
|
|
describe "trait validation" do
|
|
context "enabled" do
|
|
it "raises if a trait produces an invalid object" do
|
|
define_model "User", name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name "Yep"
|
|
trait :unnamed do
|
|
name nil
|
|
end
|
|
end
|
|
end
|
|
|
|
error_message = <<-ERROR_MESSAGE.strip
|
|
The following factories are invalid:
|
|
|
|
* user+unnamed - Validation failed: Name can't be blank (ActiveRecord::RecordInvalid)
|
|
ERROR_MESSAGE
|
|
|
|
expect do
|
|
FactoryGirl.lint traits: true
|
|
end.to raise_error FactoryGirl::InvalidFactoryError, error_message
|
|
end
|
|
|
|
it "does not raise if a trait produces a valid object" do
|
|
define_model "User", name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name "Yep"
|
|
trait :renamed do
|
|
name "Yessir"
|
|
end
|
|
end
|
|
end
|
|
|
|
expect do
|
|
FactoryGirl.lint traits: true
|
|
end.not_to raise_error
|
|
end
|
|
end
|
|
|
|
context "disabled" do
|
|
it "does not raises if a trait produces an invalid object" do
|
|
define_model "User", name: :string do
|
|
validates :name, presence: true
|
|
end
|
|
|
|
FactoryGirl.define do
|
|
factory :user do
|
|
name "Yep"
|
|
trait :unnamed do
|
|
name nil
|
|
end
|
|
end
|
|
end
|
|
|
|
expect do
|
|
FactoryGirl.lint traits: false
|
|
FactoryGirl.lint
|
|
end.not_to raise_error
|
|
end
|
|
end
|
|
end
|
|
end
|