foreman_puppet 10.0.0 → 10.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc7813b3e70470b1ede75609466fd1f395b2b24415a6fc30dea85fcf9431c10a
-  data.tar.gz: 3f6e2c28bdb0143c86e1182b201afb8eb81a9753c0fcd8198c5bf4996a3e995f
+  metadata.gz: 6897f1c5899747e2b70508bfa86bd240db54e8cef027b369941302e4de585814
+  data.tar.gz: 81360486bd6be95f0a73be4132766b4a8b561e0020e26589e4a08b90b6318ba7
 SHA512:
-  metadata.gz: 5c2df904975315458ad27f09262c0025c9240d8750b89f2325a86bcd53abbd7c9e8d4d5ded37aebe594b069b86f3e0697be806652bfba01a8a42931a2b38cbca
-  data.tar.gz: 9708de0a826955c059856924d0fd206d76b2b21f8905e450b030c8876659f7f02692520cea3888701028e330947c4e556f1741dd3f21f6397124b1dc2281b1a9
+  metadata.gz: 2391cce05ed4cf77c2f84ffa205306d563add4ad6a361002583a791debae0b8166d6ef4f196fd57b9b1896dd983d8d35f4e7e6b79fac3362a1ceb0d11a65d82e
+  data.tar.gz: 852d1503ca1443e667ccf8f70f7271ba4baaadd067878ad62a28ff0426921d6597f9f245a1232948e9ac1e2b0573b4c74634b0012ab80991895899b11a7ccff1

data/app/controllers/foreman_puppet/api/v2/hosts_bulk_actions_controller.rb

@@ -0,0 +1,102 @@
+module ForemanPuppet
+  module Api
+    module V2
+      class HostsBulkActionsController < ::ForemanPuppet::Api::V2::PuppetBaseController
+        include ::Api::V2::BulkHostsExtension
+        before_action :find_editable_hosts, only: %i[change_puppet_proxy remove_puppet_proxy]
+        before_action :find_smart_proxy, only: %i[change_puppet_proxy]
+
+        def_param_group :bulk_params do
+          param :organization_id, :number, required: true, desc: N_('ID of the organization')
+          param :included, Hash, required: true, action_aware: true do
+            param :search, String, required: false, desc: N_('Search string for hosts to perform an action on')
+            param :ids, Array, required: false, desc: N_('List of host ids to perform an action on')
+          end
+          param :excluded, Hash, required: true, action_aware: true do
+            param :ids, Array, required: false, desc: N_('List of host ids to exclude and not run an action on')
+          end
+        end
+
+        api :PUT, '/hosts/bulk/change_puppet_proxy', N_('Change Puppet (CA) Proxy')
+        param_group :bulk_params
+        param :proxy_id, :number, required: true, desc: N_('ID of the Puppet proxy to reassign the hosts to')
+        param :ca_proxy, :bool, required: true, desc: N_('True, if Puppet CA proxy should be changed instead of the Puppet proxy')
+        def change_puppet_proxy
+          error_hosts = ::BulkHostsManager.new(hosts: @hosts).change_puppet_proxy(@proxy, ca_proxy?)
+          process_bulk_puppet_proxy_response(
+            error_hosts,
+            success_message: format(n_(
+              'Updated host: changed %{proxy_type}',
+              'Updated hosts: changed %{proxy_type}',
+              @hosts.count
+            ), proxy_type: proxy_type),
+            error_message: format(n_(
+              'Failed to change %{proxy_type} for %{count} host',
+              'Failed to change %{proxy_type} for %{count} hosts',
+              error_hosts.count
+            ), proxy_type: proxy_type, count: error_hosts.count)
+          )
+        end
+
+        api :PUT, '/hosts/bulk/remove_puppet_proxy', N_('Remove Puppet (CA) Proxy')
+        param_group :bulk_params
+        param :ca_proxy, :bool, required: true, desc: N_('True, if Puppet CA proxy should be removed instead of the Puppet proxy')
+        def remove_puppet_proxy
+          error_hosts = ::BulkHostsManager.new(hosts: @hosts).change_puppet_proxy(nil, ca_proxy?)
+          process_bulk_puppet_proxy_response(
+            error_hosts,
+            success_message: format(n_(
+              'Updated host: removed %{proxy_type}',
+              'Updated hosts: removed %{proxy_type}',
+              @hosts.count
+            ), proxy_type: proxy_type),
+            error_message: format(n_(
+              'Failed to remove %{proxy_type} for %{count} host',
+              'Failed to remove %{proxy_type} for %{count} hosts',
+              error_hosts.count
+            ), proxy_type: proxy_type, count: error_hosts.count)
+          )
+        end
+
+        private
+
+        def find_editable_hosts
+          find_bulk_hosts(:edit_hosts, params)
+        end
+
+        def process_bulk_puppet_proxy_response(error_hosts, success_message:, error_message:)
+          if error_hosts.empty?
+            process_response(true, { message: success_message })
+          else
+            render_error(:bulk_hosts_error, status: :unprocessable_entity,
+              locals: { message: error_message, failed_host_ids: error_hosts })
+          end
+        end
+
+        def find_smart_proxy
+          feature = ca_proxy? ? 'Puppet CA' : 'Puppet'
+          @proxy = SmartProxy.with_features(feature).find_by(id: params[:proxy_id])
+
+          if @proxy.nil?
+            render json: {
+              error: {
+                message: format(_('A Smart Proxy with id %{id} and the %{proxy_type} feature could not be found.'), id: params[:proxy_id], proxy_type: proxy_type),
+              },
+            }, status: :unprocessable_entity
+            false
+          else
+            true
+          end
+        end
+
+        def ca_proxy?
+          Foreman::Cast.to_bool(params[:ca_proxy])
+        end
+
+        def proxy_type
+          ca_proxy? ? _('Puppet CA proxy') : _('Puppet proxy')
+        end
+      end
+    end
+  end
+end

data/app/services/concerns/foreman_puppet/extensions/bulk_hosts_manager.rb

@@ -0,0 +1,24 @@
+module ForemanPuppet
+  module Extensions
+    module BulkHostsManager
+      extend ActiveSupport::Concern
+
+      def change_puppet_proxy(proxy, is_ca_proxy)
+        error_hosts = []
+        @hosts.each do |host|
+          if is_ca_proxy
+            host.puppet_ca_proxy = proxy
+          else
+            host.puppet_proxy = proxy
+          end
+          host.save(validate: false)
+        rescue StandardError => e
+          message = format(_('Failed to set proxy for %{host}.'), host: host)
+          Foreman::Logging.exception(message, e)
+          error_hosts << host.id
+        end
+        error_hosts
+      end
+    end
+  end
+end

data/config/api_routes.rb

@@ -1,3 +1,14 @@
+Foreman::Application.routes.draw do
+  scope module: 'foreman_puppet' do
+    namespace :api, defaults: { format: 'json' } do
+      scope '(:apiv)', module: :v2, defaults: { apiv: 'v2' }, apiv: /v1|v2/, constraints: ApiConstraints.new(version: 2, default: true) do
+        match 'hosts/bulk/change_puppet_proxy', to: 'hosts_bulk_actions#change_puppet_proxy', via: [:put]
+        match 'hosts/bulk/remove_puppet_proxy', to: 'hosts_bulk_actions#remove_puppet_proxy', via: [:put]
+      end
+    end
+  end
+end
+
 ForemanPuppet::Engine.routes.draw do
   namespace :api, defaults: { format: 'json' } do
     scope '(:apiv)', module: :v2, defaults: { apiv: 'v2' }, apiv: /v1|v2/, constraints: ApiConstraints.new(version: 2, default: true) do

data/lib/foreman_puppet/engine.rb

@@ -49,6 +49,7 @@ module ForemanPuppet
       ::ProvisioningTemplate.include ForemanPuppet::Extensions::ProvisioningTemplate
 
       ::HostCounter.prepend ForemanPuppet::Extensions::HostCounter
+      ::BulkHostsManager.include ForemanPuppet::Extensions::BulkHostsManager
 
       ::Api::V2::BaseController.include ForemanPuppet::Extensions::ApiBaseController
       ::Api::V2::HostsController.include ForemanPuppet::Extensions::ApiV2HostsController

data/lib/foreman_puppet/register.rb

@@ -62,6 +62,8 @@ Foreman::Plugin.register :foreman_puppet do
       p.actions << 'hosts/update_multiple_environment'
       p.actions << 'hosts/select_multiple_puppet_proxy'
       p.actions << 'hosts/update_multiple_puppet_proxy'
+      p.actions << 'foreman_puppet/api/v2/hosts_bulk_actions/change_puppet_proxy'
+      p.actions << 'foreman_puppet/api/v2/hosts_bulk_actions/remove_puppet_proxy'
     end
     p.actions << 'foreman_puppet/puppetclasses/parameters'
   end

data/lib/foreman_puppet/version.rb

@@ -1,3 +1,3 @@
 module ForemanPuppet
-  VERSION = '10.0.0'.freeze
+  VERSION = '10.1.0'.freeze
 end

data/test/controllers/foreman_puppet/api/v2/hosts_bulk_actions_controller_test.rb

@@ -0,0 +1,171 @@
+require 'test_puppet_helper'
+
+module ForemanPuppet
+  module Api
+    module V2
+      class HostsBulkActionsControllerTest < ActionController::TestCase
+        tests ::ForemanPuppet::Api::V2::HostsBulkActionsController
+
+        setup do
+          @routes = ::Foreman::Application.routes
+        end
+
+        let(:host) { FactoryBot.create(:host, :with_puppet_enc) }
+        let(:host2) do
+          FactoryBot.create(:host, :with_puppet_enc,
+            organization: host.organization,
+            location: host.location)
+        end
+        let(:proxy) { FactoryBot.create(:puppet_and_ca_smart_proxy, organizations: [host.organization], locations: [host.location]) }
+
+        test 'changes puppet proxy for selected hosts' do
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: proxy.id, ca_proxy: false)
+
+          assert_response :success
+          assert_equal proxy.id, host2.reload.puppet_proxy_id
+          assert_equal proxy.id, host.reload.puppet_proxy_id
+        end
+
+        test 'changes puppet ca proxy for selected hosts' do
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: proxy.id, ca_proxy: true)
+
+          assert_response :success
+          assert_equal proxy.id, host2.reload.puppet_ca_proxy_id
+          assert_equal proxy.id, host.reload.puppet_ca_proxy_id
+        end
+
+        test 'removes puppet proxy for selected hosts' do
+          host.update!(puppet_proxy: proxy)
+          host2.update!(puppet_proxy: proxy)
+
+          assert_equal proxy, host.reload.puppet_proxy
+
+          put :remove_puppet_proxy,
+            params: bulk_params.merge(ca_proxy: false),
+            session: set_session_user
+
+          assert_response :success
+          assert_nil host.reload.puppet_proxy
+          assert_nil host2.reload.puppet_proxy
+        end
+
+        test 'returns error when puppet proxy is missing' do
+          missing_proxy_id = 999_999
+
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: missing_proxy_id, ca_proxy: false),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal "A Smart Proxy with id #{missing_proxy_id} and the Puppet proxy feature could not be found.",
+            response.dig('error', 'message')
+        end
+
+        test 'returns error when puppet ca proxy is missing' do
+          missing_proxy_id = 999_999
+
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: missing_proxy_id, ca_proxy: true),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal "A Smart Proxy with id #{missing_proxy_id} and the Puppet CA proxy feature could not be found.",
+            response.dig('error', 'message')
+        end
+
+        test 'returns error when smart proxy is missing puppet feature' do
+          invalid_proxy = FactoryBot.create(:smart_proxy, organizations: [host.organization], locations: [host.location])
+          invalid_proxy.smart_proxy_feature_by_name('Puppet')&.destroy!
+
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: invalid_proxy.id, ca_proxy: false),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal "A Smart Proxy with id #{invalid_proxy.id} and the Puppet proxy feature could not be found.",
+            response.dig('error', 'message')
+        end
+
+        test 'returns error when smart proxy is missing puppet ca feature' do
+          invalid_proxy = FactoryBot.create(:puppet_smart_proxy, organizations: [host.organization], locations: [host.location])
+
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: invalid_proxy.id, ca_proxy: true),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal "A Smart Proxy with id #{invalid_proxy.id} and the Puppet CA proxy feature could not be found.",
+            response.dig('error', 'message')
+        end
+
+        test 'returns error when changing puppet proxy fails for some hosts' do
+          ::BulkHostsManager.any_instance.expects(:change_puppet_proxy)
+                            .with(proxy, false)
+                            .returns([host2.id])
+
+          put :change_puppet_proxy,
+            params: bulk_params.merge(proxy_id: proxy.id, ca_proxy: false),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal 'Failed to change Puppet proxy for 1 host',
+            response.dig('error', 'message')
+          assert_equal [host2.id], response.dig('error', 'failed_host_ids')
+        end
+
+        test 'returns error when removing puppet proxy fails for some hosts' do
+          host.update!(puppet_proxy: proxy)
+          host2.update!(puppet_proxy: proxy)
+          ::BulkHostsManager.any_instance.expects(:change_puppet_proxy)
+                            .with(nil, false)
+                            .returns([host.id])
+
+          put :remove_puppet_proxy,
+            params: bulk_params.merge(ca_proxy: false),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal 'Failed to remove Puppet proxy for 1 host',
+            response.dig('error', 'message')
+          assert_equal [host.id], response.dig('error', 'failed_host_ids')
+          assert_equal proxy.id, host.reload.puppet_proxy_id
+        end
+
+        test 'returns error when removing puppet ca proxy fails for some hosts' do
+          host.update!(puppet_ca_proxy: proxy)
+          host2.update!(puppet_ca_proxy: proxy)
+          ::BulkHostsManager.any_instance.expects(:change_puppet_proxy)
+                            .with(nil, true)
+                            .returns([host.id])
+
+          put :remove_puppet_proxy,
+            params: bulk_params.merge(ca_proxy: true),
+            session: set_session_user
+
+          assert_response :unprocessable_entity
+          response = JSON.parse(@response.body)
+          assert_equal 'Failed to remove Puppet CA proxy for 1 host',
+            response.dig('error', 'message')
+          assert_equal [host.id], response.dig('error', 'failed_host_ids')
+          assert_equal proxy.id, host.reload.puppet_ca_proxy_id
+        end
+
+        def bulk_params
+          {
+            organization_id: host.organization_id,
+            included: { ids: [host.id, host2.id] },
+            excluded: { ids: [] },
+          }
+        end
+      end
+    end
+  end
+end

data/test/services/foreman_puppet/bulk_hosts_manager_test.rb

@@ -0,0 +1,33 @@
+require 'test_puppet_helper'
+
+class BulkHostsManagerTest < ActiveSupport::TestCase
+  let(:hosts) { FactoryBot.create_list(:host, 2, :with_puppet_enc) }
+  let(:manager) { ::BulkHostsManager.new(hosts: hosts) }
+  let(:proxy) { FactoryBot.create(:puppet_smart_proxy) }
+
+  test 'changes puppet proxy for hosts' do
+    manager.change_puppet_proxy(proxy, false)
+
+    hosts.each do |host|
+      assert_equal proxy.id, host.reload.puppet_proxy_id
+    end
+  end
+
+  test 'changes puppet ca proxy for hosts' do
+    manager.change_puppet_proxy(proxy, true)
+
+    hosts.each do |host|
+      assert_equal proxy.id, host.reload.puppet_ca_proxy_id
+    end
+  end
+
+  test 'clears puppet proxy when proxy is nil' do
+    hosts.each { |host| host.update!(puppet_proxy: proxy) }
+
+    manager.change_puppet_proxy(nil, false)
+
+    hosts.each do |host|
+      assert_nil host.reload.puppet_proxy
+    end
+  end
+end

data/webpack/global_index.js

@@ -1,9 +1,16 @@
+import React from 'react';
+import { addGlobalFill } from 'foremanReact/components/common/Fill/GlobalFill';
 import { registerReducer } from 'foremanReact/common/MountingService';
 import { registerColumns } from 'foremanReact/components/HostsIndex/Columns/core';
 import { translate as __ } from 'foremanReact/common/I18n';
 import reducers from './src/reducers';
 import { registerFills } from './src/Extends/Fills';
 import { registerLegacy } from './legacy';
+import HostsIndexActionsBar from './src/Extends/Hosts/ActionsBar';
+import BulkChangePuppetProxy from './src/Extends/Hosts/BulkActions/BulkChangePuppetProxy';
+import BulkChangePuppetCAProxy from './src/Extends/Hosts/BulkActions/BulkChangePuppetCAProxy';
+import BulkRemovePuppetProxy from './src/Extends/Hosts/BulkActions/BulkRemovePuppetProxy';
+import BulkRemovePuppetCAProxy from './src/Extends/Hosts/BulkActions/BulkRemovePuppetCAProxy';
 
 // register reducers
 registerReducer('puppet', reducers);
@@ -29,4 +36,39 @@ puppetHostsIndexColumns.forEach(column => {
   column.categoryKey = 'puppet';
 });
 
+addGlobalFill(
+  'hosts-index-kebab',
+  'puppet-hosts-index-kebab',
+  <HostsIndexActionsBar key="puppet-hosts-index-kebab" />,
+  100
+);
+
+addGlobalFill(
+  '_all-hosts-modals',
+  'BulkChangePuppetProxy',
+  <BulkChangePuppetProxy key="bulk-change-puppet-proxy" />,
+  100
+);
+
+addGlobalFill(
+  '_all-hosts-modals',
+  'BulkChangePuppetCAProxy',
+  <BulkChangePuppetCAProxy key="bulk-change-puppet-ca-proxy" />,
+  100
+);
+
+addGlobalFill(
+  '_all-hosts-modals',
+  'BulkRemovePuppetCAProxy',
+  <BulkRemovePuppetCAProxy key="bulk-remove-puppet-ca-proxy" />,
+  100
+);
+
+addGlobalFill(
+  '_all-hosts-modals',
+  'BulkRemovePuppetProxy',
+  <BulkRemovePuppetProxy key="bulk-remove-puppet-proxy" />,
+  100
+);
+
 registerColumns(puppetHostsIndexColumns);

data/webpack/src/Extends/Hosts/ActionsBar/ActionsBar.scss

@@ -0,0 +1,14 @@
+.disabled-menu-item-span {
+  width: 25em;
+  display: flex;
+  flex-direction: row;
+}
+
+.disabled-menu-item-p {
+  margin-left: 0.6em;
+  word-break: normal;
+}
+
+.disabled-menu-item-icon {
+  font-size: small;
+}

data/webpack/src/Extends/Hosts/ActionsBar/index.js

@@ -0,0 +1,73 @@
+import React, { useContext } from 'react';
+import { Menu, MenuItem, MenuContent, MenuList } from '@patternfly/react-core';
+import { translate as __ } from 'foremanReact/common/I18n';
+import { ForemanHostsIndexActionsBarContext } from 'foremanReact/components/HostsIndex';
+import { openBulkModal } from 'foremanReact/common/BulkModalStateHelper';
+import './ActionsBar.scss';
+
+const HostActionsBar = () => {
+  const { selectedCount, setMenuOpen } = useContext(
+    ForemanHostsIndexActionsBarContext
+  );
+
+  const handleOpenBulkModal = modalId => {
+    setMenuOpen(false);
+    setTimeout(() => openBulkModal(modalId, true), 0);
+  };
+
+  return (
+    <MenuItem
+      itemId="content-flyout-item"
+      key="content-flyout"
+      isDisabled={selectedCount === 0}
+      flyoutMenu={
+        <Menu ouiaId="content-flyout-menu" onSelect={() => setMenuOpen(false)}>
+          <MenuContent>
+            <MenuList>
+              <MenuItem
+                itemId="bulk-change-puppet-proxy-menu-item"
+                key="bulk-change-puppet-proxy-menu-item"
+                onClick={() => handleOpenBulkModal('bulk-change-puppet-proxy')}
+                isDisabled={selectedCount === 0}
+              >
+                {__('Change Puppet proxy')}
+              </MenuItem>
+              <MenuItem
+                itemId="bulk-remove-puppet-proxy-menu-item"
+                key="bulk-remove-puppet-proxy-menu-item"
+                onClick={() => handleOpenBulkModal('bulk-remove-puppet-proxy')}
+                isDisabled={selectedCount === 0}
+              >
+                {__('Remove Puppet proxy')}
+              </MenuItem>
+              <MenuItem
+                itemId="bulk-change-puppet-ca-proxy-menu-item"
+                key="bulk-change-puppet-ca-proxy-menu-item"
+                onClick={() =>
+                  handleOpenBulkModal('bulk-change-puppet-ca-proxy')
+                }
+                isDisabled={selectedCount === 0}
+              >
+                {__('Change Puppet CA proxy')}
+              </MenuItem>
+              <MenuItem
+                itemId="bulk-remove-puppet-ca-proxy-menu-item"
+                key="bulk-remove-puppet-ca-proxy-menu-item"
+                onClick={() =>
+                  handleOpenBulkModal('bulk-remove-puppet-ca-proxy')
+                }
+                isDisabled={selectedCount === 0}
+              >
+                {__('Remove Puppet CA proxy')}
+              </MenuItem>
+            </MenuList>
+          </MenuContent>
+        </Menu>
+      }
+    >
+      {__('Puppet')}
+    </MenuItem>
+  );
+};
+
+export default HostActionsBar;

data/webpack/src/Extends/Hosts/BulkActions/BulkChangeProxyCommon/__tests__/actions.test.js

@@ -0,0 +1,78 @@
+import { APIActions } from 'foremanReact/redux/API';
+import { foremanUrl } from 'foremanReact/common/helpers';
+
+import {
+  fetchSmartProxies,
+  SMART_PROXY_KEY,
+  bulkChangePuppetProxy,
+  BULK_CHANGE_PUPPET_CA_PROXY_KEY,
+  BULK_CHANGE_PUPPET_PROXY_KEY,
+} from '../actions';
+
+jest.mock('foremanReact/redux/API', () => ({
+  APIActions: {
+    get: jest.fn(),
+    put: jest.fn(),
+  },
+}));
+
+describe('BulkChangeProxyCommon actions', () => {
+  const smartProxiesUrl = foremanUrl('/api/smart_proxies');
+  const bulkChangeUrl = foremanUrl('/api/v2/hosts/bulk/change_puppet_proxy');
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('fetches smart proxies list filtered by feature search', () => {
+    fetchSmartProxies('Puppet');
+
+    expect(APIActions.get).toHaveBeenCalledWith({
+      key: SMART_PROXY_KEY,
+      url: smartProxiesUrl,
+      params: { per_page: 'all', search: 'feature = "Puppet"' },
+    });
+  });
+
+  it('calls bulk change puppet proxy endpoint for Puppet Proxy changes', () => {
+    const params = { included: { ids: [1] }, proxy_id: '1', ca_proxy: false };
+    const handleSuccess = jest.fn();
+    const handleError = jest.fn();
+
+    bulkChangePuppetProxy(
+      params,
+      handleSuccess,
+      handleError,
+      BULK_CHANGE_PUPPET_PROXY_KEY
+    );
+
+    expect(APIActions.put).toHaveBeenCalledWith({
+      key: BULK_CHANGE_PUPPET_PROXY_KEY,
+      url: bulkChangeUrl,
+      handleSuccess,
+      handleError,
+      params,
+    });
+  });
+
+  it('calls bulk change puppet proxy endpoint for Puppet CA Proxy changes', () => {
+    const params = { included: { ids: [1] }, proxy_id: '1', ca_proxy: true };
+    const handleSuccess = jest.fn();
+    const handleError = jest.fn();
+
+    bulkChangePuppetProxy(
+      params,
+      handleSuccess,
+      handleError,
+      BULK_CHANGE_PUPPET_CA_PROXY_KEY
+    );
+
+    expect(APIActions.put).toHaveBeenCalledWith({
+      key: BULK_CHANGE_PUPPET_CA_PROXY_KEY,
+      url: bulkChangeUrl,
+      handleSuccess,
+      handleError,
+      params,
+    });
+  });
+});

data/webpack/src/Extends/Hosts/BulkActions/BulkChangeProxyCommon/actions.js

@@ -0,0 +1,36 @@
+import { APIActions } from 'foremanReact/redux/API';
+import { foremanUrl } from 'foremanReact/common/helpers';
+
+export const SMART_PROXY_KEY = 'SMART_PROXY_KEY';
+export const BULK_CHANGE_PUPPET_CA_PROXY_KEY = 'BULK_CHANGE_PUPPET_CA_PROXY';
+export const BULK_CHANGE_PUPPET_PROXY_KEY = 'BULK_CHANGE_PUPPET_PROXY';
+
+export const fetchSmartProxies = feature => {
+  const url = foremanUrl('/api/smart_proxies');
+  return APIActions.get({
+    key: SMART_PROXY_KEY,
+    url,
+    params: {
+      per_page: 'all',
+      ...(feature ? { search: `feature = "${feature}"` } : {}),
+    },
+  });
+};
+
+export const bulkChangePuppetProxy = (
+  params,
+  handleSuccess,
+  handleError,
+  key
+) => {
+  const url = foremanUrl(`/api/v2/hosts/bulk/change_puppet_proxy`);
+  return APIActions.put({
+    key,
+    url,
+    handleSuccess,
+    handleError,
+    params,
+  });
+};
+
+export default fetchSmartProxies;