rubocop --auto-correct

config/routes.rb

@@ -41,7 +41,7 @@
     end
 
     scope '/r/insights' do
-      match '(/*path)(/)', :constraints => lambda { |req| !req.path.include?('view/api') }, to: 'machine_telemetries#forward_request', via: [:get, :post, :delete,:put, :patch]
+      match '(/*path)(/)', :constraints => ->(req) { !req.path.include?('view/api') }, :to => 'machine_telemetries#forward_request', :via => [:get, :post, :delete, :put, :patch]
     end
   end
 

db/seeds.d/179_ui_notifications.rb

@@ -8,4 +8,3 @@
 ]
 
 blueprints.each { |blueprint| UINotifications::Seed.new(blueprint).configure }
-

db/seeds.d/50_job_templates.rb

@@ -4,11 +4,11 @@
   JobTemplate.without_auditing do
     Dir[File.join("#{ForemanRhCloud::Engine.root}/app/views/job_templates/**/*.erb")].each do |template|
       template = JobTemplate.import_raw!(File.read(template),
-      :default => true,
-      :lock => true,
-      :update => true)
+        :default => true,
+        :lock => true,
+        :update => true)
       template.organizations = organizations if template.present?
       template.locations = locations if template.present?
     end
   end
-end
+end

foreman_rh_cloud.gemspec

@@ -18,9 +18,9 @@ Gem::Specification.new do |s|
   s.test_files = Dir['test/**/*']
   s.test_files -= Dir['test/**/*.orig']
 
-  s.add_dependency 'katello'
   s.add_dependency 'foreman_ansible'
   s.add_dependency 'foreman-tasks'
+  s.add_dependency 'katello'
 
   s.add_development_dependency 'rdoc'
   s.add_development_dependency 'rubocop'

lib/tasks/hybrid_cloud.rake

@@ -7,7 +7,7 @@ namespace :rh_cloud do |args|
   task hybridcloud_register: [:environment] do
     include ::ForemanRhCloud::CertAuth
     include ::InsightsCloud::CandlepinCache
-    
+
     def logger
       @logger ||= Logger.new(STDOUT)
     end
@@ -21,7 +21,7 @@ namespace :rh_cloud do |args|
       logger.error('ERROR: org_id needs to be specified.')
       exit(1)
     end
-    
+
     @organization = Organization.find_by(id: ENV['org_id'].to_i) # saw this coming in as a string, so making sure it gets passed as an integer.
     @uid = cp_owner_id(@organization)
     @hostname = ForemanRhCloud.foreman_host_name
@@ -33,14 +33,14 @@ namespace :rh_cloud do |args|
 
     def headers
       {
-        Authorization: "Bearer #{@token}"
+        Authorization: "Bearer #{@token}",
       }
     end
 
     def payload
       {
         "uid": @uid,
-        "display_name": "#{@hostname}+#{@organization.label}"
+        "display_name": "#{@hostname}+#{@organization.label}",
       }
     end
 

lib/tasks/rh_cloud_inventory.rake

@@ -4,10 +4,10 @@ namespace :rh_cloud_inventory do
   namespace :report do
     desc 'Generate inventory report and send it to Red Hat cloud'
     task generate_upload: [:environment, 'dynflow:client'] do
-      unless ENV['organization_id'].nil?
-        organizations = [ Organization.where(:id => ENV['organization_id']).first ]
-      else
+      if ENV['organization_id'].nil?
         organizations = Organization.unscoped.all
+      else
+        organizations = [Organization.where(:id => ENV['organization_id']).first]
       end
 
       User.as_anonymous_admin do
@@ -55,8 +55,8 @@ namespace :rh_cloud_inventory do
 
   desc "Synchronize Hosts inventory"
   task sync: [:environment, 'dynflow:client'] do
-    if ! ENV['organization_id'].nil?
-      organizations = [ Organization.where(:id => ENV['organization_id']).first ]
+    if !ENV['organization_id'].nil?
+      organizations = [Organization.where(:id => ENV['organization_id']).first]
     else
       organizations = Organization.all
     end