RubyGems - katello - Versions diffs - 3.7.0.rc1 → 3.7.0.rc2 - Mend

katello 3.7.0.rc1 → 3.7.0.rc2

Potentially problematic release.

This version of katello might be problematic. Click here for more details.

Files changed (189) hide show

data/webpack/move_to_foreman/components/common/table/components/__snapshots__/TableBody.test.js.snap ADDED Viewed

@@ -0,0 +1,28 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+exports[`TableBody renders TableBody 1`] = `
+<Body
+  onRow={[Function]}
+  rowKey={[Function]}
+  rows={
+    Array [
+      Object {
+        "data": "data-1",
+        "id": 1,
+      },
+      Object {
+        "data": "data-2",
+        "id": 2,
+      },
+    ]
+  }
+/>
+`;
+exports[`TableBody renders TableBody with message 1`] = `
+<TableBodyMessage
+  colSpan={2}
+>
+  some message
+</TableBodyMessage>
+`;

data/webpack/move_to_foreman/components/common/table/components/__snapshots__/TableBodyMessage.test.js.snap ADDED Viewed

@@ -0,0 +1,13 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+exports[`TableBodyMessage renders TableBodyMessage 1`] = `
+<tbody>
+  <tr>
+    <td
+      colSpan={2}
+    >
+      some children
+    </td>
+  </tr>
+</tbody>
+`;

data/webpack/move_to_foreman/components/common/table/components/__snapshots__/TableSelectionCell.test.js.snap ADDED Viewed

@@ -0,0 +1,16 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+exports[`TableSelectionCell renders TableSelectionCell 1`] = `
+<TableSelectionCell
+  className=""
+>
+  some before
+  <TableCheckbox
+    checked={true}
+    id="some id"
+    label="some label"
+    onChange={[Function]}
+  />
+  some after
+</TableSelectionCell>
+`;

data/webpack/move_to_foreman/components/common/table/components/__snapshots__/TableSelectionHeaderCell.test.js.snap ADDED Viewed

@@ -0,0 +1,15 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+exports[`TableSelectionHeaderCell renders TableSelectionHeaderCell 1`] = `
+<TableSelectionHeading
+  aria-label="some label"
+  className=""
+>
+  <TableCheckbox
+    checked={true}
+    id="some id"
+    label="some label"
+    onChange={[Function]}
+  />
+</TableSelectionHeading>
+`;

data/webpack/move_to_foreman/components/common/table/components/index.js ADDED Viewed

@@ -0,0 +1,6 @@
+export { default as Table } from './Table';
+export { default as TableBody } from './TableBody';
+export { default as TableBodyMessage } from './TableBodyMessage';
+export { default as TableSelectionHeaderCell } from './TableSelectionHeaderCell';
+export { default as TableSelectionCell } from './TableSelectionCell';
+export { default as CollapseSubscriptionGroupButton } from './CollapseSubscriptionGroupButton';

data/webpack/move_to_foreman/components/common/table/formatters/cellFormatter.js ADDED Viewed

@@ -0,0 +1,4 @@
+import React from 'react';
+import { Table as PfTable } from 'patternfly-react';
+export default value => <PfTable.Cell>{value}</PfTable.Cell>;

data/webpack/move_to_foreman/components/common/table/formatters/collapseableAndSelectionCellFormatter.js ADDED Viewed

@@ -0,0 +1,18 @@
+import React from 'react';
+import selectionCellFormatter from './selectionCellFormatter';
+import CollapseSubscriptionGroupButton from '../components/CollapseSubscriptionGroupButton';
+export default (collapseableController, selectionController, additionalData) => {
+  const shouldShowCollapseButton = collapseableController.isCollapseable(additionalData);
+  return selectionCellFormatter(
+    selectionController,
+    additionalData,
+    shouldShowCollapseButton && (
+      <CollapseSubscriptionGroupButton
+        collapsed={collapseableController.isCollapsed(additionalData)}
+        onClick={() => collapseableController.toggle(additionalData)}
+      />
+    ),
+  );
+};

data/webpack/move_to_foreman/components/common/table/formatters/ellipsisCellFormatter.js ADDED Viewed

@@ -0,0 +1,5 @@
+import React from 'react';
+import EllipsisWithTooltip from 'react-ellipsis-with-tooltip';
+import cellFormatter from './cellFormatter';
+export default value => cellFormatter(<EllipsisWithTooltip>{value}</EllipsisWithTooltip>);

data/webpack/move_to_foreman/components/common/table/formatters/headerFormatter.js ADDED Viewed

@@ -0,0 +1,4 @@
+import React from 'react';
+import { Table as PfTable } from 'patternfly-react';
+export default value => <PfTable.Heading>{value}</PfTable.Heading>;

data/webpack/move_to_foreman/components/common/table/formatters/index.js ADDED Viewed

@@ -0,0 +1,6 @@
+export { default as headerFormatter } from './headerFormatter';
+export { default as cellFormatter } from './cellFormatter';
+export { default as ellipsisCellFormatter } from './ellipsisCellFormatter';
+export { default as selectionHeaderCellFormatter } from './selectionHeaderCellFormatter';
+export { default as selectionCellFormatter } from './selectionCellFormatter';
+export { default as collapseableAndSelectionCellFormatter } from './collapseableAndSelectionCellFormatter';

data/webpack/move_to_foreman/components/common/table/formatters/selectionCellFormatter.js ADDED Viewed

@@ -0,0 +1,17 @@
+import React from 'react';
+import TableSelectionCell from '../components/TableSelectionCell';
+export default (
+  selectionController,
+  additionalData,
+  before,
+  after,
+) => (
+  <TableSelectionCell
+    id={`select${additionalData.rowIndex}`}
+    checked={selectionController.isSelected(additionalData)}
+    onChange={() => selectionController.selectRow(additionalData)}
+    before={before}
+    after={after}
+  />
+);

data/webpack/move_to_foreman/components/common/table/formatters/selectionHeaderCellFormatter.js ADDED Viewed

@@ -0,0 +1,10 @@
+import React from 'react';
+import TableSelectionHeaderCell from '../components/TableSelectionHeaderCell';
+export default (selectionController, label) => (
+  <TableSelectionHeaderCell
+    label={label}
+    checked={selectionController.allRowsSelected()}
+    onChange={() => selectionController.selectAllRows()}
+  />
+);

data/webpack/move_to_foreman/components/common/table/index.js CHANGED Viewed

@@ -1,88 +1,2 @@
-/* eslint-disable */
-import { Table as PfTable } from 'patternfly-react';
-import React from 'react';
-import EllipsisWithTooltip from 'react-ellipsis-with-tooltip';
-import EmptyState from '../emptyState';
-import PaginationRow from '../../../../components/PaginationRow/index';
-export const headerFormat = value => <PfTable.Heading>{value}</PfTable.Heading>;
-export const cellFormat = value => <PfTable.Cell>{value}</PfTable.Cell>;
-export const ellipsisFormat = value => (
-  <PfTable.Cell>
-    <EllipsisWithTooltip>{value}</EllipsisWithTooltip>
-  </PfTable.Cell>
-);
-export const TableBody = (props) => {
-  const { columns, rows, message } = props;
-  if (message !== undefined) {
-    return (
-      <tbody>
-        <tr>
-          <td colSpan={columns.length}>{message}</td>
-        </tr>
-      </tbody>
-    );
-  }
-  return (
-    <PfTable.Body
-      rows={rows}
-      rowKey={({ rowIndex }) => rowIndex}
-    />
-  );
-};
-export class Table extends React.Component {
-  isEmpty() {
-    return this.props.rows.length === 0 && this.props.bodyMessage === undefined;
-  }
-  render() {
-    const { columns, rows, emptyState, bodyMessage, children, itemCount, pagination, onPaginationChange, ...otherProps } = this.props;
-    let { sortingColumns } = this.props;
-    let paginationComponent;
-    if (itemCount && pagination) {
-      paginationComponent = (
-        <PaginationRow
-          viewType="table"
-          itemCount={itemCount}
-          pagination={pagination}
-          onChange={onPaginationChange}
-        />
-      );
-    }
-    if (this.isEmpty()) {
-      return (<EmptyState {...emptyState} />);
-    }
-    const table = (children !== undefined)
-      ? children
-      : [
-        <PfTable.Header key="header" />,
-        <TableBody key="body" columns={columns} rows={rows} message={bodyMessage} rowKey="id" />,
-      ];
-    sortingColumns = sortingColumns || {};
-    return (
-      <div>
-        <PfTable.PfProvider
-          className="table-fixed"
-          striped
-          bordered
-          hover
-          columns={columns}
-          {...otherProps}
-        >
-          {table}
-        </PfTable.PfProvider>
-        {paginationComponent}
-      </div>
-    );
-  }
-}
+export * from './formatters';
+export * from './components';

data/webpack/move_to_pf/LoadingState/LoadingState.js ADDED Viewed

@@ -0,0 +1,35 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { Spinner } from 'patternfly-react';
+import './LoadingState.scss';
+const LoadingState = ({
+  loading,
+  loadingText,
+  children,
+}) => {
+  if (loading) {
+    return (
+      <div className="loading-state">
+        <Spinner loading={loading} size="lg" />
+        <p>{loadingText}</p>
+      </div>
+    );
+  }
+  return children;
+};
+LoadingState.propTypes = {
+  loading: PropTypes.bool,
+  loadingText: PropTypes.string,
+  children: PropTypes.node,
+};
+LoadingState.defaultProps = {
+  loading: false,
+  loadingText: __('Loading'),
+  children: null,
+};
+export default LoadingState;

data/webpack/move_to_pf/LoadingState/LoadingState.scss ADDED Viewed

@@ -0,0 +1,12 @@
+.loading-state {
+  margin-top: 5%;
+  .spinner {
+    float: none;
+  }
+  p {
+    font-size: 15px;
+    text-align: center;
+  }
+}

data/webpack/move_to_pf/LoadingState/LoadingState.test.js ADDED Viewed

@@ -0,0 +1,28 @@
+/* eslint-disable */
+// Ignore eslint for this entire file because our rules
+// don't match patternfly-react's rules
+import React from 'react';
+import { shallow } from 'enzyme';
+import toJson from 'enzyme-to-json';
+import { LoadingState } from './index';
+test('Loading State renders properly while loading', () => {
+  const component = shallow(
+    <LoadingState loading loadingText="Loading">
+      <p>Loading Complete</p>
+    </LoadingState>
+  );
+  expect(toJson(component.render())).toMatchSnapshot();
+});
+test('Loading State renders properly while not loading', () => {
+  const component = shallow(
+    <LoadingState loading={false} loadingText="Loading">
+      <p>Loading Complete</p>
+    </LoadingState>
+  );
+  expect(toJson(component.render())).toMatchSnapshot();
+});

data/webpack/move_to_pf/LoadingState/__snapshots__/LoadingState.test.js.snap ADDED Viewed

@@ -0,0 +1,20 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+exports[`Loading State renders properly while loading 1`] = `
+<div
+  class="loading-state"
+>
+  <div
+    class="spinner spinner-lg"
+  />
+  <p>
+    Loading
+  </p>
+</div>
+`;
+exports[`Loading State renders properly while not loading 1`] = `
+<p>
+  Loading Complete
+</p>
+`;

data/webpack/move_to_pf/LoadingState/index.js ADDED Viewed

@@ -0,0 +1,3 @@
+// patternfly-react doesn't use this rule and this is how their components are structured
+// eslint-disable-next-line import/prefer-default-export
+export { default as LoadingState } from './LoadingState';

data/webpack/move_to_pf/test-utils/testHelpers.js ADDED Viewed

@@ -0,0 +1,71 @@
+import React from 'react';
+import { shallow } from 'enzyme';
+import toJson from 'enzyme-to-json';
+// a helper method for invoking a class method (for unit tests)
+// obj = a class
+// func = a tested function
+// objThis = an object's this
+// arg = function args
+export const classFunctionUnitTest = (obj, func, objThis, args) =>
+  obj.prototype[func].apply(objThis, args);
+/**
+ * Shallow render a component multipile times with fixtures
+ * @param  {ReactComponent} Component Component to shallow-render
+ * @param  {Object}         fixtures  key=fixture description, value=props to apply
+ * @return {Object}                   key=fixture description, value=shallow-rendered component
+ */
+export const shallowRenderComponentWithFixtures = (Component, fixtures) =>
+  Object.entries(fixtures).map(([description, props]) => ({
+    description,
+    component: shallow(<Component {...props} />),
+  }));
+/**
+ * Test a component with fixtures and snapshots
+ * @param  {ReactComponent} Component Component to test
+ * @param  {Object}         fixtures  key=fixture description, value=props to apply
+ */
+export const testComponentSnapshotsWithFixtures = (Component, fixtures) =>
+  shallowRenderComponentWithFixtures(Component, fixtures).forEach(({ description, component }) =>
+    it(description, () => expect(toJson(component)).toMatchSnapshot()));
+/**
+ * run an action (sync or async) and except the results to much snapshot
+ * @param  {Function}  runAction  Action runner function
+ * @return {Promise}
+ */
+export const testActionSnapshot = async (runAction) => {
+  const actionResults = runAction();
+  // if it's an async action
+  if (typeof actionResults === 'function') {
+    const dispatch = jest.fn();
+    await actionResults(dispatch);
+    expect(dispatch.mock.calls).toMatchSnapshot();
+  } else {
+    expect(actionResults).toMatchSnapshot();
+  }
+};
+/**
+ * Test actions with fixtures and snapshots
+ * @param  {Object} fixtures key=fixture description, value=action runner function
+ */
+export const testActionSnapshotWithFixtures = fixtures =>
+  Object.entries(fixtures).forEach(([description, runAction]) =>
+    it(description, () => testActionSnapshot(runAction)));
+/**
+ * Test a reducer with fixtures and snapshots
+ * @param  {Function} reducer  reducer to test
+ * @param  {Object}   fixtures key=fixture description, value=props to apply
+ */
+export const testReducerSnapshotWithFixtures = (reducer, fixtures) => {
+  const reduce = ({ state, action = {} } = {}) => reducer(state, action);
+  Object.entries(fixtures).forEach(([description, action]) =>
+    it(description, () => expect(reduce(action)).toMatchSnapshot()));
+};

data/webpack/redux/actions/RedHatRepositories/enabled.js CHANGED Viewed

@@ -23,7 +23,7 @@ export const createEnabledRepoParams = (extendedParams = {}) => {
   ]);
   const repoParams = {
-    ...{ organization_id: orgId, enabled: 'true' },
+    ...{ organization_id: orgId(), enabled: 'true' },
     ...propsToSnakeCase(extendedParams),
     search,
   };