Skip to content

UID2-6472: Replace opted-out test emails with uidapi.com addresses#136

Merged
lizk886 merged 3 commits intomainfrom
wzh-UID2-6472-update-test-user-not-opted-out
Feb 24, 2026
Merged

UID2-6472: Replace opted-out test emails with uidapi.com addresses#136
lizk886 merged 3 commits intomainfrom
wzh-UID2-6472-update-test-user-not-opted-out

Conversation

@lizk886
Copy link
Contributor

@lizk886 lizk886 commented Feb 23, 2026

Summary

The hash eVvLS/Vg+YZ6+z3i0NOpSXYyQAfEXqCZ7BTpAjFUBUc= was opted out in production, which caused testV2IdentityMapUnmapped tests to fail with unexpected opt-out results. The new email hash uses a more complex email that is unlikely to be opted out.

Replacements

Old value New value
eVvLS/Vg+YZ6+z3i0NOpSXYyQAfEXqCZ7BTpAjFUBUc= rYsn2sTT1lRlZerX+4DXZMQ+DfcejOXoMVczcF2jCLM= 9 (uid2.e2e.test.user@uidapi.com)

Test plan

  • Verify testV2IdentityMapUnmapped passes with the new test emails

The hash eVvLS/Vg+YZ6+z3i0NOpSXYyQAfEXqCZ7BTpAjFUBUc= (SHA256 of
user@example without .com) was opted out in production, causing
testV2IdentityMapUnmapped to fail — the test expects only "abc" to
be unmapped as "invalid identifier", but the opted-out hash also
appeared as unmapped with reason "optout".

Replaced with rYsn2sTT1lRlZerX+4DXZMQ+DfcejOXoMVczcF2jCLM= (SHA256
of uid2.e2e.test.user@uidapi.com) in the email_hash entries of
identityMapBatchBadEmailArgs in both operator and validator TestData.

Co-authored-by: Cursor <cursoragent@cursor.com>
@lizk886 lizk886 force-pushed the wzh-UID2-6472-update-test-user-not-opted-out branch from 9674581 to eeb4bfa Compare February 23, 2026 21:53
@lizk886 lizk886 marked this pull request as ready for review February 23, 2026 21:54
@lizk886 lizk886 merged commit 93cb776 into main Feb 24, 2026
4 checks passed
@lizk886 lizk886 deleted the wzh-UID2-6472-update-test-user-not-opted-out branch February 24, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants