Address review comments

Author: roji

src/EFCore.SqlServer/Query/Internal/SqlServerSqlTranslatingExpressionVisitor.cs

@@ -271,9 +271,9 @@ protected override Expression VisitMethodCall(MethodCallExpression methodCallExp
                             "JSON_PATH_EXISTS",
                             [translatedJson, translatedPath],
                             nullable: true,
-                            // Note that json_type() does propagate nullability; however, our query pipeline assumes that if arguments
-                            // propagate nullability, that's the *only* reason for the function to return null; this means that if the
-                            // arguments are non-nullable, the IS NOT NULL wrapping check can be optimized away.
+                            // Note that JSON_PATH_EXISTS() does propagate nullability; however, our query pipeline assumes that if
+                            // arguments propagate nullability, that's the *only* reason for the function to return null; this means that
+                            // if the arguments are non-nullable, the IS NOT NULL wrapping check can be optimized away.
                             argumentsPropagateNullability: [false, false],
                             typeof(int)),
                         _sqlExpressionFactory.Constant(1));

test/EFCore.Relational.Specification.Tests/Query/Translations/JsonTranslationsRelationalTestBase.cs

@@ -8,8 +8,8 @@ namespace Microsoft.EntityFrameworkCore.Query.Translations;
 
 // This test suite covers translations of JSON functions on EF.Functions (e.g. EF.Functions.JsonExists).
 // It does not cover general, built-in JSON support via complex type mapping, etc.
-public abstract class JsonTranslationsRelatinalTestBase<TFixture>(TFixture fixture) : QueryTestBase<TFixture>(fixture)
-    where TFixture : JsonTranslationsRelatinalTestBase<TFixture>.JsonTranslationsQueryFixtureBase, new()
+public abstract class JsonTranslationsRelationalTestBase<TFixture>(TFixture fixture) : QueryTestBase<TFixture>(fixture)
+    where TFixture : JsonTranslationsRelationalTestBase<TFixture>.JsonTranslationsQueryFixtureBase, new()
 {
     [ConditionalFact]
     public virtual Task JsonExists_on_scalar_string_column()

test/EFCore.SqlServer.FunctionalTests/Query/Translations/JsonTranslationsSqlServerTest.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.EntityFrameworkCore.Query.Translations;
 
-public class JsonTranslationsSqlServerTest : JsonTranslationsRelatinalTestBase<JsonTranslationsSqlServerTest.JsonTranslationsQuerySqlServerFixture>
+public class JsonTranslationsSqlServerTest : JsonTranslationsRelationalTestBase<JsonTranslationsSqlServerTest.JsonTranslationsQuerySqlServerFixture>
 {
     public JsonTranslationsSqlServerTest(JsonTranslationsQuerySqlServerFixture fixture, ITestOutputHelper testOutputHelper)
         : base(fixture)

test/EFCore.Sqlite.FunctionalTests/Query/Translations/JsonTranslationsSqliteTest.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.EntityFrameworkCore.Query.Translations;
 
-public class JsonTranslationsSqliteTest : JsonTranslationsRelatinalTestBase<JsonTranslationsSqliteTest.JsonTranslationsQuerySqliteFixture>
+public class JsonTranslationsSqliteTest : JsonTranslationsRelationalTestBase<JsonTranslationsSqliteTest.JsonTranslationsQuerySqliteFixture>
 {
     public JsonTranslationsSqliteTest(JsonTranslationsQuerySqliteFixture fixture, ITestOutputHelper testOutputHelper)
         : base(fixture)