Skip to content

[release/9.0] Backport FormsInputDateTest fixes - #68126

Merged
wtgodbe merged 1 commit into
dotnet:release/9.0from
ilonatommy:backport/net9-inputdate-flaky
Aug 5, 2026
Merged

[release/9.0] Backport FormsInputDateTest fixes#68126
wtgodbe merged 1 commit into
dotnet:release/9.0from
ilonatommy:backport/net9-inputdate-flaky

Conversation

@ilonatommy

@ilonatommy ilonatommy commented Jul 31, 2026

Copy link
Copy Markdown
Member

Backport of #68107 and #68115 to release/9.0

Fixes #31734, Fixes #67243, fixes #35018.

Description

The caret has to be located over the left-most segment of the 3-segment date input, otherwise SendKeys will write to wrong segment, producing non-desired date values.

The test fails deterministically before fix and passes deterministically after the fix.

Customer Impact

None, it's just a test.

Regression?

  • Yes
  • No

Risk

  • High
  • Medium
  • Low

Test fix.

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

@ilonatommy ilonatommy self-assigned this Jul 31, 2026
Copilot AI review requested due to automatic review settings July 31, 2026 09:01
@ilonatommy
ilonatommy requested a review from a team as a code owner July 31, 2026 09:01
@ilonatommy ilonatommy added the area-blazor Includes: Blazor, Razor Components label Jul 31, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Backports fixes to FormsInputDateTest on release/9.0 to reduce flakiness in native date/time <input> Selenium interactions by ensuring keystrokes land in the intended segment (caret positioning), and un-quarantines the affected tests.

Changes:

  • Replaces direct SendKeys sequences with helper methods that click and move the caret to the left-most segment before typing.
  • Adds ClearDate helper to reliably clear native date inputs and commit the change via Tab.
  • Removes [QuarantinedTest] attributes for the previously flaky tests and fixes a minor comment typo (“Stricly” → “Strictly”).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/Components/test/E2ETest/Tests/FormsInputDateTest.cs
@ilonatommy ilonatommy added this to the 9.0.x milestone Jul 31, 2026
@ilonatommy
ilonatommy requested review from rokonec and wtgodbe July 31, 2026 10:28
@ilonatommy ilonatommy added Servicing-approved Shiproom has approved the issue tell-mode Indicates a PR which is being merged during tell-mode labels Aug 3, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hi @ilonatommy. This PR was just approved to be included in the upcoming servicing release. Somebody from the @dotnet/aspnet-build team will get it merged when the branches are open. Until then, please make sure all the CI checks pass and the PR is reviewed.

This was referenced Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components Servicing-approved Shiproom has approved the issue tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants