From 34273c8a17a781e89719b017b0bc958816c65f78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Juan=20Ignacio=20S=C3=A1nchez=20Lara?= Date: Mon, 13 Mar 2017 15:34:28 +0100 Subject: [PATCH] Replace user double with value --- .../carto_gears_api/users_service_spec.rb | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/gears/carto_gears_api/spec/carto_gears_api/users_service_spec.rb b/gears/carto_gears_api/spec/carto_gears_api/users_service_spec.rb index c392572464..2fceb98e83 100644 --- a/gears/carto_gears_api/spec/carto_gears_api/users_service_spec.rb +++ b/gears/carto_gears_api/spec/carto_gears_api/users_service_spec.rb @@ -5,26 +5,23 @@ describe CartoGearsApi::Users::UsersService do describe '#logged_user' do module CartoDB; end - let(:id) { 'b51e56fb-f3c9-463f-b950-d9be188551e5' } - let(:username) { 'wadus_username' } - let(:email) { 'wadus@carto.com' } - let(:service) { CartoGearsApi::Users::UsersService.new } # This test is 100% bound to implementation. It's mostly a PoC for unit testing # within Gears and should not be used as an example. it 'returns the logged user based on subdomain and warden' do - user = double - user.stub(:id).and_return(id) - user.stub(:username).and_return(username) - user.stub(:email).and_return(email) - user.stub(:organization).and_return(nil) - user.stub(:feature_flags).and_return([]) + user = CartoGearsApi::Users::User.with( + id: 'b51e56fb-f3c9-463f-b950-d9be188551e5', + username: 'wadus_username', + email: 'wadus@carto.com', + organization: nil, + feature_flags: [], + can_change_email: true) warden = double warden.should_receive(:user).once.and_return(user) request = double request.should_receive(:env).once.and_return('warden' => warden) - CartoDB.should_receive(:extract_subdomain).with(request).and_return(username) + CartoDB.should_receive(:extract_subdomain).with(request).and_return(user.username) logged_user = service.logged_user(request) logged_user.email.should eq user.email