foreman_ansible 7.0.2 → 7.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 299049c29764a0920259664c06288f37ea56d4b920e049243baeab9ad7f2e608
-  data.tar.gz: 779264101df1f9fa6e0188c8673a435b98eb4dbd9fd814e754f991247b5bb250
+  metadata.gz: 7a8c4f263992d803c31386bc301ce1e032c680ada845db8bf07e71162c003384
+  data.tar.gz: 628c7b6952499ef5dfe912370bec8d3e936926a004acf5b7693d7706fe461ea9
 SHA512:
-  metadata.gz: '08a1b935eb0c033ff08674e4922ee597d39ed602ef351b82d9ac763af9d850e5a32ff9ea482af7f74a641552ebaf62094b6fa59170cba317b5bba67949d31a8d'
-  data.tar.gz: 3f814381b2167e370757f761a0b5d3e1217e3c3ba4293d7f1ecf1e77599c232ef8c62beae293c8593d81ab4e305ae1cced983527bc6d29e248bbda3a3f4e731e
+  metadata.gz: '09f3fd7b961edcb7abee1f6dd6de4fbf0bb80130668c39724d6c8658c2ca1570109ab6b7d172d2f9b963c6e4e5aebaa3280ebf199f9fe5ff86580ab1d8f06c7e'
+  data.tar.gz: 924b85ee81562a848c07417fd0788520c000292901f63ef45f8adbfc99ec32dae21858589c83688f461d19e49a4c6491801d53f611032a1c291ba941e9068133

data/app/models/foreman_ansible/ansible_provider.rb

@@ -115,7 +115,8 @@ if defined? ForemanRemoteExecution
         end
 
         def proxy_batch_size
-          Setting['foreman_ansible_proxy_batch_size']
+          value = Setting['foreman_ansible_proxy_batch_size']
+          value.presence && value.to_i
         end
 
         private

data/lib/foreman_ansible/version.rb

@@ -4,5 +4,5 @@
 # This way other parts of Foreman can just call ForemanAnsible::VERSION
 # and detect what version the plugin is running.
 module ForemanAnsible
-  VERSION = '7.0.2'
+  VERSION = '7.0.3'
 end

data/test/unit/ansible_provider_test.rb

@@ -47,4 +47,16 @@ class AnsibleProviderTest < ActiveSupport::TestCase
         proxy_command_options(template_invocation, dummyhost)
     end
   end
+
+  describe '#proxy_batch_size' do
+    it 'returns integer if setting is string' do
+      Setting.expects(:[]).with('foreman_ansible_proxy_batch_size').returns('10')
+      _(ForemanAnsible::AnsibleProvider.proxy_batch_size).must_equal(10)
+    end
+
+    it 'returns nil if setting is empty' do
+      Setting.expects(:[]).with('foreman_ansible_proxy_batch_size').returns('')
+      _(ForemanAnsible::AnsibleProvider.proxy_batch_size).must_equal(nil)
+    end
+  end
 end

data/webpack/components/AnsibleHostDetail/components/JobsTab/JobsTabHelper.js

@@ -7,21 +7,36 @@ import { showToast } from '../../../../toastHelper';
 export const ansiblePurpose = (resourceName, resourceId) =>
   `ansible-${resourceName}-${resourceId}`;
 
-const jobSearch = (resourceName, resourceId, statusSearch) =>
-  `recurring = true && pattern_template_name = "Ansible Roles - Ansible Default" && ${statusSearch} && recurring_logic.purpose = ${ansiblePurpose(
+const jobSearch = (resourceName, resourceId, status, hostGroupId) => {
+  const search = `recurring = true && pattern_template_name = "Ansible Roles - Ansible Default"`;
+  const searchStatus = ` && ${status}`;
+  const searchHost = ` && recurring_logic.purpose = ${ansiblePurpose(
     resourceName,
     resourceId
   )}`;
+  const searchHostGroup = hostGroupId
+    ? ` or recurring_logic.purpose = ${ansiblePurpose(
+        'hostgroup',
+        hostGroupId
+      )}`
+    : '';
 
-export const scheduledJobsSearch = (resourceName, resourceId) =>
-  jobSearch(resourceName, resourceId, 'status = queued');
-export const previousJobsSearch = (resourceName, resourceId) =>
-  jobSearch(resourceName, resourceId, 'status != queued');
+  return search + searchStatus + searchHost + searchHostGroup;
+};
+
+export const scheduledJobsSearch = (resourceName, resourceId, hostGroupId) =>
+  jobSearch(resourceName, resourceId, 'status = queued', hostGroupId);
+export const previousJobsSearch = (resourceName, resourceId, hostGroupId) =>
+  jobSearch(resourceName, resourceId, 'status != queued', hostGroupId);
 
 const fetchJobsFn = (searchFn, pagination = {}) => componentProps =>
   useQuery(jobsQuery, {
     variables: {
-      search: searchFn(componentProps.resourceName, componentProps.resourceId),
+      search: searchFn(
+        componentProps.resourceName,
+        componentProps.resourceId,
+        componentProps.hostGroupId
+      ),
       ...pagination,
     },
   });
@@ -77,3 +92,30 @@ export const useCancelMutation = (resourceName, resourceId) =>
       },
     ],
   });
+
+export const readableCron = (cron = '') => {
+  if (cron.match(/(\d+ \* \* \* \*)/)) {
+    return 'hourly';
+  }
+
+  if (cron.match(/(\d+ \d+ \* \* \*)/)) {
+    return 'daily';
+  }
+
+  if (cron.match(/(\d+ \d+ \* \* \d+)/)) {
+    return 'weekly';
+  }
+
+  if (cron.match(/(\d+ \d+ \d+ \* \*)/)) {
+    return 'monthly';
+  }
+
+  return 'custom';
+};
+
+export const readablePurpose = (purpose = '') => {
+  if (window.location.href.match(/ansible\/hostgroup/)) {
+    return '';
+  }
+  return purpose.match(/hostgroup/) ? __('(from host group)') : '';
+};

data/webpack/components/AnsibleHostDetail/components/JobsTab/PreviousJobsTable.js

@@ -17,6 +17,7 @@ import { Flex, FlexItem, Pagination } from '@patternfly/react-core';
 
 import { decodeId } from '../../../../globalIdHelper';
 import withLoading from '../../../withLoading';
+import { readableCron, readablePurpose } from './JobsTabHelper';
 import {
   preparePerPageOptions,
   refreshPage,
@@ -78,13 +79,15 @@ const PreviousJobsTable = ({ history, totalCount, jobs, pagination }) => {
                 >
                   {job.description}
                 </a>
+                &nbsp;
+                {readablePurpose(job.recurringLogic.purpose)}
               </Td>
               <Td>{job.task.result}</Td>
               <Td>{job.task.state}</Td>
               <Td>
                 <RelativeDateTime date={job.startAt} />
               </Td>
-              <Td>{job.recurringLogic.cronLine}</Td>
+              <Td>{readableCron(job.recurringLogic.cronLine)}</Td>
             </Tr>
           ))}
         </Tbody>

data/webpack/components/AnsibleHostDetail/components/JobsTab/PreviousJobsTable.js.orig

@@ -0,0 +1,151 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { translate as __ } from 'foremanReact/common/I18n';
+import { usePaginationOptions } from 'foremanReact/components/Pagination/PaginationHooks';
+
+import RelativeDateTime from 'foremanReact/components/common/dates/RelativeDateTime';
+
+import {
+  TableComposable,
+  Thead,
+  Tbody,
+  Tr,
+  Th,
+  Td,
+} from '@patternfly/react-table';
+import { Flex, FlexItem, Pagination } from '@patternfly/react-core';
+
+import { decodeId } from '../../../../globalIdHelper';
+import withLoading from '../../../withLoading';
+<<<<<<< HEAD
+import { readableCron } from './JobsTabHelper';
+import {
+  preparePerPageOptions,
+  refreshPage,
+} from '../../../../helpers/paginationHelper';
+=======
+import { readableCron, readablePurpose } from './JobsTabHelper';
+>>>>>>> 5b01704 (Fixes #34458 - Show Hostgroup jobs on the Host Detail page)
+
+const PreviousJobsTable = ({ history, totalCount, jobs, pagination }) => {
+  const columns = [
+    __('Description'),
+    __('Result'),
+    __('State'),
+    __('Executed at'),
+    __('Schedule'),
+  ];
+
+  const handlePerPageSelected = (event, perPage) => {
+    refreshPage(history, { page: 1, perPage });
+  };
+
+  const handlePageSelected = (event, page) => {
+    refreshPage(history, { ...pagination, page });
+  };
+
+  const perPageOptions = preparePerPageOptions(usePaginationOptions());
+
+  return (
+    <React.Fragment>
+      <h3>{__('Previously executed jobs')}</h3>
+<<<<<<< HEAD
+      <Flex className="pf-u-pt-md">
+=======
+      <Flex direction={{ default: 'column' }} className="pf-u-pt-md">
+        <FlexItem align={{ default: 'alignRight' }}>
+          <Pagination updateParamsByUrl itemCount={totalCount} variant="top" />
+        </FlexItem>
+        <FlexItem>
+          <TableComposable variant="compact">
+            <Thead>
+              <Tr>
+                {columns.map(col => (
+                  <Th key={col}>{col}</Th>
+                ))}
+              </Tr>
+            </Thead>
+            <Tbody>
+              {jobs.map(job => (
+                <Tr key={job.id}>
+                  <Td>
+                    <a
+                      onClick={() =>
+                        window.tfm.nav.pushUrl(
+                          `/job_invocations/${decodeId(job.id)}`
+                        )
+                      }
+                    >
+                      {job.description}
+                    </a>
+                    &nbsp;
+                    {readablePurpose(job.recurringLogic.purpose)}
+                  </Td>
+                  <Td>{job.task.result}</Td>
+                  <Td>{job.task.state}</Td>
+                  <Td>
+                    <RelativeDateTime date={job.startAt} />
+                  </Td>
+                  <Td>{readableCron(job.recurringLogic.cronLine)}</Td>
+                </Tr>
+              ))}
+            </Tbody>
+          </TableComposable>
+        </FlexItem>
+>>>>>>> 5b01704 (Fixes #34458 - Show Hostgroup jobs on the Host Detail page)
+        <FlexItem align={{ default: 'alignRight' }}>
+          <Pagination
+            itemCount={totalCount}
+            page={pagination.page}
+            perPage={pagination.perPage}
+            onSetPage={handlePageSelected}
+            onPerPageSelect={handlePerPageSelected}
+            perPageOptions={perPageOptions}
+            variant="top"
+          />
+        </FlexItem>
+      </Flex>
+      <TableComposable variant="compact">
+        <Thead>
+          <Tr>
+            {columns.map(col => (
+              <Th key={col}>{col}</Th>
+            ))}
+          </Tr>
+        </Thead>
+        <Tbody>
+          {jobs.map(job => (
+            <Tr key={job.id}>
+              <Td>
+                <a
+                  onClick={() =>
+                    window.tfm.nav.pushUrl(
+                      `/job_invocations/${decodeId(job.id)}`
+                    )
+                  }
+                >
+                  {job.description}
+                </a>
+              </Td>
+              <Td>{job.task.result}</Td>
+              <Td>{job.task.state}</Td>
+              <Td>
+                <RelativeDateTime date={job.startAt} />
+              </Td>
+              <Td>{readableCron(job.recurringLogic.cronLine)}</Td>
+            </Tr>
+          ))}
+        </Tbody>
+      </TableComposable>
+    </React.Fragment>
+  );
+};
+
+PreviousJobsTable.propTypes = {
+  jobs: PropTypes.array.isRequired,
+  history: PropTypes.object.isRequired,
+  totalCount: PropTypes.number.isRequired,
+  pagination: PropTypes.object.isRequired,
+};
+
+export default withLoading(PreviousJobsTable);

data/webpack/components/AnsibleHostDetail/components/JobsTab/RecurringJobsTable.js

@@ -14,11 +14,20 @@ import {
   Td,
 } from '@patternfly/react-table';
 
-import { useCancelMutation } from './JobsTabHelper';
+import {
+  useCancelMutation,
+  readableCron,
+  readablePurpose,
+} from './JobsTabHelper';
 import withLoading from '../../../withLoading';
 import { decodeId } from '../../../../globalIdHelper';
 
-const RecurringJobsTable = ({ jobs, resourceName, resourceId }) => {
+const RecurringJobsTable = ({
+  jobs,
+  resourceName,
+  resourceId,
+  hostGroupId,
+}) => {
   const columns = [__('Description'), __('Schedule'), __('Next Run')];
   const dispatch = useDispatch();
 
@@ -73,8 +82,10 @@ const RecurringJobsTable = ({ jobs, resourceName, resourceId }) => {
                 >
                   {job.description}
                 </a>
+                &nbsp;
+                {readablePurpose(job.recurringLogic.purpose)}
               </Td>
-              <Td>{job.recurringLogic.cronLine}</Td>
+              <Td>{readableCron(job.recurringLogic.cronLine)}</Td>
               <Td>
                 <RelativeDateTime date={job.startAt} />
               </Td>
@@ -91,6 +102,11 @@ RecurringJobsTable.propTypes = {
   jobs: PropTypes.array.isRequired,
   resourceId: PropTypes.number.isRequired,
   resourceName: PropTypes.string.isRequired,
+  hostGroupId: PropTypes.number,
+};
+
+RecurringJobsTable.defaultProps = {
+  hostGroupId: undefined,
 };
 
 export default withLoading(RecurringJobsTable);

data/webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTab.fixtures.js

@@ -23,10 +23,12 @@ const viewer = userFactory('viewer', [
 
 const firstRecurringLogicGlobalId =
   'MDE6Rm9yZW1hblRhc2tzOjpSZWN1cnJpbmdMb2dpYy0x';
+
 const firstRecurringLogic = {
   __typename: 'ForemanTasks::RecurringLogic',
   id: firstRecurringLogicGlobalId,
   cronLine: toCron(futureDate, 'weekly'),
+  purpose: '',
   meta: {
     canEdit: true,
   },
@@ -66,6 +68,7 @@ export const secondJob = {
     __typename: 'ForemanTasks::RecurringLogic',
     id: 'MDE6Rm9yZW1hblRhc2tzOjpSZWN1cnJpbmdMb2dpYy0yMw==',
     cronLine: '54 10 15 * *',
+    purpose: '',
     meta: {
       canEdit: true,
     },

data/webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTab.test.js

@@ -18,6 +18,7 @@ import {
 import * as toasts from '../../../../../toastHelper';
 
 import { toCron } from '../NewRecurringJobHelper';
+import { readableCron } from '../JobsTabHelper';
 
 import {
   tick,
@@ -49,8 +50,10 @@ describe('JobsTab', () => {
       .map(element => expect(element).toBeInTheDocument());
     expect(screen.getByText('Scheduled recurring jobs')).toBeInTheDocument();
     expect(screen.getByText('Previously executed jobs')).toBeInTheDocument();
-    expect(screen.getByText(toCron(futureDate, 'weekly'))).toBeInTheDocument();
-    expect(screen.getByText('54 10 15 * *')).toBeInTheDocument();
+    expect(
+      screen.getByText(readableCron(toCron(futureDate, 'weekly')))
+    ).toBeInTheDocument();
+    expect(screen.getByText('monthly')).toBeInTheDocument();
   });
   it('should show empty state', async () => {
     render(
@@ -129,7 +132,9 @@ describe('JobsTab', () => {
       message: 'Ansible job was successfully created.',
     });
     await waitFor(tick);
-    expect(screen.getByText(toCron(futureDate, 'weekly'))).toBeInTheDocument();
+    expect(
+      screen.getByText(readableCron(toCron(futureDate, 'weekly')))
+    ).toBeInTheDocument();
     expect(screen.getByText('in 3 days')).toBeInTheDocument();
     expect(
       screen.queryByText('No config job for Ansible roles scheduled')

data/webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTabHelper.test.js

@@ -0,0 +1,11 @@
+import { readableCron } from '../JobsTabHelper';
+
+describe('JobTabsHelper', () => {
+  it('readableCron', () => {
+    expect(readableCron('01 * * * *')).toBe('hourly');
+    expect(readableCron('01 01 * * *')).toBe('daily');
+    expect(readableCron('01 01 * * 01')).toBe('weekly');
+    expect(readableCron('01 01 01 * *')).toBe('monthly');
+    expect(readableCron()).toBe('custom');
+  });
+});

data/webpack/components/AnsibleHostDetail/components/JobsTab/index.js

@@ -15,7 +15,7 @@ import RecurringJobsTable from './RecurringJobsTable';
 import PreviousJobsTable from './PreviousJobsTable';
 import NewRecurringJobModal from './NewRecurringJobModal';
 
-const JobsTab = ({ resourceName, resourceId, history }) => {
+const JobsTab = ({ resourceName, resourceId, hostGroupId, history }) => {
   const [modalOpen, setModalOpen] = useState(false);
   const toggleModal = () => setModalOpen(!modalOpen);
 
@@ -44,6 +44,7 @@ const JobsTab = ({ resourceName, resourceId, history }) => {
         <RecurringJobsTable
           resourceId={resourceId}
           resourceName={resourceName}
+          hostGroupId={hostGroupId}
           fetchFn={fetchRecurringFn}
           renameData={renameData}
           renamedDataPath="jobs"
@@ -59,6 +60,7 @@ const JobsTab = ({ resourceName, resourceId, history }) => {
         <PreviousJobsTable
           resourceId={resourceId}
           resourceName={resourceName}
+          hostGroupId={hostGroupId}
           fetchFn={fetchPreviousFn(useParamsToVars(history))}
           renameData={renameData}
           emptyWrapper={() => null}
@@ -81,8 +83,13 @@ const JobsTab = ({ resourceName, resourceId, history }) => {
 
 JobsTab.propTypes = {
   resourceName: PropTypes.string.isRequired,
+  hostGroupId: PropTypes.number,
   resourceId: PropTypes.number.isRequired,
   history: PropTypes.object.isRequired,
 };
 
+JobsTab.defaultProps = {
+  hostGroupId: undefined,
+};
+
 export default JobsTab;

data/webpack/components/AnsibleHostDetail/components/SecondaryTabRoutes.js

@@ -44,6 +44,7 @@ const SecondaryTabRoutes = ({ response, router, history }) => (
         <JobsTab
           resourceId={response.id}
           resourceName="host"
+          hostGroupId={response.hostgroup_id}
           history={history}
         />
       </TabLayout>

data/webpack/graphql/queries/recurringJobs.gql

@@ -14,6 +14,7 @@ query($search: String, $first: Int, $last: Int) {
         meta {
           canEdit
         }
+        purpose
       }
       task {
         id

data/webpack/routes/HostgroupJobs/__test__/HostgroupJobs.test.js

@@ -24,6 +24,7 @@ import {
 } from '../../../testHelper';
 
 import { toCron } from '../../../components/AnsibleHostDetail/components/JobsTab/NewRecurringJobHelper';
+import { readableCron } from '../../../components/AnsibleHostDetail/components/JobsTab/JobsTabHelper';
 
 const TestComponent = withRedux(withRouter(withMockedProvider(HostgroupJobs)));
 
@@ -46,7 +47,7 @@ describe('HostgroupJobs', () => {
       .map(element => expect(element).toBeInTheDocument());
     expect(screen.getByText('Scheduled recurring jobs')).toBeInTheDocument();
     expect(screen.getByText('Previously executed jobs')).toBeInTheDocument();
-    expect(screen.getByText('54 10 15 * *')).toBeInTheDocument();
+    expect(screen.getByText(readableCron('54 10 15 * *'))).toBeInTheDocument();
   });
   it('should show empty state', async () => {
     render(
@@ -103,7 +104,9 @@ describe('HostgroupJobs', () => {
       type: 'success',
       message: 'Ansible job was successfully created.',
     });
-    expect(screen.getByText(toCron(futureDate, 'weekly'))).toBeInTheDocument();
+    expect(
+      screen.getByText(readableCron(toCron(futureDate, 'weekly')))
+    ).toBeInTheDocument();
     expect(screen.getByText('in 3 days')).toBeInTheDocument();
     expect(
       screen.queryByText('No config job for Ansible roles scheduled')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 7.0.2
+  version: 7.0.3
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-19 00:00:00.000000000 Z
+date: 2022-02-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: acts_as_list
@@ -327,9 +327,11 @@ files:
 - webpack/components/AnsibleHostDetail/components/JobsTab/NewRecurringJobModal.js
 - webpack/components/AnsibleHostDetail/components/JobsTab/NewRecurringJobModal.scss
 - webpack/components/AnsibleHostDetail/components/JobsTab/PreviousJobsTable.js
+- webpack/components/AnsibleHostDetail/components/JobsTab/PreviousJobsTable.js.orig
 - webpack/components/AnsibleHostDetail/components/JobsTab/RecurringJobsTable.js
 - webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTab.fixtures.js
 - webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTab.test.js
+- webpack/components/AnsibleHostDetail/components/JobsTab/__test__/JobsTabHelper.test.js
 - webpack/components/AnsibleHostDetail/components/JobsTab/index.js
 - webpack/components/AnsibleHostDetail/components/RolesTab/AllRolesModal/AllRolesTable.js
 - webpack/components/AnsibleHostDetail/components/RolesTab/AllRolesModal/index.js
@@ -447,50 +449,50 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.4
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: Ansible integration with Foreman (theforeman.org)
 test_files:
-- test/functional/ansible_variables_controller_test.rb
-- test/functional/ansible_roles_controller_test.rb
-- test/functional/api/v2/ansible_variables_controller_test.rb
+- test/functional/api/v2/hosts_controller_test.rb
 - test/functional/api/v2/ansible_roles_controller_test.rb
 - test/functional/api/v2/hostgroups_controller_test.rb
+- test/functional/api/v2/ansible_variables_controller_test.rb
 - test/functional/api/v2/ansible_inventories_controller_test.rb
-- test/functional/api/v2/hosts_controller_test.rb
-- test/functional/ui_ansible_roles_controller_test.rb
 - test/functional/hosts_controller_test.rb
-- test/graphql/mutations/hosts/assign_ansible_roles_mutation_test.rb
-- test/graphql/queries/ansible_roles_query_test.rb
-- test/foreman_ansible/helpers/ansible_roles_helper_test.rb
-- test/unit/actions/run_ansible_job_test.rb
-- test/unit/actions/run_proxy_ansible_command_test.rb
-- test/unit/services/api_roles_importer_test.rb
-- test/unit/services/ui_roles_importer_test.rb
+- test/functional/ansible_roles_controller_test.rb
+- test/functional/ansible_variables_controller_test.rb
+- test/functional/ui_ansible_roles_controller_test.rb
+- test/unit/services/inventory_creator_test.rb
 - test/unit/services/ansible_report_importer_test.rb
-- test/unit/services/insights_plan_runner_test.rb
-- test/unit/services/override_resolver_test.rb
+- test/unit/services/ui_roles_importer_test.rb
 - test/unit/services/roles_importer_test.rb
-- test/unit/services/inventory_creator_test.rb
 - test/unit/services/ansible_variables_importer_test.rb
-- test/unit/lib/proxy_api/ansible_test.rb
+- test/unit/services/override_resolver_test.rb
+- test/unit/services/api_roles_importer_test.rb
+- test/unit/services/insights_plan_runner_test.rb
 - test/unit/ansible_role_test.rb
-- test/unit/hostgroup_ansible_role_test.rb
-- test/unit/ansible_variable_test.rb
 - test/unit/host_ansible_role_test.rb
+- test/unit/hostgroup_ansible_role_test.rb
 - test/unit/helpers/ansible_reports_helper_test.rb
 - test/unit/ansible_provider_test.rb
-- test/unit/ignore_roles_test.rb
-- test/unit/import_roles_and_variables.rb
-- test/unit/concerns/config_reports_extensions_test.rb
 - test/unit/concerns/host_managed_extensions_test.rb
+- test/unit/concerns/config_reports_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
+- test/unit/ignore_roles_test.rb
+- test/unit/ansible_variable_test.rb
+- test/unit/import_roles_and_variables.rb
+- test/unit/lib/proxy_api/ansible_test.rb
+- test/unit/actions/run_ansible_job_test.rb
+- test/unit/actions/run_proxy_ansible_command_test.rb
+- test/foreman_ansible/helpers/ansible_roles_helper_test.rb
+- test/test_plugin_helper.rb
 - test/factories/ansible_proxy.rb
-- test/factories/host_ansible_enhancements.rb
 - test/factories/ansible_variables.rb
+- test/factories/host_ansible_enhancements.rb
 - test/factories/ansible_roles.rb
 - test/fixtures/report.json
 - test/fixtures/sample_facts.json
 - test/fixtures/insights_playbook.yaml
-- test/test_plugin_helper.rb
+- test/graphql/mutations/hosts/assign_ansible_roles_mutation_test.rb
+- test/graphql/queries/ansible_roles_query_test.rb