foreman_ansible 9.0.1 → 10.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf08e369d1c4b9a445123a9fab3a02b1da8c3bfe9261b7027693959d90e1e1a3
-  data.tar.gz: 13ac8e3cb9b88bf5167a2fdeb9e95409ad9a93c6f4dce12f0500cc5c5bd787a9
+  metadata.gz: 98a2c9aa6bcc8e2feb5b8eee62d111926d0dab191561f808399587d7462debbc
+  data.tar.gz: 2fc48fc5f59725cd8000229bdf08e68f5e8e53a1158271ce2d6db3766ab5826b
 SHA512:
-  metadata.gz: 8c515c517add1b91f020c63a50b6b58eaafdacfe08d52d39286ae79efd034cb8ca0ffc8ef4aa8aa2f5be9e5dcc6a797910dd9b32f444e61e525077c8239e3e56
-  data.tar.gz: 18fac58f8a1d119af822b6be9f28ce33b626b0951791d5c3ca201d6e2475f2399c52d21ba92b19335f267a183306e568cbc384910942f1d7d0de9d1d74a88ef2
+  metadata.gz: a8be9228e0b1532cdbef43e88b0f7ba84f9bd8edc5a9d9e21e1ee6d601f63bbccf16eff5e146f33d18ea06d7cdd5adc528d310d7ddb723ff3a6b91ce3a6774c6
+  data.tar.gz: 35b55ca402a155fae45abd89c6fecde949d133e50b855473a25dba13df9452a13a5b9342b93e49fefd553902909a565dd59f9d57afb07d4f8efec6f0311057a9

data/app/graphql/types/ansible_variable_override.rb

@@ -1,5 +1,5 @@
 module Types
-  class AnsibleVariableOverride < GraphQL::Types::Relay::BaseObject
+  class AnsibleVariableOverride < GraphQL::Schema::Object
     description 'Override value for Ansible Variable'
 
     field :value, ::Types::RawJson, :null => false

data/app/helpers/foreman_ansible/ansible_hostgroups_helper.rb

@@ -4,7 +4,7 @@ module ForemanAnsible
   module AnsibleHostgroupsHelper
     def ansible_hostgroups_actions(hostgroup)
       play_roles = if hostgroup.all_ansible_roles.empty?
-                     { action: (link_to _('Run all Ansible roles'), 'javascript:void(0);', disabled: true, title: 'No Roles assigned'), priority: 31 }
+                     { action: { content: (link_to _('Run all Ansible roles'), 'javascript:void(0);', disabled: true, title: 'No roles assigned', class: 'disabled'), options: { class: 'disabled' } }, priority: 31 }
                    else
                      { action: display_link_if_authorized(_('Run all Ansible roles'), hash_for_play_roles_hostgroup_path(id: hostgroup), 'data-no-turbolink': true, title: _('Run all Ansible roles on hosts belonging to this host group')), priority: 31 }
                    end

data/lib/foreman_ansible/register.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Foreman::Plugin.register :foreman_ansible do
-  requires_foreman '>= 3.4'
+  requires_foreman '>= 3.5'
 
   settings do
     category :ansible, N_('Ansible') do
@@ -188,7 +188,7 @@ Foreman::Plugin.register :foreman_ansible do
   register_global_js_file 'global'
 
   extend_graphql_type :type => '::Types::Host' do
-    field :all_ansible_roles, ::Types::InheritedAnsibleRole.connection_type, :null => true, :method => :present_all_ansible_roles
+    field :all_ansible_roles, ::Types::InheritedAnsibleRole.connection_type, :null => true, :resolver_method => :present_all_ansible_roles
     field :own_ansible_roles, ::Types::AnsibleRole.connection_type, :null => true
     field :available_ansible_roles, ::Types::AnsibleRole.connection_type, :null => true
     field :ansible_variables_with_overrides, Types::OverridenAnsibleVariable.connection_type, :null => false

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 = '9.0.1'
+  VERSION = '10.0.1'
 end

data/package.json

@@ -21,7 +21,6 @@
     "@theforeman/stories": "^10.0",
     "@theforeman/test": "^10.0",
     "@theforeman/vendor-dev": "^10.0",
-    "@testing-library/user-event": "^13.2.1",
     "babel-eslint": "^10.0.3",
     "eslint": "^6.7.2",
     "prettier": "^1.13.5"

data/test/graphql/mutations/hosts/assign_ansible_roles_mutation_test.rb

@@ -3,15 +3,15 @@ require 'test_plugin_helper'
 module Mutations
   module Hosts
     class CreateMutationTest < GraphQLQueryTestCase
-      let(:tax_location) { FactoryBot.create(:location) }
+      let(:tax_location) { as_admin { FactoryBot.create(:location) } }
       let(:location_id) { Foreman::GlobalId.for(tax_location) }
-      let(:organization) { FactoryBot.create(:organization) }
+      let(:organization) { as_admin { FactoryBot.create(:organization) } }
       let(:organization_id) { Foreman::GlobalId.for(organization) }
 
-      let(:role1) { FactoryBot.create(:ansible_role) }
-      let(:role2) { FactoryBot.create(:ansible_role) }
-      let(:role3) { FactoryBot.create(:ansible_role) }
-      let(:host) { FactoryBot.create(:host, :ansible_roles => [role1, role2, role3], :organization => organization, :location => tax_location) }
+      let(:role1) { as_admin { FactoryBot.create(:ansible_role) } }
+      let(:role2) { as_admin { FactoryBot.create(:ansible_role) } }
+      let(:role3) { as_admin { FactoryBot.create(:ansible_role) } }
+      let(:host) { as_admin { FactoryBot.create(:host, :ansible_roles => [role1, role2, role3], :organization => organization, :location => tax_location) } }
 
       let(:variables) { { id: Foreman::GlobalId.for(host), ansibleRoleIds: [role3.id, role2.id, role1.id] } }
       let(:query) do
@@ -37,7 +37,7 @@ module Mutations
       end
 
       context 'with admin permissions' do
-        let(:context_user) { FactoryBot.create(:user, :admin) }
+        let(:context_user) { as_admin { FactoryBot.create(:user, :admin) } }
         let(:data) { result['data']['assignAnsibleRoles']['host'] }
 
         it 'reorderes ansible roles' do

data/test/integration/hostgroup_js_test.rb

@@ -0,0 +1,30 @@
+require_relative '../test_plugin_helper'
+require 'integration_test_helper'
+
+class HostgroupJsTest < IntegrationTestWithJavascript
+  let(:hostgroup) { FactoryBot.create(:hostgroup, :name => 'HostgroupWithoutRoles') }
+  let(:hostgroup_with_roles) { FactoryBot.create(:hostgroup, :with_ansible_roles, :name => 'HostgroupWithRoles') }
+
+  setup do
+    FactoryBot.create(:host, :hostgroup_id => hostgroup.id)
+    FactoryBot.create(:host, :hostgroup_id => hostgroup_with_roles.id)
+  end
+
+  test 'hostgroup without roles should have disabled link' do
+    visit hostgroups_path(search: hostgroup.name)
+
+    first_row = page.find('table > tbody > tr:nth-child(1)')
+    first_row.find('td:nth-child(4) > div > a').click
+
+    assert_includes first(:link, 'Run all Ansible roles')[:class], 'disabled'
+  end
+
+  test 'hostgroup with roles should have clickable link' do
+    visit hostgroups_path(search: hostgroup_with_roles.name)
+
+    first_row = page.find('table > tbody > tr:nth-child(1)')
+    first_row.find('td:nth-child(4) > div > a').click
+
+    assert_not first(:link, 'Run all Ansible roles')[:class].include?('disabled')
+  end
+end

data/webpack/components/AnsibleHostDetail/AnsibleHostDetail.js

@@ -22,6 +22,7 @@ const AnsibleHostDetail = ({
         <>
           <Tabs
             className="ansible-host-details-tabs"
+            ouiaId="ansible-host-details-tabs"
             onSelect={(evt, subTab) => hashHistory.push(subTab)}
             activeKey={pathname?.split('/')[2]}
             isSecondary

data/webpack/components/AnsibleHostDetail/components/AnsibleVariableOverrides/AnsibleVariableOverridesTable.js

@@ -148,12 +148,17 @@ const AnsibleVariableOverridesTable = ({
     <React.Fragment>
       <Flex direction={{ default: 'column' }}>
         <FlexItem align={{ default: 'alignRight' }}>
-          <Pagination updateParamsByUrl itemCount={totalCount} variant="top" />
+          <Pagination
+            ouiaId="pagination-top"
+            updateParamsByUrl
+            itemCount={totalCount}
+            variant="top"
+          />
         </FlexItem>
         <FlexItem>
-          <TableComposable variant="compact">
+          <TableComposable ouiaId="table-composable-compact" variant="compact">
             <Thead>
-              <Tr>
+              <Tr ouiaId="row-header">
                 {columns.map(col => (
                   <Th key={col}>{col}</Th>
                 ))}
@@ -162,7 +167,7 @@ const AnsibleVariableOverridesTable = ({
             </Thead>
             <Tbody>
               {variables.map((variable, idx) => (
-                <Tr key={idx}>
+                <Tr key={idx} ouiaId={`row-${idx}`}>
                   <Td>
                     <a href={variable.path}>{variable.key}</a>
                   </Td>
@@ -202,6 +207,7 @@ const AnsibleVariableOverridesTable = ({
         </FlexItem>
         <FlexItem align={{ default: 'alignRight' }}>
           <Pagination
+            ouiaId="pagination-bottom"
             updateParamsByUrl
             itemCount={totalCount}
             variant="bottom"

data/webpack/components/AnsibleHostDetail/components/AnsibleVariableOverrides/AnsibleVariableOverridesTableHelper.js

@@ -23,6 +23,7 @@ export const formatValue = variable => {
     case 'yaml':
     case 'hash':
     case 'array':
+    case 'json':
       return JSON.stringify(value);
     default:
       return value;

data/webpack/components/AnsibleHostDetail/components/AnsibleVariableOverrides/EditableAction.js

@@ -111,6 +111,7 @@ const EditableAction = ({
             onClick={onClose}
             isDisabled={state.working}
             aria-label="Cancel editing override button"
+            ouiaId="cancel-editing-override-button"
           >
             <TimesIcon />
           </Button>
@@ -119,6 +120,7 @@ const EditableAction = ({
             onClick={onSubmit}
             isDisabled={state.working || hasError(state)}
             aria-label="Submit editing override button"
+            ouiaId="submit-editing-override-button"
           >
             <CheckIcon />
           </Button>
@@ -136,6 +138,7 @@ const EditableAction = ({
             onClick={onOpen}
             variant="plain"
             aria-label="Edit override button"
+            ouiaId="edit-override-button"
           >
             <PencilAltIcon />
           </Button>

data/webpack/components/AnsibleHostDetail/components/AnsibleVariableOverrides/EditableValue.js

@@ -17,11 +17,12 @@ const EditableValue = props => {
 
   const type = props.variable.parameterType;
 
-  if (type === 'array' || type === 'hash') {
+  if (['json', 'yaml', 'array', 'hash'].includes(type)) {
     return (
       <TextAreaField
+        aria-label="Edit override field"
         onChange={props.onChange}
-        value={props.value}
+        value={JSON.stringify(props.value)}
         validation={props.validation}
         isDisabled={props.working}
       />
@@ -31,6 +32,7 @@ const EditableValue = props => {
   if (type === 'boolean') {
     return (
       <SelectField
+        aria-label="Edit override field"
         selectItems={[
           { id: 'trueSelectOpt', value: true, name: __('true') },
           { id: 'falseSelectOpt', value: false, name: __('false') },

data/webpack/components/AnsibleHostDetail/components/AnsibleVariableOverrides/EditableValueHelper.js

@@ -23,7 +23,11 @@ const withFormGroup = Component => componentProps => {
 export const SelectField = componentProps => {
   const { selectItems, ...rest } = componentProps;
   return (
-    <FormSelect className="without_select2" {...rest}>
+    <FormSelect
+      className="without_select2"
+      ouiaId="without-form-select"
+      {...rest}
+    >
       {selectItems.map(item => (
         <FormSelectOption key={item.id} value={item.value} label={item.name} />
       ))}

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

@@ -57,6 +57,7 @@ const NewRecurringJobModal = props => {
         const actions = [
           <Button
             aria-label="submit creating job"
+            ouiaId="submit-creating-job"
             key="confirm"
             variant="primary"
             onClick={formProps.handleSubmit}
@@ -66,6 +67,7 @@ const NewRecurringJobModal = props => {
           </Button>,
           <Button
             aria-label="cancel creating job"
+            ouiaId="cancel-creating-job"
             key="cancel"
             variant="link"
             onClick={onClose}
@@ -83,6 +85,7 @@ const NewRecurringJobModal = props => {
           <Modal
             variant={ModalVariant.large}
             title="Create New Recurring Ansible Run"
+            ouiaId="modal-recurring-ansible-run"
             isOpen={props.isOpen}
             className="foreman-modal modal-high"
             showClose={false}

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

@@ -32,12 +32,17 @@ const PreviousJobsTable = ({ history, totalCount, jobs, pagination }) => {
       <h3>{__('Previously executed jobs')}</h3>
       <Flex direction={{ default: 'column' }} className="pf-u-pt-md">
         <FlexItem align={{ default: 'alignRight' }}>
-          <Pagination updateParamsByUrl itemCount={totalCount} variant="top" />
+          <Pagination
+            ouiaId="pagination-top"
+            updateParamsByUrl
+            itemCount={totalCount}
+            variant="top"
+          />
         </FlexItem>
         <FlexItem>
-          <TableComposable variant="compact">
+          <TableComposable ouiaId="table-composable-compact" variant="compact">
             <Thead>
-              <Tr>
+              <Tr ouiaId="row-header">
                 {columns.map(col => (
                   <Th key={col}>{col}</Th>
                 ))}
@@ -45,7 +50,7 @@ const PreviousJobsTable = ({ history, totalCount, jobs, pagination }) => {
             </Thead>
             <Tbody>
               {jobs.map(job => (
-                <Tr key={job.id}>
+                <Tr key={job.id} ouiaId={`row-${job.id}`}>
                   <Td>
                     <a
                       onClick={() =>
@@ -72,6 +77,7 @@ const PreviousJobsTable = ({ history, totalCount, jobs, pagination }) => {
         </FlexItem>
         <FlexItem align={{ default: 'alignRight' }}>
           <Pagination
+            ouiaId="pagination-bottom"
             updateParamsByUrl
             itemCount={totalCount}
             variant="bottom"

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

@@ -60,9 +60,9 @@ const RecurringJobsTable = ({
   return (
     <React.Fragment>
       <h3>{__('Scheduled recurring jobs')}</h3>
-      <TableComposable variant="compact">
+      <TableComposable ouiaId="table-composable-compact" variant="compact">
         <Thead>
-          <Tr>
+          <Tr ouiaId="row-header">
             {columns.map(col => (
               <Th key={col}>{col}</Th>
             ))}
@@ -71,7 +71,7 @@ const RecurringJobsTable = ({
         </Thead>
         <Tbody>
           {jobs.map(job => (
-            <Tr key={job.id}>
+            <Tr key={job.id} ouiaId={`row-${job.id}`}>
               <Td>
                 <a
                   onClick={() =>

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

@@ -33,7 +33,11 @@ const JobsTab = ({ resourceName, resourceId, hostGroupId, history }) => {
   ];
 
   const scheduleBtn = (
-    <Button aria-label="schedule recurring job" onClick={toggleModal}>
+    <Button
+      aria-label="schedule recurring job"
+      ouiaId="schedule-recurring-job"
+      onClick={toggleModal}
+    >
       {__('Schedule recurring job')}
     </Button>
   );

data/webpack/components/AnsibleHostDetail/components/RolesTab/AllRolesModal/AllRolesTable.js

@@ -21,11 +21,16 @@ const AllRolesTable = ({ allAnsibleRoles, totalCount }) => {
     <React.Fragment>
       <Flex direction={{ default: 'column' }} className="pf-u-pt-md">
         <FlexItem align={{ default: 'alignRight' }}>
-          <Pagination updateParamsByUrl itemCount={totalCount} variant="top" />
+          <Pagination
+            ouiaId="pagination-top"
+            updateParamsByUrl
+            itemCount={totalCount}
+            variant="top"
+          />
         </FlexItem>
-        <TableComposable variant="compact">
+        <TableComposable ouiaId="table-composable-compact" variant="compact">
           <Thead>
-            <Tr>
+            <Tr ouiaId="row-header">
               <Th />
               {columns.map(col => (
                 <Th key={`${col}-all`}>{col}</Th>
@@ -34,7 +39,7 @@ const AllRolesTable = ({ allAnsibleRoles, totalCount }) => {
           </Thead>
           <Tbody>
             {allAnsibleRoles.map(role => (
-              <Tr key={`${role.id}-all`} id={role.id}>
+              <Tr key={`${role.id}-all`} id={role.id} ouiaId={`row-${role.id}`}>
                 <Td />
                 <Td>{role.name}</Td>
                 <Td>
@@ -58,6 +63,7 @@ const AllRolesTable = ({ allAnsibleRoles, totalCount }) => {
         </TableComposable>
         <FlexItem align={{ default: 'alignRight' }}>
           <Pagination
+            ouiaId="pagination-bottom"
             updateParamsByUrl
             itemCount={totalCount}
             variant="bottom"

data/webpack/components/AnsibleHostDetail/components/RolesTab/AllRolesModal/index.js

@@ -15,6 +15,7 @@ import {
 
 const AllRolesModal = ({ hostGlobalId, onClose, history }) => {
   const baseModalProps = {
+    ouiaId: 'modal-ansible-roles',
     variant: ModalVariant.large,
     isOpen: true,
     onClose,

data/webpack/components/AnsibleHostDetail/components/RolesTab/EditRolesModal/EditRolesForm.js

@@ -54,6 +54,7 @@ const EditRolesForm = props => {
       onClick={() => callMutation({ variables })}
       isDisabled={loading || didNotModifyOptions()}
       aria-label="submit ansible roles"
+      ouiaId="submit-ansible-roles-button"
     >
       {__('Confirm')}
     </Button>,
@@ -65,7 +66,7 @@ const EditRolesForm = props => {
   }
 
   return (
-    <Modal {...baseModalProps} actions={formActions}>
+    <Modal ouiaId="modal-edit-roles" {...baseModalProps} actions={formActions}>
       <DualList
         availableOptions={availableOptions}
         chosenOptions={chosenOptions}

data/webpack/components/AnsibleHostDetail/components/RolesTab/EditRolesModal/index.js

@@ -20,6 +20,7 @@ const EditRolesModal = ({
   canEditHost,
 }) => {
   const baseModalProps = {
+    ouiaId: 'edit-ansible-roles-modal',
     width: '50%',
     isOpen,
     className: 'foreman-modal',
@@ -32,7 +33,12 @@ const EditRolesModal = ({
   };
 
   const actions = [
-    <Button variant="link" onClick={event => closeModal()} key="close">
+    <Button
+      ouiaId="close-button"
+      variant="link"
+      onClick={event => closeModal()}
+      key="close"
+    >
       {__('Close')}
     </Button>,
   ];

data/webpack/components/AnsibleHostDetail/components/RolesTab/RolesTable.js

@@ -32,7 +32,10 @@ const RolesTable = ({
   const editBtn = canEditHost ? (
     <FlexItem>
       <Link to="/Ansible/roles/edit">
-        <Button aria-label="edit ansible roles">
+        <Button
+          aria-label="edit ansible roles"
+          ouiaId="edit-ansible-roles-button"
+        >
           {__('Edit Ansible roles')}
         </Button>
       </Link>
@@ -53,14 +56,19 @@ const RolesTable = ({
       <Flex>
         <FlexItem>{editBtn}</FlexItem>
         <FlexItem align={{ default: 'alignRight' }}>
-          <Pagination updateParamsByUrl itemCount={totalCount} variant="top" />
+          <Pagination
+            ouiaId="pagination-top"
+            updateParamsByUrl
+            itemCount={totalCount}
+            variant="top"
+          />
         </FlexItem>
       </Flex>
       <Flex direction={{ default: 'column' }}>
         <FlexItem>
-          <TableComposable variant="compact">
+          <TableComposable variant="compact" ouiaId="table-composable-compact">
             <Thead>
-              <Tr>
+              <Tr ouiaId="row-header">
                 {columns.map(col => (
                   <Th key={col}>{col}</Th>
                 ))}
@@ -68,7 +76,7 @@ const RolesTable = ({
             </Thead>
             <Tbody>
               {ansibleRoles.map(role => (
-                <Tr key={role.id}>
+                <Tr key={role.id} ouiaId={`row-${role.id}`}>
                   <Td>
                     <a href={role.path}>{role.name}</a>
                   </Td>
@@ -88,6 +96,7 @@ const RolesTable = ({
         </FlexItem>
         <FlexItem align={{ default: 'alignRight' }}>
           <Pagination
+            ouiaId="pagination-bottom"
             updateParamsByUrl
             itemCount={totalCount}
             variant="bottom"

data/webpack/components/AnsibleHostDetail/components/RolesTab/__test__/EditRoles.test.js

@@ -21,7 +21,8 @@ import {
   assignRolesErrorMock,
 } from './RolesTab.fixtures';
 
-const TestComponent = withReactRouter(withRedux(withMockedProvider(RolesTab)));
+jest.mock('axios');
+const TestComponent = withRedux(withReactRouter(withMockedProvider(RolesTab)));
 
 describe('assigning Ansible roles', () => {
   it('should assign Ansible roles', async () => {

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

@@ -6,6 +6,7 @@ import {
   tick,
   withMockedProvider,
   withReactRouter,
+  withRedux,
 } from '../../../../../testHelper';
 
 import {
@@ -18,7 +19,8 @@ import {
 
 import RolesTab from '../';
 
-const TestComponent = withReactRouter(withMockedProvider(RolesTab));
+jest.mock('axios');
+const TestComponent = withRedux(withReactRouter(withMockedProvider(RolesTab)));
 
 describe('RolesTab', () => {
   it('should load Ansible Roles as admin', async () => {

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

@@ -2,7 +2,10 @@ import React, { useState } from 'react';
 import PropTypes from 'prop-types';
 import { useQuery } from '@apollo/client';
 import { Button } from '@patternfly/react-core';
+import { Link, Route } from 'react-router-dom';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { foremanUrl } from 'foremanReact/common/helpers';
+import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
 
 import ansibleRolesQuery from '../../../../graphql/queries/hostAnsibleRoles.gql';
 import { encodeId } from '../../../../globalIdHelper';
@@ -12,6 +15,7 @@ import {
   useCurrentPagination,
 } from '../../../../helpers/pageParamsHelper';
 import EditRolesModal from './EditRolesModal';
+import AllRolesModal from './AllRolesModal';
 
 const RolesTab = ({ hostId, history, canEditHost }) => {
   const hostGlobalId = encodeId('Host', hostId);
@@ -32,10 +36,30 @@ const RolesTab = ({ hostId, history, canEditHost }) => {
     <Button
       onClick={() => setAssignModal(true)}
       aria-label="edit ansible roles"
+      ouiaId="edit-ansible-roles-button"
     >
-      {__('Assign Ansible roles')}
+      {__('Assign roles directly to the host')}
     </Button>
   ) : null;
+
+  const url = hostId && foremanUrl(`/api/v2/hosts/${hostId}/ansible_roles`);
+  const { response: allAnsibleRoles } = useAPI('get', url, {
+    key: 'ANSIBLE_ROLES',
+  });
+  const emptyStateDescription = allAnsibleRoles.length > 0 && (
+    <>
+      <Route path="/Ansible/roles/all">
+        <AllRolesModal
+          onClose={() => history.push('/Ansible/roles')}
+          isOpen
+          hostGlobalId={hostGlobalId}
+          history={history}
+        />
+      </Route>
+      <Link to="/Ansible/roles/all">{__('View inherited roles')}</Link>
+    </>
+  );
+
   return (
     <>
       <RolesTable
@@ -46,8 +70,9 @@ const RolesTab = ({ hostId, history, canEditHost }) => {
         history={history}
         hostGlobalId={hostGlobalId}
         emptyStateProps={{
-          header: __('No Ansible roles assigned'),
+          header: __('No roles assigned directly to the host'),
           action: editBtn,
+          description: emptyStateDescription,
         }}
         pagination={pagination}
         canEditHost={canEditHost}

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

@@ -12,7 +12,7 @@ import { ANSIBLE_KEY } from '../constants';
 import { route } from '../helpers';
 
 const SecondaryTabRoutes = ({ response, router, history }) => (
-  <Switch>
+  <Switch ouiaId="switch">
     <Route exact path={`/${ANSIBLE_KEY}`}>
       <Redirect to={route('roles')} />
     </Route>

data/webpack/testHelper.js

@@ -1,15 +1,24 @@
 import React from 'react';
 import { Provider } from 'react-redux';
+import thunk from 'redux-thunk';
+import { applyMiddleware, createStore, compose, combineReducers } from 'redux';
 import { MockedProvider } from '@apollo/react-testing';
 import { Router, MemoryRouter } from 'react-router-dom';
 import { createMemoryHistory } from 'history';
 
-import store from 'foremanReact/redux';
-import ConfirmModal from 'foremanReact/components/ConfirmModal';
+import { reducers as apiReducer, APIMiddleware } from 'foremanReact/redux/API';
+import ConfirmModal, {
+  reducers as confirmModalReducers,
+} from 'foremanReact/components/ConfirmModal';
 import { getForemanContext } from 'foremanReact/Root/Context/ForemanContext';
 
+const reducers = combineReducers({ ...apiReducer, ...confirmModalReducers });
+
+export const generateStore = () =>
+  createStore(reducers, compose(applyMiddleware(thunk, APIMiddleware)));
+
 export const withRedux = Component => props => (
-  <Provider store={store}>
+  <Provider store={generateStore()}>
     <Component {...props} />
     <ConfirmModal />
   </Provider>

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 9.0.1
+  version: 10.0.1
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-05 00:00:00.000000000 Z
+date: 2022-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: acts_as_list
@@ -44,28 +44,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.4.0
+        version: 8.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.4.0
+        version: 8.0.0
 - !ruby/object:Gem::Dependency
   name: foreman-tasks
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 7.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 7.0.0
 description: Ansible integration with Foreman
 email:
 - elobatocs@gmail.com
@@ -222,7 +222,6 @@ files:
 - lib/foreman_ansible.rb
 - lib/foreman_ansible/engine.rb
 - lib/foreman_ansible/register.rb
-- lib/foreman_ansible/register.rb.orig
 - lib/foreman_ansible/remote_execution.rb
 - lib/foreman_ansible/version.rb
 - locale/Makefile
@@ -325,6 +324,7 @@ files:
 - test/graphql/mutations/hosts/assign_ansible_roles_mutation_test.rb
 - test/graphql/queries/ansible_roles_query_test.rb
 - test/graphql/queries/host_ansible_roles_query_test.rb
+- test/integration/hostgroup_js_test.rb
 - test/test_plugin_helper.rb
 - test/unit/actions/run_ansible_job_test.rb
 - test/unit/actions/run_proxy_ansible_command_test.rb
@@ -547,3 +547,4 @@ test_files:
 - test/unit/import_playbooks_test.rb
 - test/unit/import_roles_and_variables.rb
 - test/test_plugin_helper.rb
+- test/integration/hostgroup_js_test.rb

data/lib/foreman_ansible/register.rb.orig

@@ -1,257 +0,0 @@
-# frozen_string_literal: true
-
-Foreman::Plugin.register :foreman_ansible do
-  requires_foreman '>= 3.3'
-
-  settings do
-    category :ansible, N_('Ansible') do
-      setting 'ansible_ssh_private_key_file',
-              type: :string,
-              description: N_('Use this to supply a path to an SSH Private Key '\
-                               'that Ansible will use in lieu of a password '\
-                               'Override with "ansible_ssh_private_key_file" '\
-                               'host parameter'),
-              default: '',
-              full_name: N_('Private Key Path')
-      setting 'ansible_connection',
-              type: :string,
-              description: N_('Use this connection type by default when running '\
-                               'Ansible playbooks. You can override this on hosts by '\
-                               'adding a parameter "ansible_connection"'),
-              default: 'ssh',
-              full_name: N_('Connection type')
-      setting 'ansible_winrm_server_cert_validation',
-              type: :string,
-              description: N_('Enable/disable WinRM server certificate '\
-                               'validation when running Ansible playbooks. You can override '\
-                               'this on hosts by adding a parameter '\
-                               '"ansible_winrm_server_cert_validation"'),
-              default: 'validate',
-              full_name: N_('WinRM cert Validation')
-      setting 'ansible_verbosity',
-              type: :integer,
-              description: N_('Foreman will add this level of verbosity for '\
-                               'additional debugging output when running Ansible playbooks.'),
-              default: '0',
-              full_name: N_('Default verbosity level'),
-              value: nil,
-              collection: proc {
-                { '0' => N_('Disabled'),
-                  '1' => N_('Level 1 (-v)'),
-                  '2' => N_('Level 2 (-vv)'),
-                  '3' => N_('Level 3 (-vvv)'),
-                  '4' => N_('Level 4 (-vvvv)') }
-              }
-      setting 'ansible_post_provision_timeout',
-              type: :integer,
-              description: N_('Timeout (in seconds) to set when Foreman will trigger a '\
-                               'play Ansible roles task after a host is fully provisioned. '\
-                               'Set this to the maximum time you expect a host to take '\
-                               'until it is ready after a reboot.'),
-              default: '360',
-              full_name: N_('Post-provision timeout')
-      setting 'ansible_interval',
-              type: :integer,
-              description: N_('Timeout (in minutes) when hosts should have reported.'),
-              default: '30',
-              full_name: N_('Ansible report timeout')
-      setting 'ansible_out_of_sync_disabled',
-              type: :boolean,
-              description: format(N_('Disable host configuration status turning to out of'\
-                               ' sync for %{cfgmgmt} after report does not arrive within'\
-                               ' configured interval'), :cfgmgmt => 'Ansible'),
-              default: false,
-              full_name: format(N_('%{cfgmgmt} out of sync disabled'), :cfgmgmt => 'Ansible')
-      setting  'ansible_inventory_template',
-               type: :string,
-               description: N_('Foreman will use this template to schedule the report '\
-                                 'with Ansible inventory'),
-               default: 'Ansible - Ansible Inventory',
-               full_name: N_('Default Ansible inventory report template')
-      setting 'ansible_roles_to_ignore',
-              type: :array,
-              description: N_('Those roles will be excluded when importing roles from smart proxy, '\
-                'The expected input is comma separated values and you can use * wildcard metacharacters'\
-                'For example: foo*, *b*,*bar'),
-              default: [],
-              full_name: N_('Ansible roles to ignore')
-      setting 'foreman_ansible_proxy_batch_size',
-              type: :integer,
-              description: N_('Number of tasks which should be sent to the smart proxy in one request, '\
-                              'if foreman_tasks_proxy_batch_trigger is enabled. '\
-                              'If set, overrides foreman_tasks_proxy_batch_size setting for Ansible jobs.'),
-              default: nil,
-              full_name: N_('Proxy tasks batch size for Ansible')
-    end
-  end
-
-  security_block :foreman_ansible do
-    permission :play_roles_on_host,
-               { :hosts => [:play_roles, :multiple_play_roles],
-                 :'api/v2/hosts' => [:play_roles,
-                                     :multiple_play_roles] },
-               :resource_type => 'Host'
-    permission :play_roles_on_hostgroup,
-               { :hostgroups => [:play_roles],
-                 :'api/v2/hostgroups' => [:play_roles,
-                                          :multiple_play_roles] },
-               :resource_type => 'Hostgroup'
-    permission :view_ansible_roles,
-               { :ansible_roles => [:index, :auto_complete_search],
-                 :'api/v2/ansible_roles' => [:index, :show, :fetch],
-                 :ui_ansible_roles => [:index] },
-               :resource_type => 'AnsibleRole'
-    permission :destroy_ansible_roles,
-               { :ansible_roles => [:destroy],
-                 :'api/v2/ansible_roles' => [:destroy, :obsolete] },
-               :resource_type => 'AnsibleRole'
-    permission :import_ansible_roles,
-               { :ansible_roles => [:import, :confirm_import],
-                 :'api/v2/ansible_roles' => [:import, :sync] },
-               :resource_type => 'AnsibleRole'
-    permission :view_ansible_variables,
-               {
-                 :lookup_values => [:index],
-                 :ansible_variables => [:index, :auto_complete_search],
-                 :'api/v2/ansible_variables' => [:index, :show]
-               },
-               :resource_type => 'AnsibleVariable'
-    permission :edit_ansible_variables,
-               { :lookup_values => [:update],
-                 :ansible_variables => [:edit, :update],
-                 :'api/v2/ansible_variables' => [:update],
-                 :'api/v2/ansible_override_values' => [:create, :destroy] },
-               :resource_type => 'AnsibleVariable'
-    permission :destroy_ansible_variables,
-               {
-                 :lookup_values => [:destroy],
-                 :ansible_variables => [:destroy],
-                 :'api/v2/ansible_variables' => [:destroy, :obsolete]
-               },
-               :resource_type => 'AnsibleVariable'
-    permission :create_ansible_variables,
-               {
-                 :lookup_values => [:create],
-                 :ansible_variables => [:new, :create],
-                 :'api/v2/ansible_variables' => [:create]
-               },
-               :resource_type => 'AnsibleVariable'
-    permission :import_ansible_variables,
-               {
-                 :ansible_variables => [:import, :confirm_import],
-                 :'api/v2/ansible_variables' => [:import]
-               },
-               :resource_type => 'AnsibleVariable'
-    permission :view_hosts,
-               { :'api/v2/hosts' => [:ansible_roles],
-                 :'api/v2/ansible_inventories' => [:hosts] },
-               :resource_type => 'Host'
-    permission :view_hostgroups,
-               { :'api/v2/hostgroups' => [:ansible_roles],
-                 :'api/v2/ansible_inventories' => [:hostgroups] },
-               :resource_type => 'Hostgroup'
-    permission :edit_hosts,
-               { :'api/v2/hosts' => [:assign_ansible_roles] },
-               :resource_type => 'Host'
-    permission :edit_hostgroups,
-               { :'api/v2/hostgroups' => [:assign_ansible_roles] },
-               :resource_type => 'Hostgroup'
-    permission :generate_ansible_inventory,
-               { :'api/v2/ansible_inventories' => [:schedule] }
-    permission :import_ansible_playbooks,
-               { :'api/v2/ansible_playbooks' => [:sync, :fetch] }
-  end
-
-  role 'Ansible Roles Manager',
-       [:play_roles_on_host, :play_roles_on_hostgroup,
-        :create_job_invocations, :view_job_templates,      # to allow the play_roles
-        :create_template_invocations, :view_smart_proxies, # ...
-        :view_ansible_roles, :destroy_ansible_roles,
-        :import_ansible_roles, :view_ansible_variables,
-        :create_ansible_variables, :import_ansible_variables,
-        :edit_ansible_variables, :destroy_ansible_variables, :import_ansible_playbooks]
-
-  role 'Ansible Tower Inventory Reader',
-       [:view_hosts, :view_hostgroups, :view_facts, :generate_report_templates, :generate_ansible_inventory,
-        :view_report_templates],
-       'Permissions required for the user which is used by Ansible Tower Dynamic Inventory Item'
-
-  add_all_permissions_to_default_roles
-  extend_template_helpers ForemanAnsible::RendererMethods
-  allowed_template_helpers :insights_remediation
-
-  base_role_assignment_params = { :ansible_role_ids => [],
-                                  :ansible_roles => [] }
-  parameter_filter Host::Managed, base_role_assignment_params.merge(:host_ansible_roles_attributes => {})
-  parameter_filter Hostgroup, base_role_assignment_params.merge(:hostgroup_ansible_roles_attributes => {})
-
-  register_global_js_file 'global'
-
-  extend_graphql_type :type => '::Types::Host' do
-    field :all_ansible_roles, ::Types::InheritedAnsibleRole.connection_type, :null => true, :method => :present_all_ansible_roles
-    field :own_ansible_roles, ::Types::AnsibleRole.connection_type, :null => true
-    field :available_ansible_roles, ::Types::AnsibleRole.connection_type, :null => true
-    field :ansible_variables_with_overrides, Types::OverridenAnsibleVariable.connection_type, :null => false
-
-    def present_all_ansible_roles
-      inherited_ansible_roles = object.inherited_ansible_roles.map { |role| ::Presenters::AnsibleRolePresenter.new(role, true) }
-      ansible_roles = object.ansible_roles.map { |role| ::Presenters::AnsibleRolePresenter.new(role, false) }
-      (inherited_ansible_roles + ansible_roles).uniq
-    end
-
-    def ansible_variables_with_overrides
-      resolver = ::ForemanAnsible::OverrideResolver.new(object)
-      AnsibleVariable.where(:ansible_role_id => object.all_ansible_roles.pluck(:id), :override => true).map { |variable| ::Presenters::OverridenAnsibleVariablePresenter.new variable, resolver }
-    end
-  end
-
-  register_graphql_query_field :ansible_roles, '::Types::AnsibleRole', :collection_field
-  register_graphql_mutation_field :assign_ansible_roles, '::Mutations::Hosts::AssignAnsibleRoles'
-  register_graphql_mutation_field :delete_ansible_variable_override, ::Mutations::AnsibleVariableOverrides::Delete
-  register_graphql_mutation_field :update_ansible_variable_override, ::Mutations::AnsibleVariableOverrides::Update
-  register_graphql_mutation_field :create_ansible_variable_override, ::Mutations::AnsibleVariableOverrides::Create
-
-  divider :top_menu, :caption => N_('Ansible'), :parent => :configure_menu
-  menu :top_menu, :ansible_roles,
-       :caption => N_('Roles'),
-       :url_hash => { :controller => :ansible_roles, :action => :index },
-       :parent => :configure_menu
-  menu :top_menu, :ansible_variables,
-       :caption => N_('Variables'),
-       :url_hash => { :controller => :ansible_variables, :action => :index },
-       :parent => :configure_menu
-
-  apipie_documented_controllers [
-    "#{ForemanAnsible::Engine.root}/app/controllers/api/v2/*.rb"
-  ]
-  ApipieDSL.configuration.dsl_classes_matchers += [
-    "#{ForemanAnsible::Engine.root}/app/models/*.rb",
-    "#{ForemanAnsible::Engine.root}/app/services/foreman_ansible/*.rb"
-  ]
-
-  register_info_provider ForemanAnsible::AnsibleInfo
-
-  # For backwards compatiblity with 1.17
-  if respond_to?(:register_report_scanner)
-    register_report_scanner ForemanAnsible::AnsibleReportScanner
-    register_report_origin 'Ansible', 'ConfigReport'
-  end
-
-<<<<<<< HEAD
-  describe_host do
-    multiple_actions_provider :ansible_hosts_multiple_actions
-=======
-  extend_page('smart_proxies/show') do |context|
-    context.add_pagelet :smart_proxy_title_actions,
-                        :name => _('Update Smart Proxy'),
-                        :partial => 'foreman/smart_proxies/update_smart_proxy',
-                        :onlyif => ->(proxy, view) { view.can_update_proxy?(proxy) }
-  end
-  extend_page('smart_proxies/index') do |context|
-    context.add_pagelet :smart_proxy_title_actions,
-                        :name => _('Update Smart Proxy'),
-                        :partial => 'foreman/smart_proxies/update_smart_proxy',
-                        :onlyif => ->(proxy, view) { view.can_update_proxy?(proxy) }
->>>>>>> a707dab (Fixes #35143 - Add a smart proxy update button)
-  end
-end