foreman_ansible 6.0.1 → 6.3.0

data/webpack/components/AnsibleRolesAndVariables/index.js

@@ -0,0 +1,31 @@
+import React from 'react';
+import { useDispatch } from 'react-redux';
+import { push } from 'connected-react-router';
+
+import ImportRolesAndVariablesTable from './AnsibleRolesAndVariables';
+import { onSubmit } from './AnsibleRolesAndVariablesActions';
+import { ANSIBLE_ROLES_INDEX } from './AnsibleRolesAndVariablesConstants';
+
+const WrappedImportRolesAndVariables = props => {
+  const cols = [
+    { title: 'Name' },
+    { title: 'Operation' },
+    { title: 'Variables' },
+    { title: 'Hosts Count' },
+    { title: 'Hostgroups count' },
+  ];
+  const dispatch = useDispatch();
+  const submit = (rows, proxy) => dispatch(onSubmit(rows, proxy));
+  const onCancel = () => {
+    dispatch(push(ANSIBLE_ROLES_INDEX));
+  };
+  return (
+    <ImportRolesAndVariablesTable
+      {...props}
+      columnsData={cols}
+      onSubmit={submit}
+      onCancel={onCancel}
+    />
+  );
+};
+export default WrappedImportRolesAndVariables;

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcher.js

@@ -7,6 +7,7 @@ import { translate as __ } from 'foremanReact/common/I18n';
 import AvailableRolesList from './components/AvailableRolesList';
 import AssignedRolesList from './components/AssignedRolesList';
 import AnsibleRolesSwitcherError from './components/AnsibleRolesSwitcherError';
+import OrderedRolesTooltip from './components/OrderedRolesTooltip';
 import { excludeAssignedRolesSearch } from './AnsibleRolesSwitcherHelpers';
 
 class AnsibleRolesSwitcher extends React.Component {
@@ -17,7 +18,7 @@ class AnsibleRolesSwitcher extends React.Component {
       inheritedRoleIds,
       resourceId,
       resourceName,
-    } = this.props.data;
+    } = this.props;
 
     this.props.getAnsibleRoles(
       availableRolesUrl,
@@ -37,13 +38,11 @@ class AnsibleRolesSwitcher extends React.Component {
       itemCount,
       addAnsibleRole,
       removeAnsibleRole,
+      moveAnsibleRole,
       getAnsibleRoles,
-      changeAssignedPage,
-      assignedPagination,
-      assignedRolesCount,
       assignedRoles,
-      allAssignedRoles,
       unassignedRoles,
+      toDestroyRoles,
       error,
     } = this.props;
 
@@ -52,17 +51,17 @@ class AnsibleRolesSwitcher extends React.Component {
       inheritedRoleIds,
       resourceId,
       resourceName,
-    } = this.props.data;
+    } = this.props;
 
     const onListingChange = paginationArgs =>
       getAnsibleRoles(
         availableRolesUrl,
-        allAssignedRoles,
+        assignedRoles,
         inheritedRoleIds,
         resourceId,
         resourceName,
         paginationArgs,
-        excludeAssignedRolesSearch(allAssignedRoles)
+        excludeAssignedRolesSearch(assignedRoles)
       );
 
     return (
@@ -85,16 +84,18 @@ class AnsibleRolesSwitcher extends React.Component {
 
           <Col sm={6} className="assigned-roles-container">
             <div className="assigned-roles-header">
-              <h2>{__('Assigned Ansible Roles')}</h2>
+              <h2>
+                <OrderedRolesTooltip />
+                {__('Assigned Ansible Roles')}
+              </h2>
             </div>
             <AssignedRolesList
               assignedRoles={assignedRoles}
-              allAssignedRoles={allAssignedRoles}
-              pagination={assignedPagination}
-              itemCount={assignedRolesCount}
-              onPaginationChange={changeAssignedPage}
+              unassignedRoles={unassignedRoles}
               onRemoveRole={removeAnsibleRole}
+              onMoveRole={moveAnsibleRole}
               resourceName={lowerCase(resourceName || '')}
+              toDestroyRoles={toDestroyRoles}
             />
           </Col>
         </Row>
@@ -104,13 +105,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({
@@ -120,14 +119,9 @@ AnsibleRolesSwitcher.propTypes = {
   itemCount: PropTypes.number.isRequired,
   addAnsibleRole: PropTypes.func.isRequired,
   removeAnsibleRole: PropTypes.func.isRequired,
-  changeAssignedPage: PropTypes.func.isRequired,
-  assignedPagination: PropTypes.shape({
-    page: PropTypes.number,
-    perPage: PropTypes.number,
-  }).isRequired,
-  assignedRolesCount: PropTypes.number.isRequired,
+  moveAnsibleRole: PropTypes.func.isRequired,
   assignedRoles: PropTypes.arrayOf(PropTypes.object).isRequired,
-  allAssignedRoles: PropTypes.arrayOf(PropTypes.object).isRequired,
+  toDestroyRoles: PropTypes.arrayOf(PropTypes.object).isRequired,
   unassignedRoles: PropTypes.arrayOf(PropTypes.object).isRequired,
   error: PropTypes.shape({
     errorMsg: PropTypes.string,
@@ -137,6 +131,10 @@ AnsibleRolesSwitcher.propTypes = {
 
 AnsibleRolesSwitcher.defaultProps = {
   error: {},
+  resourceId: null,
+  resourceName: '',
+  initialAssignedRoles: [],
+  inheritedRoleIds: [],
 };
 
 export default AnsibleRolesSwitcher;

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcher.scss

@@ -10,7 +10,7 @@
     }
 
     .list-group-item.ansible-role-movable:hover {
-      cursor: pointer;
+      cursor: grab;
     }
   }
 

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcherActions.js

@@ -10,6 +10,7 @@ import {
   ANSIBLE_ROLES_FAILURE,
   ANSIBLE_ROLES_ADD,
   ANSIBLE_ROLES_REMOVE,
+  ANSIBLE_ROLES_MOVE,
   ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE,
 } from './AnsibleRolesSwitcherConstants';
 
@@ -21,7 +22,7 @@ export const getAnsibleRoles = (
   resourceName,
   pagination,
   search
-) => dispatch => {
+) => async dispatch => {
   dispatch({ type: ANSIBLE_ROLES_REQUEST });
 
   const params = {
@@ -30,19 +31,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) => {
@@ -63,6 +64,11 @@ export const removeAnsibleRole = role => ({
   payload: { role },
 });
 
+export const moveAnsibleRole = roles => ({
+  type: ANSIBLE_ROLES_MOVE,
+  payload: { roles },
+});
+
 export const changeAssignedPage = pagination => ({
   type: ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE,
   payload: { pagination },

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcherConstants.js

@@ -3,5 +3,4 @@ export const ANSIBLE_ROLES_SUCCESS = 'ANSIBLE_ROLES_SUCCESS';
 export const ANSIBLE_ROLES_FAILURE = 'ANSIBLE_ROLES_FAILURE';
 export const ANSIBLE_ROLES_ADD = 'ANSIBLE_ROLES_ADD';
 export const ANSIBLE_ROLES_REMOVE = 'ANSIBLE_ROLES_REMOVE';
-export const ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE =
-  'ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE';
+export const ANSIBLE_ROLES_MOVE = 'ANSIBLE_ROLE_MOVE';

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcherReducer.js

@@ -6,7 +6,7 @@ import {
   ANSIBLE_ROLES_FAILURE,
   ANSIBLE_ROLES_ADD,
   ANSIBLE_ROLES_REMOVE,
-  ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE,
+  ANSIBLE_ROLES_MOVE,
 } from './AnsibleRolesSwitcherConstants';
 
 export const initialState = Immutable({
@@ -17,12 +17,9 @@ export const initialState = Immutable({
     perPage: 10,
   },
   assignedRoles: [],
+  toDestroyRoles: [],
   inheritedRoleIds: [],
   results: [],
-  assignedPagination: {
-    page: 1,
-    perPage: 10,
-  },
   error: { errorMsg: '', status: '', statusText: '' },
 });
 
@@ -49,6 +46,9 @@ const ansibleRoles = (state = initialState, action) => {
     case ANSIBLE_ROLES_ADD:
       return state.merge({
         assignedRoles: state.assignedRoles.concat([payload.role]),
+        toDestroyRoles: state.toDestroyRoles.filter(
+          item => item.id !== payload.role.id
+        ),
         itemCount: state.itemCount - 1,
       });
     case ANSIBLE_ROLES_REMOVE:
@@ -56,11 +56,16 @@ const ansibleRoles = (state = initialState, action) => {
         assignedRoles: Immutable.flatMap(state.assignedRoles, item =>
           item.id === payload.role.id ? [] : item
         ),
-        results: state.results.concat([payload.role]),
+        results: state.results.find(item => payload.role.id === item.id)
+          ? state.results
+          : state.results.concat([payload.role]),
+        toDestroyRoles: state.toDestroyRoles.concat([
+          { ...payload.role, destroy: true },
+        ]),
         itemCount: state.itemCount + 1,
       });
-    case ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE:
-      return state.set('assignedPagination', payload.pagination);
+    case ANSIBLE_ROLES_MOVE:
+      return state.set('assignedRoles', payload.roles);
     default:
       return state;
   }

data/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcherSelectors.js

@@ -1,16 +1,7 @@
-import { differenceBy, slice, includes, uniq } from 'lodash';
+import { differenceBy, includes } from 'lodash';
 import Immutable from 'seamless-immutable';
 import { createSelector } from 'reselect';
-
-const compare = (a, b) => {
-  if (a.name < b.name) {
-    return -1;
-  }
-  if (a.name > b.name) {
-    return 1;
-  }
-  return 0;
-};
+import { propsToCamelCase } from 'foremanReact/common/helpers';
 
 const switcherState = state => state.foremanAnsible.ansibleRolesSwitcher;
 
@@ -21,7 +12,7 @@ const markInheritedRoles = (roles, inheritedRoleIds) =>
 
 export const selectResults = state =>
   Immutable(
-    Immutable.asMutable(uniq(switcherState(state).results)).sort(compare)
+    Immutable.asMutable(switcherState(state).results.map(propsToCamelCase))
   );
 
 export const selectItemCount = state => switcherState(state).itemCount;
@@ -29,16 +20,15 @@ export const selectItemCount = state => switcherState(state).itemCount;
 export const selectAssignedRoles = state =>
   Immutable.asMutable(
     markInheritedRoles(
-      switcherState(state).assignedRoles,
+      switcherState(state).assignedRoles.map(propsToCamelCase),
       switcherState(state).inheritedRoleIds
     )
-  ).sort(compare);
+  );
+
+export const selectToDestroyRoles = state =>
+  switcherState(state).toDestroyRoles;
 
-export const selectAssignedRolesCount = state =>
-  selectAssignedRoles(state).length;
 export const selectLoading = state => switcherState(state).loading;
-export const selectAssignedPagination = state =>
-  switcherState(state).assignedPagination;
 export const selectError = state => switcherState(state).error;
 export const selectPagination = state => switcherState(state).pagination;
 
@@ -56,13 +46,3 @@ export const selectUnassignedRoles = createSelector(
   selectAssignedRoles,
   (results, assignedRoles) => differenceBy(results, assignedRoles, 'id')
 );
-
-export const selectAssignedRolesPage = createSelector(
-  selectAssignedPagination,
-  selectAssignedRoles,
-  (assignedPagination, assignedRoles) => {
-    const offset = (assignedPagination.page - 1) * assignedPagination.perPage;
-
-    return slice(assignedRoles, offset, offset + assignedPagination.perPage);
-  }
-);

data/webpack/components/AnsibleRolesSwitcher/__fixtures__/ansibleRolesSwitcherReducer.fixtures.js

@@ -23,6 +23,7 @@ export const successState = Immutable({
     ...ansibleRolesLong.slice(4, 6),
   ],
   results: ansibleRolesLong,
+  toDestroyRoles: [],
   assignedPagination: {
     page: 1,
     perPage: 20,

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';
 
@@ -13,16 +13,18 @@ const fixtures = {
     itemCount: 20,
     addAnsibleRole: noop,
     removeAnsibleRole: noop,
+    moveAnsibleRole: noop,
     getAnsibleRoles: noop,
     changeAssignedPage: noop,
     assignedPagination: { page: 1, perPage: 12 },
     assignedRolesCount: 2,
     assignedRoles: [],
     unassignedRoles: [],
-    data: {
-      initialAssignedRoles: [],
-    },
+    initialAssignedRoles: [],
     error: { statusText: '', errorMsg: '' },
+    allAssignedRoles: [],
+    toDestroyRoles: [],
+    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';
@@ -15,7 +15,6 @@ import {
   ANSIBLE_ROLES_FAILURE,
   ANSIBLE_ROLES_ADD,
   ANSIBLE_ROLES_REMOVE,
-  ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE,
 } from '../AnsibleRolesSwitcherConstants';
 
 const fixtures = {
@@ -60,13 +59,6 @@ const fixtures = {
       payload: { role: ansibleRolesLong[5] },
     },
   },
-  'should change pagination for assigned roles': {
-    state: successState,
-    action: {
-      type: ANSIBLE_ROLES_ASSIGNED_PAGE_CHANGE,
-      payload: { pagination: { page: 20, perPage: 5 } },
-    },
-  },
 };
 
 describe('AnsibleRolesSwitcherReducer', () =>

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

@@ -1,13 +1,7 @@
-import { testSelectorsSnapshotWithFixtures } from 'react-redux-test-utils';
+import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
 
-import {
-  selectUnassignedRoles,
-  selectAssignedRolesPage,
-} from '../AnsibleRolesSwitcherSelectors';
-import {
-  ansibleRolesShort,
-  ansibleRolesLong,
-} from '../__fixtures__/ansibleRolesData.fixtures';
+import { selectUnassignedRoles } from '../AnsibleRolesSwitcherSelectors';
+import { ansibleRolesShort } from '../__fixtures__/ansibleRolesData.fixtures';
 
 const stateFactory = obj => ({
   foremanAnsible: {
@@ -25,18 +19,11 @@ const state2 = {
   assignedRoles: [],
 };
 
-const state3 = {
-  assignedRoles: ansibleRolesLong,
-  assignedPagination: { page: 2, perPage: 5 },
-};
-
 const fixtures = {
   'should return unassigned roles': () =>
     selectUnassignedRoles(stateFactory(state1)),
   'should return all roles when no roles assigned': () =>
     selectUnassignedRoles(stateFactory(state2)),
-  'should return requested page': () =>
-    selectAssignedRolesPage(stateFactory(state3)),
 };
 
 describe('AnsibleRolesSwitcherSelectors', () =>