diff --git a/lib/common/models/wp_user.rb b/lib/common/models/wp_user.rb index 03d0138b..467cca9a 100755 --- a/lib/common/models/wp_user.rb +++ b/lib/common/models/wp_user.rb @@ -3,7 +3,6 @@ require 'wp_user/existable' class WpUser < WpItem - include WpUser::Existable attr_accessor :id, :login, :display_name, :password @@ -23,7 +22,7 @@ class WpUser < WpItem end def ==(other) - self === (other) + self === other end def ===(other) diff --git a/spec/lib/common/models/wp_user_spec.rb b/spec/lib/common/models/wp_user_spec.rb new file mode 100644 index 00000000..30ee72de --- /dev/null +++ b/spec/lib/common/models/wp_user_spec.rb @@ -0,0 +1,57 @@ +# encoding: UTF-8 + +require 'spec_helper' + +describe WpUser do + subject(:wp_user) { WpUser.new(uri, options) } + let(:uri) { URI.parse('http://example.com') } + let(:options) { {} } + + describe '#allowed_options' do + [:id, :login, :display_name, :password].each do |sym| + its(:allowed_options) { should include sym } + end + + its(:allowed_options) { should_not include :name } + end + + describe '#uri' do + context 'when the id is not set' do + it 'raises an error' do + expect { wp_user.uri }.to raise_error('The id is nil') + end + end + + context 'when the id is set' do + it 'returns the uri to the auhor page' do + wp_user.id = 2 + + wp_user.uri.should == uri.merge('?author=2') + end + end + end + + describe '#<=>' do + it 'bases the comparaison on the :id' do + wp_user.id = 1 + other = WpUser.new(uri, id: 3) + + wp_user.<=>(other).should === 1.<=>(3) + end + end + + describe '#===, #==' do + context 'when the :id and :login are the same' do + it 'is ===, and ==' do + WpUser.new(uri, id: 1, name: 'yo').should == WpUser.new(uri, id: 1, name: 'yo') + end + end + + context 'when :id and :login are different' do + it 'is not === or ==' do + WpUser.new(uri, id: 1, name: 'yo').should_not == WpUser.new(uri, id: 2, name:'yo') + end + end + end + +end