Commit 718370d2 authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'issue_344279_omniauth_saml_spec' into 'master'

Fix Style/OpenStructUse offenses omniauth_saml_spec

See merge request gitlab-org/gitlab!75215
parents ac06692e 4b68ce23
...@@ -18,7 +18,6 @@ Style/OpenStructUse: ...@@ -18,7 +18,6 @@ Style/OpenStructUse:
- spec/controllers/import/fogbugz_controller_spec.rb - spec/controllers/import/fogbugz_controller_spec.rb
- spec/controllers/import/gitlab_controller_spec.rb - spec/controllers/import/gitlab_controller_spec.rb
- spec/controllers/projects/clusters_controller_spec.rb - spec/controllers/projects/clusters_controller_spec.rb
- spec/dependencies/omniauth_saml_spec.rb
- spec/factories/go_module_versions.rb - spec/factories/go_module_versions.rb
- spec/factories/wiki_pages.rb - spec/factories/wiki_pages.rb
- spec/features/projects/clusters_spec.rb - spec/features/projects/clusters_spec.rb
......
...@@ -7,7 +7,7 @@ RSpec.describe 'processing of SAMLResponse in dependencies' do ...@@ -7,7 +7,7 @@ RSpec.describe 'processing of SAMLResponse in dependencies' do
let(:mock_saml_response) { File.read('spec/fixtures/authentication/saml_response.xml') } let(:mock_saml_response) { File.read('spec/fixtures/authentication/saml_response.xml') }
let(:saml_strategy) { OmniAuth::Strategies::SAML.new({}) } let(:saml_strategy) { OmniAuth::Strategies::SAML.new({}) }
let(:session_mock) { {} } let(:session_mock) { {} }
let(:settings) { OpenStruct.new({ soft: false, idp_cert_fingerprint: 'something' }) } let(:settings) { double('settings', { soft: false, idp_cert_fingerprint: 'something' }) }
let(:auth_hash) { Gitlab::Auth::Saml::AuthHash.new(saml_strategy) } let(:auth_hash) { Gitlab::Auth::Saml::AuthHash.new(saml_strategy) }
subject { auth_hash.authn_context } subject { auth_hash.authn_context }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment