fhir-react 0.3.3 → 0.3.4

package/src/components/resources/Patient/Patient.test.js

@@ -7,8 +7,75 @@ import examplePatientSTU3 from '../../../fixtures/stu3/resources/patient/example
 import example2PatientSTU3 from '../../../fixtures/stu3/resources/patient/example2.json';
 import example1PatientR4 from '../../../fixtures/r4/resources/patient/example1.json';
 import example3PatientR4 from '../../../fixtures/r4/resources/patient/example3.json';
+import fhirIcons from '../../../fixtures/example-icons';
 
 describe('should render component correctly', () => {
+  it('component without a fhirIcons props should render a default icon', () => {
+    const defaultProps = {
+      fhirResource: examplePatient,
+    };
+
+    const { getByAltText } = render(<Patient {...defaultProps} />);
+    const headerIcon = getByAltText('patient');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with a false as a fhirIcons props should render a placeholder', () => {
+    const defaultProps = {
+      fhirResource: examplePatient,
+      fhirIcons: false,
+    };
+
+    const { getByTestId } = render(<Patient {...defaultProps} />);
+    const headerIcon = getByTestId('placeholder');
+
+    expect(headerIcon).toBeTruthy();
+  });
+
+  it('component with the img as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirResource: examplePatient,
+      fhirIcons: (
+        <img
+          src={require('../assets/containers/ExplanationOfBenefit/explanation-of-benefit.svg.svg')}
+          alt="patient"
+        />
+      ),
+    };
+
+    const { getByAltText } = render(<Patient {...defaultProps} />);
+    const headerIcon = getByAltText('patient');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the resources object as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirResource: examplePatient,
+      fhirIcons: fhirIcons,
+    };
+
+    const { getByAltText } = render(<Patient {...defaultProps} />);
+    const headerIcon = getByAltText('patient');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the url as a fhirIcons props should render an img', () => {
+    const avatarSrc =
+      'https://www.gravatar.com/avatar/?s=50&r=any&default=identicon&forcedefault=1';
+    const defaultProps = {
+      fhirResource: examplePatient,
+      fhirIcons: avatarSrc,
+    };
+
+    const { getByAltText } = render(<Patient {...defaultProps} />);
+    const headerIcon = getByAltText('header icon');
+
+    expect(headerIcon.getAttribute('src')).toContain(avatarSrc);
+  });
+
   it('DSTU2', () => {
     const defaultProps = {
       fhirResource: examplePatient,

package/src/components/resources/Practitioner/Practitioner.js

@@ -15,7 +15,6 @@ import UnhandledResourceDataStructure from '../UnhandledResourceDataStructure';
 import _get from 'lodash/get';
 import _has from 'lodash/has';
 import fhirVersions from '../fhirResourceVersions';
-import md5 from 'md5';
 
 const commonDTO = fhirResource => {
   const id = _get(fhirResource, 'id', '');
@@ -83,8 +82,7 @@ const resourceDTO = (fhirVersion, fhirResource) => {
   }
 };
 
-const Practitioner = props => {
-  const { fhirResource, fhirVersion } = props;
+const Practitioner = ({ fhirResource, fhirVersion, fhirIcons }) => {
   let fhirResourceData = {};
   try {
     fhirResourceData = resourceDTO(fhirVersion, fhirResource);
@@ -93,7 +91,6 @@ const Practitioner = props => {
     return <UnhandledResourceDataStructure resourceName="Practitioner" />;
   }
   const {
-    id,
     identifier,
     name,
     gender,
@@ -155,17 +152,10 @@ const Practitioner = props => {
       <Accordion
         headerContent={
           <Header
+            resourceName="Practitioner"
             additionalContent={<p className="mb-0">{`(${use})`}</p>}
             badges={status && <Badge data-testid="status">{status}</Badge>}
-            icon={
-              <img
-                className="header-icon__practitioner-avatar rounded-1"
-                src={`http://www.gravatar.com/avatar/${md5(
-                  id,
-                )}?s=30&r=any&default=identicon&forcedefault=1`}
-                alt=""
-              />
-            }
+            icon={fhirIcons}
             title={<HumanName fhirData={name} isTitle />}
           />
         }

package/src/components/resources/Practitioner/Practitioner.stories.js

@@ -11,6 +11,8 @@ import stu3Example1 from '../../../fixtures/stu3/resources/practitioner/example-
 import r4Example1 from '../../../fixtures/r4/resources/practitioner/example1.json';
 import r4Example2 from '../../../fixtures/r4/resources/practitioner/example2.json';
 import r4Example3 from '../../../fixtures/r4/resources/practitioner/example3.json';
+import PractitionerIcon from '../../../assets/containers/Practitioner/practitioner.svg';
+import fhirIcons from '../../../fixtures/example-icons';
 
 export default { title: 'Practitioner' };
 
@@ -20,6 +22,7 @@ export const DefaultVisualizationDSTU2 = () => {
     <Practitioner
       fhirVersion={fhirVersions.DSTU2}
       fhirResource={fhirResource}
+      fhirIcons={require('../../../assets/containers/Practitioner/practitioner.svg')}
     />
   );
 };
@@ -30,6 +33,7 @@ export const Example2OfDSTU2 = () => {
     <Practitioner
       fhirVersion={fhirVersions.DSTU2}
       fhirResource={fhirResource}
+      fhirIcons={PractitionerIcon}
     />
   );
 };
@@ -37,21 +41,33 @@ export const Example2OfDSTU2 = () => {
 export const ExampleOfSTU3 = () => {
   const fhirResource = object('Resource', stu3Example1);
   return (
-    <Practitioner fhirVersion={fhirVersions.STU3} fhirResource={fhirResource} />
+    <Practitioner
+      fhirVersion={fhirVersions.STU3}
+      fhirResource={fhirResource}
+      fhirIcons={fhirIcons}
+    />
   );
 };
 
 export const Example1OfR4 = () => {
   const fhirResource = object('Resource', r4Example1);
   return (
-    <Practitioner fhirVersion={fhirVersions.R4} fhirResource={fhirResource} />
+    <Practitioner
+      fhirVersion={fhirVersions.R4}
+      fhirResource={fhirResource}
+      fhirIcons={false}
+    />
   );
 };
 
 export const Example2OfR4 = () => {
   const fhirResource = object('Resource', r4Example2);
   return (
-    <Practitioner fhirVersion={fhirVersions.R4} fhirResource={fhirResource} />
+    <Practitioner
+      fhirVersion={fhirVersions.R4}
+      fhirResource={fhirResource}
+      fhirIcons={'random text'}
+    />
   );
 };
 

package/src/components/resources/Practitioner/Practitioner.test.js

@@ -7,8 +7,80 @@ import dstu2Example1 from '../../../fixtures/dstu2/resources/practitioner/exampl
 import stu3Example1 from '../../../fixtures/stu3/resources/practitioner/example-1.json';
 import r4Example1 from '../../../fixtures/r4/resources/practitioner/example1.json';
 import r4Example2 from '../../../fixtures/r4/resources/practitioner/example3.json';
+import fhirIcons from '../../../fixtures/example-icons';
 
 describe('Practitioner should render component correctly', () => {
+  it('component without a fhirIcons props should render a default icon', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+    };
+
+    const { getByAltText } = render(<Practitioner {...defaultProps} />);
+    const headerIcon = getByAltText('practitioner');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with a false as a fhirIcons props should render a placeholder', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: false,
+    };
+
+    const { getByTestId } = render(<Practitioner {...defaultProps} />);
+    const headerIcon = getByTestId('placeholder');
+
+    expect(headerIcon).toBeTruthy();
+  });
+
+  it('component with the img as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: (
+        <img
+          src={require('../assets/containers/Practitioner/practitioner.svg')}
+          alt="practitioner"
+        />
+      ),
+    };
+
+    const { getByAltText } = render(<Practitioner {...defaultProps} />);
+    const headerIcon = getByAltText('practitioner');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the resources object as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: fhirIcons,
+    };
+
+    const { getByAltText } = render(<Practitioner {...defaultProps} />);
+    const headerIcon = getByAltText('practitioner');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the url as a fhirIcons props should render an img', () => {
+    const avatarSrc =
+      'https://www.gravatar.com/avatar/?s=50&r=any&default=identicon&forcedefault=1';
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: avatarSrc,
+    };
+
+    const { getByAltText } = render(<Practitioner {...defaultProps} />);
+    const headerIcon = getByAltText('header icon');
+
+    expect(headerIcon.getAttribute('src')).toContain(avatarSrc);
+  });
+
   it('should render component correctly with DSTU2 source data', () => {
     const defaultProps = {
       fhirVersion: fhirVersions.DSTU2,

package/src/components/resources/Procedure/Procedure.js

@@ -37,7 +37,6 @@ const Procedure = props => {
   const note = _get(fhirResource, 'note', []);
   const outcome = _get(fhirResource, 'outcome');
 
-  const headerIcon = fhirIcons && fhirIcons[_get(fhirResource, 'resourceType')];
   const tableData = [
     {
       label: 'Identification',
@@ -121,7 +120,7 @@ const Procedure = props => {
               </>
             }
             badges={status && <Badge data-testid="status">{status}</Badge>}
-            icon={headerIcon}
+            icon={fhirIcons}
             title={display}
           />
         }

package/src/components/resources/Procedure/Procedure.stories.js

@@ -12,17 +12,23 @@ import r4Example2 from '../../../fixtures/r4/resources/procedure/example2.json';
 import r4Example3 from '../../../fixtures/r4/resources/procedure/example3.json';
 
 import fhirIcons from '../../../fixtures/example-icons';
+import ProcedureIcon from '../../../assets/containers/Procedure/procedure.svg';
 
 export default { title: 'Procedure' };
 
 export const DefaultVisualizationDSTU2 = () => {
   const fhirResource = object('Resource', example1);
-  return <Procedure fhirResource={fhirResource} fhirIcons={fhirIcons} />;
+  return (
+    <Procedure
+      fhirResource={fhirResource}
+      fhirIcons={require('../../../assets/containers/Procedure/procedure.svg')}
+    />
+  );
 };
 
 export const ExampleOfSTU3 = () => {
   const fhirResource = object('Resource', stu3Example1);
-  return <Procedure fhirResource={fhirResource} fhirIcons={fhirIcons} />;
+  return <Procedure fhirResource={fhirResource} fhirIcons={ProcedureIcon} />;
 };
 
 export const Example2OfSTU3 = () => {
@@ -32,15 +38,15 @@ export const Example2OfSTU3 = () => {
 
 export const Example1OfR4 = () => {
   const fhirResource = object('Resource', r4Example1);
-  return <Procedure fhirResource={fhirResource} fhirIcons={fhirIcons} />;
+  return <Procedure fhirResource={fhirResource} fhirIcons={false} />;
 };
 
 export const Example2OfR4 = () => {
   const fhirResource = object('Resource', r4Example2);
-  return <Procedure fhirResource={fhirResource} fhirIcons={fhirIcons} />;
+  return <Procedure fhirResource={fhirResource} fhirIcons={'random text'} />;
 };
 
 export const Example3OfR4 = () => {
   const fhirResource = object('Resource', r4Example3);
-  return <Procedure fhirResource={fhirResource} fhirIcons={fhirIcons} />;
+  return <Procedure fhirResource={fhirResource} />;
 };

package/src/components/resources/Procedure/Procedure.test.js

@@ -5,10 +5,80 @@ import r4Example2 from '../../../fixtures/r4/resources/procedure/example2.json';
 import r4Example3 from '../../../fixtures/r4/resources/procedure/example3.json';
 import { render } from '@testing-library/react';
 import stu3Example1 from '../../../fixtures/stu3/resources/procedure/example1.json';
-
 import fhirIcons from '../../../fixtures/example-icons';
+import fhirVersions from '../fhirResourceVersions';
 
 describe('Procedure should render component correctly', () => {
+  it('component without a fhirIcons props should render a default icon', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+    };
+
+    const { getByAltText } = render(<Procedure {...defaultProps} />);
+    const headerIcon = getByAltText('procedure');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with a false as a fhirIcons props should render a placeholder', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: false,
+    };
+
+    const { getByTestId } = render(<Procedure {...defaultProps} />);
+    const headerIcon = getByTestId('placeholder');
+
+    expect(headerIcon).toBeTruthy();
+  });
+
+  it('component with the img as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: (
+        <img
+          src={require('../assets/containers/Procedure/procedure.svg')}
+          alt="procedure"
+        />
+      ),
+    };
+
+    const { getByAltText } = render(<Procedure {...defaultProps} />);
+    const headerIcon = getByAltText('procedure');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the resources object as a fhirIcons props should render an img', () => {
+    const defaultProps = {
+      fhirVersion: fhirVersions.DSTU2,
+      fhirResource: dstu2Example1,
+      fhirIcons: fhirIcons,
+    };
+
+    const { getByAltText } = render(<Procedure {...defaultProps} />);
+    const headerIcon = getByAltText('procedure');
+
+    expect(headerIcon.getAttribute('src')).toContain('IMAGE_MOCK');
+  });
+
+  it('component with the url as a fhirIcons props should render an img', () => {
+    const avatarSrc =
+      'https://www.gravatar.com/avatar/?s=50&r=any&default=identicon&forcedefault=1';
+    const defaultProps = {
+      fhirResource: dstu2Example1,
+      fhirIcons: avatarSrc,
+    };
+
+    const { getByAltText } = render(<Procedure {...defaultProps} />);
+    const headerIcon = getByAltText('header icon');
+
+    expect(headerIcon.getAttribute('src')).toContain(avatarSrc);
+  });
+
   it('should render component correctly with DSTU2 source data', () => {
     const defaultProps = {
       fhirResource: dstu2Example1,

package/src/components/resources/ResourceCategory/ResourceCategory.js

@@ -2,6 +2,7 @@ import { Root, Title } from '../../ui';
 import HeaderIcon from '../../datatypes/HeaderIcon';
 import React from 'react';
 import PropTypes from 'prop-types';
+import { isUrl } from '../../../utils/isUrl';
 
 const ResourceCategory = props => {
   const { title, itemsCount, fhirIcons } = props;
@@ -12,7 +13,10 @@ const ResourceCategory = props => {
   const getItemsCountLabel = () =>
     `${parsedItemsCount} ${parsedItemsCount === 1 ? 'item' : 'items'}`;
 
-  const headerIcon = fhirIcons && fhirIcons['ResourceCategory'];
+  const headerIcon = isUrl(fhirIcons)
+    ? fhirIcons
+    : fhirIcons && fhirIcons['ResourceCategory'];
+
   const parsedItemsCount = parseNumber(itemsCount);
 
   return (

package/src/components/ui/index.js

@@ -3,62 +3,70 @@ import React from 'react';
 import { getBadgeColor } from '../../utils/getBadgeColor';
 import HeaderIcon from '../datatypes/HeaderIcon';
 
-export const Header = props => {
+export const Header = ({
+  resourceName,
+  isAccordionOpenable,
+  icon,
+  titleTestID,
+  title,
+  prefixBadge,
+  badges,
+  additionalBadge,
+  additionalContent,
+  rightAdditionalContent,
+  children,
+}) => {
   const rightItemsClass = 'align-items-center flex-fill d-flex';
 
   return (
     <>
       {// This condition was left due to fact, that to much changes in Header will generate many errors in tests. This condition will be removed after all changes have been made.
-      props.children || (
+      children || (
         <div
-          className={`fhir-ui__${props.resourceName}-Header w-100 p-4 position-relative`}
+          className={`fhir-ui__${resourceName}-Header w-100 p-4 position-relative`}
         >
           <div
-            className={`fhir-ui__${props.resourceName}-Header__title-data ${
-              props.isAccordionOpenable ? 'header__title-row' : ''
+            className={`fhir-ui__${resourceName}-Header__title-data ${
+              isAccordionOpenable ? 'header__title-row' : ''
             } d-flex w-100 flex-column flex-sm-row`}
           >
             <div className="d-flex">
               <div
-                className={`fhir-ui__${props.resourceName}-Header__icon flex-shrink-1 m-half me-2`}
+                className={`fhir-ui__${resourceName}-Header__icon d-flex align-items-center flex-shrink-1 m-half me-2`}
               >
-                <HeaderIcon headerIcon={props.icon} />
+                <HeaderIcon headerIcon={icon} resourceName={resourceName} />
               </div>
               <div
-                className={`fhir-ui__${props.resourceName}-Header__title flex-fill text-start`}
+                className={`fhir-ui__${resourceName}-Header__title flex-fill text-start`}
               >
-                <Title data-testid={props.titleTestID || 'title'}>
-                  {props.title || ''}
+                <Title data-testid={titleTestID || 'title'}>
+                  {title || ''}
                 </Title>
               </div>
             </div>
 
             <div
-              className={`fhir-ui__${props.resourceName}-Header__badges ps-1 ps-sm-2 mt-3 mt-sm-0 badges-max-width-sm flex-wrap flex-sm-nowrap justify-content-between justify-content-sm-end ${rightItemsClass}`}
+              className={`fhir-ui__${resourceName}-Header__badges ps-1 ps-sm-2 mt-3 mt-sm-0 badges-max-width-sm flex-wrap flex-sm-nowrap justify-content-between justify-content-sm-end ${rightItemsClass}`}
             >
-              {props.prefixBadge && (
-                <div className="me-3">{props.prefixBadge}</div>
-              )}
+              {prefixBadge && <div className="me-3">{prefixBadge}</div>}
               <div className="d-flex">
-                {props.badges}
-                {props.additionalBadge && (
-                  <div className="ms-3">{props.additionalBadge}</div>
+                {badges}
+                {additionalBadge && (
+                  <div className="ms-3">{additionalBadge}</div>
                 )}
               </div>
             </div>
           </div>
           <div
-            className={`fhir-ui__${
-              props.resourceName
-            }-Header__additional-content w-100 justify-content-start d-flex  ${
-              props.additionalContent ? ' pt-2' : ''
+            className={`fhir-ui__${resourceName}-Header__additional-content w-100 justify-content-start d-flex  ${
+              additionalContent ? ' pt-2' : ''
             }`}
           >
-            {props.additionalContent}
+            {additionalContent}
             <div
-              className={`fhir-ui__${props.resourceName}-Header__rightAdditionalContent justify-content-end ${rightItemsClass}`}
+              className={`fhir-ui__${resourceName}-Header__rightAdditionalContent justify-content-end ${rightItemsClass}`}
             >
-              {props.rightAdditionalContent}
+              {rightAdditionalContent}
             </div>
           </div>
         </div>
@@ -213,7 +221,7 @@ export const ValueSection = props => (
   </div>
 );
 
-export const MissingValue = props => (
+export const MissingValue = () => (
   <span className="fhir-ui__MissingValue">-</span>
 );