Fix Layout/ArgumentAlignment cop

theforeman · Jul 1, 2024 · 542d163 · 542d163
1 parent c65734a
commit 542d163
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 14 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -34,13 +34,6 @@ Layout/ArgumentAlignment:
     - 'db/seeds.d/10_leapp_preupgrade.rb'
     - 'lib/foreman_leapp/engine.rb'
 
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
-# URISchemes: http, https
-Layout/LineLength:
-  Max: 134
-
 # Offense count: 1
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.

diff --git a/app/controllers/api/v2/concerns/api_authorizer.rb b/app/controllers/api/v2/concerns/api_authorizer.rb
@@ -13,8 +13,8 @@ def hosts_permission
     return if User.current.can?('view_hosts')
 
     render_error 'access_denied', status: :forbidden,
-                                  locals: { details: N_('Missing one of the required permissions: view_hosts'),
-                                            missing_permissions: 'view_hosts' }
+      locals: { details: N_('Missing one of the required permissions: view_hosts'),
+                missing_permissions: 'view_hosts' }
   end
 
   def resource_scope(_options = {})

diff --git a/config/routes.rb b/config/routes.rb
@@ -5,7 +5,7 @@
 
   namespace :api, defaults: { format: 'json' } do
     scope '(:apiv)', module: :v2, defaults: { apiv: 'v2' }, apiv: /v2/,
-          constraints: ApiConstraints.new( version: 2, default: true) do
+      constraints: ApiConstraints.new( version: 2, default: true) do
       resources :preupgrade_reports, only: %i[index show]
       get 'job_invocations/:id/preupgrade_reports', to: 'preupgrade_reports#job_invocation'
     end

diff --git a/test/helpers/template_helper_test.rb b/test/helpers/template_helper_test.rb
@@ -16,8 +16,8 @@ class TemplateHelperTest < ActionView::TestCase
 
       test 'without remediation commands' do
         empty_entry = FactoryBot.create(:preupgrade_report_entry, host: host, preupgrade_report: report,
-                                                                  detail: { remediations: [{ type: 'hint',
-                                                                                             context: 'meh.' }] })
+          detail: { remediations: [{ type: 'hint',
+                                     context: 'meh.' }] })
         template = build_remediation_plan([empty_entry.id], host)
         assert_equal template, ''
       end

diff --git a/test/unit/actions/preupgrade_job_test.rb b/test/unit/actions/preupgrade_job_test.rb
@@ -9,7 +9,7 @@ class PreupgradeJobTest < ActiveSupport::TestCase
     let(:host) { FactoryBot.create(:host) }
     let(:job_template) do
       FactoryBot.create(:job_template, template: 'echo "1"', job_category: 'leapp',
-                                       provider_type: 'SSH', name: 'Leapp preupgrade')
+        provider_type: 'SSH', name: 'Leapp preupgrade')
     end
     let(:job_invocation) { FactoryBot.create(:job_invocation) }
     let(:template_invocation) do

diff --git a/test/unit/helpers/job_helper_test.rb b/test/unit/helpers/job_helper_test.rb
@@ -8,7 +8,7 @@ class JobHelperTest < ActionView::TestCase
 
     let(:job_template) do
       FactoryBot.create(:job_template, template: 'echo "1"', job_category: 'leapp',
-                                       provider_type: 'SSH', name: 'Leapp preupgrade')
+        provider_type: 'SSH', name: 'Leapp preupgrade')
     end
     let(:job_invocation) { FactoryBot.create(:job_invocation) }