Skip to content

Commit

Permalink
Fixes rubocop issues
Browse files Browse the repository at this point in the history
  • Loading branch information
davidwessman authored and stanhu committed Feb 9, 2022
1 parent 4ba473a commit c4eecad
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions test/lib/omniauth/strategies/openid_connect_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -156,10 +156,11 @@ def test_option_custom_attributes

def test_request_phase_with_allowed_params
strategy.options.issuer = 'example.com'
strategy.options.allow_authorize_params = [:name, :logo, :resource]
strategy.options.extra_authorize_params = {resource: 'xyz'}
strategy.options.allow_authorize_params = %i[name logo resource]
strategy.options.extra_authorize_params = { resource: 'xyz' }
strategy.options.client_options.host = 'example.com'
request.stubs(:params).returns('name' => 'example', 'logo' => 'example_logo', 'resource' => 'abc', 'not_allowed' => 'filter_me')
request.stubs(:params).returns('name' => 'example', 'logo' => 'example_logo', 'resource' => 'abc',
'not_allowed' => 'filter_me')

assert(strategy.authorize_uri =~ /resource=xyz/, 'URI must contain fixed param resource')
assert(strategy.authorize_uri =~ /name=example/, 'URI must contain dynamic param name')
Expand Down

0 comments on commit c4eecad

Please sign in to comment.