foreman_remote_execution 3.3.5 → 4.2.0

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

@@ -0,0 +1,82 @@
+require 'test_plugin_helper'
+
+module Api
+  module V2
+    # Tests for the extra methods to play roles on a Host
+    class RegistrationControllerTest < ActionController::TestCase
+      describe 'host registration' do
+        let(:organization) { FactoryBot.create(:organization) }
+        let(:tax_location) { FactoryBot.create(:location) }
+        let(:template_kind) { template_kinds(:registration) }
+        let(:registration_template) do
+          FactoryBot.create(
+            :provisioning_template,
+            template_kind: template_kind,
+            template: 'template content <%= @host.name %>',
+            locations: [tax_location],
+            organizations: [organization]
+          )
+        end
+        let(:os) do
+          FactoryBot.create(
+            :operatingsystem,
+            :with_associations,
+            family: 'Redhat',
+            provisioning_templates: [
+              registration_template,
+            ]
+          )
+        end
+
+        let(:host_params) do
+          { host: { name: 'centos-test.example.com',
+                    managed: false, build: false,
+                    organization_id: organization.id,
+                    location_id: tax_location.id,
+                    operatingsystem_id: os.id } }
+        end
+
+        setup do
+          FactoryBot.create(
+            :os_default_template,
+            template_kind: template_kind,
+            provisioning_template: registration_template,
+            operatingsystem: os
+          )
+        end
+
+        describe 'remote_execution_interface' do
+          setup do
+            @host = Host.create(host_params[:host])
+            @interface0 = FactoryBot.create(:nic_managed, host: @host, identifier: 'dummy0', execution: false)
+          end
+
+          test 'with existing interface' do
+            params = host_params.merge(remote_execution_interface: @interface0.identifier)
+
+            post :host, params: params, session: set_session_user
+            assert_response :success
+            assert @interface0.reload.execution
+          end
+
+          test 'with not-existing interface' do
+            params = host_params.merge(remote_execution_interface: 'dummy999')
+
+            post :host, params: params, session: set_session_user
+            assert_response :not_found
+          end
+
+          test 'with multiple interfaces' do
+            interface1 = FactoryBot.create(:nic_managed, host: @host, identifier: 'dummy1', execution: false)
+            params = host_params.merge(remote_execution_interface: interface1.identifier)
+
+            post :host, params: params, session: set_session_user
+            assert_response :success
+            refute @interface0.reload.execution
+            assert interface1.reload.execution
+          end
+        end
+      end
+    end
+  end
+end

data/test/functional/job_invocations_controller_test.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'test_plugin_helper'
+require_relative '../support/remote_execution_helper'
 
 class JobInvocationsControllerTest < ActionController::TestCase
   test 'should parse inputs coming from the URL params' do
@@ -58,4 +59,74 @@ class JobInvocationsControllerTest < ActionController::TestCase
     post :new, params: params, session: set_session_user
     assert_response :success
   end
+
+  context 'restricted access' do
+    setup do
+      @admin = users(:admin)
+      @user = FactoryBot.create(:user, mail: 'test23@test.foreman.com', admin: false)
+      @invocation = FactoryBot.create(:job_invocation, :with_template, :with_task)
+      @invocation2 = FactoryBot.create(:job_invocation, :with_template, :with_task)
+
+      @invocation.task.update(user: @admin)
+      @invocation2.task.update(user: @user)
+
+      setup_user 'view', 'hosts', nil, @user
+      setup_user 'view', 'job_invocations', 'user = current_user', @user
+      setup_user 'create', 'job_invocations', 'user = current_user', @user
+      setup_user 'cancel', 'job_invocations', 'user = current_user', @user
+    end
+
+    context 'without user filter' do
+      test '#index' do
+        get :index, session: prepare_user(@admin)
+        assert_response :success
+        assert 2, assigns(:job_invocations).size
+      end
+
+      test '#show' do
+        get :show, params: { id: @invocation2.id }, session: prepare_user(@admin)
+        assert_response :success
+      end
+
+      test '#rerun' do
+        get :rerun, params: { id: @invocation2.id }, session: prepare_user(@admin)
+        assert_response :success
+      end
+
+      test '#cancel' do
+        ForemanTasks::Task.any_instance.expects(:cancel).returns(true)
+        post :cancel, params: { id: @invocation2.id }, session: prepare_user(@admin)
+        assert_response :redirect
+      end
+    end
+
+    context 'with user filter' do
+      test '#index' do
+        get :index, session: prepare_user(@user)
+        assert_response :success
+        assert_equal 1, assigns(:job_invocations).size
+        assert_equal @invocation2, assigns(:job_invocations)[0]
+      end
+
+      test '#show' do
+        get :show, params: { id: @invocation.id }, session: prepare_user(@user)
+        assert_response :not_found
+      end
+
+      test '#rerun' do
+        get :rerun, params: { id: @invocation.id }, session: prepare_user(@user)
+        assert_response :not_found
+      end
+
+      test 'cancel' do
+        post :cancel, params: { id: @invocation.id }, session: prepare_user(@user)
+        assert_response :not_found
+      end
+    end
+  end
+
+  def prepare_user(user)
+    User.current = user
+    set_session_user(user)
+  end
 end

data/test/support/remote_execution_helper.rb

@@ -0,0 +1,5 @@
+module RemoteExecutionHelper
+  def job_invocation_task_buttons(task)
+    return []
+  end
+end

data/test/unit/actions/run_host_job_test.rb

@@ -12,7 +12,7 @@ module ForemanRemoteExecution
       let(:provider) do
         provider = ::SSHExecutionProvider
         provider.expects(:ssh_password).with(host).returns('sshpass')
-        provider.expects(:sudo_password).with(host).returns('sudopass')
+        provider.expects(:effective_user_password).with(host).returns('sudopass')
         provider.expects(:ssh_key_passphrase).with(host).returns('keypass')
         provider
       end
@@ -21,7 +21,7 @@ module ForemanRemoteExecution
         secrets = subject.secrets(host, job_invocation, provider)
 
         assert_equal 'sshpass', secrets[:ssh_password]
-        assert_equal 'sudopass', secrets[:sudo_password]
+        assert_equal 'sudopass', secrets[:effective_user_password]
         assert_equal 'keypass', secrets[:key_passphrase]
       end
 
@@ -31,7 +31,7 @@ module ForemanRemoteExecution
         secrets = subject.secrets(host, job_invocation, provider)
 
         assert_equal 'jobsshpass', secrets[:ssh_password]
-        assert_equal 'sudopass', secrets[:sudo_password]
+        assert_equal 'sudopass', secrets[:effective_user_password]
         assert_equal 'jobkeypass', secrets[:key_passphrase]
       end
     end

data/test/unit/actions/run_hosts_job_test.rb

@@ -14,7 +14,7 @@ module ForemanRemoteExecution
         invocation.description = 'Some short description'
         invocation.password = 'changeme'
         invocation.key_passphrase = 'changemetoo'
-        invocation.sudo_password = 'sudopassword'
+        invocation.effective_user_password = 'sudopassword'
         invocation.save
       end
     end
@@ -24,6 +24,7 @@ module ForemanRemoteExecution
       OpenStruct.new(:id => uuid).tap do |o|
         o.stubs(:add_missing_task_groups)
         o.stubs(:task_groups).returns([])
+        o.stubs(:pending?).returns(true)
       end
     end
     let(:action) do
@@ -73,7 +74,7 @@ module ForemanRemoteExecution
     end
 
     it 'triggers the RunHostJob actions on the resolved hosts in run phase' do
-      planned.expects(:output).returns(:planned_count => 0)
+      planned.expects(:output).at_most(5).returns(:planned_count => 0)
       planned.expects(:trigger).with { |*args| args[0] == Actions::RemoteExecution::RunHostJob }
       planned.create_sub_plans
     end

data/test/unit/job_invocation_composer_test.rb

@@ -523,15 +523,15 @@ class JobInvocationComposerTest < ActiveSupport::TestCase
         end
       end
 
-      describe '#sudo_password' do
-        let(:sudo_password) { 'password' }
+      describe '#effective_user_password' do
+        let(:effective_user_password) { 'password' }
         let(:params) do
-          { :job_invocation => { :sudo_password => sudo_password }}
+          { :job_invocation => { :effective_user_password => effective_user_password }}
         end
 
-        it 'sets the sudo password properly' do
+        it 'sets the effective_user_password password properly' do
           composer
-          _(composer.job_invocation.sudo_password).must_equal sudo_password
+          _(composer.job_invocation.effective_user_password).must_equal effective_user_password
         end
       end
 

data/test/unit/remote_execution_provider_test.rb

@@ -78,12 +78,12 @@ class RemoteExecutionProviderTest < ActiveSupport::TestCase
       end
     end
 
-    describe 'sudo password' do
-      it 'uses the remote_execution_sudo_password on the host param' do
-        host.params['remote_execution_sudo_password'] = 'mypassword'
-        host.host_parameters << FactoryBot.create(:host_parameter, :host => host, :name => 'remote_execution_sudo_password', :value => 'mypassword')
-        assert_not proxy_options.key?(:sudo_password)
-        _(secrets[:sudo_password]).must_equal 'mypassword'
+    describe 'effective user password' do
+      it 'uses the remote_execution_effective_user_password on the host param' do
+        host.params['remote_execution_effective_user_password'] = 'mypassword'
+        host.host_parameters << FactoryBot.create(:host_parameter, :host => host, :name => 'remote_execution_effective_user_password', :value => 'mypassword')
+        assert_not proxy_options.key?(:effective_user_password)
+        _(secrets[:effective_user_password]).must_equal 'mypassword'
       end
     end
 

data/webpack/__mocks__/foremanReact/components/Pagination/PaginationWrapper.js

@@ -0,0 +1,2 @@
+const PaginationWrapper = () => jest.fn();
+export default PaginationWrapper;

data/webpack/__mocks__/foremanReact/components/SearchBar.js

@@ -0,0 +1,2 @@
+const SearchBar = () => jest.fn();
+export default SearchBar;

data/webpack/__mocks__/foremanReact/constants.js

@@ -1,3 +1,24 @@
 export const STATUS = {
+  PENDING: 'PENDING',
+  RESOLVED: 'RESOLVED',
   ERROR: 'ERROR',
 };
+
+export const getControllerSearchProps = (
+  controller,
+  id = 'searchBar',
+  canCreate = true
+) => ({
+  controller,
+  autocomplete: {
+    id,
+    searchQuery: '',
+    url: `${controller}/auto_complete_search`,
+    useKeyShortcuts: true,
+  },
+  bookmarks: {
+    url: '/api/bookmarks',
+    canCreate,
+    documentationUrl: `4.1.5Searching`,
+  },
+});

data/webpack/__mocks__/foremanReact/redux/API/APISelectors.js

@@ -0,0 +1,2 @@
+export const selectAPIStatus = () => 'RESOLVED';
+export const selectAPIResponse = state => state;

data/webpack/__mocks__/foremanReact/redux/middlewares/IntervalMiddleware/IntervalSelectors.js

@@ -0,0 +1 @@
+export const selectDoesIntervalExist = () => false;

data/webpack/react_app/components/TargetingHosts/TargetingHosts.js

@@ -5,8 +5,8 @@ import { LoadingState, Alert } from 'patternfly-react';
 import { STATUS } from 'foremanReact/constants';
 import HostItem from './components/HostItem';
 
-const TargetingHosts = ({ status, items }) => {
-  if (status === STATUS.ERROR) {
+const TargetingHosts = ({ apiStatus, items }) => {
+  if (apiStatus === STATUS.ERROR) {
     return (
       <Alert type="error">
         {__(
@@ -16,8 +16,24 @@ const TargetingHosts = ({ status, items }) => {
     );
   }
 
+  const tableBodyRows = items.length ? (
+    items.map(({ name, link, status, actions }) => (
+      <HostItem
+        key={name}
+        name={name}
+        link={link}
+        status={status}
+        actions={actions}
+      />
+    ))
+  ) : (
+    <tr>
+      <td colSpan="3">{__('No hosts found.')}</td>
+    </tr>
+  );
+
   return (
-    <LoadingState loading={!items.length}>
+    <LoadingState loading={!items.length && apiStatus === STATUS.PENDING}>
       <div>
         <table className="table table-bordered table-striped table-hover">
           <thead>
@@ -27,17 +43,7 @@ const TargetingHosts = ({ status, items }) => {
               <th>{__('Actions')}</th>
             </tr>
           </thead>
-          <tbody>
-            {items.map(host => (
-              <HostItem
-                key={host.name}
-                name={host.name}
-                link={host.link}
-                status={host.status}
-                actions={host.actions}
-              />
-            ))}
-          </tbody>
+          <tbody>{tableBodyRows}</tbody>
         </table>
       </div>
     </LoadingState>
@@ -45,8 +51,12 @@ const TargetingHosts = ({ status, items }) => {
 };
 
 TargetingHosts.propTypes = {
-  status: PropTypes.string.isRequired,
+  apiStatus: PropTypes.string,
   items: PropTypes.array.isRequired,
 };
 
+TargetingHosts.defaultProps = {
+  apiStatus: null,
+};
+
 export default TargetingHosts;

data/webpack/react_app/components/TargetingHosts/TargetingHostsHelpers.js

@@ -0,0 +1,10 @@
+import { getURI } from 'foremanReact/common/urlHelpers';
+
+export const getApiUrl = (searchQuery, pagination) => {
+  const baseUrl = getURI()
+    .search('')
+    .addQuery('page', pagination.page)
+    .addQuery('per_page', pagination.perPage);
+
+  return searchQuery ? baseUrl.addQuery('search', searchQuery) : baseUrl;
+};

data/webpack/react_app/components/TargetingHosts/TargetingHostsPage.js

@@ -0,0 +1,66 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { Grid } from 'patternfly-react';
+
+import SearchBar from 'foremanReact/components/SearchBar';
+import Pagination from 'foremanReact/components/Pagination/PaginationWrapper';
+import { getControllerSearchProps } from 'foremanReact/constants';
+
+import TargetingHosts from './TargetingHosts';
+import './TargetingHostsPage.scss';
+
+const TargetingHostsPage = ({
+  handleSearch,
+  searchQuery,
+  apiStatus,
+  items,
+  totalHosts,
+  pagination,
+  handlePagination,
+}) => (
+  <div id="targeting_hosts">
+    <Grid.Row>
+      <Grid.Col md={6} className="title_filter">
+        <SearchBar
+          onSearch={query => handleSearch(query)}
+          data={{
+            ...getControllerSearchProps('hosts'),
+            autocomplete: {
+              id: 'targeting_hosts_search',
+              searchQuery,
+              url: '/hosts/auto_complete_search',
+              useKeyShortcuts: true,
+            },
+            bookmarks: {},
+          }}
+        />
+      </Grid.Col>
+    </Grid.Row>
+    <br />
+    <TargetingHosts apiStatus={apiStatus} items={items} />
+    <Pagination
+      viewType="list"
+      itemCount={totalHosts}
+      pagination={pagination}
+      onChange={args => handlePagination(args)}
+      dropdownButtonId="targeting-hosts-pagination-dropdown"
+      className="targeting-hosts-pagination"
+    />
+  </div>
+);
+
+TargetingHostsPage.propTypes = {
+  handleSearch: PropTypes.func.isRequired,
+  searchQuery: PropTypes.string.isRequired,
+  apiStatus: PropTypes.string,
+  items: PropTypes.array.isRequired,
+  totalHosts: PropTypes.number.isRequired,
+  pagination: PropTypes.object.isRequired,
+  handlePagination: PropTypes.func.isRequired,
+};
+
+TargetingHostsPage.defaultProps = {
+  apiStatus: null,
+};
+
+export default TargetingHostsPage;