Handle atom list sigils in select (#3804)

 Atom list sigils were being escaped as literals instead of going into select_expr.take.

Author: v0idpwn

lib/ecto/query/builder/select.ex

@@ -34,7 +34,7 @@ defmodule Ecto.Query.Builder.Select do
   def escape(other, vars, env) do
     cond do
       take?(other) ->
-        {{:{}, [], [:&, [], [0]]}, {[], %{0 => {:any, other}}}}
+        {{:{}, [], [:&, [], [0]]}, {[], %{0 => {:any, Macro.expand(other, env)}}}}
 
       maybe_take?(other) ->
         Builder.error! """
@@ -156,6 +156,11 @@ defmodule Ecto.Query.Builder.Select do
     end
   end
 
+  # atom list sigils
+  defp take?({name, _, [_, modifiers]}) when name in ~w(sigil_w sigil_W)a do
+    ?a in modifiers
+  end
+
   defp take?(fields) do
     is_list(fields) and Enum.all?(fields, fn
       {k, v} when is_atom(k) -> take?(List.wrap(v))

test/ecto/query/builder/select_test.exs

@@ -59,6 +59,7 @@ defmodule Ecto.Query.Builder.SelectTest do
       fields = ~w[field]a
       assert select("q", [q], map(q, ~w[field]a)).select.take == %{0 => {:map, fields}}
       assert select("q", [q], struct(q, @fields)).select.take == %{0 => {:struct, fields}}
+      assert select("q", [q], ~w[field]a).select.take == %{0 => {:any, fields}}
     end
 
     test "raises on single atom" do