@astryxdesign/core 0.6.3-canary.f22695a → 0.6.3

package/src/Icon/Icon.test.tsx

@@ -162,13 +162,10 @@ describe('Icon', () => {
     expect(icon).toHaveAttribute('aria-label', 'Home');
   });
 
-  it('uses default color and standalone size when not specified', () => {
+  it('uses default color and size when not specified', () => {
     render(<Icon icon={TestIcon} data-testid="icon" />);
-
-    const icon = screen.getByTestId('icon');
-    expect(icon).toHaveAttribute('data-size', 'md');
-    expect(getComputedStyle(icon).width).toBe('1.25rem');
-    expect(getComputedStyle(icon).height).toBe('1.25rem');
+    // The component should render without errors with defaults
+    expect(screen.getByTestId('icon')).toBeInTheDocument();
   });
 
   it('applies aria-hidden by default in string (registry) mode', () => {

package/src/Icon/Icon.tsx

@@ -31,14 +31,6 @@ import {getIcon} from './globalIconRegistry';
 import type {IconName, NamespacedIconName} from './globalIconRegistry';
 import {mergeProps} from '../utils';
 import {themeProps} from '../utils/themeProps';
-import {useIconSize} from './IconDefaultSizeContext';
-import {
-  iconBoxSizeStyles,
-  iconSizeStyles,
-  type IconSize,
-} from './IconSize.stylex';
-
-export type {IconSize} from './IconSize.stylex';
 
 // =============================================================================
 // Styles
@@ -118,11 +110,75 @@ const colorStyles = stylex.create({
   },
 });
 
+/**
+ * Size styles for direct SVG icon components.
+ * Uses width/height only — SVG components handle their own viewBox scaling.
+ *
+ * Sizes are expressed in `rem` (relative to the root font-size) so icons scale
+ * in step with text when the document font-size changes, matching the rest of
+ * the design system's rem-based type scale. Values are the px-equivalents at a
+ * 16px root: 12px → 0.75rem, 16px → 1rem, 20px → 1.25rem, 24px → 1.5rem.
+ */
+const sizeStyles = stylex.create({
+  xsm: {
+    width: '0.75rem',
+    height: '0.75rem',
+  },
+  sm: {
+    width: '1rem',
+    height: '1rem',
+  },
+  md: {
+    width: '1.25rem',
+    height: '1.25rem',
+  },
+  lg: {
+    width: '1.5rem',
+    height: '1.5rem',
+  },
+});
+
+/**
+ * Size styles for string-based (registry) icons.
+ * Includes fontSize so that 1em-based icons from the registry scale correctly.
+ *
+ * Expressed in `rem` for the same reason as {@link sizeStyles} — icons track the
+ * root font-size instead of being locked to absolute pixels.
+ */
+const spanSizeStyles = stylex.create({
+  /* eslint-disable @astryx/no-hardcoded-styles -- fontSize here sizes 1em-based
+     registry SVGs to the icon box; icons use their own 12/16/20/24 scale, not
+     the 14px-anchored textSizeVars type scale. Values are rem so icons track
+     the root font-size. */
+  xsm: {
+    width: '0.75rem',
+    height: '0.75rem',
+    fontSize: '0.75rem',
+  },
+  sm: {
+    width: '1rem',
+    height: '1rem',
+    fontSize: '1rem',
+  },
+  md: {
+    width: '1.25rem',
+    height: '1.25rem',
+    fontSize: '1.25rem',
+  },
+  lg: {
+    width: '1.5rem',
+    height: '1.5rem',
+    fontSize: '1.5rem',
+  },
+  /* eslint-enable @astryx/no-hardcoded-styles */
+});
+
 // =============================================================================
 // Types
 // =============================================================================
 
 export type IconColor = keyof typeof colorStyles;
+export type IconSize = keyof typeof sizeStyles;
 
 /**
  * Type for icon components that can be passed to Icon.
@@ -159,9 +215,7 @@ export interface IconProps extends Omit<
    * - 'sm': 1rem (16px at a 16px root)
    * - 'md': 1.25rem (20px at a 16px root)
    * - 'lg': 1.5rem (24px at a 16px root)
-   *
-   * An explicit value wins. When omitted, Icon uses the nearest default supplied
-   * by an owning Astryx component for its icon slot, then falls back to 'md'.
+   * @default 'md'
    */
   size?: IconSize;
   /**
@@ -243,7 +297,7 @@ function getIconA11yProps(
 export function Icon({
   icon,
   color = 'inherit',
-  size: sizeProp,
+  size = 'md',
   label,
   ref,
   className,
@@ -251,7 +305,6 @@ export function Icon({
   xstyle,
   ...props
 }: IconProps) {
-  const size = useIconSize(sizeProp);
   // Derive ARIA from `label`: decorative (aria-hidden) by default, or a
   // meaningful image (role="img" + aria-label) when `label` is non-empty.
   const a11yProps = getIconA11yProps(label);
@@ -288,12 +341,7 @@ export function Icon({
       // precedence as escape hatches.
       {...mergeProps(
         themeProps('icon', {size, color}),
-        stylex.props(
-          styles.root,
-          colorStyles[color],
-          iconSizeStyles[size],
-          xstyle,
-        ),
+        stylex.props(styles.root, colorStyles[color], sizeStyles[size], xstyle),
         className ?? undefined,
         style,
       )}
@@ -362,7 +410,7 @@ function IconFromRegistry({
         stylex.props(
           styles.span,
           colorStyles[color],
-          iconBoxSizeStyles[size],
+          spanSizeStyles[size],
           xstyle,
         ),
         className ?? undefined,

package/src/IconButton/IconButton.doc.mjs

@@ -20,9 +20,9 @@ export const docs = {
     {
       name: 'icon',
       type: 'ReactNode',
-      description: 'Icon element rendered inside the button. An Astryx Icon with no explicit size defaults to sm for sm/md buttons and md for lg buttons.',
+      description: 'Icon element rendered inside the button.',
       required: true,
-      slotElements: [{__element: 'Icon', props: {icon: 'check'}}],
+      slotElements: [{__element: 'Icon', props: {icon: 'check', size: 'sm'}}],
     },
     {
       name: 'variant',
@@ -137,7 +137,7 @@ export const docsDense = {
   },
   propDescriptions: {
     label: 'accessible label; used as aria-label, not rendered as visible text',
-    icon: 'icon element rendered inside button; unsized Astryx Icon defaults to sm for sm/md buttons and md for lg',
+    icon: 'icon element rendered inside button',
     variant: 'visual style variant',
     size: 'size variant',
     elevation: 'resting shadow depth: none|low|med|high; raise for a floating action button (FAB)',

package/src/IconButton/IconButton.test.tsx

@@ -12,7 +12,6 @@
 import {describe, it, expect, vi} from 'vitest';
 import {render, screen} from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import {Icon} from '../Icon/Icon';
 import {IconButton} from './IconButton';
 
 describe('IconButton', () => {
@@ -53,45 +52,6 @@ describe('IconButton', () => {
     expect(screen.getByRole('button', {name: 'Add'})).toBeInTheDocument();
   });
 
-  it.each([
-    ['sm', 'sm', '1rem'],
-    ['md', 'sm', '1rem'],
-    ['lg', 'md', '1.25rem'],
-  ] as const)(
-    'renders the %s button wrapper and inherited Icon at the same size',
-    (buttonSize, iconSize, renderedSize) => {
-      render(
-        <IconButton
-          label="Add"
-          icon={<Icon icon="check" data-testid="icon" />}
-          size={buttonSize}
-        />,
-      );
-
-      const icon = screen.getByTestId('icon');
-      const wrapper = icon.parentElement;
-      expect(wrapper).not.toBeNull();
-
-      expect(icon).toHaveAttribute('data-size', iconSize);
-      expect(getComputedStyle(icon).width).toBe(renderedSize);
-      expect(getComputedStyle(icon).height).toBe(renderedSize);
-      expect(getComputedStyle(wrapper!).width).toBe(renderedSize);
-      expect(getComputedStyle(wrapper!).height).toBe(renderedSize);
-    },
-  );
-
-  it('preserves an explicit Astryx Icon size', () => {
-    render(
-      <IconButton
-        label="Add"
-        icon={<Icon icon="check" size="lg" data-testid="icon" />}
-        size="sm"
-      />,
-    );
-
-    expect(screen.getByTestId('icon')).toHaveAttribute('data-size', 'lg');
-  });
-
   it('handles click events', async () => {
     const user = userEvent.setup();
     const handleClick = vi.fn();

package/src/Item/Item.doc.mjs

@@ -43,7 +43,7 @@ export const docs = {
         {name: 'layout', type: "'stacked' | 'inline'", description: 'How the label and description sit together. stacked puts the description on its own line below the label; inline keeps both on one line, description ellipsizing first, so the row fits a fixed-height host.', default: "'stacked'"},
         {name: 'onClick', type: '(event: MouseEvent) => void', description: 'Click handler. Makes the item clickable with button semantics.'},
         {name: 'interactiveRef', type: 'RefObject<HTMLElement | null>', description: 'Ref to a nested control (e.g. a checkbox in startContent) that owns the item\'s keyboard access and action. The row becomes an enlarged click/tap target that delegates surface clicks to it (useClickableContainer) and renders no invisible button/anchor, so the row adds no second tab stop (WCAG 4.1.2). Mutually exclusive with onClick/href; those are ignored when set.'},
-        {name: 'href', type: 'string', description: 'Link URL. Makes the item a link via an invisible anchor element. The destination follows the shared navigation rule described on the Link `href` prop.'},
+        {name: 'href', type: 'string', description: 'Link URL. Makes the item a link via an invisible anchor element.'},
         {name: 'target', type: "'_blank' | '_self'", description: 'Link target. Only used with href. target="_blank" automatically adds noopener noreferrer.'},
         {name: 'rel', type: 'string', description: 'Link relationship tokens. noopener noreferrer are merged automatically for target="_blank".'},
         {name: 'isHighlighted', type: 'boolean', description: 'Highlighted state (hover/keyboard focus appearance).', default: 'false'},
@@ -167,7 +167,7 @@ export const docsDense = {
         onClick: 'Click handler; enables button semantics.',
         interactiveRef:
           'Ref to a nested control that owns the item\'s keyboard access/action; row delegates surface clicks to it (useClickableContainer), no invisible button/anchor, no second tab stop (WCAG 4.1.2). Mutually exclusive with onClick/href.',
-        href: 'Link URL; enables anchor semantics. Follows the shared navigation rule (see Link href).',
+        href: 'Link URL; enables anchor semantics.',
         target:
           'Link target, only with href. target="_blank" auto-adds noopener noreferrer.',
         rel: 'Link relationship tokens. noopener noreferrer are merged for target="_blank".',

package/src/Link/Link.doc.mjs

@@ -38,8 +38,7 @@ export const docs = {
         {
           name: 'href',
           type: 'string',
-          description:
-            'Link destination URL. Astryx-owned links and clickable surfaces block javascript:, vbscript:, and data:text/html after control-character removal, outer-whitespace trimming, and case-insensitive scheme inspection. The same rule covers native anchors, custom routers, Markdown links, and every activation method. Relative paths, fragments, protocol-relative URLs, HTTP(S), mailto, tel, safe custom schemes, and accepted downloads keep their existing behavior. A rejected destination renders inertly without invoking a custom router. There is no bypass prop; exceptional behavior requires caller-owned custom rendering outside this guarantee. Images and other resource URLs have separate policies.',
+          description: 'Link destination URL',
         },
         {
           name: 'hasUnderline',
@@ -152,8 +151,7 @@ export const docs = {
         {
           name: 'component',
           type: 'LinkComponentType',
-          description:
-            'Component to use for all link elements. It receives accepted `href` and `to` values under the shared navigation rule described on the Link `href` prop. Supported structured destinations, including their `protocol`, are checked without changing object identity. If either supplied destination is rejected, Astryx renders inert content without invoking this component; it does not pass undefined or fall back to the other destination.',
+          description: 'Component to use for all link elements',
           required: true,
         },
         {
@@ -357,7 +355,7 @@ export const docsDense = {
       propDescriptions: {
         as: 'Custom component to render instead of <a>',
         label: 'Accessible label (aria-label). Only for non-text content like icon-only links.',
-        href: 'Link destination URL. Shared rule blocks javascript:, vbscript:, data:text/html after scheme normalization on every Astryx-owned navigation path. Ordinary URLs, fragments, safe custom schemes, and accepted downloads are preserved. Rejected => inert, no custom router. No bypass prop; exceptional caller-owned rendering is outside the guarantee. Image/resource policy is separate.',
+        href: 'Link destination URL',
         hasUnderline: 'Always show underline',
         isDisabled: 'Disables link',
         isExternalLink: 'Opens new tab w/ external icon and safe rel tokens',
@@ -378,7 +376,7 @@ export const docsDense = {
       description:
         'Provider setting default link component for all Astryx links in subtree.',
       propDescriptions: {
-        component: 'Component for all link elements. Accepted href/to follow the Link href rule; structured fields including protocol are checked, preserving identity. Either rejected => inert content, no router invocation or fallback.',
+        component: 'Component for all link elements',
         children: 'Subtree',
       },
     },

package/src/Link/LinkProvider.doc.mjs

@@ -13,7 +13,7 @@ export const docs = {
     description: 'Wraps your app to replace the default <a> tag with a framework-specific link component (e.g. Next.js Link) for client-side routing across all Astryx components.',
   },
   props: [
-    {name: 'component', type: 'LinkComponentType', required: true, description: 'Link component to use for all link elements in the subtree (e.g. Next.js Link). It receives accepted `href` and `to` values under the shared navigation rule described on the Link `href` prop. Supported structured destinations, including their `protocol`, are checked without changing object identity. If either supplied destination is rejected, Astryx renders inert content without invoking this component; it does not pass undefined or fall back to the other destination.'},
+    {name: 'component', type: 'LinkComponentType', required: true, description: 'Link component to use for all link elements in the subtree (e.g. Next.js Link).'},
     {name: 'children', type: 'ReactNode', required: true, description: 'Content to render with the link provider.'},
   ],
 };
@@ -25,6 +25,6 @@ export const docsDense = {
     description: 'Wraps app to replace default <a> tag w/ framework-specific link component (e.g. Next.js Link) for client-side routing across all Astryx components.',
   },
   propDescriptions: {
-    component: 'link component for all link elements in subtree (e.g. Next.js Link). Accepted href/to follow the Link href rule; structured fields including protocol are checked, preserving identity. Either rejected => inert content, no router invocation or fallback.',
+    component: 'link component for all link elements in subtree (e.g. Next.js Link)',
   },
 };

package/src/Link/useLinkComponent.test.tsx

@@ -208,339 +208,6 @@ describe('useLinkComponent — to prop', () => {
   });
 });
 
-// =============================================================================
-// href scheme rule
-// =============================================================================
-
-describe('useLinkComponent — href scheme rule', () => {
-  // Kept in a const so the JSX stays lint-clean — the test is ABOUT this URL.
-  const scriptUrl = 'javascript:alert(1)';
-
-  function HrefConsumer({href}: {href: string}) {
-    const LinkComponent = useLinkComponent();
-    return (
-      <LinkComponent href={href} data-testid="resolved-link">
-        Link
-      </LinkComponent>
-    );
-  }
-
-  function propSpy() {
-    return vi.fn(
-      ({
-        children,
-        ...props
-      }: {
-        children?: React.ReactNode;
-        [key: string]: unknown;
-      }) => (
-        <a data-testid="spy-link" href={props.href as string}>
-          {children}
-        </a>
-      ),
-    );
-  }
-
-  it('forwards ordinary hrefs to a custom component untouched', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        <HrefConsumer href="/docs/button" />
-      </LinkProvider>,
-    );
-    expect(spy).toHaveBeenCalledWith(
-      expect.objectContaining({href: '/docs/button', to: '/docs/button'}),
-    );
-  });
-
-  it('withholds a script-scheme href from a custom component (href and to alike)', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        {/* eslint-disable-next-line @eslint-react/dom-no-script-url -- the test proves this URL never navigates */}
-        <HrefConsumer href={scriptUrl} />
-      </LinkProvider>,
-    );
-    expect(spy).not.toHaveBeenCalled();
-    expect(screen.getByTestId('resolved-link')).not.toHaveAttribute('href');
-  });
-
-  it('an explicit `to` passes the same check instead of riding through ...rest', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    function ToConsumer({to}: {to: string}) {
-      const LinkComponent = useLinkComponent();
-      return (
-        <LinkComponent href="/fallback" to={to} data-testid="resolved-link">
-          Link
-        </LinkComponent>
-      );
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        {/* eslint-disable-next-line @eslint-react/dom-no-script-url -- the test proves this URL never reaches the router */}
-        <ToConsumer to={scriptUrl} />
-      </LinkProvider>,
-    );
-    expect(spy).not.toHaveBeenCalled();
-    expect(screen.getByTestId('resolved-link')).not.toHaveAttribute('href');
-  });
-
-  it('an ordinary explicit `to` keeps its precedence over href', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    function ToConsumer() {
-      const LinkComponent = useLinkComponent();
-      return (
-        <LinkComponent
-          href="/fallback"
-          to="/routed"
-          data-testid="resolved-link">
-          Link
-        </LinkComponent>
-      );
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        <ToConsumer />
-      </LinkProvider>,
-    );
-    expect(spy).toHaveBeenCalledWith(
-      expect.objectContaining({href: '/fallback', to: '/routed'}),
-    );
-  });
-});
-
-// =============================================================================
-// structured router `to`
-// =============================================================================
-
-describe('useLinkComponent — structured router `to`', () => {
-  const scriptUrl = 'javascript:alert(1)';
-
-  function propSpy() {
-    return vi.fn(
-      ({
-        children,
-        ...props
-      }: {
-        children?: React.ReactNode;
-        [key: string]: unknown;
-      }) => (
-        <a data-testid="spy-link" href={props.href as string}>
-          {children}
-        </a>
-      ),
-    );
-  }
-
-  function ToConsumer({to}: {to: unknown}) {
-    const LinkComponent = useLinkComponent();
-    return (
-      <LinkComponent href="/fallback" to={to} data-testid="resolved-link">
-        Link
-      </LinkComponent>
-    );
-  }
-
-  it('preserves a structured destination for the router (no crash, same object)', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    const to = {pathname: '/docs', search: '?q=1', hash: '#top'};
-    render(
-      <LinkProvider component={SpyLink}>
-        <ToConsumer to={to} />
-      </LinkProvider>,
-    );
-    expect(spy).toHaveBeenCalledWith(
-      expect.objectContaining({href: '/fallback', to}),
-    );
-    expect(spy.mock.calls[0][0].to).toBe(to);
-  });
-
-  it('preserves a structured destination that carries no pathname', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    const to = {search: '?page=2'};
-    render(
-      <LinkProvider component={SpyLink}>
-        <ToConsumer to={to} />
-      </LinkProvider>,
-    );
-    expect(spy).toHaveBeenCalledWith(expect.objectContaining({to}));
-  });
-
-  it('withholds a structured destination whose pathname is a script scheme', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        <ToConsumer to={{pathname: scriptUrl}} />
-      </LinkProvider>,
-    );
-    expect(spy).not.toHaveBeenCalled();
-    expect(screen.getByTestId('resolved-link')).not.toHaveAttribute('href');
-  });
-
-  it('withholds a structured destination whose href is a script scheme', () => {
-    const spy = propSpy();
-    function SpyLink(props: {[key: string]: unknown}) {
-      return spy(props);
-    }
-    render(
-      <LinkProvider component={SpyLink}>
-        <ToConsumer to={{href: scriptUrl}} />
-      </LinkProvider>,
-    );
-    expect(spy).not.toHaveBeenCalled();
-    expect(screen.getByTestId('resolved-link')).not.toHaveAttribute('href');
-  });
-});
-
-// =============================================================================
-// Shared native and router boundary
-// =============================================================================
-
-describe('useLinkComponent — rejected destinations are inert', () => {
-  function Consumer({
-    href,
-    to,
-    as,
-    onClick,
-    linkRef,
-  }: {
-    href?: unknown;
-    to?: unknown;
-    as?: LinkComponentType;
-    onClick?: () => void;
-    linkRef?: React.Ref<HTMLAnchorElement>;
-  }) {
-    const Component = useLinkComponent(as);
-    return (
-      <Component
-        href={href}
-        to={to}
-        ref={linkRef}
-        onClick={onClick}
-        className="consumer-class"
-        aria-label="Destination"
-        data-testid="destination">
-        Go
-      </Component>
-    );
-  }
-
-  for (const mode of ['native', 'provider', 'as'] as const) {
-    it.each([
-      {href: 'javascript:alert(1)'},
-      {href: 'vbscript:MsgBox(1)'},
-      {href: 'data:text/html,<b>x</b>'},
-      {href: 'java\nscript:alert(1)'},
-      {href: '/safe', to: 'javascript:alert(1)'},
-      {href: 'vbscript:MsgBox(1)', to: '/safe'},
-      {href: {protocol: 'javascript:', pathname: 'alert(1)'}},
-      {href: '/safe', to: {protocol: 'data:', pathname: 'text/html,x'}},
-    ])(`${mode}: rejects %j without calling a router`, props => {
-      const router = vi.fn(() => {
-        throw new Error('rejected router invoked');
-      });
-      const onClick = vi.fn();
-      const linkRef = vi.fn();
-      render(
-        <LinkProvider component={mode === 'provider' ? router : 'a'}>
-          <Consumer
-            {...props}
-            as={mode === 'as' ? router : undefined}
-            onClick={onClick}
-            linkRef={linkRef}
-          />
-        </LinkProvider>,
-      );
-      const node = screen.getByTestId('destination');
-      expect(router).not.toHaveBeenCalled();
-      expect(node.tagName).toBe('A');
-      expect(node).not.toHaveAttribute('href');
-      expect(node).not.toHaveAttribute('to');
-      expect(node).not.toHaveAttribute('aria-disabled');
-      expect(node).toHaveAttribute('aria-label', 'Destination');
-      expect(node).toHaveClass('consumer-class');
-      expect(linkRef.mock.calls[0][0]).toBe(node);
-      node.click();
-      expect(onClick).toHaveBeenCalledOnce();
-    });
-  }
-
-  it.each([
-    '/relative',
-    '#fragment',
-    'https://example.com',
-    'http://example.com',
-    'mailto:a@example.com',
-    'tel:+1234567890',
-    'custom:document',
-    'data:text/plain,download',
-  ])('preserves native href and download affordances for %s', href => {
-    function Download() {
-      const Component = useLinkComponent();
-      return (
-        <Component
-          href={href}
-          download="note.txt"
-          target="_blank"
-          rel="noopener"
-          data-testid="download">
-          Download
-        </Component>
-      );
-    }
-    render(<Download />);
-    const node = screen.getByTestId('download');
-    expect(node).toHaveAttribute('href', href);
-    expect(node).toHaveAttribute('download', 'note.txt');
-    expect(node).toHaveAttribute('target', '_blank');
-    expect(node).toHaveAttribute('rel', 'noopener');
-  });
-
-  it('preserves both structured prop identities and safe explicit precedence', () => {
-    const router = vi.fn<(props: {href?: unknown; to?: unknown}) => null>(
-      () => null,
-    );
-    const href = {
-      protocol: 'https:',
-      hostname: 'example.com',
-      pathname: '/docs',
-    };
-    const to = {pathname: '/next', search: '?q=1', hash: '#top'};
-    render(<Consumer as={router} href={href} to={to} />);
-    expect(router.mock.calls[0][0].href).toBe(href);
-    expect(router.mock.calls[0][0].to).toBe(to);
-  });
-
-  it('keeps the native node stable while accepted props update', () => {
-    const {rerender} = render(<Consumer href="/first" />);
-    const node = screen.getByTestId('destination');
-    rerender(<Consumer href="/second" />);
-    expect(screen.getByTestId('destination')).toBe(node);
-    expect(node).toHaveAttribute('href', '/second');
-  });
-});
-
 // =============================================================================
 // LinkProvider
 // =============================================================================