foreman_templates 7.0.1 → 7.0.2

data/webpack/components/TemplateSyncResult/components/SyncedTemplate/helpers.js

@@ -0,0 +1,128 @@
+import React from 'react';
+import { isEmpty } from 'lodash';
+import { Icon } from 'patternfly-react';
+
+import IconInfoItem from './IconInfoItem';
+import EmptyInfoItem from './EmptyInfoItem';
+import StringInfoItem from './StringInfoItem';
+import LinkInfoItem from './LinkInfoItem';
+
+export const itemIteratorId = (template, attr) =>
+  `${template.templateFile}-${attr}`;
+
+export const additionalInfo = (template, editPath) => {
+  const infoAttrs = [
+    'name',
+    'locked',
+    'snippet',
+    'humanizedClassName',
+    'kind',
+    'templateFile',
+  ];
+
+  return infoAttrs.map(attr => {
+    const key = itemIteratorId(template, attr);
+
+    const classNameMap = { Ptable: 'Partition Table' };
+
+    if (!template[attr]) {
+      return <EmptyInfoItem template={template} attr={attr} key={key} />;
+    }
+
+    switch (attr) {
+      case 'locked':
+        return (
+          <IconInfoItem
+            template={template}
+            attr={attr}
+            iconName="lock"
+            tooltipText="Locked"
+            key={key}
+          />
+        );
+      case 'snippet':
+        return (
+          <IconInfoItem
+            template={template}
+            attr={attr}
+            iconName="check"
+            tooltipText="Snippet"
+            key={key}
+          />
+        );
+      case 'humanizedClassName':
+        return (
+          <StringInfoItem
+            template={template}
+            attr={attr}
+            translate
+            mapAttr={(templateObj, attribute) =>
+              classNameMap[templateObj[attribute]]
+                ? classNameMap[templateObj[attribute]]
+                : templateObj[attribute]
+            }
+            key={key}
+          />
+        );
+      case 'kind':
+        return <StringInfoItem template={template} attr={attr} key={key} />;
+      case 'templateFile':
+        return (
+          <StringInfoItem template={template} attr={attr} key={key} elipsed />
+        );
+      case 'name':
+        return (
+          <LinkInfoItem template={template} editPath={editPath} attr={attr} />
+        );
+      default:
+        return '';
+    }
+  });
+};
+
+export const itemLeftContentIcon = template => {
+  let iconName = template.additionalInfo ? 'warning-triangle-o' : undefined;
+
+  if (!iconName) {
+    iconName = isEmpty(aggregatedErrors(template)) ? 'ok' : 'error-circle-o';
+  }
+  return <Icon name={iconName} size="sm" type="pf" />;
+};
+
+export const expandableContent = template => {
+  if (Object.keys(aggregatedMessages(template)).length !== 0) {
+    const res = Object.keys(aggregatedMessages(template)).map(key => (
+      <li key={itemIteratorId(template, key)}>
+        {formatError(key, aggregatedMessages(template)[key])}
+      </li>
+    ));
+    return <ul>{res}</ul>;
+  }
+  return <span>There were no errors.</span>;
+};
+
+const aggregatedErrors = template => {
+  const err = { ...template.errors } || {};
+  if (template.additionalErrors) {
+    err.additional = template.additionalErrors;
+  }
+
+  return err;
+};
+
+const aggregatedMessages = template => {
+  const errors = aggregatedErrors(template);
+  if (template.additionalInfo) {
+    errors.info = template.additionalInfo;
+  }
+  return errors;
+};
+
+const formatError = (key, value) => {
+  const omitKeys = ['base', 'additional', 'info'];
+  if (omitKeys.reduce((memo, item) => memo || key === item, false)) {
+    return value;
+  }
+
+  return `${key}: ${value}`;
+};

data/webpack/components/TemplateSyncResult/components/SyncedTemplate/index.js

@@ -0,0 +1,33 @@
+import React from 'react';
+import { ListView } from 'patternfly-react';
+import PropTypes from 'prop-types';
+
+import {
+  expandableContent,
+  itemLeftContentIcon,
+  additionalInfo,
+} from './helpers';
+
+const SyncedTemplate = ({ template, editPath }) => (
+  <ListView.Item
+    key={template.id}
+    additionalInfo={additionalInfo(template, editPath)}
+    className="listViewItem--listItemVariants"
+    leftContent={itemLeftContentIcon(template)}
+    hideCloseIcon
+    stacked
+  >
+    {expandableContent(template)}
+  </ListView.Item>
+);
+
+SyncedTemplate.propTypes = {
+  template: PropTypes.object.isRequired,
+  editPath: PropTypes.string,
+};
+
+SyncedTemplate.defaultProps = {
+  editPath: '',
+};
+
+export default SyncedTemplate;

data/webpack/components/TemplateSyncResult/components/__tests__/SyncResultList.test.js

@@ -0,0 +1,27 @@
+import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+
+import SyncResultList from '../SyncResultList';
+import { importTemplates } from '../../__fixtures__/templateSyncResult.fixtures';
+
+const noop = () => {};
+
+const editPaths = {
+  JobTemplate: '',
+  Ptable: '',
+  ProvisioningTemplate: '',
+};
+
+const fixtures = {
+  'should render': {
+    pageChange: noop,
+    templates: importTemplates,
+    editPaths,
+    pagination: {
+      page: 1,
+      perPage: 20,
+    },
+  },
+};
+
+describe('SyncResultList', () =>
+  testComponentSnapshotsWithFixtures(SyncResultList, fixtures));

data/webpack/components/TemplateSyncResult/components/__tests__/SyncedTemplate.test.js

@@ -0,0 +1,30 @@
+import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import SyncedTemplate from '../SyncedTemplate';
+import {
+  noName,
+  epel,
+  coreos,
+  filteredOut,
+} from '../../__fixtures__/templateSyncResult.fixtures';
+
+const fixtures = {
+  'should render template with invalid metadata': {
+    template: noName,
+    editPath: '',
+  },
+  'should render template with validation errors': {
+    template: epel,
+    editPath: '/templates/:id/edit',
+  },
+  'should render template without errros': {
+    template: coreos,
+    editPath: '/ptables/:id/edit',
+  },
+  'should render skipped template': {
+    template: filteredOut,
+    editPath: '/ptables/:id/edit',
+  },
+};
+
+describe('SyncedTemplate', () =>
+  testComponentSnapshotsWithFixtures(SyncedTemplate, fixtures));

data/webpack/components/TemplateSyncResult/components/__tests__/__snapshots__/SyncResultList.test.js.snap

@@ -0,0 +1,102 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`SyncResultList should render 1`] = `
+<ListView
+  className=""
+>
+  <ListViewHeader />
+  <SyncedTemplate
+    editPath=""
+    key="null"
+    template={
+      Object {
+        "additionalErrors": "No \\"name\\" found in metadata",
+        "errors": null,
+        "name": null,
+        "templateFile": "random_template.erb",
+      }
+    }
+  />
+  <SyncedTemplate
+    editPath=""
+    key="EPEL"
+    template={
+      Object {
+        "additionalErrors": null,
+        "className": "ProvisioningTemplate",
+        "errors": Object {
+          "base": Array [
+            "This template is locked",
+          ],
+        },
+        "humanizedClassName": "Provisioning Template",
+        "kind": "Provision",
+        "locked": true,
+        "name": "EPEL",
+        "snippet": true,
+        "templateFile": "epel.erb",
+      }
+    }
+  />
+  <SyncedTemplate
+    editPath=""
+    key="CoreOS default"
+    template={
+      Object {
+        "additionalErrors": null,
+        "className": "Ptable",
+        "errors": Object {},
+        "humanizedClassName": "Ptable",
+        "locked": false,
+        "name": "CoreOS default",
+        "snippet": false,
+        "templateFile": "coreos_default.erb",
+      }
+    }
+  />
+  <SyncedTemplate
+    editPath=""
+    key="null"
+    template={
+      Object {
+        "additionalErrors": "No \\"model\\" found in metadata",
+        "errors": null,
+        "name": null,
+        "templateFile": "no_model.erb",
+      }
+    }
+  />
+  <SyncedTemplate
+    editPath=""
+    key="Kickstart fake"
+    template={
+      Object {
+        "additionalErrors": null,
+        "className": "ProvisioningTemplate",
+        "errors": Object {
+          "name": Array [
+            "has already been taken",
+          ],
+        },
+        "humanizedClassName": "Provisioning Template",
+        "locked": true,
+        "name": "Kickstart fake",
+        "snippet": true,
+        "templateFile": "kickstart_fake.erb",
+      }
+    }
+  />
+  <PaginationWrapper
+    dropdownButtonId="template-sync-result-dropdown"
+    itemCount={5}
+    onChange={[Function]}
+    pagination={
+      Object {
+        "page": 1,
+        "perPage": 20,
+      }
+    }
+    viewType="list"
+  />
+</ListView>
+`;