updates to address coderabbit comments

Author: tlabaj

packages/react-icons/src/__tests__/createIcon.test.tsx

@@ -1,5 +1,3 @@
-// eslint-disable-next-line no-restricted-imports -- test file excluded from package tsconfig; default import satisfies TS/JSX
-import React from 'react';
 import { render, screen } from '@testing-library/react';
 import { IconDefinition, CreateIconProps, createIcon, LegacyFlatIconDefinition, SVGPathObject } from '../createIcon';
 
@@ -197,21 +195,23 @@ describe('rh-ui mapping: nested SVGs, set prop, and warnings', () => {
 
   test('set="rh-ui" with no rhUiIcon mapping falls back to default and warns', () => {
     const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
-    const IconNoRhMapping = createIcon({
-      name: 'NoRhMappingIcon',
-      icon: defaultIconDef,
-      rhUiIcon: null
-    });
-
-    render(<IconNoRhMapping set="rh-ui" />);
-
-    expect(warnSpy).toHaveBeenCalledWith(
-      'Set "rh-ui" was provided for NoRhMappingIcon, but no rh-ui icon data exists for this icon. The default icon will be rendered.'
-    );
-    const root = screen.getByRole('img', { hidden: true });
-    expect(root.querySelector('path')).toHaveAttribute('d', defaultPath);
-    expect(root.querySelectorAll('svg')).toHaveLength(0);
-
-    warnSpy.mockRestore();
+    try {
+      const IconNoRhMapping = createIcon({
+        name: 'NoRhMappingIcon',
+        icon: defaultIconDef,
+        rhUiIcon: null
+      });
+
+      render(<IconNoRhMapping set="rh-ui" />);
+
+      expect(warnSpy).toHaveBeenCalledWith(
+        'Set "rh-ui" was provided for NoRhMappingIcon, but no rh-ui icon data exists for this icon. The default icon will be rendered.'
+      );
+      const root = screen.getByRole('img', { hidden: true });
+      expect(root.querySelector('path')).toHaveAttribute('d', defaultPath);
+      expect(root.querySelectorAll('svg')).toHaveLength(0);
+    } finally {
+      warnSpy.mockRestore();
+    }
   });
 });

packages/react-icons/src/createIcon.tsx

@@ -29,6 +29,7 @@ export interface IconDefinitionWithSvgPath extends IconDefinitionBase {
 /** Describes SVG path content for one icon variant (default or rh-ui). */
 export type IconDefinition = IconDefinitionWithSvgPathData | IconDefinitionWithSvgPath;
 
+/** When passing `icon` or `rhUiIcon` keys (nested form), `icon` is required at runtime. */
 export interface CreateIconProps {
   name?: string;
   icon?: IconDefinition;