Permalink
Browse files

Suggestions

  • Loading branch information...
joshuaclayton committed Aug 19, 2011
1 parent d296a22 commit 744177cb8451a90f6cc44c930ed46efaa45c942c
@@ -135,7 +135,7 @@
end
[:build, :create, :attributes_for, :stub].each do |method|
- it "should raise an ArgumentError on #{method} with a nonexistant factory" do
+ it "should raise an ArgumentError on #{method} with a nonexistent factory" do
lambda { Factory.send(method, :bogus) }.should raise_error(ArgumentError)
end
@@ -34,7 +34,8 @@
end
describe "accessing an undefined constant on Factory that is defined on FactoryGirl" do
- it { Factory::VERSION.should == FactoryGirl::VERSION }
+ subject { Factory::VERSION }
+ it { should == FactoryGirl::VERSION }
end
describe "accessing an undefined constant on Factory that is undefined on FactoryGirl" do
@@ -261,7 +261,7 @@
end
describe FactoryGirl::Factory do
- let(:factory_with_non_existant_strategy) do
+ let(:factory_with_non_existent_strategy) do
FactoryGirl::Factory.new(:object, :default_strategy => :nonexistent) { }
end
@@ -283,7 +283,7 @@
end
it "raises an ArgumentError when trying to use a non-existent strategy" do
- expect { factory_with_non_existant_strategy }.to raise_error(ArgumentError)
+ expect { factory_with_non_existent_strategy }.to raise_error(ArgumentError)
end
it "creates a new factory with a specified default strategy" do

0 comments on commit 744177c

Please sign in to comment.