foreman_remote_execution 18.0.1 → 18.1.0

data/webpack/react_app/components/TargetingHosts/TargetingHostsLabelsRow.scss

@@ -1,30 +0,0 @@
-@import 'foremanReact/common/variables';
-
-.tasks-labels-row {
-  margin: 0;
-  padding: 10px;
-
-  .title {
-    font-weight: 600;
-    font-size: 13px;
-  }
-
-  .label {
-    font-size: 100%;
-    margin-left: 5px;
-    margin-right: 5px;
-
-    a {
-      padding-left: 10px;
-    }
-  }
-
-  .pficon-close {
-    color: $color-pf-white;
-  }
-
-  .compound-label-pf {
-    margin: 10px;
-    margin-left: 0;
-  }
-}

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

@@ -1,78 +0,0 @@
-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';
-import { getControllerSearchProps } from 'foremanReact/constants';
-import { noop } from 'foremanReact/common/helpers';
-
-import TargetingHosts from './TargetingHosts';
-import { TARGETING_HOSTS_AUTOCOMPLETE } from './TargetingHostsConsts';
-import './TargetingHostsPage.scss';
-import TargetingHostsLabelsRow from './TargetingHostsLabelsRow';
-
-const TargetingHostsPage = ({
-  handleSearch,
-  searchQuery,
-  statusFilter,
-  statusFilterReset,
-  apiStatus,
-  items,
-  totalHosts,
-  handlePagination,
-  page,
-}) => (
-  <div id="targeting_hosts">
-    <Grid.Row>
-      <Grid.Col md={6}>
-        <SearchBar
-          onSearch={query => handleSearch(query, statusFilter)}
-          data={{
-            ...getControllerSearchProps('hosts', TARGETING_HOSTS_AUTOCOMPLETE),
-            autocomplete: {
-              id: TARGETING_HOSTS_AUTOCOMPLETE,
-              searchQuery,
-              url: '/hosts/auto_complete_search',
-              useKeyShortcuts: true,
-            },
-          }}
-          onBookmarkClick={handleSearch}
-        />
-      </Grid.Col>
-    </Grid.Row>
-    <TargetingHostsLabelsRow
-      query={statusFilter}
-      updateQuery={statusFilterReset}
-    />
-    <br />
-    <TargetingHosts apiStatus={apiStatus} items={items} />
-    <Pagination
-      page={page}
-      itemCount={totalHosts}
-      onChange={args => handlePagination(args)}
-      className="targeting-hosts-pagination"
-      updateParamsByUrl={false}
-    />
-  </div>
-);
-
-TargetingHostsPage.propTypes = {
-  handleSearch: PropTypes.func.isRequired,
-  searchQuery: PropTypes.string.isRequired,
-  apiStatus: PropTypes.string,
-  statusFilter: PropTypes.object,
-  statusFilterReset: PropTypes.func,
-  items: PropTypes.array.isRequired,
-  totalHosts: PropTypes.number.isRequired,
-  handlePagination: PropTypes.func.isRequired,
-  page: PropTypes.number.isRequired,
-};
-
-TargetingHostsPage.defaultProps = {
-  apiStatus: null,
-  statusFilter: {},
-  statusFilterReset: noop,
-};
-
-export default TargetingHostsPage;

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

@@ -1,3 +0,0 @@
-#targeting_hosts {
-  min-height: 350px;
-}

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

@@ -1,25 +0,0 @@
-import {
-  selectAPIStatus,
-  selectAPIResponse,
-} from 'foremanReact/redux/API/APISelectors';
-import { selectDoesIntervalExist } from 'foremanReact/redux/middlewares/IntervalMiddleware/IntervalSelectors';
-
-import { TARGETING_HOSTS } from './TargetingHostsConsts';
-
-export const selectItems = state =>
-  selectAPIResponse(state, TARGETING_HOSTS).hosts || [];
-
-export const selectAutoRefresh = state =>
-  selectAPIResponse(state, TARGETING_HOSTS).autoRefresh || '';
-
-export const selectApiStatus = state => selectAPIStatus(state, TARGETING_HOSTS);
-export const selectTotalHosts = state =>
-  selectAPIResponse(state, TARGETING_HOSTS).total_hosts || 0;
-
-export const selectIntervalExists = state =>
-  selectDoesIntervalExist(state, TARGETING_HOSTS);
-
-const defaultStatusFilter = {};
-export const selectStatusFilter = state =>
-  state.foremanRemoteExecutionReducers.jobInvocations
-    .jobInvocationStateFilter || defaultStatusFilter;

data/webpack/react_app/components/TargetingHosts/__tests__/HostItem.test.js

@@ -1,76 +0,0 @@
-import React from 'react';
-import { render, screen } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import '@testing-library/jest-dom';
-import HostItem from '../components/HostItem';
-import { HostItemFixtures } from './fixtures';
-
-jest.unmock('foremanReact/components/common/ActionButtons/ActionButtons');
-
-const renderHostItem = props =>
-  render(
-    <table>
-      <tbody>
-        <HostItem {...props} />
-      </tbody>
-    </table>
-  );
-
-describe('HostItem', () => {
-  it('renders the host name as a link with the provided URL', () => {
-    renderHostItem(HostItemFixtures.renders);
-
-    expect(screen.getByRole('link', { name: 'Host1' })).toHaveAttribute(
-      'href',
-      '/host1'
-    );
-  });
-
-  it('renders the host status', () => {
-    renderHostItem(HostItemFixtures.renders);
-
-    expect(screen.getByText('Succeeded')).toBeInTheDocument();
-  });
-
-  it('renders no action buttons when actions are empty', () => {
-    renderHostItem(HostItemFixtures.renders);
-
-    expect(screen.queryByRole('button')).not.toBeInTheDocument();
-  });
-
-  it('renders a disabled link when link is not provided', () => {
-    renderHostItem({
-      ...HostItemFixtures.renders,
-      link: '',
-    });
-
-    expect(screen.getByRole('link', { name: 'Host1' })).toHaveAttribute(
-      'href',
-      '#'
-    );
-  });
-
-  it('renders action buttons and calls action on click', async () => {
-    const onClick = jest.fn();
-
-    renderHostItem({
-      ...HostItemFixtures.renders,
-      actions: [{ title: 'View output', action: { onClick } }],
-    });
-
-    await userEvent.click(
-      screen.getByRole('button', { name: 'View output' })
-    );
-
-    expect(onClick).toHaveBeenCalledTimes(1);
-  });
-
-  it('renders failed status text for error status', () => {
-    renderHostItem({
-      ...HostItemFixtures.renders,
-      status: 'error',
-    });
-
-    expect(screen.getByText('Failed')).toBeInTheDocument();
-  });
-});

data/webpack/react_app/components/TargetingHosts/__tests__/HostStatus.test.js

@@ -1,6 +0,0 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-import HostStatus from '../components/HostStatus';
-import { HostStatusFixtures } from './fixtures';
-
-describe('HostStatus', () =>
-  testComponentSnapshotsWithFixtures(HostStatus, HostStatusFixtures));

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

@@ -1,6 +0,0 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-import TargetingHosts from '../TargetingHosts';
-import { TargetingHostsFixtures } from './fixtures';
-
-describe('TargetingHosts', () =>
-  testComponentSnapshotsWithFixtures(TargetingHosts, TargetingHostsFixtures));

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

@@ -1,9 +0,0 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-import TargetingHostsPage from '../TargetingHostsPage';
-import { TargetingHostsPageFixtures } from './fixtures';
-
-describe('TargetingHostsPage', () =>
-  testComponentSnapshotsWithFixtures(
-    TargetingHostsPage,
-    TargetingHostsPageFixtures
-  ));

data/webpack/react_app/components/TargetingHosts/__tests__/TargetingHostsSelectors.test.js

@@ -1,31 +0,0 @@
-import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
-
-import {
-  selectItems,
-  selectAutoRefresh,
-  selectApiStatus,
-  selectTotalHosts,
-  selectIntervalExists,
-} from '../TargetingHostsSelectors';
-
-const state = {
-  TARGETING_HOSTS: {
-    status: 'RESOLVED',
-    response: {
-      hosts: [1, 2],
-      total_hosts: 2,
-      autoRefresh: 'true',
-    },
-  },
-};
-
-const fixtures = {
-  'should return hosts': () => selectItems(state),
-  'should return autoRefresh': () => selectAutoRefresh(state),
-  'should return apiStatus': () => selectApiStatus(state),
-  'should return totalHosts': () => selectTotalHosts(state),
-  'should return intervalExists': () => selectIntervalExists(state),
-};
-
-describe('TargetingHostsSelectors', () =>
-  testSelectorsSnapshotWithFixtures(fixtures));

data/webpack/react_app/components/TargetingHosts/__tests__/__snapshots__/HostStatus.test.js.snap

@@ -1,12 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`HostStatus renders 1`] = `
-<div>
-  <Icon
-    name="ok"
-    type="pf"
-  />
-   
-  Succeeded
-</div>
-`;

data/webpack/react_app/components/TargetingHosts/__tests__/__snapshots__/TargetingHosts.test.js.snap

@@ -1,96 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`TargetingHosts renders 1`] = `
-<LoadingState
-  additionalClasses=""
-  loading={false}
-  loadingText="Loading"
-  size="lg"
-  timeout={300}
->
-  <div>
-    <table
-      className="table table-bordered table-striped table-hover"
-    >
-      <thead>
-        <tr>
-          <th>
-            Host
-          </th>
-          <th>
-            Status
-          </th>
-          <th>
-            Actions
-          </th>
-        </tr>
-      </thead>
-      <tbody>
-        <HostItem
-          actions={Array []}
-          key="host"
-          link="/link"
-          name="host"
-          status="success"
-        />
-        <HostItem
-          actions={Array []}
-          key="host2"
-          link="/link2"
-          name="host2"
-          status="success"
-        />
-      </tbody>
-    </table>
-  </div>
-</LoadingState>
-`;
-
-exports[`TargetingHosts renders with error 1`] = `
-<Alert
-  className=""
-  onDismiss={null}
-  type="error"
->
-  There was an error while updating the status, try refreshing the page.
-</Alert>
-`;
-
-exports[`TargetingHosts renders with loading 1`] = `
-<LoadingState
-  additionalClasses=""
-  loading={true}
-  loadingText="Loading"
-  size="lg"
-  timeout={300}
->
-  <div>
-    <table
-      className="table table-bordered table-striped table-hover"
-    >
-      <thead>
-        <tr>
-          <th>
-            Host
-          </th>
-          <th>
-            Status
-          </th>
-          <th>
-            Actions
-          </th>
-        </tr>
-      </thead>
-      <tbody>
-        <tr>
-          <td
-            colSpan="3"
-          >
-            No hosts found.
-          </td>
-        </tr>
-      </tbody>
-    </table>
-  </div>
-</LoadingState>
-`;

data/webpack/react_app/components/TargetingHosts/__tests__/__snapshots__/TargetingHostsPage.test.js.snap

@@ -1,75 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`TargetingHostsPage renders 1`] = `
-<div
-  id="targeting_hosts"
->
-  <Row
-    bsClass="row"
-    componentClass="div"
-  >
-    <Col
-      bsClass="col"
-      componentClass="div"
-      md={6}
-    >
-      <SearchBar
-        bookmarksPosition="left"
-        data={
-          Object {
-            "autocomplete": Object {
-              "id": "targeting_hosts_search",
-              "searchQuery": "",
-              "url": "/hosts/auto_complete_search",
-              "useKeyShortcuts": true,
-            },
-            "bookmarks": Object {
-              "canCreate": true,
-              "documentationUrl": "/links/manual/4.1.5Searching",
-              "id": "targeting_hosts_search",
-              "url": "/api/bookmarks",
-            },
-            "controller": "hosts",
-          }
-        }
-        initialQuery=""
-        name={null}
-        onBookmarkClick={[Function]}
-        onSearch={[Function]}
-        onSearchChange={[Function]}
-        restrictedSearchQuery={[Function]}
-      />
-    </Col>
-  </Row>
-  <TargetingHostsLabelsRow
-    query={Object {}}
-  />
-  <br />
-  <TargetingHosts
-    apiStatus="RESOLVED"
-    items={
-      Array [
-        Object {
-          "actions": Array [],
-          "link": "/link",
-          "name": "host",
-          "status": "success",
-        },
-        Object {
-          "actions": Array [],
-          "link": "/link2",
-          "name": "host2",
-          "status": "success",
-        },
-      ]
-    }
-  />
-  <Pagination
-    className="targeting-hosts-pagination"
-    itemCount={1}
-    onChange={[Function]}
-    page={1}
-    updateParamsByUrl={false}
-  />
-</div>
-`;

data/webpack/react_app/components/TargetingHosts/__tests__/__snapshots__/TargetingHostsSelectors.test.js.snap

@@ -1,16 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`TargetingHostsSelectors should return apiStatus 1`] = `"RESOLVED"`;
-
-exports[`TargetingHostsSelectors should return autoRefresh 1`] = `"true"`;
-
-exports[`TargetingHostsSelectors should return hosts 1`] = `
-Array [
-  1,
-  2,
-]
-`;
-
-exports[`TargetingHostsSelectors should return intervalExists 1`] = `false`;
-
-exports[`TargetingHostsSelectors should return totalHosts 1`] = `2`;

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

@@ -1,56 +0,0 @@
-const items = [
-  {
-    name: 'host',
-    link: '/link',
-    status: 'success',
-    actions: [],
-  },
-  {
-    name: 'host2',
-    link: '/link2',
-    status: 'success',
-    actions: [],
-  },
-];
-
-export const HostItemFixtures = {
-  renders: {
-    name: 'Host1',
-    link: '/host1',
-    status: 'success',
-    actions: [],
-  },
-};
-
-export const HostStatusFixtures = {
-  renders: {
-    status: 'success',
-  },
-};
-
-export const TargetingHostsFixtures = {
-  renders: {
-    apiStatus: 'RESOLVED',
-    items,
-  },
-  'renders with error': {
-    apiStatus: 'ERROR',
-    items,
-  },
-  'renders with loading': {
-    apiStatus: 'PENDING',
-    items: [],
-  },
-};
-
-export const TargetingHostsPageFixtures = {
-  renders: {
-    handleSearch: () => {},
-    searchQuery: '',
-    apiStatus: 'RESOLVED',
-    items,
-    totalHosts: 1,
-    page: 1,
-    handlePagination: () => {},
-  },
-};

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

@@ -1,39 +0,0 @@
-import React from 'react';
-import PropTypes from 'prop-types';
-import { ActionButtons } from 'foremanReact/components/common/ActionButtons/ActionButtons';
-import HostStatus from './HostStatus';
-
-const HostItem = ({ name, link, status, actions }) => {
-  const hostLink = link ? (
-    <a href={link}>{name}</a>
-  ) : (
-    <a href="#" className="disabled">
-      {name}
-    </a>
-  );
-
-  return (
-    <tr id={`targeting-host-${name}`}>
-      <td className="host_name">{hostLink}</td>
-      <td className="host_status">
-        <HostStatus status={status} />
-      </td>
-      <td className="host_actions">
-        <ActionButtons buttons={[...actions]} />
-      </td>
-    </tr>
-  );
-};
-
-HostItem.propTypes = {
-  name: PropTypes.string.isRequired,
-  link: PropTypes.string.isRequired,
-  status: PropTypes.string.isRequired,
-  actions: PropTypes.array,
-};
-
-HostItem.defaultProps = {
-  actions: [],
-};
-
-export default HostItem;

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

@@ -1,54 +0,0 @@
-import React from 'react';
-import { Icon } from 'patternfly-react';
-import PropTypes from 'prop-types';
-import { translate as __ } from 'foremanReact/common/I18n';
-
-const HostStatus = ({ status }) => {
-  switch (status) {
-    case 'cancelled':
-      return (
-        <div>
-          <Icon type="pf" name="warning-triangle-o" /> {__('Cancelled')}
-        </div>
-      );
-    case 'N/A':
-      return (
-        <div>
-          <Icon type="fa" name="question" /> {__('Awaiting start')}
-        </div>
-      );
-    case 'running' || 'pending':
-      return (
-        <div>
-          <Icon type="pf" name="running" /> {__('Pending')}
-        </div>
-      );
-    case 'planned':
-      return (
-        <div>
-          <Icon type="pf" name="build" /> {__('Scheduled')}
-        </div>
-      );
-    case 'warning':
-    case 'error':
-      return (
-        <div>
-          <Icon type="pf" name="error-circle-o" /> {__('Failed')}
-        </div>
-      );
-    case 'success':
-      return (
-        <div>
-          <Icon type="pf" name="ok" /> {__('Succeeded')}
-        </div>
-      );
-    default:
-      return <span>{status}</span>;
-  }
-};
-
-HostStatus.propTypes = {
-  status: PropTypes.string.isRequired,
-};
-
-export default HostStatus;