Merge pull request #17 from niftyn8/gender

Include gender in info
jjbohn · May 1, 2014 · 88ba033 · 88ba033
2 parents 0f1842b + 95c6b4e
commit 88ba033
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/lib/omniauth/strategies/openid_connect.rb b/lib/omniauth/strategies/openid_connect.rb
@@ -40,6 +40,7 @@ class OpenIDConnect
           nickname: user_info.preferred_username,
           first_name: user_info.given_name,
           last_name: user_info.family_name,
+          gender: user_info.gender,
           image: user_info.picture,
           phone: user_info.phone_number,
           urls: { website: user_info.website }

diff --git a/test/lib/omniauth/strategies/openid_connect_test.rb b/test/lib/omniauth/strategies/openid_connect_test.rb
@@ -41,6 +41,7 @@ def test_info
     assert_equal user_info.preferred_username, info[:nickname]
     assert_equal user_info.given_name, info[:first_name]
     assert_equal user_info.family_name, info[:last_name]
+    assert_equal user_info.gender, info[:gender]
     assert_equal user_info.picture, info[:image]
     assert_equal user_info.phone_number, info[:phone]
     assert_equal({ website: user_info.website }, info[:urls])

diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -37,6 +37,7 @@ def user_info
       preferred_username: Faker::Internet.user_name,
       given_name: Faker::Name.first_name,
       family_name: Faker::Name.last_name,
+      gender: 'female',
       picture: Faker::Internet.url + ".png",
       phone_number: Faker::PhoneNumber.phone_number,
       website: Faker::Internet.url,