@patternfly/react-core 4.235.4 → 4.235.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.235.4",
+  "version": "4.235.5",
   "description": "This library provides a set of common React components for use with the PatternFly reference implementation.",
   "main": "dist/js/index.js",
   "module": "dist/esm/index.js",
@@ -44,9 +44,9 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.86.4",
-    "@patternfly/react-styles": "^4.85.4",
-    "@patternfly/react-tokens": "^4.87.4",
+    "@patternfly/react-icons": "^4.86.5",
+    "@patternfly/react-styles": "^4.85.5",
+    "@patternfly/react-tokens": "^4.87.5",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
@@ -70,5 +70,5 @@
     "react": "^16.8.0 || ^17.0.0",
     "react-dom": "^16.8.0 || ^17.0.0"
   },
-  "gitHead": "35bd0846f74042fe9a5cb755ac8bf89ae4716bce"
+  "gitHead": "6ec6b7c439ec2475f03ddfdc9ad494396aecb6a6"
 }

package/src/components/NumberInput/NumberInput.tsx

@@ -10,7 +10,7 @@ import { TextInput } from '../TextInput';
 
 export interface NumberInputProps extends React.HTMLProps<HTMLDivElement> {
   /** Value of the number input */
-  value?: number;
+  value?: number | null;
   /** Additional classes added to the number input */
   className?: string;
   /** Sets the width of the number input to a number of characters */
@@ -89,6 +89,7 @@ export const NumberInput: React.FunctionComponent<NumberInputProps> = ({
   plusBtnProps,
   ...props
 }: NumberInputProps) => {
+  value = value || 0;
   const numberInputUnit = <div className={css(styles.numberInputUnit)}>{unit}</div>;
   const keyDownHandler =
     inputProps && inputProps.onKeyDown ? inputProps.onKeyDown : defaultKeyDownHandler({ inputName, onMinus, onPlus });

package/src/components/NumberInput/__tests__/NumberInput.test.tsx

@@ -169,4 +169,22 @@ describe('numberInput', () => {
   
     expect(input).toHaveDisplayValue('47.01');
   });
+
+  test('renders 0 if no value passed', () => {
+    render(<NumberInput />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
+
+  test('renders 0 if undefined value passed', () => {
+    render(<NumberInput value={undefined} />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
+
+  test('renders 0 if null value passed', () => {
+    render(<NumberInput value={null} />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
 });