Fix not truncating at minutes the first time in <TimePicker> (#24305)

WordPress · Jul 31, 2020 · ab5eedf · ab5eedf
1 parent 67dce6d
commit ab5eedf
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/packages/components/src/date-time/test/time.js b/packages/components/src/date-time/test/time.js
@@ -200,4 +200,23 @@ describe( 'TimePicker', () => {
 		pmButton.simulate( 'click' );
 		expect( onChangeSpy ).toHaveBeenCalledWith( '1986-10-18T11:00:00' );
 	} );
+
+	it( 'should truncate at the minutes on change', () => {
+		const onChangeSpy = jest.fn();
+
+		const wrapper = shallow(
+			<TimePicker
+				currentTime="1986-10-18T23:12:35"
+				onChange={ onChangeSpy }
+				is12Hour
+			/>
+		);
+
+		const minuteInput = wrapper.find( 'input[aria-label="Minutes"]' );
+
+		minuteInput.simulate( 'change', { target: { value: '22' } } );
+		minuteInput.simulate( 'blur' );
+
+		expect( onChangeSpy ).toHaveBeenCalledWith( '1986-10-18T23:22:00' );
+	} );
 } );
diff --git a/packages/components/src/date-time/time.js b/packages/components/src/date-time/time.js
@@ -73,7 +73,9 @@ class TimePicker extends Component {
 	changeDate( newDate ) {
 		const dateWithStartOfMinutes = newDate.clone().startOf( 'minute' );
 		this.setState( { date: dateWithStartOfMinutes } );
-		this.props.onChange( newDate.format( TIMEZONELESS_FORMAT ) );
+		this.props.onChange(
+			dateWithStartOfMinutes.format( TIMEZONELESS_FORMAT )
+		);
 	}
 
 	getMaxHours() {