Delete PossibleIndexSubmission when anonymizing the case (#1899)

jshmrtn · Mar 10, 2023 · 717638c · 717638c
1 parent 6fa7f97
commit 717638c
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 4 deletions.
diff --git a/lib/hygeia/case_context.ex b/lib/hygeia/case_context.ex
@@ -2485,7 +2485,8 @@ defmodule Hygeia.CaseContext do
         :auto_tracing,
         :received_transmissions,
         :propagated_transmissions,
-        :tests
+        :tests,
+        :possible_index_submissions
       ])
 
     attrs = %{
@@ -2501,6 +2502,7 @@ defmodule Hygeia.CaseContext do
       tests: Anonymization.anonymize_test_params(case.tests),
       anonymized: true,
       anonymization_date: Date.utc_today(),
+      possible_index_submissions: [],
       monitoring: Anonymization.anonymize_monitoring_params(case.monitoring)
     }
 

diff --git a/lib/hygeia/case_context/case.ex b/lib/hygeia/case_context/case.ex
@@ -138,7 +138,10 @@ defmodule Hygeia.CaseContext.Case do
       foreign_key: :propagator_case_uuid,
       on_replace: :delete
 
-    has_many :possible_index_submissions, PossibleIndexSubmission, foreign_key: :case_uuid
+    has_many :possible_index_submissions, PossibleIndexSubmission,
+      foreign_key: :case_uuid,
+      on_replace: :delete
+
     has_many :emails, Email, foreign_key: :case_uuid, on_replace: :delete
     has_many :sms, SMS, foreign_key: :case_uuid, on_replace: :delete
     has_many :notes, Note, foreign_key: :case_uuid, on_replace: :delete
@@ -207,6 +210,7 @@ defmodule Hygeia.CaseContext.Case do
     |> cast_assoc(:propagated_transmissions)
     |> cast_assoc(:sms)
     |> cast_assoc(:tests)
+    |> cast_assoc(:possible_index_submissions)
     |> validate_monitoring()
     |> cast_assoc(:notes)
     |> cast_embed(:phases, required: true)

diff --git a/priv/repo/migrations/20230310101526_anonymization_delete_possible_index_submissions.exs b/priv/repo/migrations/20230310101526_anonymization_delete_possible_index_submissions.exs
@@ -0,0 +1,26 @@
+# credo:disable-for-this-file Credo.Check.Readability.Specs
+defmodule Hygeia.Repo.Migrations.AnonymizationDeletePossibleIndexSubmissions do
+  @moduledoc false
+
+  use Hygeia, :migration
+
+  def up do
+    execute(fn ->
+      :ok = run_authentication(repo(), origin: :anonymization_job, originator: :noone)
+    end)
+
+    execute("""
+    DELETE
+      FROM possible_index_submissions
+      WHERE case_uuid IN (
+        SELECT
+          uuid
+          FROM cases
+          WHERE anonymized
+      )
+    """)
+  end
+
+  def down do
+  end
+end
diff --git a/test/hygeia/case_context_test.exs b/test/hygeia/case_context_test.exs
@@ -830,6 +830,8 @@ defmodule Hygeia.CaseContextTest do
         }
       })
 
+      possible_index_submission_fixture(case_main, %{})
+
       assert %Case{
                uuid: ^case_main_uuid,
                notes: [%Note{}],
@@ -877,6 +879,7 @@ defmodule Hygeia.CaseContextTest do
                    }
                  }
                ],
+               possible_index_submissions: [_submission],
                anonymized: false,
                anonymization_date: nil
              } =
@@ -888,7 +891,8 @@ defmodule Hygeia.CaseContextTest do
                  :auto_tracing,
                  :received_transmissions,
                  :propagated_transmissions,
-                 :tests
+                 :tests,
+                 :possible_index_submissions
                ])
 
       assert {:ok, anonymized_case} = CaseContext.anonymize_case(case_main)
@@ -936,6 +940,7 @@ defmodule Hygeia.CaseContextTest do
                    }
                  }
                ],
+               possible_index_submissions: [],
                anonymized: true,
                anonymization_date: ^today
              } =
@@ -947,7 +952,8 @@ defmodule Hygeia.CaseContextTest do
                  :auto_tracing,
                  :received_transmissions,
                  :propagated_transmissions,
-                 :tests
+                 :tests,
+                 :possible_index_submissions
                ])
     end