Skip to content
代码片段 群组 项目
未验证 提交 8995cfec 编辑于 作者: Dylan Griffith's avatar Dylan Griffith 提交者: GitLab
浏览文件

Merge branch...

Merge branch '474650-gerardo-navarro-step-up-auth-with-oidc-for-admin-mode-pre-work-login-helper-additional-info' into 'master' 

test: Refactor login helper to allow additional info for raw_info

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/173300



Merged-by: default avatarDylan Griffith <dyl.griffith@gmail.com>
Approved-by: default avatarShreyas Agarwal <sagarwal@gitlab.com>
Approved-by: default avatarDrew Blessing <drew@gitlab.com>
Approved-by: default avatarDylan Griffith <dyl.griffith@gmail.com>
Reviewed-by: default avatarGerardo Navarro <gerardo@b310.de>
Co-authored-by: default avatarGerardo Navarro <gerardo@b310.de>
No related branches found
No related tags found
无相关合并请求
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe Gitlab::Auth::GroupSaml::AuthHash do RSpec.describe Gitlab::Auth::GroupSaml::AuthHash, feature_category: :system_access do
let(:omniauth_auth_hash) do let(:omniauth_auth_hash) do
OmniAuth::AuthHash.new(extra: { raw_info: OneLogin::RubySaml::Attributes.new(raw_info_attr) }) OmniAuth::AuthHash.new(extra: { raw_info: OneLogin::RubySaml::Attributes.new(raw_info_attr) })
end end
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe Gitlab::Auth::Saml::User do RSpec.describe Gitlab::Auth::Saml::User, feature_category: :system_access do
include LdapHelpers include LdapHelpers
include LoginHelpers include LoginHelpers
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe Gitlab::Auth::Saml::AuthHash do RSpec.describe Gitlab::Auth::Saml::AuthHash, feature_category: :system_access do
include LoginHelpers include LoginHelpers
let(:raw_info_attr) { { 'groups' => %w[Developers Freelancers] } } let(:raw_info_attr) { { 'groups' => %w[Developers Freelancers] } }
......
...@@ -181,7 +181,7 @@ def configure_mock_auth(provider, uid, email, response_object: nil, additional_i ...@@ -181,7 +181,7 @@ def configure_mock_auth(provider, uid, email, response_object: nil, additional_i
), ),
response_object: response_object response_object: response_object
} }
}).merge(additional_info) { |_, old_hash, new_hash| old_hash.merge(new_hash) } }).merge(additional_info)
end end
def mock_auth_hash(provider, uid, email, additional_info: {}, response_object: nil, name: 'mockuser', groups: []) def mock_auth_hash(provider, uid, email, additional_info: {}, response_object: nil, name: 'mockuser', groups: [])
......
0% 加载中 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册