Skip to content

Fix DateTimeWithRange validation in test#64940

Merged
BrennanConroy merged 1 commit intomainfrom
BrennanConroy-patch-4
Jan 5, 2026
Merged

Fix DateTimeWithRange validation in test#64940
BrennanConroy merged 1 commit intomainfrom
BrennanConroy-patch-4

Conversation

@BrennanConroy
Copy link
Member

No description provided.

@github-actions github-actions bot added the needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically label Jan 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds missing test coverage for the DateTimeWithRange property validation in the parsable properties test. The property was defined in the test class with a range validation attribute but was not being tested.

Key Changes

  • Added test input data for DateTimeWithRange with an out-of-range value
  • Added corresponding assertion to verify the validation error message

@BrennanConroy BrennanConroy enabled auto-merge (squash) January 5, 2026 22:14
@BrennanConroy BrennanConroy merged commit 1998be8 into main Jan 5, 2026
31 of 32 checks passed
@BrennanConroy BrennanConroy deleted the BrennanConroy-patch-4 branch January 5, 2026 22:22
@dotnet-policy-service dotnet-policy-service bot added this to the 11.0-preview1 milestone Jan 5, 2026
@wtgodbe
Copy link
Member

wtgodbe commented Jan 6, 2026

/backport to release/10.0

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Started backporting to release/10.0 (link to workflow run)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants