Comment out tests that no longer compile

Because of the new .NET Contains overload with an optional parameter,
which is incompatible with LINQ expression trees.

See #35547

Author: roji

test/EFCore.Cosmos.FunctionalTests/Query/NorthwindAggregateOperatorsQueryCosmosTest.cs

@@ -1442,29 +1442,32 @@ WHERE ARRAY_CONTAINS(@ids, c["EmployeeID"])
 """);
             });
 
-    public override Task Contains_with_local_nullable_uint_array_closure(bool async)
-        => Fixture.NoSyncTest(
-            async, async a =>
-            {
-                await base.Contains_with_local_nullable_uint_array_closure(a);
-
-                AssertSql(
-                    """
-@ids='[0,1]'
-
-SELECT VALUE c
-FROM root c
-WHERE ARRAY_CONTAINS(@ids, c["EmployeeID"])
-""",
-                    //
-                    """
-@ids='[0]'
-
-SELECT VALUE c
-FROM root c
-WHERE ARRAY_CONTAINS(@ids, c["EmployeeID"])
-""");
-            });
+// TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
+// optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
+//
+//     public override Task Contains_with_local_nullable_uint_array_closure(bool async)
+//         => Fixture.NoSyncTest(
+//             async, async a =>
+//             {
+//                 await base.Contains_with_local_nullable_uint_array_closure(a);
+//
+//                 AssertSql(
+//                     """
+// @ids='[0,1]'
+//
+// SELECT VALUE c
+// FROM root c
+// WHERE ARRAY_CONTAINS(@ids, c["EmployeeID"])
+// """,
+//                     //
+//                     """
+// @ids='[0]'
+//
+// SELECT VALUE c
+// FROM root c
+// WHERE ARRAY_CONTAINS(@ids, c["EmployeeID"])
+// """);
+//             });
 
     public override Task Contains_with_local_array_inline(bool async)
         => Fixture.NoSyncTest(
@@ -2004,21 +2007,25 @@ FROM root c
         }
     }
 
-    public override async Task Contains_with_local_tuple_array_closure(bool async)
-    {
-        // Contains over subquery. Issue #17246.
-        await AssertTranslationFailed(() => base.Contains_with_local_tuple_array_closure(async));
-
-        AssertSql();
-    }
-
-    public override async Task Contains_with_local_anonymous_type_array_closure(bool async)
-    {
-        // Contains over subquery. Issue #17246.
-        await AssertTranslationFailed(() => base.Contains_with_local_anonymous_type_array_closure(async));
-
-        AssertSql();
-    }
+    // TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
+    // optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
+    //
+    //
+    // public override async Task Contains_with_local_tuple_array_closure(bool async)
+    // {
+    //     // Contains over subquery. Issue #17246.
+    //     await AssertTranslationFailed(() => base.Contains_with_local_tuple_array_closure(async));
+    //
+    //     AssertSql();
+    // }
+    //
+    // public override async Task Contains_with_local_anonymous_type_array_closure(bool async)
+    // {
+    //     // Contains over subquery. Issue #17246.
+    //     await AssertTranslationFailed(() => base.Contains_with_local_anonymous_type_array_closure(async));
+    //
+    //     AssertSql();
+    // }
 
     public override async Task OfType_Select(bool async)
     {

test/EFCore.Cosmos.FunctionalTests/Query/NorthwindWhereQueryCosmosTest.cs

@@ -1627,13 +1627,16 @@ public override async Task Where_collection_navigation_ToArray_Count(bool async)
         AssertSql();
     }
 
-    public override async Task Where_collection_navigation_ToArray_Contains(bool async)
-    {
-        // Cosmos client evaluation. Issue #17246.
-        await AssertTranslationFailed(() => base.Where_collection_navigation_ToArray_Contains(async));
-
-        AssertSql();
-    }
+    // TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
+    // optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
+    //
+    // public override async Task Where_collection_navigation_ToArray_Contains(bool async)
+    // {
+    //     // Cosmos client evaluation. Issue #17246.
+    //     await AssertTranslationFailed(() => base.Where_collection_navigation_ToArray_Contains(async));
+    //
+    //     AssertSql();
+    // }
 
     public override async Task Where_collection_navigation_AsEnumerable_Count(bool async)
     {
@@ -1691,21 +1694,24 @@ FROM root c
 """);
             });
 
-    public override Task Where_array_of_object_contains_over_value_type(bool async)
-        => Fixture.NoSyncTest(
-            async, async a =>
-            {
-                await base.Where_array_of_object_contains_over_value_type(a);
-
-                AssertSql(
-                    """
-@orderIds='[10248,10249]'
-
-SELECT VALUE c
-FROM root c
-WHERE ((c["$type"] = "Order") AND ARRAY_CONTAINS(@orderIds, c["OrderID"]))
-""");
-            });
+// TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
+// optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
+//
+//     public override Task Where_array_of_object_contains_over_value_type(bool async)
+//         => Fixture.NoSyncTest(
+//             async, async a =>
+//             {
+//                 await base.Where_array_of_object_contains_over_value_type(a);
+//
+//                 AssertSql(
+//                     """
+// @orderIds='[10248,10249]'
+//
+// SELECT VALUE c
+// FROM root c
+// WHERE ((c["$type"] = "Order") AND ARRAY_CONTAINS(@orderIds, c["OrderID"]))
+// """);
+//             });
 
     public override Task Filter_with_EF_Property_using_closure_for_property_name(bool async)
         => Fixture.NoSyncTest(