Fix issues with the compiled model

Fixes #31727
Fixes #31720

Also fixed a formatting issue introduced in #31709

Added tests for all common types on SQLite and SQL Server.

Author: ajcvickers

src/EFCore.Design/Query/Internal/LinqToCSharpSyntaxTranslator.cs

@@ -1417,6 +1417,13 @@ protected override Expression VisitMethodCall(MethodCallExpression call)
                     methodIdentifier),
                 ArgumentList(SeparatedList(arguments[1..])));
         }
+        else if (call.Method is { Name: "op_Equality", IsHideBySig: true, IsSpecialName: true })
+        {
+            Result = BinaryExpression(
+                SyntaxKind.EqualsExpression,
+                Translate<ExpressionSyntax>(call.Arguments[0]),
+                Translate<ExpressionSyntax>(call.Arguments[1]));
+        }
         else
         {
             ExpressionSyntax expression;
@@ -1440,12 +1447,23 @@ ExpressionSyntax GetMemberAccessesForAllDeclaringTypes(Type type)
                 expression = Translate<ExpressionSyntax>(call.Object);
             }
 
-            Result = InvocationExpression(
-                MemberAccessExpression(
-                    SyntaxKind.SimpleMemberAccessExpression,
+            if (call.Method.Name.StartsWith("get_", StringComparison.Ordinal)
+                     && call.Method.GetParameters().Length == 1
+                     && call.Method is { IsHideBySig: true, IsSpecialName: true })
+            {
+                Result = ElementAccessExpression(
                     expression,
-                    methodIdentifier),
-                ArgumentList(SeparatedList(arguments)));
+                    BracketedArgumentList(SeparatedList(arguments)));
+            }
+            else
+            {
+                Result = InvocationExpression(
+                    MemberAccessExpression(
+                        SyntaxKind.SimpleMemberAccessExpression,
+                        expression,
+                        methodIdentifier),
+                    ArgumentList(SeparatedList(arguments)));
+            }
         }
 
         if (call.Method.DeclaringType.Namespace is { } ns)
@@ -1904,7 +1922,11 @@ protected override Expression VisitTypeBinary(TypeBinaryExpression node)
     /// <inheritdoc />
     protected override Expression VisitUnary(UnaryExpression unary)
     {
-        if (unary.Method is not null)
+        if (unary.Method is not null
+            && !unary.Method.IsHideBySig
+            && !unary.Method.IsSpecialName
+            && unary.Method.Name != "op_Implicit"
+            && unary.Method.Name != "op_Explicit")
         {
             throw new NotImplementedException("Unary node with non-null method");
         }

src/EFCore.Relational/Storage/ByteTypeMapping.cs

@@ -27,7 +27,7 @@ public class ByteTypeMapping : RelationalTypeMapping
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static ByteArrayTypeMapping Default { get; } = new("tinyint");
+    public static ByteTypeMapping Default { get; } = new("tinyint");
 
     /// <summary>
     ///     Initializes a new instance of the <see cref="ByteTypeMapping" /> class.

src/EFCore.Sqlite.Core/Storage/Internal/SqliteGuidTypeMapping.cs

@@ -20,7 +20,7 @@ public class SqliteGuidTypeMapping : GuidTypeMapping
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static new readonly SqliteGuidTypeMapping Default = new(SqliteTypeMappingSource.TextTypeName);
+    public static new SqliteGuidTypeMapping Default { get; } = new(SqliteTypeMappingSource.TextTypeName);
 
     /// <summary>
     ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to

src/EFCore.Sqlite.Core/Storage/Internal/SqliteJsonTypeMapping.cs

@@ -32,7 +32,7 @@ private static readonly ConstructorInfo MemoryStreamConstructor
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static readonly SqliteJsonTypeMapping Default = new(SqliteTypeMappingSource.TextTypeName);
+    public static SqliteJsonTypeMapping Default { get; } = new(SqliteTypeMappingSource.TextTypeName);
 
     /// <summary>
     ///     Initializes a new instance of the <see cref="SqliteJsonTypeMapping" /> class.

src/EFCore.Sqlite.Core/Storage/Internal/SqliteStringTypeMapping.cs

@@ -21,7 +21,7 @@ public class SqliteStringTypeMapping : StringTypeMapping
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static new readonly SqliteStringTypeMapping Default = new(SqliteTypeMappingSource.TextTypeName);
+    public static new SqliteStringTypeMapping Default { get; } = new(SqliteTypeMappingSource.TextTypeName);
 
     /// <summary>
     ///     Initializes a new instance of the <see cref="SqliteStringTypeMapping" /> class.

src/EFCore.Sqlite.Core/Storage/Internal/SqliteTimeOnlyTypeMapping.cs

@@ -20,7 +20,7 @@ public class SqliteTimeOnlyTypeMapping : TimeOnlyTypeMapping
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static new readonly SqliteTimeOnlyTypeMapping Default = new(SqliteTypeMappingSource.TextTypeName);
+    public static new SqliteTimeOnlyTypeMapping Default { get; } = new(SqliteTypeMappingSource.TextTypeName);
 
     /// <summary>
     ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to

src/EFCore.Sqlite.Core/Storage/Internal/SqliteULongTypeMapping.cs

@@ -19,7 +19,7 @@ public class SqliteULongTypeMapping : ULongTypeMapping
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static new readonly SqliteULongTypeMapping Default = new(SqliteTypeMappingSource.IntegerTypeName);
+    public static new SqliteULongTypeMapping Default { get; } = new(SqliteTypeMappingSource.IntegerTypeName);
 
     /// <summary>
     ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to

src/EFCore/Design/Internal/CSharpRuntimeAnnotationCodeGenerator.cs

@@ -392,8 +392,7 @@ public static void Create(
             CreateJsonValueReaderWriter((JsonValueReaderWriter)jsonReaderWriterProperty.GetValue(converter)!, parameters, codeHelper);
 
             mainBuilder
-                .Append(")")
-                .DecrementIndent();
+                .Append(")");
         }
     }
 
@@ -443,8 +442,7 @@ public static void Create(
             Create((ValueComparer)elementComparerProperty.GetValue(comparer)!, parameters, codeHelper);
 
             mainBuilder
-                .Append(")")
-                .DecrementIndent();
+                .Append(")");
         }
     }