refactor test to use let and rm unused variables
This commit is contained in:
parent
d86a768590
commit
a9919fabd1
1 changed files with 3 additions and 9 deletions
|
|
@ -6,20 +6,14 @@ require 'spec_helper'
|
||||||
|
|
||||||
describe Aspect, :type => :model do
|
describe Aspect, :type => :model do
|
||||||
describe 'creation' do
|
describe 'creation' do
|
||||||
before do
|
let(:name) { alice.aspects.first.name }
|
||||||
@name = alice.aspects.first.name
|
|
||||||
end
|
|
||||||
|
|
||||||
it 'does not allow duplicate names' do
|
it 'does not allow duplicate names' do
|
||||||
expect {
|
expect { alice.aspects.create(:name => name) }.not_to change(Aspect, :count)
|
||||||
invalid_aspect = alice.aspects.create(:name => @name)
|
|
||||||
}.not_to change(Aspect, :count)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'validates case insensitiveness on names' do
|
it 'validates case insensitiveness on names' do
|
||||||
expect {
|
expect { alice.aspects.create(:name => name.titleize) }.not_to change(Aspect, :count)
|
||||||
invalid_aspect = alice.aspects.create(:name => @name.titleize)
|
|
||||||
}.not_to change(Aspect, :count)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'has a 20 character limit on names' do
|
it 'has a 20 character limit on names' do
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue