Added logic to push resetIdentities event to all registered modules#190
Added logic to push resetIdentities event to all registered modules#190addb merged 3 commits intoadobe:consentfrom
Conversation
…andleResetIdentities
kevinlind
left a comment
There was a problem hiding this comment.
Can you add an additional test case which
- sets the location hint and state store
- calls resetIdentities
- sends a network request and verifies the request does not have the location hint and state store?
Your current test is good in that it verifies the delete function is called, but you should also verify the actual value is deleted.
I will add the case for the steps you mentioned. For verifying it actually deleted, I believe stateStore and locationHint unit tests would have covered that. And can be covered in Integration tests. |
|
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: