-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Create GraphUpdatesTests for owned entities #23212
Conversation
test/EFCore.SqlServer.FunctionalTests/GraphUpdates/GraphUpdatesSqlServerTest.cs
Outdated
Show resolved
Hide resolved
You just added 1622 new tests 😮 |
Can I piggy-back on this PR? 🐖 (Can also submit the fix to main directly)
|
Feel free to do either one |
@AndriySvyryd So happy to see this testing! It's been on my TODO list forever. |
86b290d
to
0ba7d14
Compare
I disabled special-casing for owned entities that allowed reparenting of non-added entities with identifying FKs. #19135 would be the proper way of doing that, but if we don't revert the behavior I also filed dotnet/EntityFramework.Docs#2859 |
0ba7d14
to
af53e2e
Compare
e472efe
to
13f8955
Compare
Disable owned entity reparenting if it leads to key modification Fix default SQL Server value generation strategy for shared columns Fixes #16454
13f8955
to
f481b97
Compare
Disable owned entity reparenting if it leads to key modification
Fix default SQL Server value generation strategy for shared columns
Query: Detect parent identifier correctly when parent has owned navs
Fixes #16454