diff --git a/app/models/user.rb b/app/models/user.rb index bbf9c9bd5774f490b864bc50046f258827dc24a3..db4de26a5bc38a532cc98a4c2a8529619adbe36e 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -250,6 +250,7 @@ class User return self.person if id == self.person.id result = friends.detect{|x| x.id == id } result = visible_people.detect{|x| x.id == id } unless result + result end def group_by_id( id ) @@ -272,7 +273,7 @@ class User end def setup_person - self.person.serialized_key ||= generate_key.export + self.person.serialized_key ||= User.generate_key.export self.person.email ||= email self.person.save! end @@ -281,9 +282,6 @@ class User self.groups.all.collect{|x| x.id} end protected - def generate_key - OpenSSL::PKey::RSA::generate 1024 - end def self.generate_key OpenSSL::PKey::RSA::generate 1024 diff --git a/spec/controllers/groups_controller_spec.rb b/spec/controllers/groups_controller_spec.rb index 36773fd98bf58274520544ac6e3847f59a408627..590739e6a4273d6fc41c0bf11ad7870314664594 100644 --- a/spec/controllers/groups_controller_spec.rb +++ b/spec/controllers/groups_controller_spec.rb @@ -6,7 +6,7 @@ describe GroupsController do @user = Factory.create(:user) @user.group(:name => "lame-os") @person = Factory.create(:person) - request.env['warden'] = mock_model(Warden, :authenticate? => @user, :authenticate! => @user, :authenticate => @user) + sign_in :user, @user end it "on index sets a variable containing all a user's friends when a user is signed in" do diff --git a/spec/controllers/people_controller_spec.rb b/spec/controllers/people_controller_spec.rb index 0bca73e18bbc9034a41c740e9dcb6ec89f0a4016..bc5348052b724750868b20855da3ba8386252f6a 100644 --- a/spec/controllers/people_controller_spec.rb +++ b/spec/controllers/people_controller_spec.rb @@ -4,7 +4,6 @@ describe PeopleController do render_views before do @user = Factory.create(:user) - request.env['warden'] = mock_model(Warden, :authenticate? => @user, :authenticate! => @user, :authenticate => @user) sign_in :user, @user @user.group(:name => "lame-os") diff --git a/spec/controllers/publics_controller_spec.rb b/spec/controllers/publics_controller_spec.rb index dfe1e461b730b5c380afe90def9478eb54d5a33f..cc6bd85b5f31a66dd45eca4ce446cfb733c4eaf5 100644 --- a/spec/controllers/publics_controller_spec.rb +++ b/spec/controllers/publics_controller_spec.rb @@ -6,7 +6,7 @@ describe PublicsController do before do @user = Factory.create(:user) @user.person.save - request.env['warden'] = mock_model(Warden, :authenticate? => @user, :authenticate! => @user, :authenticate => @user) + sign_in :user, @user end describe 'receive endpoint' do diff --git a/spec/controllers/sockets_controller_spec.rb b/spec/controllers/sockets_controller_spec.rb index 23a02d61e22c2103149bf8abf216c285b8ee12aa..26318dad6356343007c8be72026604c6417a10d3 100644 --- a/spec/controllers/sockets_controller_spec.rb +++ b/spec/controllers/sockets_controller_spec.rb @@ -6,15 +6,11 @@ class SocketsController end end -describe 'SocketsController' do +describe SocketsController do render_views before do @user = Factory.create(:user) - SocketsController.unstub!(:new) @controller = SocketsController.new - @controller.request = mock_model(Request, :env => - {'warden' => mock_model(Warden, :authenticate? => @user, :authenticate! => @user, :authenticate => @user)}) - stub_sockets_controller end it 'should unstub the websockets' do diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index 3b80d4c50ffdac8b9e57ce6ecd42a9d373c22d13..ff5cb46f33ccd14f2856b63dbd5730a4b4260251 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -21,24 +21,22 @@ RSpec.configure do |config| config.before(:suite) do DatabaseCleaner.clean_with(:truncation) stub_signature_verification + end config.before(:each) do DatabaseCleaner.start - stub_sockets_controller + stub_sockets end config.after(:each) do DatabaseCleaner.clean end end - def stub_sockets_controller - mock_sockets_controller = mock('sockets mock') - mock_sockets_controller.stub!(:incoming).and_return(true) - mock_sockets_controller.stub!(:new_subscriber).and_return(true) - mock_sockets_controller.stub!(:outgoing).and_return(true) - mock_sockets_controller.stub!(:delete_subscriber).and_return(true) - SocketsController.stub!(:new).and_return(mock_sockets_controller) + def stub_sockets + Diaspora::WebSocket.stub!(:push_to_user).and_return(true) + Diaspora::WebSocket.stub!(:subscribe).and_return(true) + Diaspora::WebSocket.stub!(:unsubscribe).and_return(true) end def stub_signature_verification