From: Toshi MARUYAMA Date: Sat, 23 Nov 2019 15:12:35 +0000 (+0000) Subject: cleanup: rubocop: fix Layout/IndentationConsistency in test/mocks/open_id_authenticat... X-Git-Tag: 4.2.0~1420 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=caa88a2ed308224edc66f411b1466a7e622994c3;p=redmine.git cleanup: rubocop: fix Layout/IndentationConsistency in test/mocks/open_id_authentication_mock.rb git-svn-id: http://svn.redmine.org/redmine/trunk@19220 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index c8ea9ab15..b3ee67d06 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -171,13 +171,6 @@ Layout/IndentFirstArrayElement: Layout/IndentFirstHashElement: Enabled: false -# Cop supports --auto-correct. -# Configuration parameters: EnforcedStyle. -# SupportedStyles: normal, indented_internal_methods -Layout/IndentationConsistency: - Exclude: - - 'test/mocks/open_id_authentication_mock.rb' - # Cop supports --auto-correct. # Configuration parameters: Width, IgnoredPatterns. Layout/IndentationWidth: diff --git a/test/mocks/open_id_authentication_mock.rb b/test/mocks/open_id_authentication_mock.rb index d2c04c4bc..dcc8ed441 100644 --- a/test/mocks/open_id_authentication_mock.rb +++ b/test/mocks/open_id_authentication_mock.rb @@ -15,7 +15,7 @@ module OpenIdAuthentication protected - def authenticate_with_open_id(identity_url = params[:openid_url], options = {}) + def authenticate_with_open_id(identity_url = params[:openid_url], options = {}) if User.find_by_identity_url(identity_url) || identity_url.include?('good') extension_response_fields = {} @@ -38,12 +38,12 @@ module OpenIdAuthentication logger.info "OpenID authentication failed: #{identity_url}" yield Result[:failed], identity_url, nil end - end + end private - def add_simple_registration_fields(open_id_response, fields) + def add_simple_registration_fields(open_id_response, fields) open_id_response.add_extension_arg('sreg', 'required', [ fields[:required] ].flatten * ',') if fields[:required] open_id_response.add_extension_arg('sreg', 'optional', [ fields[:optional] ].flatten * ',') if fields[:optional] - end + end end