Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: user deletion handlers test cases #1723

Conversation

anantjain45823
Copy link
Contributor

@anantjain45823 anantjain45823 commented Dec 21, 2022

Description of the change

Added mock test cases for userDeletion

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • "Ready for review" label attached to the PR and reviewers mentioned in a comment
  • Changes have been reviewed by at least one other engineer
  • Issue from task tracker has a link to this pull request

@anantjain45823 anantjain45823 requested review from a team and sivashanmukh as code owners December 21, 2022 10:52
@anantjain45823 anantjain45823 self-assigned this Dec 21, 2022
@anantjain45823 anantjain45823 changed the base branch from master to fix.user-deletion-handlers December 21, 2022 10:53
@anantjain45823 anantjain45823 changed the title Fix.user deletion handlers update test cases fix:user deletion handlers update test cases Dec 21, 2022
@saikumarrs saikumarrs changed the title fix:user deletion handlers update test cases fix: user deletion handlers test cases Dec 21, 2022
@saikumarrs saikumarrs merged commit 8315fc4 into fix.user-deletion-handlers Dec 21, 2022
saikumarrs added a commit that referenced this pull request Jan 5, 2023
* fix: make all the destination user deletion handlers uniform

* fix: update the metadata fetch code to send to bugsnag

* fix: req metadata function in user deletion handler

* fix: correction for the MAX_BATCH_SIZE value (#1720)

Co-authored-by: Anant <[email protected]>

* fix: update to add robust error handling (#1719)

* fix: update deletion flows

* fix: update test cases

* fix: update test cases to remove the ones which require mocking

* fix: handled specific success sattus for braze and intercom

* fix: make config error for af as retryable

* fix: user deletion handlers test cases (#1723)

* feat: mock cases for clevertap

* feat: mock cases for engage

Co-authored-by: Anant <[email protected]>

* fix(destination): avoid creating parallel delete user requests in clevertap (#1751)

* fix: user deletion module update (#1750)

* chore: coverage and testing for mixpanel and engage

* fix: updated api and test coverage for intercom

* refactor: coverage and testing for sendgrid

* feat: Batch Support with testing and mocking for Amplitude

* feat: Batch Support with testing and mocking for Braze

* fix: conflicts

* fix: conflicts+1

* fix: conflicts and resolved comments

* fix: added comments

* fix: resolved Comments

* fix: resolved Conflicts

* fix: resolved Comments

* fix: resolved Comments+1

* fix: resolved Comments+2

Co-authored-by: Anant <[email protected]>

* refactor: af user deletion (#1735)

* refactor: af user deletion

* fix: conflicts

Co-authored-by: Anant <[email protected]>

Co-authored-by: Utsab Chowdhury <[email protected]>
Co-authored-by: Anant Jain <[email protected]>
Co-authored-by: Anant <[email protected]>
@github-actions github-actions bot deleted the fix.user-deletion-handlers-updateTestCases branch March 22, 2023 02: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.

3 participants