Update explicit throw for navigation ignored warning (#24486)

Cleanup from earlier PR
dotnet · Mar 24, 2021 · 12d4057 · 12d4057
1 parent 4722fe3
commit 12d4057
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/EFCore/Infrastructure/CoreOptionsExtension.cs b/src/EFCore/Infrastructure/CoreOptionsExtension.cs
@@ -48,7 +48,8 @@ private WarningsConfiguration _warningsConfiguration
                 .TryWithExplicit(CoreEventId.ManyServiceProvidersCreatedWarning, WarningBehavior.Throw)
                 .TryWithExplicit(CoreEventId.LazyLoadOnDisposedContextWarning, WarningBehavior.Throw)
                 .TryWithExplicit(CoreEventId.DetachedLazyLoadingWarning, WarningBehavior.Throw)
-                .TryWithExplicit(CoreEventId.InvalidIncludePathError, WarningBehavior.Throw);
+                .TryWithExplicit(CoreEventId.InvalidIncludePathError, WarningBehavior.Throw)
+                .TryWithExplicit(CoreEventId.NavigationBaseIncludeIgnored, WarningBehavior.Throw);
 
         /// <summary>
         ///     Creates a new set of options with everything set to default values.

diff --git a/test/EFCore.SqlServer.FunctionalTests/Query/QueryBugsTest.cs b/test/EFCore.SqlServer.FunctionalTests/Query/QueryBugsTest.cs
@@ -9422,7 +9422,6 @@ protected override DbContextOptionsBuilder AddOptions(DbContextOptionsBuilder bu
                 {
                     w.Log(SqlServerEventId.ByteIdentityColumnWarning);
                     w.Log(SqlServerEventId.DecimalTypeKeyWarning);
-                    w.Throw(CoreEventId.NavigationBaseIncludeIgnored);
                 });
 
         protected override TestStore CreateTestStore()