foreman_ansible 1.5.2 → 2.0.0

data/test/functional/api/v2/hosts_controller_test.rb

@@ -4,45 +4,37 @@ module Api
   module V2
     # Tests for the extra methods to play roles on a Host
     class HostsControllerTest < ActionController::TestCase
-      include ::Dynflow::Testing
-
       setup do
-        @host1 = FactoryGirl.create(:host)
-        @host2 = FactoryGirl.create(:host)
-      end
-
-      after do
-        ::ForemanTasks::Task::DynflowTask.all.each do |task|
-          task.destroy
-          task.delete
-        end
+        @host1 = FactoryBot.create(:host)
+        @host2 = FactoryBot.create(:host)
       end
 
       test 'should return an not_found due to non-existent host_id' do
-        post :play_roles, :id => 'non-existent'
+        post :play_roles, :params => { :id => 'non-existent' }
         response = JSON.parse(@response.body)
         refute_empty response
         assert_response :not_found
       end
 
       test 'should trigger task on host' do
-        post :play_roles, :id => @host1.id
+        load File.join(ForemanAnsible::Engine.root,
+                       '/db/seeds.d/75_job_templates.rb')
+        ::JobInvocationComposer.any_instance.expects(:trigger!).returns(true)
+        targets = @host1.id
+        post :play_roles, :params => { :id => targets }
         response = JSON.parse(@response.body)
-
-        assert response['message']['foreman_tasks'].key?('id'),
-               'task id not contained in response'
-        assert_equal response['message']['host']['name'],
-                     @host1.name,
-                     'host name not contained in response'
-        assert_response :success
+        assert_job_invocation_is_ok(response, targets)
       end
 
       test 'should trigger two host tasks' do
-        post :multiple_play_roles, :id => [@host1.id, @host2.id]
-        response = JSON.parse(@response.body)
+        load File.join(ForemanAnsible::Engine.root,
+                       '/db/seeds.d/75_job_templates.rb')
+        ::JobInvocationComposer.any_instance.expects(:trigger!).returns(true)
+        targets = [@host1.id, @host2.id]
 
-        assert response['message'].length == 2, 'should trigger two tasks'
-        assert_response :success
+        post :multiple_play_roles, :params => { :id => targets }
+        response = JSON.parse(@response.body)
+        assert_job_invocation_is_ok(response, targets)
       end
     end
   end

data/test/functional/hosts_controller_test.rb

@@ -1,44 +1,45 @@
 require 'test_plugin_helper'
 require 'dynflow/testing'
 Mocha::Mock.send :include, Dynflow::Testing::Mimic
-require 'support/foreman_tasks/task'
 
 # Ensure Hosts controller can CRUD ansible roles
 class HostsControllerExtensionsTest < ActionController::TestCase
-  include Support::ForemanTasks::Task
-
   tests ::HostsController
 
-  # rubocop:disable Metrics/BlockLength
   context 'role assignment' do
     setup do
-      @role = FactoryGirl.create(:ansible_role)
+      @role = FactoryBot.create(:ansible_role)
     end
 
     test 'create a host with ansible roles' do
       host = { :name => 'foo',
                :managed => false,
                :ansible_role_ids => [@role.id] }
-      post :create, { :host => host }, set_session_user
+      post :create, :params => { :host => host }, :session => set_session_user
       assert_redirected_to host_url(assigns('host'))
       assert assigns('host').ansible_roles, [@role]
     end
 
     test 'update a host with ansible roles' do
-      host = FactoryGirl.create(:host, :managed => false)
-      post :update, { :id => host.id,
-                      :host => { :ansible_role_ids => [@role.id] } },
-           set_session_user
+      host = FactoryBot.create(:host, :managed => false)
+      post :update,
+           :params => {
+             :id => host.id,
+             :host => { :ansible_role_ids => [@role.id] }
+           },
+           :session => set_session_user
       assert_redirected_to host_url(assigns('host'))
       assert assigns('host').ansible_roles, [@role]
     end
 
     test 'delete a host with ansible roles' do
-      host = FactoryGirl.create(:host,
-                                :managed => false,
-                                :ansible_roles => [@role])
+      host = FactoryBot.create(:host,
+                               :managed => false,
+                               :ansible_roles => [@role])
       assert_include @role.hosts, host
-      delete :destroy, { :id => host.id }, set_session_user
+      delete :destroy,
+             :params => { :id => host.id },
+             :session => set_session_user
       assert_redirected_to hosts_url
       assert @role.hosts.empty?
     end
@@ -46,33 +47,36 @@ class HostsControllerExtensionsTest < ActionController::TestCase
 
   context 'playing roles' do
     setup do
-      @host = FactoryGirl.create(:host, :managed => false)
+      @host = FactoryBot.create(:host, :managed => false)
     end
 
     test 'redirect to task if successful' do
-      task_stub = new_task_stub(@host)
+      load File.join(ForemanAnsible::Engine.root,
+                     '/db/seeds.d/75_job_templates.rb')
+
       task_to_redirect = ForemanTasks::Task.new
-      task_stub.stubs(:to_model).returns(task_to_redirect)
       task_to_redirect.stubs(:persisted?).returns(true)
       task_to_redirect.stubs(:id).returns(1)
-      get :play_roles, { :id => @host.id }, set_session_user
+      # We don't actually want to run the job, so trigger juts returns true
+      ::JobInvocationComposer.any_instance.expects(:trigger).returns(true)
+      get :play_roles,
+          :params => { :id => @host.id },
+          :session => set_session_user
+      assert_response :redirect
+      assert_redirected_to(:controller => 'job_invocations',
+                           :action => 'show',
+                           :id => JobInvocation.last.id)
+      assert flash['error'].empty?
     end
 
     test 'shows errors when not successful' do
-      HostsController.any_instance.expects(:async_task).
+      HostsController.any_instance.expects(:job_composer).
         raises(::Foreman::Exception.new('Oh foo'))
-      get :play_roles, { :id => @host.id }, set_session_user
+      get :play_roles,
+          :params => { :id => @host.id },
+          :session => set_session_user
       assert flash[:error].present?
       assert_redirected_to host_path(@host)
     end
   end
-
-  private
-
-  def new_task_stub(host)
-    assert_async_task(::Actions::ForemanAnsible::PlayHostRoles,
-                      host) do |action_host|
-      assert_equal host, action_host
-    end
-  end
 end

data/test/test_plugin_helper.rb

@@ -1,4 +1,5 @@
 require 'test_helper'
+require 'facets'
 
 def sample_facts_file
   File.read(
@@ -12,6 +13,15 @@ def facts_json
   HashWithIndifferentAccess.new(JSON.parse(sample_facts_file))
 end
 
+def assert_job_invocation_is_ok(response, targets)
+  as_admin do
+    targeting = JobInvocation.find(response['id']).targeting
+    targeting.resolve_hosts!
+    assert_equal [targets].flatten.sort, targeting.hosts.map(&:id).sort
+  end
+  assert_equal 'Ansible Playbook', response['job_category']
+  assert_response :created
+end
 plugin_factories_path = File.join(File.dirname(__FILE__), 'factories')
-FactoryGirl.definition_file_paths << plugin_factories_path
-FactoryGirl.reload
+FactoryBot.definition_file_paths << plugin_factories_path
+FactoryBot.reload

data/test/unit/concerns/host_managed_extensions_test.rb

@@ -3,14 +3,14 @@ require 'test_plugin_helper'
 # Tests for the behavior of Host with roles, checks inheritance, etc
 class HostManagedExtensionsTest < ActiveSupport::TestCase
   before do
-    @role1 = FactoryGirl.create(:ansible_role)
-    @role2 = FactoryGirl.create(:ansible_role)
-    @role3 = FactoryGirl.create(:ansible_role)
+    @role1 = FactoryBot.create(:ansible_role)
+    @role2 = FactoryBot.create(:ansible_role)
+    @role3 = FactoryBot.create(:ansible_role)
 
-    @hostgroup_parent = FactoryGirl.create(:hostgroup,
-                                           :ansible_roles => [@role2])
-    @hostgroup = FactoryGirl.create(:hostgroup, :parent => @hostgroup_parent)
-    @host = FactoryGirl.build_stubbed(:host, :ansible_roles => [@role1])
+    @hostgroup_parent = FactoryBot.create(:hostgroup,
+                                          :ansible_roles => [@role2])
+    @hostgroup = FactoryBot.create(:hostgroup, :parent => @hostgroup_parent)
+    @host = FactoryBot.build_stubbed(:host, :ansible_roles => [@role1])
   end
 
   describe '#all_ansible_roles' do

data/test/unit/concerns/hostgroup_extensions_test.rb

@@ -3,13 +3,13 @@ require 'test_plugin_helper'
 # Tests for the behavior of Hostgroup with roles, checks inheritance, etc
 class HostgroupExtensionsTest < ActiveSupport::TestCase
   before do
-    @role1 = FactoryGirl.create(:ansible_role)
-    @role2 = FactoryGirl.create(:ansible_role)
-    @role3 = FactoryGirl.create(:ansible_role)
+    @role1 = FactoryBot.create(:ansible_role)
+    @role2 = FactoryBot.create(:ansible_role)
+    @role3 = FactoryBot.create(:ansible_role)
 
-    @hostgroup_parent = FactoryGirl.create(:hostgroup,
-                                           :ansible_roles => [@role2])
-    @hostgroup = FactoryGirl.create(:hostgroup, :ansible_roles => [@role1])
+    @hostgroup_parent = FactoryBot.create(:hostgroup,
+                                          :ansible_roles => [@role2])
+    @hostgroup = FactoryBot.create(:hostgroup, :ansible_roles => [@role1])
   end
 
   describe '#all_ansible_roles' do

data/test/unit/helpers/foreman_ansible/ansible_reports_helper_test.rb

@@ -11,11 +11,11 @@ module ForemanAnsible
       end
 
       test 'ansible_report? returns false when log is from Puppet' do
-        refute ansible_report?(FactoryGirl.build(:log))
+        refute ansible_report?(FactoryBot.build(:log))
       end
 
       test 'ansible_report? returns true when log contains module_name' do
-        log = FactoryGirl.build(:log)
+        log = FactoryBot.build(:log)
         log.source.value = '{ "module_name" : "foo" }'
         assert ansible_report?(log)
       end

data/test/unit/host_ansible_role_test.rb

@@ -9,8 +9,8 @@ class HostAnsibleRoleTest < ActiveSupport::TestCase
 
   describe 'uniqueness' do
     subject do
-      HostAnsibleRole.new(:host => FactoryGirl.build(:host),
-                          :ansible_role => FactoryGirl.build(:ansible_role))
+      HostAnsibleRole.new(:host => FactoryBot.build(:host),
+                          :ansible_role => FactoryBot.build(:ansible_role))
     end
     should validate_uniqueness_of(:ansible_role_id).scoped_to(:host_id)
   end

data/test/unit/hostgroup_ansible_role_test.rb

@@ -10,8 +10,8 @@ class HostgroupAnsibleRoleTest < ActiveSupport::TestCase
   describe 'uniqueness' do
     subject do
       HostgroupAnsibleRole.new(
-        :hostgroup => FactoryGirl.build(:hostgroup),
-        :ansible_role => FactoryGirl.build(:ansible_role)
+        :hostgroup => FactoryBot.build(:hostgroup),
+        :ansible_role => FactoryBot.build(:ansible_role)
       )
     end
     should validate_uniqueness_of(:ansible_role_id).scoped_to(:hostgroup_id)

data/test/unit/lib/foreman_ansible_core/roles_reader_test.rb

@@ -29,16 +29,14 @@ class RolesReaderTest < ActiveSupport::TestCase
   describe '#list_roles' do
     test 'reads roles from paths' do
       expect_content_config ["roles_path = #{ROLES_PATH}"]
-      ForemanAnsibleCore::RolesReader.expects(:read_roles)
-                                     .with(ROLES_PATH)
+      ForemanAnsibleCore::RolesReader.expects(:read_roles).with(ROLES_PATH)
       ForemanAnsibleCore::RolesReader.list_roles
     end
 
     test 'reads roles from paths' do
       roles_paths = ['/mycustom/roles/path', '/another/path']
       roles_paths.each do |path|
-        ForemanAnsibleCore::RolesReader.expects(:read_roles)
-                                       .with(path)
+        ForemanAnsibleCore::RolesReader.expects(:read_roles).with(path)
       end
       expect_content_config ["roles_path = #{roles_paths.join(':')}"]
       ForemanAnsibleCore::RolesReader.list_roles
@@ -88,6 +86,7 @@ class RolesReaderTest < ActiveSupport::TestCase
   private
 
   def expect_content_config(ansible_cfg_content)
-    ForemanAnsibleCore::RolesReader.expects(:roles_path_from_config).returns(ansible_cfg_content)
+    ForemanAnsibleCore::RolesReader.expects(:roles_path_from_config).
+      returns(ansible_cfg_content)
   end
 end

data/test/unit/services/fact_importer_test.rb

@@ -1,11 +1,9 @@
-require 'test_plugin_helper'
-
 module ForemanAnsible
   # Test for the facts importer - only verify that given
   # a set of facts it's able to import them
   class FactImporterTest < ActiveSupport::TestCase
     setup do
-      @host = FactoryGirl.build(:host)
+      @host = FactoryBot.build(:host)
     end
 
     test 'add new facts adds all fact names in the fixture' do
@@ -28,13 +26,13 @@ module ForemanAnsible
         @fact_importer = FactImporter.new(@host, :ansible_facts => {})
       end
       test 'does not add existing facts' do
-        existing_fact = FactoryGirl.build(:fact_name)
+        existing_fact = FactoryBot.build(:fact_name)
         @fact_importer.expects(:missing_facts).returns([])
         assert_nil @fact_importer.send(:add_fact_value, nil, existing_fact)
       end
 
       test 'adds missing facts' do
-        missing_fact = FactoryGirl.build(:fact_name)
+        missing_fact = FactoryBot.build(:fact_name)
         @fact_importer.expects(:missing_facts).returns([missing_fact.name])
         @fact_importer.counters[:added] = 0
         assert_difference('@host.fact_values.count', 1) do

data/test/unit/services/fact_parser_test.rb

@@ -1,5 +1,3 @@
-require 'test_plugin_helper'
-
 module ForemanAnsible
   # Checks sample Ansible facts to see if it can assign them to
   # Host properties

data/test/unit/services/fact_sparser_test.rb

@@ -1,5 +1,3 @@
-require 'test_plugin_helper'
-
 module ForemanAnsible
   # Tests for checking if FactSparser can sparse a hash and unsparse it
   class FactSparserTest < ActiveSupport::TestCase

data/test/unit/services/inventory_creator_test.rb

@@ -5,7 +5,7 @@ module ForemanAnsible
   # inventory variables and connection options
   class InventoryCreatorTest < ActiveSupport::TestCase
     setup do
-      @host = FactoryGirl.build(:host)
+      @host = FactoryBot.build(:host)
     end
 
     test 'ansible_ parameters get turned into host variables' do

data/test/unit/services/proxy_selector_test.rb

@@ -3,7 +3,7 @@ require 'test_plugin_helper'
 # Tests for the Proxy Selector service
 class ProxySelectorTest < ActiveSupport::TestCase
   setup do
-    @host = FactoryGirl.create(:host)
+    @host = FactoryBot.create(:host)
   end
 
   test 'reuses proxies already assigned to host' do
@@ -17,10 +17,10 @@ class ProxySelectorTest < ActiveSupport::TestCase
 
   test 'only finds proxies that are within host taxonomies' do
     @host.organization = taxonomies(:organization1)
-    @ansible_proxy = FactoryGirl.create(:smart_proxy, :with_ansible,
-                                        :organizations => [@host.organization])
+    @ansible_proxy = FactoryBot.create(:smart_proxy, :with_ansible,
+                                       :organizations => [@host.organization])
     # Unreachable proxy, because of the organizations mismatch with Host
-    FactoryGirl.create(:smart_proxy, :with_ansible, :organizations => [])
+    FactoryBot.create(:smart_proxy, :with_ansible, :organizations => [])
     proxy_selector = ForemanAnsible::ProxySelector.new
     setup_user('view', 'smart_proxies')
     assert_equal [@ansible_proxy],

data/test/unit/services/roles_importer_test.rb

@@ -2,12 +2,12 @@ require 'test_plugin_helper'
 # unit tests for RolesImporter
 class RolesImporterTest < ActiveSupport::TestCase
   setup do
-    @role = FactoryGirl.create(:ansible_role)
+    @role = FactoryBot.create(:ansible_role)
     @importer = ForemanAnsible::RolesImporter.new
   end
 
   test 'should detect changes when importing roles' do
-    role = FactoryGirl.build(:ansible_role)
+    role = FactoryBot.build(:ansible_role)
     changes = @importer.detect_changes [role]
     assert_equal 1, changes[:new].length
     assert_equal 1, changes[:obsolete].length

data/test/unit/services/structured_fact_importer_test.rb

@@ -14,7 +14,7 @@ module ForemanAnsible
     end
 
     test 'if host is in Foreman, use hostname provided by Ansible' do
-      ansible_fqdn_host = FactoryGirl.build(:host)
+      ansible_fqdn_host = FactoryBot.build(:host)
       ansible_fqdn_host.name = facts_json[:ansible_facts][:ansible_fqdn]
       ansible_fqdn_host.save
       importer = ForemanAnsible::StructuredFactImporter.new(

data/test/unit/services/ui_roles_importer_test.rb

@@ -21,7 +21,7 @@ class UiRolesImporterTest < ActiveSupport::TestCase
   private
 
   def changed_roles
-    @role = FactoryGirl.create(:ansible_role)
+    @role = FactoryBot.create(:ansible_role)
     new_role_name = 'test_role.foreman'
     @new_role = { :id => nil, :name => new_role_name }
     @changes = { 'new' => { 'test_role.foreman' => @new_role.to_json },

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 1.5.2
+  version: 2.0.0
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-12 00:00:00.000000000 Z
+date: 2018-02-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.42'
+        version: '0.52'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.42'
+        version: '0.52'
 - !ruby/object:Gem::Dependency
   name: deface
   requirement: !ruby/object:Gem::Requirement
@@ -38,6 +38,20 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: dynflow
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.8.14
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.8.14
 - !ruby/object:Gem::Dependency
   name: foreman-tasks
   requirement: !ruby/object:Gem::Requirement
@@ -45,9 +59,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -55,37 +66,40 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.12'
 - !ruby/object:Gem::Dependency
-  name: dynflow
+  name: foreman_ansible_core
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.14
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.14
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: foreman_ansible_core
+  name: foreman_remote_execution
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.2
+        version: 1.4.4
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.2
+        version: 1.4.4
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 description: Ansible integration with Foreman
 email:
 - elobatocs@gmail.com
@@ -103,20 +117,19 @@ files:
 - app/controllers/foreman_ansible/api/v2/hosts_controller_extensions.rb
 - app/controllers/foreman_ansible/concerns/hostgroups_controller_extensions.rb
 - app/controllers/foreman_ansible/concerns/hosts_controller_extensions.rb
+- app/controllers/foreman_ansible/concerns/job_invocation_helper.rb
 - app/helpers/foreman_ansible/ansible_plugin_helper.rb
 - app/helpers/foreman_ansible/ansible_reports_helper.rb
 - app/helpers/foreman_ansible/ansible_roles_helper.rb
 - app/helpers/foreman_ansible/hosts_helper_extensions.rb
 - app/lib/actions/foreman_ansible/helpers/host_common.rb
 - app/lib/actions/foreman_ansible/helpers/play_roles_description.rb
-- app/lib/actions/foreman_ansible/play_host_roles.rb
-- app/lib/actions/foreman_ansible/play_hostgroup_roles.rb
-- app/lib/actions/foreman_ansible/play_hosts_roles.rb
 - app/lib/proxy_api/ansible.rb
 - app/models/ansible_role.rb
 - app/models/concerns/foreman_ansible/has_many_ansible_roles.rb
 - app/models/concerns/foreman_ansible/host_managed_extensions.rb
 - app/models/concerns/foreman_ansible/hostgroup_extensions.rb
+- app/models/foreman_ansible/ansible_provider.rb
 - app/models/foreman_ansible/fact_name.rb
 - app/models/host_ansible_role.rb
 - app/models/hostgroup_ansible_role.rb
@@ -151,6 +164,7 @@ files:
 - app/views/foreman_ansible/api/v2/ansible_roles/show.json.rabl
 - app/views/foreman_ansible/config_reports/_ansible.html.erb
 - app/views/foreman_ansible/config_reports/_output.html.erb
+- app/views/foreman_ansible/job_templates/ansible_roles.erb
 - config/routes.rb
 - db/migrate/20160705082036_create_ansible_role.rb
 - db/migrate/20160706074540_create_join_table_hosts_ansible_roles.rb
@@ -160,8 +174,11 @@ files:
 - db/migrate/20160805094233_add_primary_key_hostgroup_ansible_roles.rb
 - db/migrate/20161122154057_automatically_set_role_timestamps.rb
 - db/seeds.d/62_ansible_proxy_feature.rb
+- db/seeds.d/75_job_templates.rb
 - lib/foreman_ansible.rb
 - lib/foreman_ansible/engine.rb
+- lib/foreman_ansible/register.rb
+- lib/foreman_ansible/remote_execution.rb
 - lib/foreman_ansible/version.rb
 - locale/Makefile
 - locale/en/foreman_ansible.po
@@ -169,17 +186,15 @@ files:
 - locale/gemspec.rb
 - test/factories/ansible_proxy.rb
 - test/factories/ansible_roles.rb
-- test/fixtures/ansible_permissions.yml
 - test/fixtures/sample_facts.json
 - test/functional/ansible_roles_controller_test.rb
 - test/functional/api/v2/ansible_roles_controller_test.rb
 - test/functional/api/v2/hostgroups_controller_test.rb
 - test/functional/api/v2/hosts_controller_test.rb
 - test/functional/hosts_controller_test.rb
-- test/support/fixture_support.rb
-- test/support/foreman_tasks/task.rb
-- test/support/foreman_test_helper_additions.rb
 - test/test_plugin_helper.rb
+- test/unit/actions/run_ansible_job_test.rb
+- test/unit/actions/run_proxy_ansible_command_test.rb
 - test/unit/ansible_role_test.rb
 - test/unit/concerns/host_managed_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
@@ -233,24 +248,22 @@ test_files:
 - test/unit/ansible_role_test.rb
 - test/unit/concerns/host_managed_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
+- test/unit/helpers/foreman_ansible/ansible_reports_helper_test.rb
 - test/unit/lib/foreman_ansible_core/playbook_runner_test.rb
 - test/unit/lib/foreman_ansible_core/roles_reader_test.rb
 - test/unit/lib/proxy_api/ansible_test.rb
 - test/unit/services/api_roles_importer_test.rb
+- test/unit/services/inventory_creator_test.rb
 - test/unit/services/roles_importer_test.rb
-- test/unit/services/structured_fact_importer_test.rb
 - test/unit/services/ui_roles_importer_test.rb
 - test/unit/services/fact_importer_test.rb
-- test/unit/services/fact_parser_test.rb
 - test/unit/services/fact_sparser_test.rb
-- test/unit/services/inventory_creator_test.rb
 - test/unit/services/proxy_selector_test.rb
-- test/unit/helpers/foreman_ansible/ansible_reports_helper_test.rb
+- test/unit/services/structured_fact_importer_test.rb
+- test/unit/services/fact_parser_test.rb
 - test/unit/host_ansible_role_test.rb
 - test/unit/hostgroup_ansible_role_test.rb
+- test/unit/actions/run_ansible_job_test.rb
+- test/unit/actions/run_proxy_ansible_command_test.rb
 - test/fixtures/sample_facts.json
-- test/fixtures/ansible_permissions.yml
-- test/support/fixture_support.rb
-- test/support/foreman_tasks/task.rb
-- test/support/foreman_test_helper_additions.rb
 - test/test_plugin_helper.rb