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 and explaination for failing createOrUpdateUser test #553

Closed
colbya opened this issue Mar 7, 2023 · 0 comments · Fixed by #554
Closed

Fix and explaination for failing createOrUpdateUser test #553

colbya opened this issue Mar 7, 2023 · 0 comments · Fixed by #554

Comments

@colbya
Copy link

colbya commented Mar 7, 2023

Describe the bug
The createOrUpdateUser test is failing after attempting to update a phone number field on a user. It appears that the Zendesk api has additional rules for updating identity fields that are exposed on the user resource (i.e. email and phone). These rules include not allowing a duplicate value for an email field and not replacing but adding phone numbers when updated.

To Reproduce
Run the createOrUpdateUser test. The user modified ends up with two phone numbers instead of having the single phone number updated. The user object returned only displays the first phone number set making it appear as if the updated did not work.

Expected behavior
The field updated should be reflected on the object returned from the update api call. This expected behavior occurs when updating most fields. Recommended change is to update a non-identity and test for that change in the smoke test.

Additional context
This behavior was discovered while attempting to synchronize a large set of users with Zendesk. There are unexpected (and undocumented) behaviors with the Zendesk User endpoint including the ability to create an end-user without an email or phone number but not being able to clear both email and phone after they have been set.

PR to be submitted

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 a pull request may close this issue.

1 participant