Skip to content

Commit

Permalink
Query: Merge processing of eager loaded include with normal include
Browse files Browse the repository at this point in the history
Resolves #16619

This avoids issue of adding includes when includes are not part of projection.
Fixed in new nav rewrite
  • Loading branch information
smitpatel committed Aug 1, 2019
1 parent a631e1b commit 0d76ab2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -221,9 +221,6 @@ public override void No_ignored_include_warning_when_implicit_load()
private void AssertSql(params string[] expected)
=> Fixture.TestSqlLoggerFactory.AssertBaseline(expected);

private void AssertContainsSql(params string[] expected)
=> Fixture.TestSqlLoggerFactory.AssertBaseline(expected, assertOrder: false);

private void ClearLog()
=> Fixture.TestSqlLoggerFactory.Clear();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ public virtual void Query_for_base_type_loads_all_owned_navs()
}
}

[ConditionalFact(Skip = "Issue#16619")]
[ConditionalFact]
public virtual void No_ignored_include_warning_when_implicit_load()
{
using (var context = CreateContext())
Expand Down

0 comments on commit 0d76ab2

Please sign in to comment.