Skip to content

Fix Sequence Tests#151

Open
n7studios wants to merge 1 commit into
masterfrom
fix-sequence-tests
Open

Fix Sequence Tests#151
n7studios wants to merge 1 commit into
masterfrom
fix-sequence-tests

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

Fixes the create/update/delete sequence email test since the API changes rightly no longer permit setting a position of an existing email in a sequence to one that already exists.

1) ConvertKitAPIKeyTest::testCreateGetUpdateAndDeleteSequenceEmail
GuzzleHttp\Exception\ClientException: Client error: `PUT [https://api.kit.com/v4/sequences/1030824/emails/10087666`](https://api.kit.com/v4/sequences/1030824/emails/10087666%60) resulted in a `422 Unprocessable Content` response:
{"errors":["position 1 is already taken in this sequence"]}

Testing

Existing tests pass.

Checklist

@n7studios n7studios self-assigned this Jul 16, 2026
@n7studios n7studios added the bug label Jul 16, 2026
@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team July 16, 2026 09:50
@n7studios n7studios marked this pull request as ready for review July 16, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants