Functional tests 2 Kirill: Next patch (#18)

KirillKurdyukov · KirillKurdyukov · commit c14b7e5db2c7 · 2025-05-19T16:32:40.000+03:00
* fixes

* next patch
diff --git a/src/EFCore.Ydb/src/Query/Internal/YdbMethodCallTranslatorProvider.cs b/src/EFCore.Ydb/src/Query/Internal/YdbMethodCallTranslatorProvider.cs
@@ -14,7 +14,6 @@ public YdbMethodCallTranslatorProvider(RelationalMethodCallTranslatorProviderDep
             [
                 new YdbDateTimeMethodTranslator(sqlExpressionFactory),
                 new YdbMathTranslator(sqlExpressionFactory),
-                new YdbMathTranslator(sqlExpressionFactory),
                 new YdbByteArrayMethodTranslator(sqlExpressionFactory)
             ]
         );
diff --git a/src/EFCore.Ydb/src/Query/Internal/YdbSqlAliasManager.cs b/src/EFCore.Ydb/src/Query/Internal/YdbSqlAliasManager.cs
@@ -76,7 +76,7 @@ private Expression VisitLeftJoin(LeftJoinExpression leftJoinExpression)
                 leftJoinExpression.JoinPredicate
             );
 
-        private Expression VisitTable(TableExpressionBase tableExpression) => tableExpression;
+        private static Expression VisitTable(TableExpressionBase tableExpression) => tableExpression;
 
         private IReadOnlyList<ProjectionExpression> AdjustAliases(
             IReadOnlyList<ProjectionExpression> projections,
diff --git a/src/EFCore.Ydb/src/Storage/Internal/Mapping/YdbDecimalTypeMapping.cs b/src/EFCore.Ydb/src/Storage/Internal/Mapping/YdbDecimalTypeMapping.cs
@@ -27,5 +27,5 @@ protected override RelationalTypeMapping Clone(RelationalTypeMappingParameters p
 
     protected override string ProcessStoreType(
         RelationalTypeMappingParameters parameters, string storeType, string storeTypeNameBase
-    ) => $"Decimal({parameters.Precision ?? DefaultPrecision}, {parameters.Scale ?? DefaultScale})";
+    ) => $"{storeType}({parameters.Precision ?? DefaultPrecision}, {parameters.Scale ?? DefaultScale})";
 }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ public YdbMethodCallTranslatorProvider(RelationalMethodCallTranslatorProviderDep`
`14`	`14`	`[`
`15`	`15`	`new YdbDateTimeMethodTranslator(sqlExpressionFactory),`
`16`	`16`	`new YdbMathTranslator(sqlExpressionFactory),`
`17`		`- new YdbMathTranslator(sqlExpressionFactory),`
`18`	`17`	`new YdbByteArrayMethodTranslator(sqlExpressionFactory)`
`19`	`18`	`]`
`20`	`19`	`);`
Original file line number	Diff line number	Diff line change
`@@ -27,5 +27,5 @@ protected override RelationalTypeMapping Clone(RelationalTypeMappingParameters p`
`27`	`27`
`28`	`28`	`protected override string ProcessStoreType(`
`29`	`29`	`RelationalTypeMappingParameters parameters, string storeType, string storeTypeNameBase`
`30`		`- ) => $"Decimal({parameters.Precision ?? DefaultPrecision}, {parameters.Scale ?? DefaultScale})";`
	`30`	`+ ) => $"{storeType}({parameters.Precision ?? DefaultPrecision}, {parameters.Scale ?? DefaultScale})";`
`31`	`31`	`}`