-
Notifications
You must be signed in to change notification settings - Fork 257
Sync to EF 11.0.0-preview.4.26215.121 #3814
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -155,14 +155,14 @@ public override Task Can_read_write_nullable_TimeSpan_JSON_values(string? value, | |
| // Cannot override since the base test contains [InlineData] attributes which still apply, and which contain data we need | ||
| // to override. See Can_read_write_nullable_TimeSpan_JSON_values_npgsql instead. | ||
| // TODO: Implement Can_read_write_collection_of_TimeSpan_JSON_values_npgsql | ||
| public override Task Can_read_write_collection_of_TimeSpan_JSON_values() | ||
| public override Task Can_read_write_collection_of_TimeSpan_JSON_values(string expected) | ||
| => Task.CompletedTask; | ||
|
|
||
| // Cannot override since the base test contains [InlineData] attributes which still apply, and which contain data we need | ||
| // to override. See Can_read_write_nullable_TimeSpan_JSON_values_npgsql instead. | ||
| // TODO: Implement Can_read_write_collection_of_nullable_TimeSpan_JSON_values_npgsql | ||
| public override Task Can_read_write_collection_of_nullable_TimeSpan_JSON_values() | ||
| => Task.CompletedTask; | ||
| public override Task Can_read_write_collection_of_nullable_TimeSpan_JSON_values(string expected) | ||
| => Task.CompletedTask; | ||
|
Comment on lines
161
to
+165
|
||
|
|
||
| #endregion TimeSpan | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment says to see
Can_read_write_nullable_TimeSpan_JSON_values_npgsqlinstead, but no such method exists in this class (and the TODO indicates it’s not implemented yet). This cross-reference is misleading—either implement the referenced*_npgsqltest method or update the comment to point at the correct helper/placeholder for the TimeSpan collection case.