add test

Author: greg-rychlewski

test/ecto/repo/has_assoc_test.exs

@@ -45,6 +45,7 @@ defmodule Ecto.Repo.HasAssocTest do
       has_one :on_delete_nilify_assoc, MyAssoc, on_delete: :nilify_all
       has_many :assocs, MyAssoc, on_replace: :delete
       has_many :ordered_assocs, MyAssoc, on_replace: :delete, change_order: :unique_safe
+      has_many :ordered_nilify_assocs, MyAssoc, on_replace: :nilify, change_order: :unique_safe
       has_many :delete_assocs, MyAssoc
       has_many :on_delete_delete_assocs, MyAssoc, on_delete: :delete_all
       has_many :on_delete_nilify_assocs, MyAssoc, on_delete: :nilify_all
@@ -435,6 +436,30 @@ defmodule Ecto.Repo.HasAssocTest do
     assert tu < ti
   end
 
+  test "updating assocs with change_order: :unique_safe and on_replace: :nilify" do
+    sample1 = %MyAssoc{id: 10, x: "xyz", my_schema_id: 1} |> Ecto.put_meta(state: :loaded)
+    sample2 = %MyAssoc{id: 11, x: "abc", my_schema_id: 1} |> Ecto.put_meta(state: :loaded)
+
+    changeset =
+      %MySchema{id: 1, ordered_nilify_assocs: [sample1, sample2]}
+      |> Ecto.Changeset.cast(
+        %{x: "abc", ordered_nilify_assocs: [%{id: 12, x: "xyz"}, %{id: 10, x: "tuv"}]},
+        [:x]
+      )
+      |> Ecto.Changeset.cast_assoc(:ordered_nilify_assocs)
+
+    TestRepo.update!(changeset)
+    # Parent
+    assert_received {:update, _}
+    # Assoc changes
+    assert_received {:update, %{filters: [id: 10], order: tu1}}
+    assert_received {:update, %{filters: [id: 11], order: tu2}}
+    assert_received {:insert, %{order: ti}}
+
+    assert tu1 < ti
+    assert tu2 < ti
+  end
+
   test "replacing assocs on update on_replace: :delete" do
     sample = %MyAssoc{id: 10, x: "xyz"} |> Ecto.put_meta(state: :loaded)