foreman_remote_execution 4.5.6 → 5.0.0

data/webpack/JobWizard/steps/AdvancedFields/AdvancedFields.js

@@ -5,7 +5,6 @@ import { Form } from '@patternfly/react-core';
 import {
   selectEffectiveUser,
   selectAdvancedTemplateInputs,
-  selectTemplateInputs,
 } from '../../JobWizardSelectors';
 import {
   EffectiveUserField,
@@ -22,10 +21,13 @@ import { DescriptionField } from './DescriptionField';
 import { WIZARD_TITLES } from '../../JobWizardConstants';
 import { WizardTitle } from '../form/WizardTitle';
 
-export const AdvancedFields = ({ advancedValues, setAdvancedValues }) => {
+export const AdvancedFields = ({
+  templateValues,
+  advancedValues,
+  setAdvancedValues,
+}) => {
   const effectiveUser = useSelector(selectEffectiveUser);
   const advancedTemplateInputs = useSelector(selectAdvancedTemplateInputs);
-  const templateInputs = useSelector(selectTemplateInputs);
   return (
     <>
       <WizardTitle
@@ -49,7 +51,7 @@ export const AdvancedFields = ({ advancedValues, setAdvancedValues }) => {
           />
         )}
         <DescriptionField
-          inputs={templateInputs}
+          inputValues={{ ...templateValues, ...advancedValues.templateValues }}
           value={advancedValues.description}
           setValue={newValue => setAdvancedValues({ description: newValue })}
         />
@@ -117,5 +119,6 @@ export const AdvancedFields = ({ advancedValues, setAdvancedValues }) => {
 AdvancedFields.propTypes = {
   advancedValues: PropTypes.object.isRequired,
   setAdvancedValues: PropTypes.func.isRequired,
+  templateValues: PropTypes.object.isRequired,
 };
 export default AdvancedFields;

data/webpack/JobWizard/steps/AdvancedFields/DescriptionField.js

@@ -1,10 +1,19 @@
-import React, { useState } from 'react';
+import React, { useState, useEffect, useCallback } from 'react';
+import { useSelector } from 'react-redux';
 import PropTypes from 'prop-types';
-import { FormGroup, TextInput, Button } from '@patternfly/react-core';
+import { FormGroup, TextInput, Tooltip, Button } from '@patternfly/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
+import {
+  selectTemplateInputs,
+  selectAdvancedTemplateInputs,
+} from '../../JobWizardSelectors';
 
-export const DescriptionField = ({ inputs, value, setValue }) => {
-  const generateDesc = () => {
+export const DescriptionField = ({ inputValues, value, setValue }) => {
+  const inputs = [
+    ...useSelector(selectTemplateInputs),
+    ...useSelector(selectAdvancedTemplateInputs),
+  ].map(input => input.name);
+  const generateDesc = useCallback(() => {
     let newDesc = value;
     if (value) {
       const re = new RegExp('%\\{([^\\}]+)\\}', 'gm');
@@ -15,16 +24,19 @@ export const DescriptionField = ({ inputs, value, setValue }) => {
       results.forEach(result => {
         newDesc = newDesc.replace(
           result.text,
-          // TODO: Replace with the value of the input from Target Hosts step
-          inputs.find(input => input.name === result.name)?.name || result.text
+          inputValues[result.name] ||
+            (inputs.includes(result.name) ? '' : result.text)
         );
       });
     }
     return newDesc;
-  };
+  }, [inputs, value, inputValues]);
   const [generatedDesc, setGeneratedDesc] = useState(generateDesc());
   const [isPreview, setIsPreview] = useState(true);
 
+  useEffect(() => {
+    setGeneratedDesc(generateDesc());
+  }, [generateDesc]);
   const togglePreview = () => {
     setGeneratedDesc(generateDesc());
     setIsPreview(v => !v);
@@ -43,9 +55,20 @@ export const DescriptionField = ({ inputs, value, setValue }) => {
       }
     >
       {isPreview ? (
-        <TextInput id="description-preview" value={generatedDesc} isDisabled />
+        <Tooltip content={generatedDesc}>
+          <div>
+            {/* div wrapper so the tooltip will be shown in chrome */}
+            <TextInput
+              aria-label="description preview"
+              id="description-preview"
+              value={generatedDesc}
+              isDisabled
+            />
+          </div>
+        </Tooltip>
       ) : (
         <TextInput
+          aria-label="description edit"
           type="text"
           autoComplete="description"
           id="description"
@@ -58,7 +81,7 @@ export const DescriptionField = ({ inputs, value, setValue }) => {
 };
 
 DescriptionField.propTypes = {
-  inputs: PropTypes.array.isRequired,
+  inputValues: PropTypes.object.isRequired,
   value: PropTypes.string,
   setValue: PropTypes.func.isRequired,
 };

data/webpack/JobWizard/steps/AdvancedFields/__tests__/AdvancedFields.test.js

@@ -1,14 +1,19 @@
+/* eslint-disable max-lines */
 import React from 'react';
 import { Provider } from 'react-redux';
 import { mount } from '@theforeman/test';
 import { fireEvent, screen, render, act } from '@testing-library/react';
+import { MockedProvider } from '@apollo/client/testing';
 import * as api from 'foremanReact/redux/API';
 import { JobWizard } from '../../../JobWizard';
 import * as selectors from '../../../JobWizardSelectors';
 import {
-  jobTemplateResponse as jobTemplate,
+  jobTemplateResponse,
+  jobTemplate,
   testSetup,
   mockApi,
+  jobCategories,
+  gqlMock,
 } from '../../../__tests__/fixtures';
 import { WIZARD_TITLES } from '../../../JobWizardConstants';
 
@@ -18,14 +23,16 @@ mockApi(api);
 jest.spyOn(selectors, 'selectEffectiveUser');
 
 selectors.selectEffectiveUser.mockImplementation(
-  () => jobTemplate.effective_user
+  () => jobTemplateResponse.effective_user
 );
 describe('AdvancedFields', () => {
   it('should save data between steps for advanced fields', async () => {
     const wrapper = mount(
-      <Provider store={store}>
-        <JobWizard advancedValues={{}} setAdvancedValues={jest.fn()} />
-      </Provider>
+      <MockedProvider mocks={gqlMock} addTypename={false}>
+        <Provider store={store}>
+          <JobWizard />
+        </Provider>
+      </MockedProvider>
     );
     // setup
     wrapper.find('.pf-c-button.pf-c-select__toggle-button').simulate('click');
@@ -78,9 +85,11 @@ describe('AdvancedFields', () => {
   });
   it('fill template fields', async () => {
     render(
-      <Provider store={store}>
-        <JobWizard />
-      </Provider>
+      <MockedProvider mocks={gqlMock} addTypename={false}>
+        <Provider store={store}>
+          <JobWizard />
+        </Provider>
+      </MockedProvider>
     );
     await act(async () => {
       fireEvent.click(screen.getByText(WIZARD_TITLES.advanced));
@@ -91,7 +100,10 @@ describe('AdvancedFields', () => {
     const textField = screen.getByLabelText('adv plain hidden', {
       selector: 'textarea',
     });
-    const selectField = screen.getByText('option 1');
+    const selectField = screen.getByLabelText('adv plain select toggle');
+    const resourceSelectField = screen.getByLabelText(
+      'adv resource select toggle'
+    );
     const searchField = screen.getByPlaceholderText('Filter...');
     const dateField = screen.getByLabelText('adv date', {
       selector: 'input',
@@ -101,6 +113,10 @@ describe('AdvancedFields', () => {
     await act(async () => {
       await fireEvent.click(screen.getByText('option 2'));
       fireEvent.click(screen.getAllByText(WIZARD_TITLES.advanced)[0]); // to remove focus
+
+      fireEvent.click(resourceSelectField);
+      await fireEvent.click(screen.getByText('resource2'));
+
       await fireEvent.change(textField, {
         target: { value: textValue },
       });
@@ -134,12 +150,16 @@ describe('AdvancedFields', () => {
     expect(dateField.value).toBe(dateValue);
     expect(screen.queryAllByText('option 1')).toHaveLength(0);
     expect(screen.queryAllByText('option 2')).toHaveLength(1);
+    expect(screen.queryAllByDisplayValue('resource1')).toHaveLength(0);
+    expect(screen.queryAllByDisplayValue('resource2')).toHaveLength(1);
   });
   it('fill defaults into fields', async () => {
     render(
-      <Provider store={store}>
-        <JobWizard />
-      </Provider>
+      <MockedProvider mocks={gqlMock} addTypename={false}>
+        <Provider store={store}>
+          <JobWizard />
+        </Provider>
+      </MockedProvider>
     );
     await act(async () => {
       fireEvent.click(screen.getByText('Advanced Fields'));
@@ -155,5 +175,189 @@ describe('AdvancedFields', () => {
         selector: 'input',
       }).value
     ).toBe('2');
+
+    expect(
+      screen.getByLabelText('description preview', {
+        selector: 'input',
+      }).value
+    ).toBe(
+      'template1 with inputs adv plain hidden="Default val" adv plain select="" adv resource select="" adv search="" adv date="" plain hidden="Default val"'
+    );
+  });
+  it('DescriptionField', async () => {
+    render(
+      <Provider store={store}>
+        <JobWizard />
+      </Provider>
+    );
+    await act(async () => {
+      fireEvent.click(screen.getByText(WIZARD_TITLES.advanced));
+    });
+
+    const textField = screen.getByLabelText('adv plain hidden', {
+      selector: 'textarea',
+    });
+    await act(async () => {
+      await fireEvent.change(textField, {
+        target: { value: 'test command' },
+      });
+    });
+    const descriptionValue = 'Run %{adv plain hidden} %{wrong command name}';
+
+    await act(async () => {
+      fireEvent.click(screen.getByText('Edit job description template'));
+    });
+
+    const editText = screen.getByLabelText('description edit', {
+      selector: 'input',
+    });
+    await fireEvent.change(editText, {
+      target: { value: descriptionValue },
+    });
+    await act(async () => {
+      fireEvent.click(screen.getByText('Preview job description'));
+    });
+    expect(
+      screen.getByLabelText('description preview', {
+        selector: 'input',
+      }).value
+    ).toBe('Run test command %{wrong command name}');
+  });
+
+  it('DescriptionField with no inputs', async () => {
+    jest.spyOn(api, 'get');
+
+    jest.spyOn(selectors, 'selectTemplateInputs');
+    jest.spyOn(selectors, 'selectAdvancedTemplateInputs');
+    selectors.selectTemplateInputs.mockImplementation(() => []);
+    selectors.selectAdvancedTemplateInputs.mockImplementation(() => []);
+    api.get.mockImplementation(({ handleSuccess, ...action }) => {
+      if (action.key === 'JOB_CATEGORIES') {
+        handleSuccess &&
+          handleSuccess({ data: { job_categories: jobCategories } });
+      } else if (action.key === 'JOB_TEMPLATE') {
+        handleSuccess &&
+          handleSuccess({
+            data: {
+              ...jobTemplateResponse,
+              advanced_template_inputs: [],
+              template_inputs: [],
+            },
+          });
+      } else if (action.key === 'JOB_TEMPLATES') {
+        handleSuccess &&
+          handleSuccess({
+            data: { results: [jobTemplate] },
+          });
+      }
+      return { type: 'get', ...action };
+    });
+    render(
+      <Provider store={store}>
+        <JobWizard />
+      </Provider>
+    );
+    await act(async () => {
+      fireEvent.click(screen.getByText(WIZARD_TITLES.advanced));
+    });
+    expect(
+      screen.getByLabelText('description preview', {
+        selector: 'input',
+      }).value
+    ).toBe('template1');
+  });
+
+  it('DescriptionField with description_format', async () => {
+    jest.spyOn(api, 'get');
+    jest.spyOn(selectors, 'selectTemplateInputs');
+    selectors.selectTemplateInputs.mockImplementation(() => [
+      {
+        name: 'command',
+        required: true,
+        input_type: 'user',
+        description: 'some Description',
+        advanced: true,
+        value_type: 'plain',
+        resource_type: 'ansible_roles',
+        default: 'Default val',
+        hidden_value: true,
+      },
+    ]);
+    api.get.mockImplementation(({ handleSuccess, ...action }) => {
+      if (action.key === 'JOB_CATEGORIES') {
+        handleSuccess &&
+          handleSuccess({ data: { job_categories: jobCategories } });
+      } else if (action.key === 'JOB_TEMPLATE') {
+        handleSuccess &&
+          handleSuccess({
+            data: {
+              ...jobTemplateResponse,
+              job_template: {
+                ...jobTemplateResponse.jobTemplate,
+                description_format: 'Run %{command}',
+              },
+
+              template_inputs: [
+                {
+                  name: 'command',
+                  required: true,
+                  input_type: 'user',
+                  description: 'some Description',
+                  advanced: true,
+                  value_type: 'plain',
+                  resource_type: 'ansible_roles',
+                  default: 'Default val',
+                  hidden_value: true,
+                },
+              ],
+            },
+          });
+      } else if (action.key === 'JOB_TEMPLATES') {
+        handleSuccess &&
+          handleSuccess({
+            data: { results: [jobTemplate] },
+          });
+      }
+      return { type: 'get', ...action };
+    });
+    render(
+      <Provider store={store}>
+        <JobWizard />
+      </Provider>
+    );
+    await act(async () => {
+      fireEvent.click(screen.getByText(WIZARD_TITLES.advanced));
+    });
+    expect(
+      screen.getByLabelText('description preview', {
+        selector: 'input',
+      }).value
+    ).toBe('Run Default val');
+  });
+
+  it('search resources action', async () => {
+    jest.useFakeTimers();
+    mockApi(api);
+    const newStore = testSetup(selectors, api);
+    render(
+      <Provider store={newStore}>
+        <JobWizard />
+      </Provider>
+    );
+    await act(async () => {
+      fireEvent.click(screen.getByText(WIZARD_TITLES.advanced));
+    });
+    const resourceSelectField = screen.getByLabelText(
+      'adv resource select typeahead input'
+    );
+
+    await act(async () => {
+      await fireEvent.change(resourceSelectField, {
+        target: { value: 'some search' },
+      });
+
+      await jest.runAllTimers();
+    });
+    expect(newStore.getActions()).toMatchSnapshot('resource search');
   });
 });

data/webpack/JobWizard/steps/AdvancedFields/__tests__/__snapshots__/AdvancedFields.test.js.snap

@@ -0,0 +1,82 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`AdvancedFields search resources action: resource search 1`] = `
+Array [
+  Object {
+    "key": "JOB_CATEGORIES",
+    "type": "get",
+    "url": "/ui_job_wizard/categories",
+  },
+  Object {
+    "key": "JOB_TEMPLATES",
+    "type": "get",
+    "url": URI {
+      "_deferred_build": true,
+      "_parts": Object {
+        "duplicateQueryParameters": false,
+        "escapeQuerySpace": true,
+        "fragment": null,
+        "hostname": null,
+        "password": null,
+        "path": "foreman/api/v2/job_templates",
+        "port": null,
+        "preventInvalidHostname": false,
+        "protocol": null,
+        "query": "search=job_category%3D%22Ansible+Commands%22&per_page=all",
+        "urn": null,
+        "username": null,
+      },
+      "_string": "",
+    },
+  },
+  Object {
+    "key": "JOB_TEMPLATE",
+    "type": "get",
+    "url": "/ui_job_wizard/template/178",
+  },
+  Object {
+    "key": "ForemanTasksTask",
+    "type": "get",
+    "url": URI {
+      "_deferred_build": true,
+      "_parts": Object {
+        "duplicateQueryParameters": false,
+        "escapeQuerySpace": true,
+        "fragment": null,
+        "hostname": null,
+        "password": null,
+        "path": "/ui_job_wizard/resources",
+        "port": null,
+        "preventInvalidHostname": false,
+        "protocol": null,
+        "query": "resource=ForemanTasks%3A%3ATask&name=",
+        "urn": null,
+        "username": null,
+      },
+      "_string": "",
+    },
+  },
+  Object {
+    "key": "ForemanTasksTask",
+    "type": "get",
+    "url": URI {
+      "_deferred_build": true,
+      "_parts": Object {
+        "duplicateQueryParameters": false,
+        "escapeQuerySpace": true,
+        "fragment": null,
+        "hostname": null,
+        "password": null,
+        "path": "/ui_job_wizard/resources",
+        "port": null,
+        "preventInvalidHostname": false,
+        "protocol": null,
+        "query": "resource=ForemanTasks%3A%3ATask&name=some+search",
+        "urn": null,
+        "username": null,
+      },
+      "_string": "",
+    },
+  },
+]
+`;

data/webpack/JobWizard/steps/CategoryAndTemplate/CategoryAndTemplate.js

@@ -53,6 +53,7 @@ export const CategoryAndTemplate = ({
           value={selectedCategory}
           placeholderText={categoryError ? __('Error') : ''}
           isDisabled={!!categoryError}
+          isRequired
         />
         <GroupedSelectField
           label={__('Job template')}

data/webpack/JobWizard/steps/CategoryAndTemplate/index.js

@@ -54,10 +54,11 @@ const ConnectedCategoryAndTemplate = ({
             search: `job_category="${category}"`,
             per_page: 'all',
           }),
-          handleSuccess: response =>
+          handleSuccess: response => {
             setJobTemplate(
               Number(filterJobTemplates(response?.data?.results)[0]?.id) || null
-            ),
+            );
+          },
         })
       );
     }

data/webpack/JobWizard/steps/HostsAndInputs/HostPreviewModal.js

@@ -0,0 +1,62 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { useSelector } from 'react-redux';
+import URI from 'urijs';
+import { List, ListItem, Modal, Button } from '@patternfly/react-core';
+import { translate as __, sprintf } from 'foremanReact/common/I18n';
+import { foremanUrl } from 'foremanReact/common/helpers';
+import { selectHosts, selectHostCount } from '../../JobWizardSelectors';
+import { HOSTS_TO_PREVIEW_AMOUNT } from '../../JobWizardConstants';
+
+export const HostPreviewModal = ({ isOpen, setIsOpen, searchQuery }) => {
+  const hosts = useSelector(selectHosts);
+  const hostsCount = useSelector(selectHostCount);
+  const url = new URI(foremanUrl('/hosts'));
+
+  return (
+    <Modal
+      title={__('Preview Hosts')}
+      isOpen={isOpen}
+      onClose={() => setIsOpen(false)}
+      appendTo={() => document.getElementsByClassName('job-wizard')[0]}
+    >
+      <List isPlain>
+        {hosts.map(host => (
+          <ListItem key={host}>
+            <Button
+              component="a"
+              href={foremanUrl(`/hosts/${host}`)}
+              variant="link"
+              target="_blank"
+              rel="noreferrer"
+            >
+              {host}
+            </Button>
+          </ListItem>
+        ))}
+        {hostsCount > HOSTS_TO_PREVIEW_AMOUNT && (
+          <ListItem>
+            <Button
+              component="a"
+              href={url.addSearch({ search: searchQuery })}
+              variant="link"
+              target="_blank"
+              rel="noreferrer"
+            >
+              {sprintf(
+                __('...and %s more'),
+                hostsCount - HOSTS_TO_PREVIEW_AMOUNT
+              )}
+            </Button>
+          </ListItem>
+        )}
+      </List>
+    </Modal>
+  );
+};
+
+HostPreviewModal.propTypes = {
+  isOpen: PropTypes.bool.isRequired,
+  setIsOpen: PropTypes.func.isRequired,
+  searchQuery: PropTypes.string.isRequired,
+};

data/webpack/JobWizard/steps/HostsAndInputs/HostSearch.js

@@ -0,0 +1,54 @@
+import React, { useEffect } from 'react';
+import { useSelector, useDispatch } from 'react-redux';
+import PropTypes from 'prop-types';
+import SearchBar from 'foremanReact/components/SearchBar';
+import { getControllerSearchProps } from 'foremanReact/constants';
+import { getResults } from 'foremanReact/components/AutoComplete/AutoCompleteActions';
+import { TRIGGERS } from 'foremanReact/components/AutoComplete/AutoCompleteConstants';
+import { hostsController, hostQuerySearchID } from '../../JobWizardConstants';
+import { noop } from '../../../helpers';
+
+export const HostSearch = ({ value, setValue }) => {
+  const searchQuery = useSelector(
+    state => state.autocomplete?.hostsSearch?.searchQuery
+  );
+  useEffect(() => {
+    setValue(searchQuery || '');
+  }, [setValue, searchQuery]);
+  const dispatch = useDispatch();
+  const setSearch = newSearchQuery => {
+    dispatch(
+      getResults({
+        url: '/hosts/auto_complete_search',
+        searchQuery: newSearchQuery,
+        controller: 'hostsController',
+        trigger: TRIGGERS.INPUT_CHANGE,
+        id: hostQuerySearchID,
+      })
+    );
+  };
+
+  const props = getControllerSearchProps(hostsController, hostQuerySearchID);
+  return (
+    <div className="foreman-search-field">
+      <SearchBar
+        data={{
+          ...props,
+          autocomplete: {
+            id: hostQuerySearchID,
+            url: '/hosts/auto_complete_search',
+            useKeyShortcuts: true,
+          },
+        }}
+        onSearch={noop}
+        initialQuery={value}
+        onBookmarkClick={search => setSearch(search)}
+      />
+    </div>
+  );
+};
+
+HostSearch.propTypes = {
+  value: PropTypes.string.isRequired,
+  setValue: PropTypes.func.isRequired,
+};

data/webpack/JobWizard/steps/HostsAndInputs/SelectAPI.js

@@ -0,0 +1,33 @@
+import React from 'react';
+import { useSelector, useDispatch } from 'react-redux';
+import URI from 'urijs';
+import { SelectVariant } from '@patternfly/react-core';
+import { get } from 'foremanReact/redux/API';
+import { selectResponse, selectIsLoading } from '../../JobWizardSelectors';
+import { SearchSelect } from '../form/SearchSelect';
+
+export const useNameSearchAPI = (apiKey, url) => {
+  const dispatch = useDispatch();
+  const uri = new URI(url);
+  const onSearch = search =>
+    dispatch(
+      get({
+        key: apiKey,
+        url: uri.addSearch({
+          search: `name~"${search}"`,
+        }),
+      })
+    );
+
+  const response = useSelector(state => selectResponse(state, apiKey));
+  const isLoading = useSelector(state => selectIsLoading(state, apiKey));
+  return [onSearch, response, isLoading];
+};
+
+export const SelectAPI = props => (
+  <SearchSelect
+    {...props}
+    variant={SelectVariant.typeaheadMulti}
+    useNameSearch={useNameSearchAPI}
+  />
+);