Fixes #37451 - Support Zeitwerk loader

app/controllers/ansible_roles_controller.rb

@@ -46,7 +46,7 @@ def default_order
   end
 
   def create_importer
-    @importer = ForemanAnsible::UiRolesImporter.new(@proxy)
+    @importer = ForemanAnsible::UIRolesImporter.new(@proxy)
     @variables_importer = ForemanAnsible::VariablesImporter.new(@proxy)
   end
 

app/controllers/ansible_variables_controller.rb

@@ -92,7 +92,7 @@ def import_new_roles
 
   def create_importer
     @importer = ForemanAnsible::VariablesImporter.new(@proxy)
-    @importer_roles = ForemanAnsible::UiRolesImporter.new(@proxy)
+    @importer_roles = ForemanAnsible::UIRolesImporter.new(@proxy)
   end
 
   def find_required_proxy

app/controllers/api/v2/ansible_roles_controller.rb

@@ -98,7 +98,7 @@ def find_proxy
       # rubocop:enable Layout/DotPosition
 
       def create_importer
-        @roles_importer = ForemanAnsible::UiRolesImporter.new(@proxy)
+        @roles_importer = ForemanAnsible::UIRolesImporter.new(@proxy)
         @variables_importer = ForemanAnsible::VariablesImporter.new(@proxy)
         @importer = ForemanAnsible::ApiRolesImporter.new(@proxy)
       end

app/controllers/ui_ansible_roles_controller.rb

@@ -1,4 +1,4 @@
-class UiAnsibleRolesController < ::Api::V2::BaseController
+class UIAnsibleRolesController < ::Api::V2::BaseController
   def resource_name(resource = 'AnsibleRole')
     super resource
   end

app/jobs/sync_roles_and_variables.rb

@@ -2,7 +2,7 @@ class SyncRolesAndVariables < ::ApplicationJob
   queue_as :default
 
   def perform(changed, proxy)
-    roles_importer = ForemanAnsible::UiRolesImporter.new(proxy)
+    roles_importer = ForemanAnsible::UIRolesImporter.new(proxy)
     variables_importer = ForemanAnsible::VariablesImporter.new(proxy)
     roles_importer.finish_import(changed)
     variables_importer.import_variables_roles(changed) if changed['new'] || changed['old']

app/services/foreman_ansible/api_roles_importer.rb

@@ -6,7 +6,7 @@ class ApiRolesImporter < RolesImporter
     include ::ForemanAnsible::AnsibleRolesDataPreparations
 
     def import!(role_names)
-      @roles_importer = ForemanAnsible::UiRolesImporter.new(@ansible_proxy)
+      @roles_importer = ForemanAnsible::UIRolesImporter.new(@ansible_proxy)
       @variables_importer = ForemanAnsible::VariablesImporter.new(@ansible_proxy)
       params = { 'changed' => {} }
       roles = prepare_ansible_import_rows(@roles_importer.import!, @variables_importer, false)

app/services/foreman_ansible/ui_roles_importer.rb

@@ -2,7 +2,7 @@
 
 module ForemanAnsible
   # imports ansible roles through UI
-  class UiRolesImporter < RolesImporter
+  class UIRolesImporter < RolesImporter
     def import!
       import_role_names
     end

foreman_ansible.gemspec

@@ -20,8 +20,8 @@ Gem::Specification.new do |s|
 
   s.add_dependency 'acts_as_list', '~> 1.0.3'
   s.add_dependency 'deface', '< 2.0'
-  s.add_dependency 'foreman_remote_execution', '>= 9.0', '< 14'
-  s.add_dependency 'foreman-tasks', '>= 7.0', '< 10'
+  s.add_dependency 'foreman_remote_execution', '>= 14.0', '< 15'
+  s.add_dependency 'foreman-tasks', '>= 10.0', '< 11'
 
   s.add_development_dependency 'theforeman-rubocop', '~> 0.1.0'
 end

lib/foreman_ansible/engine.rb

@@ -11,16 +11,10 @@ module ForemanAnsible
   class Engine < ::Rails::Engine
     engine_name 'foreman_ansible'
 
-    config.autoload_paths += Dir["#{config.root}/app/controllers/concerns"]
-    config.autoload_paths += Dir["#{config.root}/app/models"]
-    config.autoload_paths += Dir["#{config.root}/app/helpers"]
-    config.autoload_paths += Dir["#{config.root}/app/overrides"]
-    config.autoload_paths += Dir["#{config.root}/app/services"]
-    config.autoload_paths += Dir["#{config.root}/app/views"]
-    config.autoload_paths += Dir["#{config.root}/app/lib"]
-
-    initializer 'foreman_ansible.register_plugin', :before => :finisher_hook do
-      require 'foreman_ansible/register'
+    initializer 'foreman_ansible.register_plugin', :before => :finisher_hook do |app|
+      app.reloader.to_prepare do
+        require 'foreman_ansible/register'
+      end
     end
 
     initializer('foreman_ansible.require_dynflow',

lib/foreman_ansible/register.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Foreman::Plugin.register :foreman_ansible do
-  requires_foreman '>= 3.10.0'
+  requires_foreman '>= 3.13.0'
   register_gettext
 
   settings do

test/functional/ansible_variables_controller_test.rb

@@ -25,7 +25,7 @@ class AnsibleVariablesControllerTest < ActionController::TestCase
   test 'there are no problems when the import hash is empty' do
     ForemanAnsible::VariablesImporter.any_instance.
       expects(:import_variable_names).returns({})
-    ForemanAnsible::UiRolesImporter.any_instance.
+    ForemanAnsible::UIRolesImporter.any_instance.
       expects(:import_role_names).returns({})
 
     get :import,

test/functional/ui_ansible_roles_controller_test.rb

@@ -1,6 +1,6 @@
 require 'test_plugin_helper'
 
-class UiAnsibleRolesControllerTest < ActionController::TestCase
+class UIAnsibleRolesControllerTest < ActionController::TestCase
   setup do
     @role = FactoryBot.create(:ansible_role)
   end

test/unit/services/ui_roles_importer_test.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
 require 'test_plugin_helper'
-# unit tests for UiRolesImporter
-class UiRolesImporterTest < ActiveSupport::TestCase
+# unit tests for UIRolesImporter
+class UIRolesImporterTest < ActiveSupport::TestCase
   setup do
     changed_roles
-    @importer = ForemanAnsible::UiRolesImporter.new
+    @importer = ForemanAnsible::UIRolesImporter.new
   end
 
   test 'should create new role' do