From 8e8a85005934ef517f8a8f5834434d6002657c6d Mon Sep 17 00:00:00 2001
From: fboldo <filipeboldo@gmail.com>
Date: Fri, 22 Sep 2023 15:01:56 +0100
Subject: [PATCH] update snapshots

---
 .../__snapshots__/Breadcrumb.test.tsx.snap    |  4 +--
 src/components/button/Button.tsx              | 24 +++++------------
 src/components/card/Card.tsx                  |  2 +-
 .../__snapshots__/Card.test.tsx.snap          | 24 +++++------------
 .../__snapshots__/Checkboxes.test.tsx.snap    | 11 --------
 .../__snapshots__/DateInput.test.tsx.snap     |  9 -------
 src/components/footer/Footer.tsx              |  2 +-
 .../__snapshots__/Hero.test.tsx.snap          |  4 +--
 .../__snapshots__/Icons.test.tsx.snap         | 27 +------------------
 src/components/pagination/Pagination.tsx      | 19 ++++++++-----
 .../__snapshots__/Radios.test.tsx.snap        |  4 +--
 .../WarningCallout.test.tsx.snap              |  4 +--
 12 files changed, 33 insertions(+), 101 deletions(-)

diff --git a/src/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap b/src/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
index cd463980..12818f76 100644
--- a/src/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
+++ b/src/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
@@ -14,9 +14,7 @@ exports[`Breadcrumb base element matches snapshot: BaseBreadcrumbs 1`] = `
 `;
 
 exports[`Breadcrumb renders with children: BreadcrumbWithChildren 1`] = `
-<Breadcrumb
-  aria-label="Breadcrumb"
->
+<Breadcrumb>
   <nav
     aria-label="Breadcrumb"
     className="nhsuk-breadcrumb"
diff --git a/src/components/button/Button.tsx b/src/components/button/Button.tsx
index 82fc57d3..840e7ee9 100644
--- a/src/components/button/Button.tsx
+++ b/src/components/button/Button.tsx
@@ -21,10 +21,12 @@ export const Button: React.FC<ButtonProps> = ({
   disabled,
   secondary,
   reverse,
+  type = 'submit',
   ...rest
 }) => (
   // eslint-disable-next-line react/button-has-type
   <button
+    type={type}
     className={classNames(
       'nhsuk-button',
       { 'nhsuk-button--disabled': disabled },
@@ -40,13 +42,12 @@ export const Button: React.FC<ButtonProps> = ({
 
 export const ButtonLink: React.FC<ButtonLinkProps> = ({
   className,
-  role,
-  draggable,
+  role = 'button',
+  draggable = false,
   children,
   disabled,
   secondary,
   reverse,
-  type = 'submit',
   ...rest
 }) => (
   <a
@@ -57,7 +58,6 @@ export const ButtonLink: React.FC<ButtonLinkProps> = ({
       { 'nhsuk-button--reverse': reverse },
       className,
     )}
-    type={type}
     role={role}
     aria-disabled={disabled ? 'true' : 'false'}
     draggable={draggable}
@@ -67,26 +67,16 @@ export const ButtonLink: React.FC<ButtonLinkProps> = ({
   </a>
 );
 
-const ButtonWrapper: React.FC<ButtonLinkProps | ButtonProps> = ({
-  href,
-  as,
-  role = 'button',
-  draggable = false,
-  ...rest
-}) => {
+const ButtonWrapper: React.FC<ButtonLinkProps | ButtonProps> = ({ href, as, ...rest }) => {
   const buttonProps = { type: 'submit', ...rest } as ButtonProps;
   if (as === 'a') {
-    return (
-      <ButtonLink role={role} draggable={draggable} href={href} {...(rest as ButtonLinkProps)} />
-    );
+    return <ButtonLink href={href} {...(rest as ButtonLinkProps)} />;
   }
   if (as === 'button') {
     return <Button {...buttonProps} />;
   }
   if (href) {
-    return (
-      <ButtonLink role={role} draggable={draggable} href={href} {...(rest as ButtonLinkProps)} />
-    );
+    return <ButtonLink href={href} {...(rest as ButtonLinkProps)} />;
   }
   return <Button {...buttonProps} />;
 };
diff --git a/src/components/card/Card.tsx b/src/components/card/Card.tsx
index edfe679b..57e1d225 100644
--- a/src/components/card/Card.tsx
+++ b/src/components/card/Card.tsx
@@ -34,7 +34,7 @@ const Card: ICard = ({ className, clickable, children, feature = false, ...rest
     )}
     {...rest}
   >
-    <CardContext.Provider value={{ feature: Boolean(feature) }}>{children}</CardContext.Provider>
+    <CardContext.Provider value={{ feature: feature }}>{children}</CardContext.Provider>
   </div>
 );
 
diff --git a/src/components/card/__tests__/__snapshots__/Card.test.tsx.snap b/src/components/card/__tests__/__snapshots__/Card.test.tsx.snap
index c54628ac..b47cabf3 100644
--- a/src/components/card/__tests__/__snapshots__/Card.test.tsx.snap
+++ b/src/components/card/__tests__/__snapshots__/Card.test.tsx.snap
@@ -18,9 +18,7 @@ exports[`Card Card.Group matches snapshot 1`] = `
           <div
             className="nhsuk-grid-column-one-half nhsuk-card-group__item"
           >
-            <Card
-              feature={false}
-            >
+            <Card>
               <div
                 className="nhsuk-card"
               >
@@ -28,9 +26,7 @@ exports[`Card Card.Group matches snapshot 1`] = `
                   <div
                     className="nhsuk-card__content"
                   >
-                    <CardHeading
-                      headingLevel="h2"
-                    >
+                    <CardHeading>
                       <HeadingLevel
                         className="nhsuk-card__heading"
                         headingLevel="h2"
@@ -66,9 +62,7 @@ exports[`Card Card.Group matches snapshot 1`] = `
           <div
             className="nhsuk-grid-column-one-half nhsuk-card-group__item"
           >
-            <Card
-              feature={false}
-            >
+            <Card>
               <div
                 className="nhsuk-card"
               >
@@ -76,9 +70,7 @@ exports[`Card Card.Group matches snapshot 1`] = `
                   <div
                     className="nhsuk-card__content"
                   >
-                    <CardHeading
-                      headingLevel="h2"
-                    >
+                    <CardHeading>
                       <HeadingLevel
                         className="nhsuk-card__heading"
                         headingLevel="h2"
@@ -110,9 +102,7 @@ exports[`Card Card.Group matches snapshot 1`] = `
 `;
 
 exports[`Card matches snapshot 1`] = `
-<Card
-  feature={false}
->
+<Card>
   <div
     className="nhsuk-card"
   >
@@ -130,9 +120,7 @@ exports[`Card matches snapshot 1`] = `
       <div
         className="nhsuk-card__content"
       >
-        <CardHeading
-          headingLevel="h2"
-        >
+        <CardHeading>
           <HeadingLevel
             className="nhsuk-card__heading"
             headingLevel="h2"
diff --git a/src/components/checkboxes/__tests__/__snapshots__/Checkboxes.test.tsx.snap b/src/components/checkboxes/__tests__/__snapshots__/Checkboxes.test.tsx.snap
index 3717c750..2221080d 100644
--- a/src/components/checkboxes/__tests__/__snapshots__/Checkboxes.test.tsx.snap
+++ b/src/components/checkboxes/__tests__/__snapshots__/Checkboxes.test.tsx.snap
@@ -18,7 +18,6 @@ exports[`Checkboxes matches snapshot 1`] = `
         id="example"
       >
         <Box
-          type="checkbox"
           value="animal"
         >
           <div
@@ -54,7 +53,6 @@ exports[`Checkboxes matches snapshot 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="mines"
         >
           <div
@@ -90,7 +88,6 @@ exports[`Checkboxes matches snapshot 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="farm"
         >
           <div
@@ -151,7 +148,6 @@ exports[`Checkboxes matches snapshot with boolean error 1`] = `
         id="example"
       >
         <Box
-          type="checkbox"
           value="animal"
         >
           <div
@@ -187,7 +183,6 @@ exports[`Checkboxes matches snapshot with boolean error 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="mines"
         >
           <div
@@ -223,7 +218,6 @@ exports[`Checkboxes matches snapshot with boolean error 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="farm"
         >
           <div
@@ -281,8 +275,6 @@ exports[`Checkboxes matches snapshot with string error 1`] = `
     >
       <ErrorMessage
         id="example--error-message"
-        role="alert"
-        visuallyHiddenText="Error: "
       >
         <span
           className="nhsuk-error-message"
@@ -302,7 +294,6 @@ exports[`Checkboxes matches snapshot with string error 1`] = `
         id="example"
       >
         <Box
-          type="checkbox"
           value="animal"
         >
           <div
@@ -338,7 +329,6 @@ exports[`Checkboxes matches snapshot with string error 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="mines"
         >
           <div
@@ -374,7 +364,6 @@ exports[`Checkboxes matches snapshot with string error 1`] = `
           </div>
         </Box>
         <Box
-          type="checkbox"
           value="farm"
         >
           <div
diff --git a/src/components/date-input/__tests__/__snapshots__/DateInput.test.tsx.snap b/src/components/date-input/__tests__/__snapshots__/DateInput.test.tsx.snap
index b8aa1b1d..dd072e59 100644
--- a/src/components/date-input/__tests__/__snapshots__/DateInput.test.tsx.snap
+++ b/src/components/date-input/__tests__/__snapshots__/DateInput.test.tsx.snap
@@ -19,10 +19,7 @@ exports[`DateInput matches snapshot 1`] = `
       >
         <DayInput>
           <IndividualDateInput
-            inputMode="numeric"
             inputType="day"
-            pattern="[0-9]*"
-            type="text"
           >
             <div
               className="nhsuk-date-input__item"
@@ -65,10 +62,7 @@ exports[`DateInput matches snapshot 1`] = `
         </DayInput>
         <MonthInput>
           <IndividualDateInput
-            inputMode="numeric"
             inputType="month"
-            pattern="[0-9]*"
-            type="text"
           >
             <div
               className="nhsuk-date-input__item"
@@ -111,10 +105,7 @@ exports[`DateInput matches snapshot 1`] = `
         </MonthInput>
         <YearInput>
           <IndividualDateInput
-            inputMode="numeric"
             inputType="year"
-            pattern="[0-9]*"
-            type="text"
           >
             <div
               className="nhsuk-date-input__item"
diff --git a/src/components/footer/Footer.tsx b/src/components/footer/Footer.tsx
index 90704625..a85f4cfd 100644
--- a/src/components/footer/Footer.tsx
+++ b/src/components/footer/Footer.tsx
@@ -42,7 +42,7 @@ interface Footer extends React.FC<FooterProps> {
   Copyright: React.FC<HTMLProps<HTMLParagraphElement>>;
 }
 
-const Footer: Footer = ({ className, children, visuallyHiddenText, ...rest }) => (
+const Footer: Footer = ({ className, children, visuallyHiddenText = 'Support links', ...rest }) => (
   <footer {...rest}>
     <div className={classNames('nhsuk-footer', className)}>
       <Container>
diff --git a/src/components/hero/__tests__/__snapshots__/Hero.test.tsx.snap b/src/components/hero/__tests__/__snapshots__/Hero.test.tsx.snap
index 8275f8c2..2188aa97 100644
--- a/src/components/hero/__tests__/__snapshots__/Hero.test.tsx.snap
+++ b/src/components/hero/__tests__/__snapshots__/Hero.test.tsx.snap
@@ -1,9 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`Hero Hero.Heading matches snapshot: Hero.Heading 1`] = `
-<HeroHeading
-  headingLevel="h1"
->
+<HeroHeading>
   <HeadingLevel
     className="nhsuk-u-margin-bottom-3"
     headingLevel="h1"
diff --git a/src/components/icons/__tests__/__snapshots__/Icons.test.tsx.snap b/src/components/icons/__tests__/__snapshots__/Icons.test.tsx.snap
index 328901cf..335d8987 100644
--- a/src/components/icons/__tests__/__snapshots__/Icons.test.tsx.snap
+++ b/src/components/icons/__tests__/__snapshots__/Icons.test.tsx.snap
@@ -3,9 +3,7 @@
 exports[`Icons all icons match snapshots: ArrowLeft 1`] = `
 <ArrowLeft>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__arrow-left"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -26,9 +24,7 @@ exports[`Icons all icons match snapshots: ArrowLeft 1`] = `
 exports[`Icons all icons match snapshots: ArrowRight 1`] = `
 <ArrowRight>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__arrow-right"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -49,9 +45,7 @@ exports[`Icons all icons match snapshots: ArrowRight 1`] = `
 exports[`Icons all icons match snapshots: ArrowRightCircle 1`] = `
 <ArrowRightCircle>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__arrow-right-circle"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -74,10 +68,7 @@ exports[`Icons all icons match snapshots: ArrowRightCircle 1`] = `
 `;
 
 exports[`Icons all icons match snapshots: BaseIconSVG 1`] = `
-<BaseIconSVG
-  height={32}
-  width={32}
->
+<BaseIconSVG>
   <svg
     aria-hidden="true"
     className="nhsuk-icon"
@@ -92,9 +83,7 @@ exports[`Icons all icons match snapshots: BaseIconSVG 1`] = `
 exports[`Icons all icons match snapshots: ChevronLeft 1`] = `
 <ChevronLeft>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__chevron-left"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -115,9 +104,7 @@ exports[`Icons all icons match snapshots: ChevronLeft 1`] = `
 exports[`Icons all icons match snapshots: ChevronRight 1`] = `
 <ChevronRight>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__chevron-right"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -138,9 +125,7 @@ exports[`Icons all icons match snapshots: ChevronRight 1`] = `
 exports[`Icons all icons match snapshots: Close 1`] = `
 <Close>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__close"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -161,9 +146,7 @@ exports[`Icons all icons match snapshots: Close 1`] = `
 exports[`Icons all icons match snapshots: Cross 1`] = `
 <Cross>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__cross"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -210,9 +193,7 @@ exports[`Icons all icons match snapshots: Emdash 1`] = `
 exports[`Icons all icons match snapshots: Minus 1`] = `
 <Minus>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__minus"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -243,9 +224,7 @@ exports[`Icons all icons match snapshots: Minus 1`] = `
 exports[`Icons all icons match snapshots: Plus 1`] = `
 <Plus>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__minus"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -276,9 +255,7 @@ exports[`Icons all icons match snapshots: Plus 1`] = `
 exports[`Icons all icons match snapshots: Search 1`] = `
 <Search>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__search"
-    width={32}
   >
     <svg
       aria-hidden="true"
@@ -322,9 +299,7 @@ exports[`Icons all icons match snapshots: SmallEmdash 1`] = `
 exports[`Icons all icons match snapshots: Tick 1`] = `
 <Tick>
   <BaseIconSVG
-    height={32}
     iconType="nhsuk-icon__tick"
-    width={32}
   >
     <svg
       aria-hidden="true"
diff --git a/src/components/pagination/Pagination.tsx b/src/components/pagination/Pagination.tsx
index 613041d6..da88883d 100644
--- a/src/components/pagination/Pagination.tsx
+++ b/src/components/pagination/Pagination.tsx
@@ -12,8 +12,6 @@ const PaginationLink: React.FC<PaginationLinkProps> = ({
   children,
   previous,
   next,
-  role = 'navigation',
-  'aria-label': ariaLabel = 'Pagination',
   ...rest
 }) => (
   <li
@@ -23,14 +21,12 @@ const PaginationLink: React.FC<PaginationLinkProps> = ({
     )}
   >
     <a
-      role={role}
       className={classNames(
         'nhsuk-pagination__link',
         { 'nhsuk-pagination__link--prev': previous },
         { 'nhsuk-pagination__link--next': next },
         className,
       )}
-      aria-label={ariaLabel}
       {...rest}
     >
       <span className="nhsuk-pagination__title">
@@ -49,8 +45,19 @@ interface Pagination extends React.FC<HTMLProps<HTMLDivElement>> {
   Link: React.FC<PaginationLinkProps>;
 }
 
-const Pagination: Pagination = ({ className, children, ...rest }) => (
-  <nav className={classNames('nhsuk-pagination', className)} {...rest}>
+const Pagination: Pagination = ({
+  className,
+  children,
+  role = 'navigation',
+  'aria-label': ariaLabel = 'Pagination',
+  ...rest
+}) => (
+  <nav
+    className={classNames('nhsuk-pagination', className)}
+    role={role}
+    aria-label={ariaLabel}
+    {...rest}
+  >
     <ul className="nhsuk-list nhsuk-pagination__list">{children}</ul>
   </nav>
 );
diff --git a/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap b/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap
index c7c2500f..7a5bdd18 100644
--- a/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap
+++ b/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap
@@ -19,7 +19,6 @@ exports[`Radios matches snapshot 1`] = `
       >
         <Radio
           id="example-1"
-          type="radio"
           value="yes"
         >
           <div
@@ -57,7 +56,6 @@ exports[`Radios matches snapshot 1`] = `
         </Radio>
         <Radio
           id="example-2"
-          type="radio"
           value="no"
         >
           <div
@@ -97,4 +95,4 @@ exports[`Radios matches snapshot 1`] = `
     </div>
   </FormGroup>
 </Radios>
-`;
\ No newline at end of file
+`;
diff --git a/src/components/warning-callout/__tests__/__snapshots__/WarningCallout.test.tsx.snap b/src/components/warning-callout/__tests__/__snapshots__/WarningCallout.test.tsx.snap
index ffd32284..ff433cfe 100644
--- a/src/components/warning-callout/__tests__/__snapshots__/WarningCallout.test.tsx.snap
+++ b/src/components/warning-callout/__tests__/__snapshots__/WarningCallout.test.tsx.snap
@@ -5,9 +5,7 @@ exports[`WarningCallout matches snapshot 1`] = `
   <div
     className="nhsuk-warning-callout"
   >
-    <WarningCalloutLabel
-      visuallyHiddenText="Important: "
-    >
+    <WarningCalloutLabel>
       <HeadingLevel
         className="nhsuk-warning-callout__label"
       >