summaryrefslogtreecommitdiff
path: root/spec/lib/gitlab/oauth
diff options
context:
space:
mode:
authorJan-Willem van der Meer <mail@jewilmeer.nl>2014-09-01 16:31:27 +0200
committerJan-Willem van der Meer <mail@jewilmeer.nl>2014-09-01 16:31:27 +0200
commit26b14dd2d597d7bd5579cfcbad456abb0af6a5e5 (patch)
treeda3b5a77b009962753b7914a84889954279e844a /spec/lib/gitlab/oauth
parent92a9964940784063810d068f230088d7f297ba54 (diff)
downloadgitlab-ce-26b14dd2d597d7bd5579cfcbad456abb0af6a5e5.tar.gz
Get uid from auth instead of info hash
As found in the omniauth specs: https://github.com/intridea/omniauth/wiki/Auth-Hash-Schema
Diffstat (limited to 'spec/lib/gitlab/oauth')
-rw-r--r--spec/lib/gitlab/oauth/user_spec.rb17
1 files changed, 8 insertions, 9 deletions
diff --git a/spec/lib/gitlab/oauth/user_spec.rb b/spec/lib/gitlab/oauth/user_spec.rb
index 7dcc849454b..c241e198609 100644
--- a/spec/lib/gitlab/oauth/user_spec.rb
+++ b/spec/lib/gitlab/oauth/user_spec.rb
@@ -4,7 +4,6 @@ describe Gitlab::OAuth::User do
let(:gl_auth) { Gitlab::OAuth::User }
let(:info) do
double(
- uid: 'my-uid',
nickname: 'john',
name: 'John',
email: 'john@mail.com'
@@ -24,32 +23,32 @@ describe Gitlab::OAuth::User do
end
it "finds an existing user based on nested uid and provider" do
- auth = double(info: info, provider: 'my-provider')
+ auth = double(info: info, uid: 'my-uid', provider: 'my-provider')
assert gl_auth.find(auth)
end
end
describe :create do
it "should create user from LDAP" do
- auth = double(info: info, provider: 'ldap')
+ auth = double(info: info, uid: 'my-uid', provider: 'ldap')
user = gl_auth.create(auth)
user.should be_valid
- user.extern_uid.should == info.uid
+ user.extern_uid.should == auth.uid
user.provider.should == 'ldap'
end
it "should create user from Omniauth" do
- auth = double(info: info, provider: 'twitter')
+ auth = double(info: info, uid: 'my-uid', provider: 'twitter')
user = gl_auth.create(auth)
user.should be_valid
- user.extern_uid.should == info.uid
+ user.extern_uid.should == auth.uid
user.provider.should == 'twitter'
end
it "should apply defaults to user" do
- auth = double(info: info, provider: 'ldap')
+ auth = double(info: info, uid: 'my-uid', provider: 'ldap')
user = gl_auth.create(auth)
user.should be_valid
@@ -63,7 +62,7 @@ describe Gitlab::OAuth::User do
nickname: 'john',
name: 'John'
)
- auth = double(info: info, provider: 'my-provider')
+ auth = double(info: info, uid: 'my-uid', provider: 'my-provider')
user = gl_auth.create(auth)
expect(user.email).to_not be_empty
@@ -75,7 +74,7 @@ describe Gitlab::OAuth::User do
name: 'John',
email: 'john@example.com'
)
- auth = double(info: info, provider: 'my-provider')
+ auth = double(info: info, uid: 'my-uid', provider: 'my-provider')
user = gl_auth.create(auth)
expect(user.username).to eql 'john'