fix validation entry for unsafe_validate_unique/4 (fixes #3752) (#3753)

Author: mathieuprog

lib/ecto/changeset.ex

@@ -1868,7 +1868,7 @@ defmodule Ecto.Changeset do
         %Ecto.Changeset{} ->
           raise ArgumentError, "unsafe_validate_unique/4 does not work with schemaless changesets"
       end
-    changeset = %{changeset | validations: [{:unsafe_unique, fields} | validations]}
+    changeset = %{changeset | validations: [{hd(fields), {:unsafe_unique, fields: fields}} | validations]}
 
     where_clause = for field <- fields do
       {field, get_field(changeset, field)}
@@ -3073,7 +3073,7 @@ defimpl Inspect, for: Ecto.Changeset do
     end
 
     redacted_fields = case data do
-      %type{} -> 
+      %type{} ->
         if function_exported?(type, :__schema__, 1) do
           type.__schema__(:redact_fields)
         else

test/ecto/changeset_test.exs

@@ -1349,8 +1349,8 @@ defmodule Ecto.ChangesetTest do
     end
   end
 
-  test "validate_number/3 with bad value" do 
-    assert_raise ArgumentError, "expected value to be of type Decimal, Integer or Float, got: \"Oops\"", fn -> 
+  test "validate_number/3 with bad value" do
+    assert_raise ArgumentError, "expected value to be of type Decimal, Integer or Float, got: \"Oops\"", fn ->
       validate_number(changeset(%{"virtual" => "Oops"}), :virtual, greater_than: 0)
     end
   end
@@ -1502,6 +1502,8 @@ defmodule Ecto.ChangesetTest do
       assert changeset.errors ==
                [title: {"has already been taken", validation: :unsafe_unique, fields: [:title]}]
 
+      assert changeset.validations == [title: {:unsafe_unique, fields: [:title]}]
+
       Process.put(:test_repo_all_results, context.no_dup_result)
       changeset = unsafe_validate_unique(context.base_changeset, :title, TestRepo)
       assert changeset.valid?
@@ -1517,6 +1519,8 @@ defmodule Ecto.ChangesetTest do
                    {"has already been taken", validation: :unsafe_unique, fields: [:title, :body]}
                ]
 
+      assert changeset.validations == [title: {:unsafe_unique, fields: [:title, :body]}]
+
       Process.put(:test_repo_all_results, context.no_dup_result)
       changeset = unsafe_validate_unique(context.base_changeset, [:title, :body], TestRepo)
       assert changeset.valid?