foreman_ansible 6.0.0 → 6.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3ee1b4acb26925c919a3d2b3c8b9efe6bd355dc603d896d642a0150d9364f86b
-  data.tar.gz: b5fb045a0cfddf957c238f311b29f97a56b0b5fc91999b56ca25ee3f4c0eeda4
+  metadata.gz: d95899e15050cbf3f25d75a8f3c9bc5a3059a9dd0ffa90f4690ce84cf49d3408
+  data.tar.gz: 466754ac90d1e80046fca70ca8db372e90d2b61a0d26a726f0807f7e81227728
 SHA512:
-  metadata.gz: 529107a643fb300fc1755a3efe8bbb1bb94b726c822c93f568e3001d2acc71658499343952bce1191ca2e142d1abc3548525a3fdfbde2c8fe1a2f645923b366c
-  data.tar.gz: 2874526de96c18b4aafd6fcaa29006c1d910ccb0e87b1f3326d64312c510cc496fc4e49fd176fafb3f465bded12a2b9c26768dfc0833b628c1538659279cd6a6
+  metadata.gz: 9f79b560e9e01e8719c6e705ff5fef6a9f9e7b91bc9ec1b9b79c81750b572c4de6929d2ab07049caefdf4f555a5087c31eb76543d52b87622038107a1e02ea63
+  data.tar.gz: 357b61ffdb94b61a37ecf87afba07f7e0c2158d4af14d39b701fb31152803692a8feaac09294bb2bc8c0ae8fe8d9daa66eb8eb6f0869c4901315dca688f06dab

data/app/controllers/api/v2/ansible_inventories_controller.rb

@@ -72,8 +72,8 @@ module Api
           process_resource_error(:resource => @ansible_inventory)
         end
       rescue StandardError => e
-        render_error 'standard_error', :status => :internal_error,
-                                       :locals => { :exception => e }
+        render_error :custom_error, :status => :unprocessable_entity,
+                                    :locals => { :message => _("Scheduling Report template failed for: #{e.message}") }
       end
 
       def action_permission

data/app/helpers/foreman_ansible/ansible_reports_helper.rb

@@ -42,10 +42,7 @@ module ForemanAnsible
     end
 
     def report_json_viewer(json)
-      uid = "reportjson-viewer-#{json.object_id}"
-      viewer = content_tag :div, '', :id => uid
-      viewer << mount_react_component('ReportJsonViewer',
-                                      "##{uid}", json.to_json)
+      react_component('ReportJsonViewer', data: json)
     end
 
     private
@@ -68,7 +65,7 @@ module ForemanAnsible
       prettyp.gsub!(/^(\[|\])/, '')
       prettyp.gsub!(/^[\s]*$\n/, '')
       paragraph_style = 'white-space:pre;padding: 2em 0'
-      content_tag(:p, prettyp, :style => paragraph_style)
+      tag(:p, prettyp, :style => paragraph_style)
     end
 
     def hash_with_keys_removed(hash)

data/app/models/concerns/foreman_ansible/host_managed_extensions.rb

@@ -68,9 +68,9 @@ end
 module Host
   class Managed
     apipie :class do
-      property :all_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns all ansible roles assigned to the host, both its own and inherited'
-      property :ansible_roles, array_of: 'AnsibleRole', desc: 'Returns ansible roles assigned to the host'
-      property :inherited_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns inherited ansible roles assigned to the host'
+      property :all_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns all ansible roles assigned to the host, both its own and inherited from the host group and its parents'
+      property :ansible_roles, array_of: 'AnsibleRole', desc: 'Returns ansible roles directly assigned to the host'
+      property :inherited_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns only ansible roles assigned to the host\'s host group and its parents'
     end
     # Methods to be allowed in any template with safemode enabled
     class Jail < Safemode::Jail

data/app/models/concerns/foreman_ansible/hostgroup_extensions.rb

@@ -36,3 +36,15 @@ module ForemanAnsible
     end
   end
 end
+
+class Hostgroup
+  apipie :class do
+    property :all_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns all ansible roles assigned to the host group, both its own and inherited from parent host groups'
+    property :ansible_roles, array_of: 'AnsibleRole', desc: 'Returns ansible roles directly assigned to the host group'
+    property :inherited_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns only the inherited ansible roles assigned to the host group\'s parents'
+  end
+  # Methods to be allowed in any template with safemode enabled
+  class Jail < Safemode::Jail
+    allow :all_ansible_roles, :ansible_roles, :inherited_ansible_roles
+  end
+end

data/app/models/foreman_ansible/ansible_provider.rb

@@ -35,7 +35,7 @@ if defined? ForemanRemoteExecution
           {
             'per-host' => {
               host.name => {
-                'ansible_ssh_pass' => rex_ssh_password(host),
+                'ansible_password' => rex_ssh_password(host),
                 'ansible_become_password' => rex_effective_user_password(host)
               }
             }
@@ -66,14 +66,6 @@ if defined? ForemanRemoteExecution
           'ForemanAnsibleCore::TaskLauncher::Playbook::PlaybookRunnerAction'
         end
 
-        def required_proxy_selector_for(template)
-          if template.remote_execution_features.where(:label => 'ansible_run_capsule_upgrade').any?
-            ::DefaultProxyProxySelector.new
-          else
-            super
-          end
-        end
-
         private
 
         def ansible_command?(template)

data/app/services/foreman_ansible/operating_system_parser.rb

@@ -80,8 +80,14 @@ module ForemanAnsible
         facts[:ansible_os_name].tr(" \n\t", '') ||
           facts[:ansible_distribution].tr(" \n\t", '')
       else
-        facts[:ansible_distribution] ||
-          facts[:ansible_lsb] && facts[:ansible_lsb]['id']
+        distribution = facts[:ansible_lsb].try(:[], 'id') || facts[:ansible_distribution]
+
+        if distribution == 'RedHat' &&
+           facts[:ansible_lsb].try(:[], 'id') == 'RedHatEnterpriseWorkstation'
+          distribution += '_Workstation'
+        end
+
+        distribution
       end
     end
 

data/app/services/foreman_ansible/variables_importer.rb

@@ -51,7 +51,8 @@ module ForemanAnsible
           :key => variable_name,
           :ansible_role_id => role.id
         )
-        variable.assign_attributes(:default_value => variable_default,
+        variable.assign_attributes(:hidden_value => false,
+                                   :default_value => variable_default,
                                    :key_type => infer_key_type(variable_default))
         variable.imported = true if variable.new_record?
         variable.valid? ? variable : nil
@@ -78,8 +79,10 @@ module ForemanAnsible
     def create_new_variables(variables)
       iterate_over_variables(variables) do |role, memo, attrs|
         variable = AnsibleVariable.new(
-          JSON.parse(attrs)['ansible_variable']
+          JSON.parse(attrs)
         )
+        # Make sure, newly created variables are not hidden
+        variable.hidden_value = false
         variable.ansible_role = ::AnsibleRole.find_by(:name => role)
         variable.save
         memo << variable
@@ -88,7 +91,9 @@ module ForemanAnsible
 
     def update_variables(variables)
       iterate_over_variables(variables) do |_role, memo, attrs|
-        attributes = JSON.parse(attrs)['ansible_variable']
+        attributes = JSON.parse(attrs)
+        # Make sure to let the flag if a variable is hidden untouched
+        attributes.delete('hidden_value')
         var = AnsibleVariable.find attributes['id']
         var.update(attributes)
         memo << var
@@ -98,7 +103,7 @@ module ForemanAnsible
     def delete_old_variables(variables)
       iterate_over_variables(variables) do |_role, memo, attrs|
         variable = AnsibleVariable.find(
-          JSON.parse(attrs)['ansible_variable']['id']
+          JSON.parse(attrs)['id']
         )
         memo << variable.key
         variable.destroy

data/app/views/ansible_roles/welcome.html.erb

@@ -4,7 +4,7 @@
     <%= icon_text("play", "", :kind => "fa") %>
   </div>
   <h1><%= _('Ansible Roles') %></h1>
-  <p><%= _('No ansible roles were found in Foreman. If you want to assign roles to your hosts,
+  <p><%= _('No Ansible Roles were found in Foreman. If you want to assign roles to your hosts,
              you have to import them first.').html_safe %>
   </p>
   <p><%= link_to(_('Learn more about this in the documentation.'), documentation_url('#4.1ImportingRoles', :root_url => ansible_doc_url), target: '_blank') %></p>

data/app/views/foreman_ansible/ansible_roles/_select_tab_content.html.erb

@@ -4,20 +4,12 @@
 <div class='tab-pane' id='ansible_roles'>
   <% roles = f.object.is_a?(Hostgroup) ? roles_attrs(f.object.inherited_and_own_ansible_roles) : roles_attrs(f.object.all_ansible_roles) %>
   <% class_name = f.object.is_a?(Hostgroup) ? 'Hostgroup' : 'Host' %>
-  <%= content_tag(:div, '',
-                  :id => 'ansible_roles_switcher',
-                  'data-roles' => {
-                    :initialAssignedRoles => roles,
-                    :inheritedRoleIds => f.object.inherited_ansible_roles.map(&:id),
-                    :availableRolesUrl => ui_ansible_roles_path,
-                    :resourceId => f.object.id,
-                    :resourceName => class_name,
-                    :canView => User.current.can?(:view_ansible_roles)
-                  }.to_json
-  ) %>
-  <% unless request.xhr? %>
-    <script type="text/javascript">
-      tfm.initAnsibleRoleSwitcher();
-    </script>
-  <% end %>
+  <%= react_component('AnsibleRolesSwitcher', {
+                                                :initialAssignedRoles => roles,
+                                                :inheritedRoleIds => f.object.inherited_ansible_roles.map(&:id),
+                                                :availableRolesUrl => ui_ansible_roles_path,
+                                                :resourceId => f.object.id,
+                                                :resourceName => class_name,
+                                                :canView => User.current.can?(:view_ansible_roles)
+                                              }) %>
 </div>

data/app/views/foreman_ansible/job_templates/ansible_roles_-_install_from_galaxy.erb

@@ -25,7 +25,7 @@ model: JobTemplate
 ---
 - hosts: all
   tasks:
-    - command: ansible-galaxy install {{ item }} <% "-p #{input('location')}" if input('location').present? %>
+    - command: ansible-galaxy install {{ item }} -p <%= input('location').present? ? input('location') : '/etc/ansible/roles' %>
       register: out
       with_items:
         - <%= input('ansible_roles_list') %>

data/app/views/foreman_ansible/job_templates/capsule_upgrade_-_ansible_default.erb

@@ -34,11 +34,10 @@ feature: ansible_run_capsule_upgrade
         msg: "This playbook cannot be executed on a Satellite server. Use only on a Capsule server."
       when: "'satellite' in ansible_facts.packages"
 
-    - name: Install satellite-maintain if not present
+    - name: Install|Update satellite-maintain if not present
       package:
         name: rubygem-foreman_maintain
-        state: present
-      when: "'rubygem-foreman_maintain' not in ansible_facts.packages"
+        state: latest
 
     - block:
       <%- whitelist_option = if input('whitelist_options').present?

data/app/views/foreman_ansible/job_templates/configure_cloud_connector_-_ansible_default.erb

@@ -26,6 +26,7 @@ job_category: Maintenance Operations
 description_format: "%{template_name}"
 provider_type: Ansible
 kind: job_template
+feature: ansible_configure_cloud_connector
 %>
 
 ---

data/app/views/foreman_ansible/job_templates/service_action_-_enable_web_console.erb

@@ -0,0 +1,16 @@
+<%#
+name: Service Action - Enable Web Console
+job_category: Ansible Services
+snippet: false
+provider_type: Ansible
+kind: job_template
+model: JobTemplate
+feature: ansible_enable_web_console
+%>
+---
+- hosts: all
+  tasks:
+  - name: ensure cockpit is installed
+    package:
+      name: "cockpit-system"
+      state: present

data/lib/foreman_ansible/remote_execution.rb

@@ -45,7 +45,14 @@ module ForemanAnsible
       RemoteExecutionFeature.register(
         :ansible_run_capsule_upgrade,
         N_('Upgrade Capsules on given hosts'),
-        :description => N_('Upgrade Capsules on given Capsule server hosts')
+        :description => N_('Upgrade Capsules on given Capsule server hosts'),
+        :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
+      )
+      RemoteExecutionFeature.register(
+        :ansible_configure_cloud_connector,
+        N_('Configure Cloud Connector on given hosts'),
+        :description => N_('Configure Cloud Connector on given hosts'),
+        :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
       )
     end
   end

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 = '6.0.0'
+  VERSION = '6.2.0'
 end

data/package.json

@@ -7,33 +7,24 @@
     "test": "test"
   },
   "peerDependencies": {
-    "@theforeman/vendor": ">= 3.3.2"
+    "@theforeman/vendor": ">= 6.0.0"
   },
   "dependencies": {
     "react-json-tree": "^0.11.0"
   },
   "devDependencies": {
     "@babel/core": "^7.7.0",
-    "@theforeman/builder": "^4.0.2",
-    "@theforeman/vendor-dev": "^3.3.2",
-    "babel-eslint": "^10.0.0",
-    "babel-jest": "^24.9.0",
-    "enzyme": "^3.7.0",
-    "enzyme-adapter-react-16": "^1.7.0",
-    "enzyme-to-json": "^3.3.5",
-    "eslint": "^6.0.0",
-    "eslint-plugin-patternfly-react": "^0.2.1",
-    "identity-obj-proxy": "^3.0.0",
-    "jest": "^24.9.0",
-    "prettier": "^1.19.0",
-    "react-redux": "^5.0.7",
-    "react-redux-test-utils": "^0.1.1",
-    "redux": "^3.7.2",
-    "redux-thunk": "^2.3.0"
+    "@theforeman/builder": "^6.0.0",
+    "@theforeman/eslint-plugin-foreman": "6.0.0",
+    "@theforeman/test": "^6.0.0",
+    "@theforeman/vendor-dev": "^6.0.0",
+    "babel-eslint": "^10.0.3",
+    "eslint": "^6.7.2",
+    "prettier": "^1.13.5"
   },
   "scripts": {
-    "test": "node node_modules/.bin/jest webpack",
-    "lint": "eslint ./webpack"
+    "test": "tfm-test --plugin",
+    "lint": "tfm-lint --plugin -d webpack"
   },
   "repository": {
     "type": "git",
@@ -44,27 +35,5 @@
   "bugs": {
     "url": "https://projects.theforeman.org/projects/ansible"
   },
-  "homepage": "https://theforeman.org/plugins/foreman_ansible/",
-  "jest": {
-    "verbose": true,
-    "moduleDirectories": [
-      "node_modules/@theforeman/vendor-core/node_modules",
-      "node_modules",
-      "webpack"
-    ],
-    "transform": {
-      "^.+\\.js$": "babel-jest"
-    },
-    "setupFiles": [
-      "raf/polyfill",
-      "./webpack/test_setup.js"
-    ],
-    "testPathIgnorePatterns": [
-      "/node_modules/",
-      "<rootDir>/foreman/"
-    ],
-    "moduleNameMapper": {
-      "^.+\\.(css|scss)$": "identity-obj-proxy"
-    }
-  }
+  "homepage": "https://theforeman.org/plugins/foreman_ansible/"
 }

data/test/unit/ansible_provider_test.rb

@@ -40,7 +40,7 @@ class AnsibleProviderTest < ActiveSupport::TestCase
         host.expects(:params).twice.returns(params)
         secrets = ForemanAnsible::AnsibleProvider.secrets(host)
         host_secrets = secrets['per-host'][host.name]
-        assert_equal host_secrets['ansible_ssh_pass'], 'password'
+        assert_equal host_secrets['ansible_password'], 'password'
         assert_equal host_secrets['ansible_become_password'], 'letmein'
       end
     end

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

@@ -24,7 +24,7 @@ module ForemanAnsibleCore
             '_meta' => { 'hostvars' => { 'foreman.example.com' => {} } } }
         end
         let(:input) do
-          host_secrets = { 'ansible_ssh_pass' => 'letmein', 'ansible_become_password' => 'iamroot' }
+          host_secrets = { 'ansible_password' => 'letmein', 'ansible_become_password' => 'iamroot' }
           secrets = { 'per-host' => { 'foreman.example.com' => host_secrets } }
           host_input = { 'input' => { 'action_input' => { 'secrets' => secrets } } }
           { 'foreman.example.com' => host_input }
@@ -35,14 +35,14 @@ module ForemanAnsibleCore
           test_inventory = inventory.merge('ssh_password' => 'sshpass', 'effective_user_password' => 'mypass')
           rebuilt = runner.send(:rebuild_secrets, test_inventory, input)
           host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-          assert_equal 'sshpass', host_vars['ansible_ssh_pass']
+          assert_equal 'sshpass', host_vars['ansible_password']
           assert_equal 'mypass', host_vars['ansible_become_password']
         end
 
         test 'host secrets are used when not overriden by inventory secrest' do
           rebuilt = runner.send(:rebuild_secrets, inventory, input)
           host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-          assert_equal 'letmein', host_vars['ansible_ssh_pass']
+          assert_equal 'letmein', host_vars['ansible_password']
           assert_equal 'iamroot', host_vars['ansible_become_password']
         end
       end

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

@@ -87,7 +87,7 @@ class PlaybookRunnerTest < ActiveSupport::TestCase
         '_meta' => { 'hostvars' => { 'foreman.example.com' => {} } } }
     end
     let(:secrets) do
-      host_secrets = { 'ansible_ssh_pass' => 'letmein', 'ansible_become_password' => 'iamroot' }
+      host_secrets = { 'ansible_password' => 'letmein', 'ansible_become_password' => 'iamroot' }
       { 'per-host' => { 'foreman.example.com' => host_secrets } }
     end
     let(:runner) { ForemanAnsibleCore::Runner::Playbook.allocate }
@@ -96,14 +96,14 @@ class PlaybookRunnerTest < ActiveSupport::TestCase
       test_inventory = inventory.merge('ssh_password' => 'sshpass', 'effective_user_password' => 'mypass')
       rebuilt = runner.send(:rebuild_secrets, test_inventory, secrets)
       host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-      assert_equal 'sshpass', host_vars['ansible_ssh_pass']
+      assert_equal 'sshpass', host_vars['ansible_password']
       assert_equal 'mypass', host_vars['ansible_become_password']
     end
 
     test 'host secrets are used when not overriden by inventory secrest' do
       rebuilt = runner.send(:rebuild_secrets, inventory, secrets)
       host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-      assert_equal 'letmein', host_vars['ansible_ssh_pass']
+      assert_equal 'letmein', host_vars['ansible_password']
       assert_equal 'iamroot', host_vars['ansible_become_password']
     end
   end

data/test/unit/services/ansible_variables_importer_test.rb

@@ -59,6 +59,19 @@ class AnsibleVariablesImporterTest < ActiveSupport::TestCase
       assert_empty changes['update']
     end
 
+    test 'sets hidden value to false by default' do
+      role = FactoryBot.create(:ansible_role)
+      variable = FactoryBot.create(:ansible_variable)
+      api_response = {
+        role.name => { variable.key => 'new value' }
+      }
+      changes = importer.import_variables(api_response, [role.name])
+      assert_empty changes['update']
+      assert_not_empty changes['new']
+      assert_equal variable.key, changes['new'].first.key
+      assert_not changes['new'].first.hidden_value?
+    end
+
     test 'reimports variable with same key for different role' do
       role = FactoryBot.create(:ansible_role)
       variable = FactoryBot.create(:ansible_variable)

data/test/unit/services/inventory_creator_test.rb

@@ -69,7 +69,7 @@ module ForemanAnsible
                    connection_params['ansible_winrm_server_cert_validation']
       assert_equal Setting['remote_execution_effective_user_method'],
                    connection_params['ansible_become_method']
-      refute connection_params.key?('ansible_ssh_pass')
+      refute connection_params.key?('ansible_password')
       refute connection_params.key?('ansible_become_password')
     end
 

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcher.js

@@ -17,7 +17,7 @@ class AnsibleRolesSwitcher extends React.Component {
       inheritedRoleIds,
       resourceId,
       resourceName,
-    } = this.props.data;
+    } = this.props;
 
     this.props.getAnsibleRoles(
       availableRolesUrl,
@@ -52,7 +52,7 @@ class AnsibleRolesSwitcher extends React.Component {
       inheritedRoleIds,
       resourceId,
       resourceName,
-    } = this.props.data;
+    } = this.props;
 
     const onListingChange = paginationArgs =>
       getAnsibleRoles(
@@ -104,13 +104,11 @@ class AnsibleRolesSwitcher extends React.Component {
 }
 
 AnsibleRolesSwitcher.propTypes = {
-  data: PropTypes.shape({
-    initialAssignedRoles: PropTypes.arrayOf(PropTypes.object),
-    availableRolesUrl: PropTypes.string,
-    inheritedRoleIds: PropTypes.arrayOf(PropTypes.number),
-    resourceId: PropTypes.number,
-    resourceName: PropTypes.string,
-  }).isRequired,
+  initialAssignedRoles: PropTypes.arrayOf(PropTypes.object),
+  availableRolesUrl: PropTypes.string.isRequired,
+  inheritedRoleIds: PropTypes.arrayOf(PropTypes.number),
+  resourceId: PropTypes.number,
+  resourceName: PropTypes.string,
   getAnsibleRoles: PropTypes.func.isRequired,
   loading: PropTypes.bool.isRequired,
   pagination: PropTypes.shape({
@@ -137,6 +135,10 @@ AnsibleRolesSwitcher.propTypes = {
 
 AnsibleRolesSwitcher.defaultProps = {
   error: {},
+  resourceId: null,
+  resourceName: '',
+  initialAssignedRoles: [],
+  inheritedRoleIds: [],
 };
 
 export default AnsibleRolesSwitcher;

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcherActions.js

@@ -21,7 +21,7 @@ export const getAnsibleRoles = (
   resourceName,
   pagination,
   search
-) => dispatch => {
+) => async dispatch => {
   dispatch({ type: ANSIBLE_ROLES_REQUEST });
 
   const params = {
@@ -30,19 +30,19 @@ export const getAnsibleRoles = (
     ...propsToSnakeCase({ resourceId, resourceName }),
   };
 
-  return api
-    .get(url, {}, params)
-    .then(({ data }) =>
-      dispatch({
-        type: ANSIBLE_ROLES_SUCCESS,
-        payload: {
-          initialAssignedRoles,
-          inheritedRoleIds,
-          ...propsToCamelCase(data),
-        },
-      })
-    )
-    .catch(error => dispatch(errorHandler(ANSIBLE_ROLES_FAILURE, error)));
+  try {
+    const res = await api.get(url, {}, params);
+    return dispatch({
+      type: ANSIBLE_ROLES_SUCCESS,
+      payload: {
+        initialAssignedRoles,
+        inheritedRoleIds,
+        ...propsToCamelCase(res.data),
+      },
+    });
+  } catch (error) {
+    return dispatch(errorHandler(ANSIBLE_ROLES_FAILURE, error));
+  }
 };
 
 const errorHandler = (msg, err) => {

data/webpack/components/AnsibleRolesSwitcher/__tests__/AnsibleRolesSwitcher.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import AnsibleRolesSwitcher from '../AnsibleRolesSwitcher';
 
@@ -19,10 +19,10 @@ const fixtures = {
     assignedRolesCount: 2,
     assignedRoles: [],
     unassignedRoles: [],
-    data: {
-      initialAssignedRoles: [],
-    },
+    initialAssignedRoles: [],
     error: { statusText: '', errorMsg: '' },
+    allAssignedRoles: [],
+    availableRolesUrl: 'http://test/roles',
   },
 };
 

data/webpack/components/AnsibleRolesSwitcher/__tests__/AnsibleRolesSwitcherReducer.test.js

@@ -1,4 +1,4 @@
-import { testReducerSnapshotWithFixtures } from 'react-redux-test-utils';
+import { testReducerSnapshotWithFixtures } from '@theforeman/test';
 
 import reducer, { initialState } from '../AnsibleRolesSwitcherReducer';
 import { ansibleRolesLong } from '../__fixtures__/ansibleRolesData.fixtures';

data/webpack/components/AnsibleRolesSwitcher/__tests__/AnsibleRolesSwitcherSelectors.test.js

@@ -1,4 +1,4 @@
-import { testSelectorsSnapshotWithFixtures } from 'react-redux-test-utils';
+import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
 
 import {
   selectUnassignedRoles,

data/webpack/components/AnsibleRolesSwitcher/__tests__/__snapshots__/AnsibleRolesSwitcher.test.js.snap

@@ -61,6 +61,7 @@ exports[`AnsibleRolesSwitcher should render 1`] = `
         </h2>
       </div>
       <AssignedRolesList
+        allAssignedRoles={Array []}
         assignedRoles={Array []}
         itemCount={2}
         onPaginationChange={[Function]}

data/webpack/components/AnsibleRolesSwitcher/components/AnsiblePermissionDenied.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 import AnsiblePermissionDenied from './AnsiblePermissionDenied';
 
 jest.mock('foremanReact/components/common/EmptyState');

data/webpack/components/AnsibleRolesSwitcher/components/AnsibleRole.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import AnsibleRole from './AnsibleRole';
 

data/webpack/components/AnsibleRolesSwitcher/components/AssignedRolesList.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import AssignedRolesList from './AssignedRolesList';
 

data/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import AvailableRolesList from './AvailableRolesList';
 

data/webpack/components/AnsibleRolesSwitcher/index.js

@@ -37,5 +37,5 @@ const mapDispatchToProps = dispatch =>
 export default withProtectedView(
   connect(mapStateToProps, mapDispatchToProps)(AnsibleRolesSwitcher),
   AnsiblePermissionDenied,
-  props => props.data && props.data.canView
+  props => props.canView
 );

data/webpack/index.js

@@ -1,12 +1,9 @@
-import $ from 'jquery';
 import componentRegistry from 'foremanReact/components/componentRegistry';
 import injectReducer from 'foremanReact/redux/reducers/registerReducer';
 import ReportJsonViewer from './components/ReportJsonViewer';
 import AnsibleRolesSwitcher from './components/AnsibleRolesSwitcher';
 import reducer from './reducer';
 
-const { tfm } = window;
-
 componentRegistry.register({
   name: 'ReportJsonViewer',
   type: ReportJsonViewer,
@@ -17,13 +14,3 @@ componentRegistry.register({
 });
 
 injectReducer('foremanAnsible', reducer);
-
-tfm.initAnsibleRoleSwitcher = () => {
-  $(document).on('ContentLoad', evt => {
-    tfm.reactMounter.mount(
-      'AnsibleRolesSwitcher',
-      '#ansible_roles_switcher',
-      $('#ansible_roles_switcher').data('roles')
-    );
-  });
-};

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.2.0
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-01 00:00:00.000000000 Z
+date: 2021-02-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foreman_ansible_core
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.0.0
+        version: 4.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.0.0
+        version: 4.2.0
 - !ruby/object:Gem::Dependency
   name: ipaddress
   requirement: !ruby/object:Gem::Requirement
@@ -182,6 +182,7 @@ files:
 - app/views/foreman_ansible/job_templates/run_command_-_ansible_default.erb
 - app/views/foreman_ansible/job_templates/run_playbook-ansible_default.erb
 - app/views/foreman_ansible/job_templates/service_action_-_ansible_default.erb
+- app/views/foreman_ansible/job_templates/service_action_-_enable_web_console.erb
 - app/views/ui_ansible_roles/index.json.rabl
 - app/views/ui_ansible_roles/main.json.rabl
 - app/views/ui_ansible_roles/show.json.rabl
@@ -327,7 +328,6 @@ files:
 - webpack/components/ReportJsonViewer.js
 - webpack/index.js
 - webpack/reducer.js
-- webpack/test_setup.js
 homepage: https://github.com/theforeman/foreman_ansible
 licenses:
 - GPL-3.0
@@ -347,28 +347,31 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Ansible integration with Foreman (theforeman.org)
 test_files:
 - test/factories/ansible_proxy.rb
-- test/factories/ansible_roles.rb
 - test/factories/ansible_variables.rb
+- test/factories/ansible_roles.rb
 - test/fixtures/insights_playbook.yaml
-- test/fixtures/sample_facts.json
 - test/fixtures/report.json
+- test/fixtures/sample_facts.json
+- test/functional/ansible_roles_controller_test.rb
+- test/functional/api/v2/ansible_inventories_controller_test.rb
 - test/functional/api/v2/ansible_roles_controller_test.rb
+- test/functional/api/v2/ansible_variables_controller_test.rb
 - test/functional/api/v2/hostgroups_controller_test.rb
 - test/functional/api/v2/hosts_controller_test.rb
-- test/functional/api/v2/ansible_inventories_controller_test.rb
-- test/functional/api/v2/ansible_variables_controller_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/functional/ansible_variables_controller_test.rb
 - test/functional/hosts_controller_test.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/ansible_variable_test.rb
 - test/unit/concerns/config_reports_extensions_test.rb
 - test/unit/concerns/host_managed_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
@@ -377,20 +380,17 @@ test_files:
 - test/unit/lib/foreman_ansible_core/ansible_runner_test.rb
 - test/unit/lib/foreman_ansible_core/playbook_runner_test.rb
 - test/unit/lib/proxy_api/ansible_test.rb
+- test/unit/services/ansible_report_importer_test.rb
 - test/unit/services/api_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/insights_plan_runner_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/ansible_report_importer_test.rb
-- test/unit/services/fact_importer_test.rb
-- test/unit/services/fact_parser_test.rb
 - test/unit/services/ansible_variables_importer_test.rb
 - test/unit/services/inventory_creator_test.rb
-- test/unit/ansible_role_test.rb
+- test/unit/services/ui_roles_importer_test.rb
+- test/unit/ansible_provider_test.rb
 - test/unit/host_ansible_role_test.rb
 - test/unit/hostgroup_ansible_role_test.rb
-- test/unit/ansible_variable_test.rb
-- test/unit/ansible_provider_test.rb
-- test/test_plugin_helper.rb

data/webpack/test_setup.js

@@ -1,12 +0,0 @@
-import 'core-js/shim';
-import 'regenerator-runtime/runtime';
-
-import { configure } from 'enzyme';
-import Adapter from 'enzyme-adapter-react-16';
-
-configure({ adapter: new Adapter() });
-
-// Mocking translation function
-global.__ = str => str;
-global.n__ = str => str;
-global.Jed = { sprintf: str => str };