Add test for ExecuteUpdate with two inner joins

And fixup for #29349
dotnet · Nov 9, 2022 · b93b368 · b93b368
1 parent d4aa1ea
commit b93b368
Show file tree

Hide file tree

Showing 10 changed files with 71 additions and 12 deletions.
diff --git a/src/EFCore.SqlServer/Query/Internal/SqlServerObjectToStringTranslator.cs b/src/EFCore.SqlServer/Query/Internal/SqlServerObjectToStringTranslator.cs
@@ -67,7 +67,7 @@ public SqlServerObjectToStringTranslator(ISqlExpressionFactory sqlExpressionFact
             return null;
         }
 
-        if(instance.TypeMapping is not null && instance.TypeMapping.ClrType == typeof(string))
+        if (instance.TypeMapping?.ClrType == typeof(string))
         {
             return instance;
         }

diff --git a/src/EFCore.Sqlite.Core/Query/Internal/SqliteObjectToStringTranslator.cs b/src/EFCore.Sqlite.Core/Query/Internal/SqliteObjectToStringTranslator.cs
@@ -66,7 +66,7 @@ public SqliteObjectToStringTranslator(ISqlExpressionFactory sqlExpressionFactory
             return null;
         }
 
-        if (instance.TypeMapping is not null && instance.TypeMapping.ClrType == typeof(string))
+        if (instance.TypeMapping?.ClrType == typeof(string))
         {
             return instance;
         }

diff --git a/test/EFCore.Relational.Specification.Tests/BulkUpdates/NorthwindBulkUpdatesTestBase.cs b/test/EFCore.Relational.Specification.Tests/BulkUpdates/NorthwindBulkUpdatesTestBase.cs
@@ -1075,6 +1075,19 @@ public virtual Task Update_Where_Join_set_property_from_joined_single_result_sca
                     }
                 }));
 
+    [ConditionalTheory]
+    [MemberData(nameof(IsAsyncData))]
+    public virtual Task Update_with_two_inner_joins(bool async)
+        => AssertUpdate(
+            async,
+            ss => ss
+                .Set<OrderDetail>()
+                .Where(od => od.Product.Discontinued && od.Order.OrderDate > new DateTime(1990, 1, 1)),
+            e => e,
+            s => s.SetProperty(od => od.Quantity, 1),
+            rowsAffectedCount: 228,
+            (b, a) => Assert.All(a, od => Assert.Equal(1, od.Quantity)));
+
     protected string NormalizeDelimitersInRawString(string sql)
         => Fixture.TestStore.NormalizeDelimitersInRawString(sql);
 

diff --git a/test/EFCore.SqlServer.FunctionalTests/BulkUpdates/NorthwindBulkUpdatesSqlServerTest.cs b/test/EFCore.SqlServer.FunctionalTests/BulkUpdates/NorthwindBulkUpdatesSqlServerTest.cs
@@ -1,6 +1,8 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+using Microsoft.EntityFrameworkCore.TestModels.Northwind;
+
 namespace Microsoft.EntityFrameworkCore.BulkUpdates;
 
 public class NorthwindBulkUpdatesSqlServerTest : NorthwindBulkUpdatesTestBase<NorthwindBulkUpdatesSqlServerFixture<NoopModelCustomizer>>
@@ -1458,6 +1460,23 @@ WHERE [c].[CustomerID] LIKE N'F%'
 """);
     }
 
+    [ConditionalTheory]
+    [MemberData(nameof(IsAsyncData))]
+    public override async Task Update_with_two_inner_joins(bool async)
+    {
+        await base.Update_with_two_inner_joins(async);
+
+        AssertExecuteUpdateSql(
+"""
+UPDATE [o]
+SET [o].[Quantity] = CAST(1 AS smallint)
+FROM [Order Details] AS [o]
+INNER JOIN [Products] AS [p] ON [o].[ProductID] = [p].[ProductID]
+INNER JOIN [Orders] AS [o0] ON [o].[OrderID] = [o0].[OrderID]
+WHERE [p].[Discontinued] = CAST(1 AS bit) AND [o0].[OrderDate] > '1990-01-01T00:00:00.000'
+""");
+    }
+
     private void AssertSql(params string[] expected)
         => Fixture.TestSqlLoggerFactory.AssertBaseline(expected);
 

diff --git a/test/EFCore.SqlServer.FunctionalTests/Query/GearsOfWarQuerySqlServerTest.cs b/test/EFCore.SqlServer.FunctionalTests/Query/GearsOfWarQuerySqlServerTest.cs
@@ -4099,8 +4099,11 @@ public override async Task ToString_string_property_projection(bool async)
     {
         await base.ToString_string_property_projection(async);
 
-        AssertSql(@"SELECT [w].[Name]
-FROM [Weapons] AS [w]");
+        AssertSql(
+"""
+SELECT [w].[Name]
+FROM [Weapons] AS [w]
+""");
     }
 
 

diff --git a/test/EFCore.SqlServer.FunctionalTests/Query/TPCGearsOfWarQuerySqlServerTest.cs b/test/EFCore.SqlServer.FunctionalTests/Query/TPCGearsOfWarQuerySqlServerTest.cs
@@ -12499,8 +12499,11 @@ public override async Task ToString_string_property_projection(bool async)
     {
         await base.ToString_string_property_projection(async);
 
-        AssertSql(@"SELECT [w].[Name]
-FROM [Weapons] AS [w]");
+        AssertSql(
+"""
+SELECT [w].[Name]
+FROM [Weapons] AS [w]
+""");
     }
 
     public override async Task ToString_boolean_property_non_nullable(bool async)

diff --git a/test/EFCore.SqlServer.FunctionalTests/Query/TPTGearsOfWarQuerySqlServerTest.cs b/test/EFCore.SqlServer.FunctionalTests/Query/TPTGearsOfWarQuerySqlServerTest.cs
@@ -10738,8 +10738,11 @@ public override async Task ToString_string_property_projection(bool async)
     {
         await base.ToString_string_property_projection(async);
 
-        AssertSql(@"SELECT [w].[Name]
-FROM [Weapons] AS [w]");
+        AssertSql(
+"""
+SELECT [w].[Name]
+FROM [Weapons] AS [w]
+""");
     }
 
     public override async Task ToString_boolean_property_non_nullable(bool async)

diff --git a/test/EFCore.SqlServer.FunctionalTests/Query/TemporalGearsOfWarQuerySqlServerTest.cs b/test/EFCore.SqlServer.FunctionalTests/Query/TemporalGearsOfWarQuerySqlServerTest.cs
@@ -9008,8 +9008,11 @@ public override async Task ToString_string_property_projection(bool async)
     {
         await base.ToString_string_property_projection(async);
 
-        AssertSql(@"SELECT [w].[Name]
-FROM [Weapons] FOR SYSTEM_TIME AS OF '2010-01-01T00:00:00.0000000' AS [w]");
+        AssertSql(
+"""
+SELECT [w].[Name]
+FROM [Weapons] FOR SYSTEM_TIME AS OF '2010-01-01T00:00:00.0000000' AS [w]
+""");
     }
 
     public override async Task ToString_boolean_property_non_nullable(bool async)

diff --git a/test/EFCore.Sqlite.FunctionalTests/BulkUpdates/NorthwindBulkUpdatesSqliteTest.cs b/test/EFCore.Sqlite.FunctionalTests/BulkUpdates/NorthwindBulkUpdatesSqliteTest.cs
@@ -2,6 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using Microsoft.EntityFrameworkCore.Sqlite.Internal;
+using Microsoft.EntityFrameworkCore.TestModels.Northwind;
 
 namespace Microsoft.EntityFrameworkCore.BulkUpdates;
 
@@ -1365,6 +1366,17 @@ public override async Task Update_Where_Join_set_property_from_joined_single_res
     ORDER BY "o"."OrderDate" DESC
     LIMIT 1)) AS INTEGER) AS TEXT)
 WHERE "c"."CustomerID" LIKE 'F%'
+""");
+    }
+
+    [ConditionalTheory(Skip = "Issue#28886")]
+    public override async Task Update_with_two_inner_joins(bool async)
+    {
+        await base.Update_with_two_inner_joins(async);
+
+        AssertExecuteUpdateSql(
+"""
+
 """);
     }
 

diff --git a/test/EFCore.Sqlite.FunctionalTests/Query/GearsOfWarQuerySqliteTest.cs b/test/EFCore.Sqlite.FunctionalTests/Query/GearsOfWarQuerySqliteTest.cs
@@ -3095,8 +3095,11 @@ public override async Task ToString_string_property_projection(bool async)
     {
         await base.ToString_string_property_projection(async);
 
-        AssertSql(@"SELECT ""w"".""Name""
-FROM ""Weapons"" AS ""w""");
+        AssertSql(
+"""
+SELECT "w"."Name"
+FROM "Weapons" AS "w"
+""");
     }
 
     public override async Task ToString_boolean_property_non_nullable(bool async)