nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

443- patch trim on validation + spec

Browse source
This commit is contained in:
Raphael 2010-10-28 20:03:49 -07:00
parent fd2fac865b
commit f1ae95fac9
2 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/models/aspect.rb
View file

@ -18,7 +18,11 @@ class Aspect
validates_presence_of :name validates_presence_of :name
validates_uniqueness_of :name, :scope => :user_id validates_uniqueness_of :name, :scope => :user_id
attr_accessible :name attr_accessible :name
before_validation do
name.strip!
end
timestamps! timestamps!
def to_s def to_s

8
spec/models/aspect_spec.rb
View file

@ -18,11 +18,17 @@ describe Aspect do
let(:aspect3) {user3.aspect(:name => "lala")} let(:aspect3) {user3.aspect(:name => "lala")}
describe 'creation' do describe 'creation' do
let(:aspect){user.aspect(:name => 'losers')} let!(:aspect){user.aspect(:name => 'losers')}
it 'should have a name' do it 'should have a name' do
aspect.name.should == "losers" aspect.name.should == "losers"
end end
it 'should not allow duplicate names' do
lambda {
invalid_aspect = user.aspect(:name => "losers ")
}.should_not change(Aspect, :count)
end
it 'should not be creatable with people' do it 'should not be creatable with people' do
aspect = user.aspect(:name => 'losers', :people => [friend, friend_2]) aspect = user.aspect(:name => 'losers', :people => [friend, friend_2])
aspect.people.size.should == 0 aspect.people.size.should == 0