Use `AnonymizedFaker` for primary keys
CU-86azdzv1n, Resolve #1807.
Task linked: CU-86azdzv1n SDV - Contextual Anonymization transformers shouldn't be used for primary keys #1807
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 97.32%. Comparing base (
6222922) to head (d0da7c5). Report is 13 commits behind head on main.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## main #1841 +/- ##
=======================================
Coverage 97.32% 97.32%
=======================================
Files 50 50
Lines 4821 4823 +2
=======================================
+ Hits 4692 4694 +2
Misses 129 129
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Results of the change:
Could we add a test for this as well?
I can't write a test which actually verifies the logic, since that requires SDV-Enterprise. I can add a unit test that runs through the logic though.