maybe_paren_for_expr_inside_fragment

Author: dkuku

test/ecto/adapters/myxql_test.exs

@@ -681,6 +681,12 @@ defmodule Ecto.Adapters.MyXQLTest do
     assert_raise Ecto.QueryError, fn ->
       all(query)
     end
+
+    query = Schema |> select([r], fragment("CAST(? AS INT)", r.x and r.y)) |> plan()
+    assert all(query) == ~s{SELECT CAST((s0.`x` AND s0.`y`) AS INT) FROM `schema` AS s0}
+
+    query = Schema |> select([r], fragment("CAST(? AS INT)", r.x or r.y)) |> plan()
+    assert all(query) == ~s{SELECT CAST((s0.`x` OR s0.`y`) AS INT) FROM `schema` AS s0}
   end
 
   test "literals" do

test/ecto/adapters/postgres_test.exs

@@ -861,6 +861,14 @@ defmodule Ecto.Adapters.PostgresTest do
     assert_raise Ecto.QueryError, fn ->
       all(query)
     end
+
+    query = Schema |> select([r], fragment("?::integer", r.x and r.y)) |> plan()
+    # Boolean operations inside fragments should be wrapped in parentheses
+    # to ensure correct precedence with surrounding SQL
+    assert all(query) == ~s{SELECT (s0."x" AND s0."y")::integer FROM "schema" AS s0}
+
+    query = Schema |> select([r], fragment("?::integer", r.x or r.y)) |> plan()
+    assert all(query) == ~s{SELECT (s0."x" OR s0."y")::integer FROM "schema" AS s0}
   end
 
   test "literals" do

test/ecto/adapters/tds_test.exs

@@ -719,6 +719,12 @@ defmodule Ecto.Adapters.TdsTest do
                  fn ->
                    all(query)
                  end
+
+    query = Schema |> select([r], fragment("CAST(? AS INT)", r.x and r.y)) |> plan()
+    assert all(query) == ~s{SELECT CAST((s0.[x] AND s0.[y]) AS INT) FROM [schema] AS s0}
+
+    query = Schema |> select([r], fragment("CAST(? AS INT)", r.x or r.y)) |> plan()
+    assert all(query) == ~s{SELECT CAST((s0.[x] OR s0.[y]) AS INT) FROM [schema] AS s0}
   end
 
   test "literals" do