fix(material/timepicker): text field in parse error not up to date

Fixes that the `text` field in the `matTimepickerParse` error wasn't up-to-date with the element's value.

Author: crisbeto

src/material/timepicker/timepicker-input.ts

@@ -251,8 +251,14 @@ export class MatTimepickerInput<D> implements ControlValueAccessor, Validator, O
     const date = this._dateAdapter.parseTime(value, this._dateFormats.parse.timeInput);
     const hasChanged = !this._dateAdapter.sameTime(date, currentValue);
 
-    // We need to fire the CVA change event for all nulls, otherwise the validators won't run.
-    this._assignUserSelection(date, !date || hasChanged || !!(value && !currentValue));
+    if (!date || hasChanged || !!(value && !currentValue)) {
+      // We need to fire the CVA change event for all nulls, otherwise the validators won't run.
+      this._assignUserSelection(date, true);
+    } else {
+      // Call the validator even if the value hasn't changed since
+      // some fields change depending on what the user has entered.
+      this._validatorOnChange?.();
+    }
   }
 
   /** Handles the `blur` event. */

src/material/timepicker/timepicker.spec.ts

@@ -1007,7 +1007,22 @@ describe('MatTimepicker', () => {
       clearElement(input);
       typeInElement(input, 'not a valid date');
       fixture.detectChanges();
-      expect(control.errors?.['matTimepickerParse']).toBeTruthy();
+      expect(control.errors?.['matTimepickerParse']).toEqual(
+        jasmine.objectContaining({
+          text: 'not a valid date',
+        }),
+      );
+      expect(control.value).toBeTruthy();
+      expect(adapter.isValid(control.value!)).toBe(false);
+
+      // Change from one invalid value to the other to make sure that the object stays in sync.
+      typeInElement(input, ' (changed)');
+      fixture.detectChanges();
+      expect(control.errors?.['matTimepickerParse']).toEqual(
+        jasmine.objectContaining({
+          text: 'not a valid date (changed)',
+        }),
+      );
       expect(control.value).toBeTruthy();
       expect(adapter.isValid(control.value!)).toBe(false);