Skip to content
This repository has been archived by the owner on Mar 10, 2024. It is now read-only.

feat: Sequence integration tests #1843

Merged
merged 5 commits into from
Nov 3, 2023
Merged

feat: Sequence integration tests #1843

merged 5 commits into from
Nov 3, 2023

Conversation

tonyxiao
Copy link
Contributor

@tonyxiao tonyxiao commented Nov 2, 2023

Fixes: ####

[Describe your change here]

Test Plan

[Describe test plan here]

Deployment instructions

[Add any special deployment instructions here]

Copy link

vercel bot commented Nov 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
management-ui ⬜️ Ignored (Inspect) Visit Preview Nov 3, 2023 5:20pm
supaglue-docs ⬜️ Ignored (Inspect) Visit Preview Nov 3, 2023 5:20pm

@tonyxiao tonyxiao marked this pull request as ready for review November 2, 2023 18:28
Copy link
Contributor

@asdfryan asdfryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also test sequenceState.

Something new that we decided to commit to was to also test invalid request bodies and test that the error responses include relevant context.

@tonyxiao
Copy link
Contributor Author

tonyxiao commented Nov 2, 2023

Done for both

);
expect(response.status).toEqual(201);
expect(response.data.record?.id).toBeTruthy();
addedObjects.push({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this get cleaned up at the end even if there are active contacts on it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
Yes, it appears so.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Though delete is only supported in outreach api though.

Copy link
Contributor Author

@tonyxiao tonyxiao Nov 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is salesloft

image

And for apollo I don't think there is delete at all, only archive.

@tonyxiao tonyxiao merged commit 68cde18 into main Nov 3, 2023
11 checks passed
@tonyxiao tonyxiao deleted the tony/integration-tests branch November 3, 2023 20:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants