@patternfly/react-core 4.258.6 → 4.258.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.258.6",
+  "version": "4.258.7",
   "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",
@@ -70,5 +70,5 @@
     "react": "^16.8 || ^17 || ^18",
     "react-dom": "^16.8 || ^17 || ^18"
   },
-  "gitHead": "418db80c9d90e4a39bac9b24c7869a9ab7d985f5"
+  "gitHead": "69c78cfc668577eb1b005710b0c77522cbabe9c6"
 }

package/src/components/TimePicker/TimePicker.tsx

@@ -143,6 +143,8 @@ export class TimePicker extends React.Component<TimePickerProps, TimePickerState
     document.addEventListener('mousedown', this.onDocClick);
     document.addEventListener('touchstart', this.onDocClick);
     document.addEventListener('keydown', this.handleGlobalKeys);
+
+    this.setState({ isInvalid: !this.isValid(this.state.timeState) });
   }
 
   componentWillUnmount() {
@@ -198,7 +200,7 @@ export class TimePicker extends React.Component<TimePickerProps, TimePickerState
 
   componentDidUpdate(prevProps: TimePickerProps, prevState: TimePickerState) {
     const { timeState, isTimeOptionsOpen, isInvalid, timeRegex } = this.state;
-    const { time, is24Hour, delimiter, includeSeconds, isOpen } = this.props;
+    const { time, is24Hour, delimiter, includeSeconds, isOpen, minTime, maxTime } = this.props;
     if (prevProps.isOpen !== isOpen) {
       this.onToggle(isOpen);
     }
@@ -212,8 +214,22 @@ export class TimePicker extends React.Component<TimePickerProps, TimePickerState
       });
     }
     if (time !== '' && time !== prevProps.time) {
+      const parsedTime = parseTime(time, timeRegex, delimiter, !is24Hour, includeSeconds);
+
+      this.setState({
+        timeState: parsedTime,
+        isInvalid: !this.isValid(parsedTime)
+      });
+    }
+    if (minTime !== '' && minTime !== prevProps.minTime) {
+      this.setState({
+        minTimeState: parseTime(minTime, timeRegex, delimiter, !is24Hour, includeSeconds)
+      });
+    }
+
+    if (maxTime !== '' && maxTime !== prevProps.maxTime) {
       this.setState({
-        timeState: parseTime(time, timeRegex, delimiter, !is24Hour, includeSeconds)
+        maxTimeState: parseTime(maxTime, timeRegex, delimiter, !is24Hour, includeSeconds)
       });
     }
   }

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

@@ -107,24 +107,27 @@ describe('TimePicker', () => {
     });
 
     test('should be invalid after onBlur', async () => {
-      const validateTime = (_time: string) => {
-        return false;
-      };
       const user = userEvent.setup();
 
       render(
         <>
           <div>Other element</div>
-          <TimePicker value={'00:00'} validateTime={validateTime} aria-label="time picker" />
+          <TimePicker value={'00:00'} aria-label="time picker" />
         </>
       );
 
-      await user.type(screen.getByLabelText('time picker'), '01:00');
+      await user.type(screen.getByLabelText('time picker'), '14:00');
       expect(screen.queryByText('Invalid time format')).toBeNull();
 
       await user.click(screen.getByText('Other element'));
       expect(screen.getByText('Invalid time format')).toBeInTheDocument();
     });
+
+    test('should be invalid when invalid time prop is passed', () => {
+      render(<TimePicker time="00:00" aria-label="time picker" />);
+
+      expect(screen.getByText('Invalid time format')).toBeInTheDocument();
+    });
   });
 
   describe('test includeSeconds', () => {